[BUGFIX] Get correct page id for a shortcut

[KrohnMi]

When creating a shortcut in TYPO3 that points to a different root page, an incorrect link is generated.

I hope I have fixed it correctly.

[coveralls]

Pull Request Test Coverage Report for Build 6850008836

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

• 3 of 4 (75.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.008%) to 63.336%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
Classes/Event/Listener/AfterLinkIsGeneratedListener.php	3	4	75.0%

Totals
Change from base Build 6785936861:	0.008%
Covered Lines:	824
Relevant Lines:	1301

---

💛 - Coveralls

[lukaszuznanski]

Could you cover this with tests?
By the way: is this really an issue? I mean, it's OK to have id of shortcut page, as it's redirected by 307 HTTP to other page, or I don't understand your issue.

[KrohnMi]

@lukaszuznanski

My use case involves two Typo3 root pages in one backend, let's call them Page A and Page B.

On Page A, i'm building a menu using the Menubuilder, and within this menu, i have links to Page B. In the Typo3 site config, i have defined the frontend URL as a.site.com and the backend URL as api.a.site.com. Similarly, Page B has frontend URL b.site.com and backend URL api.b.site.com.

When i build the menu, the links are generated as a.site.com instead of the expected b.site.com. This is happening because the Site Configuration of Page A is being used to get the frontend url, and Page B's Site Configuration is not being used as expected.

[twoldanski]

Hi @KrohnMi thanks for PR, but can you please test you issue with this PR #675 (should solve you issue as well)

[lukaszuznanski]

@KrohnMi please check if #675 fixes your issue

[KrohnMi]

@twoldanski No, unfortunately, your merge request could not resolve my issue.

[twoldanski]

Hi @KrohnMi could you provide example typoscript to reproduce your issue?

[lukaszuznanski]

@KrohnMi please provide some examples so we can reproduce this issue