When logging in and accepting the T&C, users should be logged in normally. Current code attempts to do this, but it is implemented incorrectly.

The following patch fixes the issue. Users are now properly logged in after accepting T&C.

CommentFileSizeAuthor
#9 legal.560548_1.patch277 bytesAnonymous (not verified)
#3 legal-6.x-2.2-beta4.patch467 bytescizar
legal.module.patch1.03 KBentendu

Comments

a.bond’s picture

a.bond commented

I'm not sure if my problem is the same as this, but I've had some customers report that when they login and accept the T&C, they get a “User Id cannot be verified” error. Also, when someone who is not logged in attempts to go to the login page, the T&C flashes briefly before redirecting them to the login page.

christefano’s picture

christefano commented
Status: Patch (to be ported) » Reviewed & tested by the community

This works for me. Thanks, entendu!

a.bond, I think you're having a separate issue. In any case, please review this patch if you can and report back.

cizar’s picture

cizar commented
StatusFileSize
new legal-6.x-2.2-beta4.patch467 bytes

I'm having the same problem with Presflow-6.19. Here is my solution, it works fine.

a.bond’s picture

a.bond commented

Christefano, I'm not seeing the problem anymore. Not sure what changed.

recidive’s picture

recidive commented
Status: Reviewed & tested by the community » Postponed (maintainer needs more info)

Can you guys tell me if this problem still occurs in HEAD?

seakayjay’s picture

seakayjay commented
Version: 6.x-2.2-beta4 » 6.x-2.3-rc1

I'm using Pressflow 6.19. I'm having the same issue with the 6.x-2.3-rc1 release.

seakayjay’s picture

seakayjay commented

Everything works fine with me after inserting the code as given in #3

Right after
cache_clear_all($user->uid, 'cache_menu', TRUE);

Insert
sess_regenerate();

wonder95’s picture

wonder95 commented

Was having this same issue and the suggestion by @kuanjian in #7 fixed it.

Anonymous’s picture

Anonymous (not verified) commented
Assigned: Unassigned »
Status: Postponed (maintainer needs more info) » Reviewed & tested by the community
StatusFileSize
new legal.560548_1.patch277 bytes

I can confirm that the patch against the legal module (DRUPAL-6--2-3-RC1) fixes the problem on the latest version of pressflow downloaded from http://fourkitchens.com/downloads and also does not result in any problems for the legal module on ordinary installations of Drupal.

Best,
Paul Booker
Appcoast

robert castelo’s picture

robert castelo commented
Status: Reviewed & tested by the community » Fixed

Thanks for patches and testing.

Fixed in Legal 6.x-2.4-rc2

Status: Fixed » Closed (fixed)

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