I use the module to create simple url's so I can create domain.com/twitter and will redirect to twitter url.

Was working fine but then stopped and now returns a message:

The requested page "/twitter" could not be found.

Tried uninstalling it, removed it, re-installed, deleted and re-entered all entries and same behavior.

The only recent change was to adjust the replacement rules on path-auto.

I've read the other "stopped working" threads but no solution worked for me.

CommentFileSizeAuthor
#12 1444506-check-maintenance-mode-permission.patch604 bytesdave reid
#8 redirect.miscellaneous.1444506-8.patch687 byteshelmo
#6 redirect.miscellaneous.1444506-6.patch562 bytesclemens.tolboom

Comments

chris-cmfi’s picture

chris-cmfi commented

Also cleared all cache (no pages are being cached), ran cron, and updates.

Will try dev version along with this patch (http://drupal.org/node/1198028) to see if that works.

chris-cmfi’s picture

chris-cmfi commented

The module is set to not redirect when in maintenance mode though my site has been in maintenance mode the whole time (not yet live).

Updating the variable from FALSE to TRUE alleviated my issue.

elseif (variable_get('maintenance_mode', 0) || defined('MAINTENANCE_MODE')) {
// Do not redirect in offline or maintenance mode.
$can_redirect = TRUE;

Line 1093 'redirect.module' in DEV version

nhck’s picture

nhck commented
Title: The Module Stopped Working » Redirect behaviour in maintenance mode is confusing
Priority: Normal » Minor

As outlined above the module does not redirect when the site is in maintenance mode. Whilst this behavior makes perfect sense Redirect shouldn't provide a 404 page to logged in users. In my humble opinion it would be nice to display a message to users that are logged in (permission to use site in maintenance mode) and a 404 (or maintenance message) to users that are not logged in.

obleser’s picture

obleser commented

Aggree with all of the above!
It took me 1 day (!) to find this post (Tried all of the things mentioned by #1)

#2:

my site has been in maintenance mode the whole time (not yet live).

same here :/

The proposal of #3 sounds great.

By the way: A simple system warning message on the redirect config page in maintenance mode were enough...

alforddm’s picture

alforddm commented

I vote for #3 as well. So glad I found this post!

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
commented
Version: 7.x-1.0-beta4 » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new redirect.miscellaneous.1444506-6.patch562 bytes
clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
commented

See also #1795224: Redirect not working in Maintenance mode

+++ b/redirect.module
@@ -1168,6 +1168,7 @@ function redirect_can_redirect() {
+      drupal_set_message(t('Redirect does not work in maintenance mode'));

Call to watchdog should add 'warning' and no repeat.

      drupal_set_message(t('Redirect does not work in maintenance mode'), 'warning', FALSE);
helmo’s picture

helmo commented
StatusFileSize
new redirect.miscellaneous.1444506-8.patch687 bytes

Or even better... let an admin with the right permissions follow the redirect.

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented

Is there any situation when this redirect can cause problems?

helmo’s picture

helmo commented

I doubt it. It's working fine here. And anyone with the 'access site in maintenance mode' permission should be aware of ongoing maintenance :)

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented
Status: Needs review » Reviewed & tested by the community

I also do not see any problem with this change..

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Issue summary: View changes
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 1444506-check-maintenance-mode-permission.patch604 bytes

Re-rolled to move the user_access() check up to the if statement rather than inside of it, so that the additional checks can still execute.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

Committed #12 to 7.x-1.x.

  • Dave Reid committed 20542c1 on 7.x-1.x 
    Issue #1444506 by helmo, clemens.tolboom, Dave Reid: Fixed users with...

Status: Fixed » Closed (fixed)

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