When we have a "destination" in the query parameters (cf drupal_get_destination()), the admin paths are not renamed. So it causes 404 errors.

When renaming the paths, the module should check if there is a destination, and if so, rename the destination too.

CommentFileSizeAuthor
#8 rename_admin_paths-rename-query-destination-2804667-3-D7.patch1.37 KBfonski
#3 rename_admin_paths-rename-query-destination-2804667-2-D7.patch1.37 KBb-prod
#2 rename_admin_paths-rename-query-destination-2804667-1-D7.patch1.21 KBfonski

Comments

Fonski created an issue. See original summary.

fonski’s picture

fonski commented
Status: Active » Needs review
StatusFileSize
new rename_admin_paths-rename-query-destination-2804667-1-D7.patch1.21 KB
b-prod’s picture

b-prod commented
Version: 7.x-2.1 » 7.x-2.x-dev
StatusFileSize
new rename_admin_paths-rename-query-destination-2804667-2-D7.patch1.37 KB

This patch is almost the same as the one in comment #1, but it also take car of the presence of %2F in the query string, as it is the URI-converted forward slash.

  • Raphael Apard committed 398a069 on 7.x-2.x authored by B-Prod 
    Issue #2804667 by Fonski, B-Prod: Rename not working with query...
raphael apard’s picture

raphael apard commented

Make sense. Thanks

raphael apard’s picture

raphael apard commented
Status: Needs review » Fixed
raphael apard’s picture

raphael apard commented
Status: Fixed » Closed (fixed)
fonski’s picture

fonski commented
StatusFileSize
new rename_admin_paths-rename-query-destination-2804667-3-D7.patch1.37 KB

Fix an error for user path (it was checking for "admin" instead of "user").

raphael apard’s picture

raphael apard commented

Thanks ! It's committed