I installed Login Destination on a Drupal-6.13 and it worked great including the php code provided in the forums to allow role-based destination. I then put Login Destination on a Drupal-6.12 installation and the module doesn't do anything. Does any one have any ideas of what I could check?

Comments

ebremer’s picture

ebremer commented

OK, just playing around...I turned off "Preserve destination" and it now works. However, on my other site that worked "Preserve destination" is turned on. I'm good for now, but this is weird.

zahor’s picture

zahor commented

It's just not working for me.

pizzapotamus’s picture

pizzapotamus commented

change login_destination.module at line 147 from

  if ($_GET['destination'] != $_GET['q'] && variable_get('ld_destination', TRUE)) {
    return FALSE;
  }

to

  if (!empty($_GET['destination']) && variable_get('ld_destination', TRUE)) {
    return FALSE;
  }

The original seems to say that if preserve destination is enabled and the current page isn't equal to the destination then ignore our settings. This of course doesn't work because if destination is empty it will never be equal to the current page so the module quits.

francewhoa’s picture

francewhoa
he/him
Primary language French
Location Sept-Îles, Québec, 🇨🇦
commented

FYI: Login Destination module seems abandoned. Read more at http://drupal.org/node/552602

They have added a note in the first paragraph of module project page. Read more at http://drupal.org/project/login_destination

rsvelko’s picture

rsvelko commented

read the project page again - it is no more abandoned.

rsvelko’s picture

rsvelko commented
Status: Active » Closed (duplicate)

please test the new 2.5 rel.

open a new issue if not fixed... This should be the 10th issue for "not working at all" - I need to diminish their numbers....

francewhoa’s picture

francewhoa
he/him
Primary language French
Location Sept-Îles, Québec, 🇨🇦
commented

Indeed. This is a duplicate of http://drupal.org/node/341683

Confirming that 6.x-2.5 works.

Thanks rsvelko