I am using Domain Access.

My main site is www.mydomain.com.
I have multiple subdomains.

Issue:
1. I was logged in as an admin.
2. I was viewing the front page (I could still see the admin toolbar).
3. When I tried to go to my test environment, I got this URL:
https://subdomain.test.mydomain.comhttps//subdomain.mydomain.com/content/standing-rock-community-resources

I'm not sure if it's the fact that I was viewing the front page, so there was no additional path to append to the test base URL, or the fact that I'm using Domains.

I'll try to diagnose it further when I have a bit more time.

Comments

arnaldop created an issue. See original summary.

richa_porwal’s picture

Assigned: Unassigned » richa_porwal
richa_porwal’s picture

StatusFileSize
new581 bytes

Fixed the url issue .

richa_porwal’s picture

Assigned: richa_porwal » Unassigned
Status: Active » Needs review
ambuj_gupta’s picture

Assigned: Unassigned » ambuj_gupta
ambuj_gupta’s picture

Assigned: ambuj_gupta » Unassigned
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new527.58 KB
new455.71 KB

Test Steps:
1. Install Drupal 8 with environment_indicator module.
2. Go to "admin/config/development/environment-indicator".
3. Apply patch #4
4. And add other environments with valid URLs.
5. Open php.settings in any of the editors.
6. And add below lines. And Save:
$config['environment_indicator.indicator']['bg_color'] = '#0D0D0D';
$config['environment_indicator.indicator']['fg_color'] = '#15f512';
$config['environment_indicator.indicator']['name'] = 'Locals';
7. Clear and cache and go to sites.
8. And check the added environment URLs

Results:
Tested and Reviewed by applying patch #4. And it's working as expected.
The user is able to navigate to the added environments.

jaypan’s picture

Version: 8.x-3.7 » 4.x-dev
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new573 bytes

Re-rolling for 4.x branch.

jaypan’s picture

StatusFileSize
new646 bytes

I discovered this was cutting the the current path on pages that needed it. As such, I've set a conditional whereby the current path is only set if there is a current path.

devkinetic’s picture

Status: Needs review » Needs work

This patch needs a reroll now that #2610208: Move procedural functions to a manager has landed.

ankitjhakal’s picture

Assigned: Unassigned » ankitjhakal
ankitjhakal’s picture

Status: Needs work » Needs review
StatusFileSize
new683 bytes

Hi @devkinetic, Rerolled the patch, Attached patch working fine on my local machine.

ankitjhakal’s picture

Assigned: ankitjhakal » Unassigned
yevko’s picture

The patch does not apply anymore. Plus, as I mentioned in https://www.drupal.org/project/environment_indicator/issues/3065135#comm... the problem reported here is fixed by the solution in that issue.

trackleft2’s picture

trackleft2’s picture

trackleft2’s picture

Status: Needs review » Closed (duplicate)

Fixed in #3065135: Add support for multilingual domain sites where I've added credit.