Stuff still visible at /user when site is offline (Maintenance mode)

Drupal is supposed to work this way. This is so an administrator can log on to take the site off maintenance if they were logged off for whatever reason.

You're right, I apologize. I'll patch it up tonight.

It's actually somewhat complicated of a problems and I'll need all of you to vote on which solution you prefer... Patches coming soon.

This is bugging me too. I would prefer the main theme to be invisible if a user goes to ?q=user to log in when the site is in offline mode. Just a plain admin themed login block would be ideal when in offline maintenace. Maybe someone can suggest a simple way to do this for D6?

I've been trying to find a way around this too for D6.
I made custom maintenance-page.tpl.php, and maintenance-page-offline.tpl.php, and that helps. The user login form still bypasses it though. I poked around the documentation looking to see if maybe there's a php variable that get's set when in maintenance mode. If so maybe then a bit of php code could be added to the main page.tpl.php in your theme that bypasses the standard design, and goes to a "offline" design. Kind of like how the $page= flag is used.

@#6 - you can set your own variable, I had to do this on the weekend for D6 site and wrote a tutorial about it also because it was so annoyingly complicated to get done (not set the var, I mean the whole she-bang of theming the user login page when offline).

function template_preprocess(&$vars, $hook) {
  $vars['in_maintenance'] = MAINTENANCE_MODE;
}

Thank you. That's just what I was picturing.

This has become a thread about Drupal 6 work arounds.

Is this patch still being considered? Or should this go to D8?

Maintenance mode:

What you see at /user in maintenance mode

Sorry for holding this up for so long. I ended up getting busy/forgetting about this.

I'll write a something if it hasn't been done by the time I'm less busy.

I doubt it will get into Drupal 7, but we should definitely get it into Drupal 8.

I think you're right, this is probably a D8 thing. In fact, this will likely roll into the numerous proposed changes to Maintenance mode. Though I'm wary of moving it out myself, heh.

Not sure about hiding the logo and sitename etc, many of my clients want this page branded.

Garrett -in what was is it a security issue, definitely a WTF I'd agree (as in why isn't maintenance page tpl used for all pages when in maintenance mode?)

Even if it isn't a security issue, it's definitely a webmaster's sanity one. I have a site offline now, and I have to deal with people emailing, asking why they can't get to the parts of the menu they can see. For some reason the big notice saying the site's off line means nothing.

Well, the patch works as required. RTBC from me, pending it getting tested.

Yet... I have this sneeking suspicion this is a feature request, and will be pushed to D8.

- h

Looks elegant.
The main debatable point here is the new menu constant. Moving this to the menu subsystem so that one of the maintainers can voice his opinion.

+++ modules/simpletest/tests/menu.test	10 Sep 2010 03:12:29 -0000
@@ -82,6 +82,12 @@ class MenuRouterTestCase extends DrupalW
+    ¶
+    // Assure that the user login and password reset pages are using the
+    // maintenance theme too (in Drupal 6, the main site theme was used). The
+    // presence of the form itself is tested in modules/system/system.test.
+    $this->drupalGet('user/login');
+    $this->assertRaw('bartik/css/style.css', t("The maintenance theme's CSS appears on the page."));    ¶

whitespace issues, tabs?

// Assure that the user login and password reset pages use the
// maintenance theme. The presence of the form is tested in 
// modules/system/system.test.

Thinking this comment needs a make-over...

The maintenance theme's CSS appears on the page.

I don't quite follow what this means?

Powered by Dreditor.

Seems verbose and too passive. My rework of the comment in #24 is more active and succinct.

+++ modules/simpletest/tests/menu.test	15 Sep 2010 17:21:13 -0000
@@ -83,6 +83,12 @@ class MenuRouterTestCase extends DrupalW
+    // maintenance theme. The presence of the form is tested in ¶

one white space

other than that RTBC, looks good and works as expected, great work Garrett.

Powered by Dreditor.

With Dreditor - it will highlight them :)

#29: 722434-site-offline.patch queued for re-testing.

+++ includes/menu.inc	16 Sep 2010 02:43:00 -0000
@@ -244,6 +244,14 @@ define('MENU_SITE_OFFLINE', 4);
 /**
+ * Internal menu status code -- Site is offline, but show this content using the
+ * "maintenance" page template anyway. Useful for showing the user login and
+ * password retrieval forms while the site is in maintenance mode, but not
+ * showing any other page content besides that (such as sidebar content, etc).
+ */
+define('MENU_SITE_ONLINE_MAINTENANCE_TPL', 6);

This looks a bit odd to me.

1) Why not simply MENU_SITE_MAINTENANCE?

2) Why is no implementation of hook_menu_site_status_alter() not changed in this patch?

3) Not sure I understand why $page_callback_result is renamed to $offline_status in this patch?

Overall, I have the impression that this patch doesn't take all system components for this logic into account. At the very least, it requires in-depth reviews of base system maintainers.

Powered by Dreditor.

I was suprised to find that an entire "book" is viewable on the net, and listed in google, even though site is in maintenance mode. D7. http://cafelehibourecollections.ca

Is there now a solution in D7 to show only content variable, containing only the login form, without a patch ?
I mean, it seems obvious not to show other blocks and stuff at /user page while in offline mode...