Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The following line of code is in the tests:
$path['options']['base_url'] = $GLOBALS['base_url'] . base_path();
Concatenating $GLOBALS['base_url']
and base_path()
shouldn't ever be done as the subdirectory will appear twice:
Example:
$GLOBALS['base_url'] == 'http://example.com/drupal';
base_bath() == '/drupal/';
$GLOBALS['base_url'] . base_bath() == 'http://example.com/drupal/drupal/';
Comment | File | Size | Author |
---|---|---|---|
#2 | globalredirect-2285209-testfixbaseurl-2.patch | 1.46 KB | stefan.r |
#1 | globalredirect-2285209-testfixbaseurl-1.patch | 949 bytes | stefan.r |
Comments
Comment #1
stefan.r CreditAttribution: stefan.r commentedComment #2
stefan.r CreditAttribution: stefan.r commentedThere is a further issue in the tests if they are run from a URL with a language prefix that does not exist in the sandbox (same as #2284727).
Updated patch attached.
Comment #3
DamienMcKennaThis is a duplicate of #304025: not working when drupal is installed in a subdirectory.
Comment #4
stefan.r CreditAttribution: stefan.r commented@DamienMcKenna, I actually don't think this is a duplicate. This issue refers to two bugs in the Global Redirect automated test itself, neither of which is caused by the (similar) issue in #304025: not working when drupal is installed in a subdirectory
Comment #5
DamienMcKennaFair enough.
Comment #6
Chris Matthews CreditAttribution: Chris Matthews commentedThe 5 year old patch in #2 to globalredirect.test applied cleanly to the latest 7.x-1.x-dev, but still needs community review & testing.