Skip to content

[TASK] improve UrlUtility#882

Merged
twoldanski merged 1 commit into
feature/typo3-v14from
feature/v14-utility
May 8, 2026
Merged

[TASK] improve UrlUtility#882
twoldanski merged 1 commit into
feature/typo3-v14from
feature/v14-utility

Conversation

@twoldanski

Copy link
Copy Markdown
Collaborator

No description provided.

@twoldanski twoldanski merged commit 71de898 into feature/typo3-v14 May 8, 2026
9 of 10 checks passed
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 25548099539

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Warning

No base build found for commit 6155992 on feature/typo3-v14.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 74.7%

Details

  • Patch coverage: 1 uncovered change across 1 file (0 of 1 lines covered, 0.0%).

Uncovered Changes

File Changed Covered %
Classes/Form/Finisher/JsonRedirectFinisher.php 1 0 0.0%

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 1581
Covered Lines: 1181
Line Coverage: 74.7%
Coverage Strength: 6.63 hits per line

💛 - Coveralls

@twoldanski twoldanski deleted the feature/v14-utility branch May 8, 2026 14:28
@develth

develth commented Jul 9, 2026

Copy link
Copy Markdown

Hi,
i recognized, that with that change all URLs are rendered with frontend URL prefixed.
That causes nuxtJS in the frontend, to load the full page and handle it as an external link - so you lose speed and do unnecessary requests.
Is there an option to let the links generate without prefix for menu etc.?

Thanks!

@twoldanski

Copy link
Copy Markdown
Collaborator Author

hi @develth if you are on v14 try this branch #893, it is fixed there (with additional improvements), if you are using v4 downgrade to 4.7.x or use this branch #892

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants