So in the System category #564562: Configuration page: System category we have now removed all items under system, however we still need to remove the Site Configuration category as a whole. This issue is all about the possible complications that this change would bring, they are quite a couple of links we left and we still need to take a look at.

This patch is in coherence with a larger movement of #546956: [meta-issue] Overhaul of Information Architecture

CommentFileSizeAuthor
#1 die-admin-settings-die-v1.patch15.63 KBjoshmiller

Comments

joshmiller’s picture

StatusFileSize
new15.63 KB

Attached is a first run... My local Drupal installation has died, but that was after enabling all core modules and clearing the cache (could be another issue)

Props to Bojhan for keeping me on this.

Be brutal, and then roll your own ;P

joshmiller’s picture

Status: Active » Needs review

*cough*

Bojhan’s picture

Waiting for some tests from boombatower, to confirm the greenishness of this test.

gábor hojtsy’s picture

I think these links fixed by this issue were not left there but rather added in parallel patches, which were not aware of the path changes. Nice catches though :)

catch’s picture

Status: Needs review » Reviewed & tested by the community
joshmiller’s picture

If anyone is interested in writing a test to keep these kind of links from getting committed, #569332: Need test to confirm there are no dead links in code -- I created an issue. Boombatower has some working code, but hasn't posted anything yet. Bug him to see if he kept it.

josh

catch’s picture

bump. I keep thinking I need to rebuild my menu because site configuration is still there.

gábor hojtsy’s picture

Yeah, a few people jumped on me in Paris that they found a bug: "an empty Site configuration area".

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -Drupal, -IA, -D7UX

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

Issue tags: +, +IA, +D7UX