Hi all,

When i access the users/admin page as an anonymous user, it will redirect to the login page with the following url.
http://[domain]/user/login?destination=/users/admin

and after login, the it will return to
http://[domain]//users/admin

as you can see, there is a extra "/" on the final url. although it will render the same page as http://[domain]/users/admin, i still want to fix it.

Regards,
Kit

Comments

lotyrin’s picture

lotyrin commented
Version: 7.x-1.5 » 7.x-1.x-dev
Status: Active » Postponed (maintainer needs more info)

Can you still reproduce this? The code for generating destinations is completely re-done.

verblet’s picture

verblet commented

I also get this problem using 7.x-1.5. My Drupal installation is in a subfolder so all redirects are going to domain.com/subfolder//subfolder/node

pipep’s picture

pipep commented

+1

D.7.12

pipep’s picture

pipep commented

Ok, quick & dirty solution for my case:

on r4032login.module

modify line #90 from

header('Location: ' . url('user/login', array('query' => array('destination' => urldecode($path)), 'absolute' => TRUE)), TRUE, 302);

to

header('Location: ' . url('user/login', array('query' => array('destination' => urldecode('?q=home')), 'absolute' => TRUE)), TRUE, 302);

where

'?q=home'

stands for my custom home page after logging in.

lotyrin’s picture

lotyrin commented

#2 is definitely using the released version (before my improvements). #3/#4 is also, based on the line of code changed.

Please try the latest -dev snapshot.

pipep’s picture

pipep commented

yep, but the -dev isn't available for download...

lotyrin’s picture

lotyrin commented

It's not listed on the project page (and I don't have the access to add them there), but they exist (under "view all releases"):

6.x-1.x: http://drupal.org/node/316860
7.x-1.x: http://drupal.org/node/316861

deekayen’s picture

deekayen commented

I just turned on the -dev releases.

pipep’s picture

pipep commented

cool, thanks mate !

ambientdrup’s picture

ambientdrup commented

I've tested and can confirm that the dev version fixes this.

-Trevor

lotyrin’s picture

lotyrin commented
Status: Postponed (maintainer needs more info) » Fixed

Status: Fixed » Closed (fixed)

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