Closed (fixed)
Project:
Lightbox2
Version:
7.x-1.0-beta1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 Jan 2009 at 08:35 UTC
Updated:
6 Jul 2012 at 20:43 UTC
Jump to comment: Most recent file
Comments
Comment #1
wmnnd commentedI have the very same problem with the current D6 version.
Comment #2
hankpalan.com commentedHaving same issue.
Comment #3
elanghout commentedSame issue here. In version 6.x-1.9 and the v6.x DEV from feb 9.
Comment #4
floris-1 commentedanyone figured this out ?
i seem to have the same problem, havent found a solution just yet
Comment #5
floris-1 commentedComment #6
wmnnd commentedWill this be fixed?
Comment #7
stella commentedeventually - just rather busy at the moment. If you feel like investigating it yourself and creating a patch, then it's more likely to get done quicker.
Cheers,
Stella
Comment #8
gnat commentedThis bug is caused by the fact that lightbox2 prints the login form directly to get a clean version used in the pop-up. The problem is, that when you get a failed form validation, like a bad password, the form gets printed cleanly, and you miss out on the drupal_set_message() letting you know about the error. Since Drupal spools those messages until the next usable page, you will see it when you go back to a fully rendered page.
This patch solves the problem by only printing a clean form if there is no POST information. This allows successful logins to proceed as normal, and failed logins to occur on the standard Drupal generated login page, with errors noted and flagged.
This is for the D6 version.
Comment #9
gnat commentedAnd the D5 version.
Comment #10
gnat commentedComment #11
wmnnd commentedThe patch works great, thank you very much.
Comment #12
marcvangendThe patch works, but it's not the most beautiful method. Am I asking too much when I say: Can't I go back to the same page with a lightbox over it?
Comment #13
stella commentedCommitted, thanks!! The fix will be included in the next dev release of the module later today.
Cheers,
Stella
Comment #15
csc4 commentedSorry for reopening this - but I have the latest dev version with this patch installed and I'm getting redirected on a successful login to user/login/lightbox2 which is throwing an access denied error.
Comment #16
redijedi commentedsubscribing
Comment #17
nicktech commentedsubscribing
Comment #18
SeanBannister commentedI'm running the current dev and it's working great, no errors.
Comment #19
stella commented@csc4 : are you logging in via the login block, the full login page, or something else?
Comment #20
stella commentedComment #22
vardas commentedCan anyone point me at the documentation that describes how to enable Lightbox2 for login. I have searched but can't find any mention of how to do this.
Comment #23
marcvangendPlease don't hijack other issues for support questions, but open a new one instead.
Comment #24
dorijrs commentedhow do i apply that patch into the lightbox 2 module?
Comment #25
rpataca commentedThis still appears to be an issue in D7???