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.

CommentFileSizeAuthor
#1 1296906-1-menu-path.patch375 bytesEvanDonovan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

EvanDonovan’s picture

EvanDonovan CreditAttribution: EvanDonovan commented
Status: Active » Needs review
FileSize
1296906-1-menu-path.patch375 bytes

Here's the patch.

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
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

EvanDonovan CreditAttribution: EvanDonovan commented

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

EvanDonovan CreditAttribution: EvanDonovan commented
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

mesr01 CreditAttribution: mesr01 commented

Actually, it's a duplicate of issue 1096294.