hi guys,

$items['admin/config/administration/admin_menu'] = array(

I think '_' in url is a Invalid URL.In Ie Browser it will course some problems.

CommentFileSizeAuthor
#1 admin_menu-invalid_url-2123625-1.patch617 bytesycshen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ycshen’s picture

Status: Active » Needs review
FileSize
617 bytes

hi,

I have patched it. and need review.

@@ -78,7 +78,7 @@ function admin_menu_menu() {
     'file' => 'system.admin.inc',
     'file path' => drupal_get_path('module', 'system'),
   );
-  $items['admin/config/administration/admin_menu'] = array(
+  $items['admin/config/administration/admin-menu'] = array(
renatog’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Good catch.

Really makes sense.

Works good.

  • RenatoG committed 182488f on 7.x-3.x authored by ycshen
    Issue #2123625 by ycshen: Invalid url define in hook_menu
    
renatog’s picture

Status: Reviewed & tested by the community » Fixed

Thank you very much for your contribution.

Best,

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

ron_s’s picture

Please review this related patch: https://www.drupal.org/project/admin_menu/issues/3061018

There are a number of places where the "admin_menu" path is referenced in the module. The patch committed in this thread will lead to errors without the patch I created in #3061018.

Thanks.