Using a dash (-) in a path causes a white screen when saving the new path for both /admin and /user. Using a new path without a dash works as expected.

stderr: PHP Fatal error:  Call to undefined function Drupal\\rename_admin_paths\\Form\\form_error() in 
/modules/contrib/rename_admin_paths/src/Form/RenameAdminPathsCallbacks.php on line 20,

Drupal core 8.0.5 and the only other contrib modules I have are ctools, pathauto, rename_admin_paths, restui, token.

CommentFileSizeAuthor
#5 reaname-admin-paths-dashes.jpg79.22 KBkingfisher64
#4 2687371-4.patch1.34 KBheykarthikwithu

Comments

orangecoat-ciallella created an issue. See original summary.

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Status: Active » Needs work

Tested this case,

If site-admin is using dash (-) in a path setting of "Rename admin path", we will be able to see the WSOD.

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Assigned: Unassigned » heykarthikwithu
heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Assigned: heykarthikwithu » Unassigned
Status: Needs work » Needs review
Issue tags: +Quick fix
StatusFileSize
new 2687371-4.patch1.34 KB

Actually the module is not providing support to lowercase letters, numbers, and underscores in field values.

This was a quick fix for showing up the error on the field validation.

kingfisher64’s picture

kingfisher64 commented
StatusFileSize
new reaname-admin-paths-dashes.jpg79.22 KB

Tested patch outputs message in screenshot.

raphael apard’s picture

raphael apard commented
raphael apard’s picture

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

raphael apard commented
Status: Fixed » Closed (fixed)