In testing, this module always redirected users to their user information page if they where trying to go to the homepage, for example 'example.com'. In testing the issue, I have come across the issue in AnonymousLoginSubscriber.php.
- On lines 177-180 the current path is stored. In case for the front page of the site, it comes back as
/.
- On line 205, the front page setting is fetched. In my case it comes back as something like
/node/4.
On line 206, the module tries to check to see if the current path matches the front page setting, but it never does. This is because the URL 'example.com' always has a path of /, and the set front pate will always have /node/4. This will cause <front> to never be set, and / will be the set as $current['path'].
The second part of the issue comes on line 214 where the path is sent to the pathSlashCut function that strips the leading and trailing slashes form the path. Since the path only has /, the slash is stripped, leaving an empty path. In my instances of Drupal, when an empty path is requested, it forces the system to redirect the user to their information page. The user doesn't end up at the front page as expected.
One possible solution that I thought of is to have the pathSlashCut function look to see if the path only contains a /, and if it does, then don't strip the slashes:
if($path_string !== '/') {
// Remove the leading slash.
if (substr($path_string, 0, 1) == '/') {
$path_string = substr($path_string, 1);
}
// Remove the trailer slash.
if (substr($path_string, -1) == '/') {
$path_string = substr($path_string, 0, strlen($path_string) - 1);
}
}
The other solution would be to make the comparison on line 206 looking for the front page smarter by having it understand that '/' means the front page, and thus setting the path as <front>:
$current['path'] = ($current['path'] != $front_page && $current['path'] !== '/') ? $current['path'] : '<front>';
In my testing, both work well. I'm thinking that the second solution would be the more elegant solution. I'll have a patch file ready shortly with the change.