First of all sorry for my english =)

So in template.php i use - 

function gonobobel_theme() {
  return array(
    'user_login' => array(
      'template' => 'user-login',
      'arguments' => array('form' => NULL),
    ),
  );
}
function gonobobel_user_login($form) {
  return drupal_render($form);
}

in user-login.tpl.php i use - 

function loginform() {
        $loginform=user_login_block();
        $loginform['name']['#size']=15;
        $loginform['name']['#title']="Name";
        $loginform['name']['#description']=drupal_get_destination();
        $loginform['pass']['#size']=8;
        $loginform['pass']['#title']="Passwort";
        $loginform['pass']['#description']="";
        //print_r($loginform);
        return $loginform;
}

print drupal_get_form('loginform');

in page.tpl.php i use -

print l("Увійти", 'user', array('attributes' => array('class' => 'popups')));

After submitting name\password as in picture 1 a see dot whill loading and then - picture 2. So main page stays the same, not logged in. but in popup "window" i am logged in.

Please help me to fix my problem!

Thanks!

CommentFileSizeAuthor
#12 popups.js_.patch753 bytesenzipher
#10 popups.js_.patch560 bytesenzipher
2.jpg13.5 KBgonobobel
1.jpg14.5 KBgonobobel

Comments

gonobobel’s picture

gonobobel commented
Category: task » support
starbow’s picture

starbow commented 
print l("Увійти", 'user', array('attributes' => array('class' => 'popups-form-reload', 'query' => array('destination' => $_GET['q'])));
gonobobel’s picture

gonobobel commented

nothing new happens. I used 

print l("Увійти", 'user', array('attributes' => array('class' => 'popups-form-reload', 'query' => array('destination' => $_GET['q'])));

in page.tpl.php, but all happens the same. please, help me!

phpsharma’s picture

phpsharma commented

removed..

gonobobel’s picture

gonobobel commented

So, people, how can I solve my problem?

starbow’s picture

starbow commented

read comment #4

pdcarto’s picture

pdcarto commented

Don't read comment #4 - it doesn't work! The problem is that after submitting the login form from a popup, the page is not reloaded, so no redirect action happens.

frankcarey’s picture

frankcarey commented

#2 seems to work fine for me in 6.x.2.0-rc-5.

alextronic’s picture

alextronic commented

I have exactly the same issue. #2 doesn't work for me... maybe there is another workaround? I'm using module Login destination, but doesn't seem to be the problem.

enzipher’s picture

enzipher commented
Version: 6.x-1.1-rc6 » 6.x-2.0-beta0
Component: Miscellaneous » Code
Assigned: gonobobel » Unassigned
Category: support » bug
Priority: Critical » Major
StatusFileSize
new popups.js_.patch560 bytes

After all this time I ran into this issue as well, that the page does not reload using "popups-form-reload" for the path "user". I got it working in my case by changing line 909 in popups.js as follows:

Before:

if (!done) { // Not done yet, so show new page in new popups.

After:

if (!done && popup.options.updateMethod != 'reload') { // Not done yet, so show new page in new popups.

This is not tested extensively and might have a negative impact on something else.

Cheers,

joseph28’s picture

joseph28 commented

the patch solve the problem if the username and password are correct, but what if the password or the username are wrong?

enzipher’s picture

enzipher commented
StatusFileSize
new popups.js_.patch753 bytes

I should have posted an update. I did some more testing and you are right, the patch pretty much bypasses the error check. I have updated the code I am using to check for an error in the returned content. This is based on the standard Drupal class names, so if you have modified the "error" class this will fail. That said, here is the updated code which I haven't encountered any problems with yet.

What the patch does is that instead of using the isDone() function, a check is made to see if the requested content contains class="error".

Cheers,

drewish’s picture

drewish commented
Status: Active » Needs work

I didn't test it but just looking at the code it could be a bit tighter:

  var done = data.content.match(/class="[\w-\s]*?error[\w-\s]*?"/gm);

That said I think there might be a better way to do this.

enzipher’s picture

enzipher commented

Yes, some optimization would not hurt, however, in order to return true/false I needed to type cast the result. I changed the variable to make things more logical as well.

var error = Boolean(data.content.match(/class="[\w-\s]*?error[\w-\s]*?"/gm));

However, as you say there are probably better ways of doing this, but this works for my needs atm.

qilen’s picture

qilen commented

Just add

location.reload();
at the end of function: Popups.formSuccess = function(popup, data) {

it will be like this:

Popups.formSuccess = function(popup, data) {
blablabla...

location.reload();
};

/**
* Get a jQuery object for the content of a layer.
* @param layer
* Either a popup, or null to signify the original page.
*/