This incredibly small module seems like it should be rolled into system.module to me. Admin.module can be removed once this patch for system.module is applied.

CommentFileSizeAuthor
system.module_1.patch1.85 KBdrumm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented

+1 on removing the admin module, but why not name the admin page function system_admin_main_page() or something prefixed with 'system', to be consistent?

Dries’s picture

Dries CreditAttribution: Dries commented

Committed to HEAD. Renamed the callback as per Goba's suggestion.

Steven’s picture

Steven CreditAttribution: Steven commented

There were several other mentions of admin.module around, including an entry in the system table. I updated HEAD to remove them.

Anonymous’s picture

(not verified) CreditAttribution: commented
apaderno’s picture

apaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
CreditAttribution: apaderno commented
Version: » 7.x-dev
Component: admin.module » other

I am changing the component for the issues using admin.module.