fixes a similar issue to: node/154568

This occurs on lines 238, 234, 230. I think it may be on one other one, but you can run a diff.

This also fixes the issue described at: node/155205 in post 4, and at: node/186396 for the whole issue.

Basically, the uid was hardcoded in the module to 7. If your 7th user was all filled out profile wise, you had no issues.

This also fixes some grammar issues in debug mode.

For anyone that wants this prior to it being included, just download the attached file, and overwrite your existing one in the profile_enforcer module directory.

CommentFileSizeAuthor
#3 profile_enforcer.module.tar_.gz3.02 KBxieliwei
#1 profile_enforcer.module.tar_.gz2.95 KBdragonfyre6
profile_enforcer.module.tar_.gz2.95 KBdragonfyre6

Comments

dragonfyre6’s picture

dragonfyre6 commented
StatusFileSize
new profile_enforcer.module.tar_.gz2.95 KB

Crap. That included a line of my own debugging code, and will screw up your redirects. This is the one I meant to upload.

xieliwei’s picture

xieliwei commented

Confirm patch works, but causes infinite redirect loop when an unauthorised user visits the site.

I've turned off content access for unauthorised users, this probably causes all redirects by profile enforcer to redirect to the main login page, which profile enforcer tries to redirect to profile edit page again, causing the loop.

Confirmed problem goes away when profile enforcer is disabled.

xieliwei’s picture

xieliwei commented
StatusFileSize
new profile_enforcer.module.tar_.gz3.02 KB

Did a quick patch. Wrapped a uid check before line 114 and after line 164. Also added suitable debug statements and comments. Please review.

dragonfyre6’s picture

dragonfyre6 commented

Very nice. Much better.

I had no block appearing for unauthorized users, so it wasn't a problem for me. Thanks for catching that.

christefano’s picture

christefano commented

#186396 is a duplicate of this issue.

christefano’s picture

christefano commented

#243116 is a duplicate of this issue.

christefano’s picture

christefano commented
Status: Reviewed & tested by the community » Needs work

We need a patch.

1kenthomas’s picture

1kenthomas commented

Status? What version is the above made from (there is mention elsewhere-- http://drupal.org/node/186396-- of a newer version in "bugtracker")? Is this module currently maintained?

1kenthomas’s picture

1kenthomas commented
Status: Needs work » Closed (won't fix)

Closing. // outdated, evidently abandoned module.