Closed (fixed)
Project:
Front Page
Version:
5.x-1.6
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
2 Nov 2006 at 05:14 UTC
Updated:
6 Feb 2007 at 11:25 UTC
Jump to comment: Most recent file
Using 5b1 with frontpage cvs, redirect for authenticated user and themed for anon user results in the authenticated user getting the same page as the anon user (that is they both get the anon users page).
| Comment | File | Size | Author |
|---|---|---|---|
| #20 | front_page.info | 184 bytes | dublin drupaller |
| #19 | front_page_1.module | 15.7 KB | dublin drupaller |
| #8 | front_page.module.txt | 14.95 KB | dublin drupaller |
Comments
Comment #1
dublin drupaller commentedthanks for spotting that Ac. I'll get right on it.
Dub
Comment #2
dublin drupaller commentedI've fixed the redirect problem and updated the module. Apologies for taking so long.
Download the updated module using this link. I can't assign the correct tags for the CVS at the moment...
http://cvs.drupal.org/viewcvs/drupal/contributions/modules/front/front_p...
dub
Comment #3
acNice one. I still can't get roles to redirect properly though. The anon role redirects properly but authenticated and any created roles redirect to the anon redirect page.
Comment #4
dublin drupaller commentedSorry Ac. I left in a typo.
use the link below to download the latest version
********************************************************
IF YOU ARE USING DRUPAL 5 PLEASE DOWNLOAD THE LATEST
VERSION OF THE FRONT PAGE MODULE FROM THIS LINK:
********************************************************
After you have downloaded the daily snapshot tar.gz file, overwrite the module file with the latest version at this link:
http://cvs.drupal.org/viewcvs/drupal/contributions/modules/front/front_p...
Apologies for the inconvenience. The CVS system keeps returning error messages and I'm unable to tag the module properly with the new updates.
Comment #5
acI can't see the typo fix using diff, and the redirects still dont work for me using version 1.44. Am I using the wrong version? Sorry for the bother.
Comment #6
dublin drupaller commentedtry this version:
http://cvs.drupal.org/viewcvs/drupal/contributions/modules/front/front_p...
Dub
Comment #7
acYeh thats what I am on
// $Id: front_page.module,v 1.44 2006/12/06 13:33:54 jasonm3m Exp $
I just update the module from cvs against HEAD. Authenticated and custom role redirects still not working.
Comment #8
dublin drupaller commentedit's working for me at this end Ac. try the attached file instead. I think there are problems with the CVS system - I wasn't able to tag the file properly earlier.
Comment #9
acStill can't get this working. I checkouted a fresh copy of HEAD and used this attached version and it still won't redirect authorised users or custom roles. Not sure what to do. Is there any other information I can provide to help?
Comment #10
ac'checkouted' -> checked out
Comment #11
dublin drupaller commentedI'm not sure Ac. I have it working on a fresh install of Drupal 5 beta 2, with custom roles and I've tried almost every variation of redirects and mixed themed/redirects for alternate roles and it works fine.
Drop me an email with a login so I can see how you have setup the front page...maybe you have overlooked something simple.
Dub
Comment #12
Ian Ward commentedI may see the issue here but need to debug a little more - but if you look at
you'll see that statement encapsulates the case statement where a redirect would be fired. The issue is, unless you have something in _text (something in the textarea for the given role) it will not get to the case statement - because the if statement says && ... != "" for the given _text area.
Make sense?
Ian
Comment #13
dublin drupaller commentedwell spotted Ian. I'm up to my eyes at the moment, but, I think you're right about the way the logic is put together.
Will have a more in depth look at it when I get a chance.
thanks for flagging that.
Dub
Comment #14
daffy commentedI can confirm these observations and the root cause analysis.
Simply put, if a text/body field for a role is empty, "redirect" will not work.
Comment #15
lxndr commentedMe too.
No role-depending redirection without text in the body-field.
I'm lucky that I found this thread here.
Comment #16
dublin drupaller commentedI updated the module last night with a fix for the redirect problem when the text field is empty.
Download from here:
http://cvs.drupal.org/viewcvs/drupal/contributions/modules/front/
I've tested it on the latest version of Drupal 5 (Jan 18th) and appears to work fine.
Marking as fixed. Reopen if you have any problems with it.
Cheers
Dub
Comment #17
aglasser commentedSorry to say, just tested the latest version on a brand spanking new install of Drupal 5 and the redirect is still not working.
Comment #18
dublin drupaller commentedMake sure you're downloading the latest version:
http://cvs.drupal.org/viewcvs/drupal/contributions/modules/front/front_p...
I've tried it on a fresh install of Drupal 5 and it works as expected..including the redirects.
Dub
Comment #19
dublin drupaller commentedI have attached the very latest version to this post...
I notice the .info file is not being picked up by the normal download, so make sure you download both files..i.e. front_page.module and front_page.info
Dub
Comment #20
dublin drupaller commentedhere's the .info file. (the normal download isn't including this for some reason)
Dub
Comment #21
dublin drupaller commentedJust correcting the link to download the latest version
CLICK HERE TO DOWNLOAD A ZIPPED VERSION OF THE MODULE FOR DRUPAL 5.X (IN ZIP FILE FORMAT) - January 22nd
Apologies for the inconvenience. The CVS system keeps returning error messages and I'm unable to tag or branch the module properly with the new updates.
Dub
Comment #22
mjarkiver commentedI'm also having the redirect problem. Quicklogin on the front page doesn't work at all, but the login at /user works fine (or ?q=user). I downloaded both files from the comments in this thread and updated my drupal install (latest version of 5.0), but it is still not working.
Comment #23
Phillip Mc commentedthere's a newer update of the module on the main page for this project.
You could try that, but, I don't think that will solve your problem. The redirects are working fine for me using Drupal 5.x.
Are you having the same problem to what Pknag experienced at this thread?
http://drupal.org/node/111828#comment-192461
Phil
Comment #24
dublin drupaller commentedjust a quick note regarding using redirects on the front page for anon users:
In previous versions, the front page redirect would automatically pick up those redirect settings, but, there have been changes made to the Drupal core that affects the login process which means that it doesn't work as it used to:
http://drupal.org/node/83822
those changes to core have thrown up a lot of problems in other areas, that are probably more pressing than the impact on the front_page.module, so I haven't been screaming about it. Once those problems are sorted, I'll revisit the front page redirects to see how I can get the front_page to work with the new changes.
In the meantime, use the Front_page setting for ANONYMOUS USERS to setup your anonymous node. In other words, don't use direct for anonymous users.
i.e. what will work is the following:
(a) for your 'Anonymous User' front page go to ADMINISTER -> SETTINGS -> FRONT PAGE and in the text area setting for anonymous users, paste in the content you want anonymous users to see and select themed.
(b) Redirecting for authenticated users (assuming they are able to login) will work as it should.
I hope that helps and makes sense. I'll revist the front_page.module redirects when the login issues are sorted. In the meantime, just use the procedure outlined.
Dub
p.s. I'm closing off this issue as it's a duplicate of others. open this issue back up by posting a reply and selecting ACTIVE in the status option if you're still having problems after following the procedure outlined.