Closed (fixed)
Project:
Maintenance Helper
Version:
7.x-2.x-dev
Component:
Code
Priority:
Critical
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
11 May 2010 at 06:38 UTC
Updated:
1 Jun 2013 at 20:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
catchComment #2
gregglesLooks good. Don't we need to rebuild the menu cache in the _submit if we want the menu_alter to work?
Comment #3
catchWe would, if I'd added a UI for the variable in the settings form, which it appears I didn't.
Comment #4
claar commentedThis will need re-rolled for 6.x-2.x -- also, I suggest making this issue a straight D7 port. Toward that end, I've created #1096334: Add feature to (optionally?) hide login methods.
Comment #5
jonhattanHere's a patch for a straight port from 6.x-2.x. I've added three minimal changes not related to the port:
* Implement hook_uninstall().
* Use url() to build the link to the Maintenance Helper from the Maintenance Mode page.
* Move the check to show the message inside the user conditions check. That is, show the message iff the user is restricted to login.
Comment #6
claar commentedTested, committed, new D7 release created. Thanks, Jonhattan!
Comment #7
jonhattanThanks claar. Please make the release visible in the project page (Administer releases > Show snapshot release).
Comment #8
claar commentedDone; thanks.