Currently the config path in D7 is still under admin/settings, which no longer exists.

It should be admin/config/aes.

Line 17 of aes.module should change to:

$items['admin/config/aes'] = array(

I can supply a patch if needed.

#1 1296906-1-menu-path.patch375 bytesEvanDonovan
Members fund testing for the Drupal project. Drupal Association Learn more


EvanDonovan’s picture

Status: Active » Needs review
375 bytes

Here's the patch.

tstoeckler’s picture

Status: Needs review » Needs work

In D7, modules shouldn't take paths directly under admin/config. We should choose a category for these settings to live under, e.g. "admin/config/security/aes" or something.

EvanDonovan’s picture

Is that an already-existing category? I wouldn't want to have the module create a category if no other modules use it.

To me, that just seems like burying the configuration.

EvanDonovan’s picture

Status: Needs work » Closed (duplicate)

Realized this was actually duplicate of an earlier issue - #1296906: Config path should be admin/config/aes.

mesr01’s picture

Actually, it's a duplicate of issue 1096294.