Skip to content

[BUGFIX] Remove outdated language settings causing empty links#1494

Merged
benjaminkott merged 2 commits intobenjaminkott:masterfrom
linawolf:bugfix/language-settings
Oct 8, 2024
Merged

[BUGFIX] Remove outdated language settings causing empty links#1494
benjaminkott merged 2 commits intobenjaminkott:masterfrom
linawolf:bugfix/language-settings

Conversation

@linawolf
Copy link
Copy Markdown
Contributor

Since TYPO3 9.5 language settings are handled by the site config.

The linkVars setting causes hard to find bugs where all links on a page are empty, see:

https://forge.typo3.org/issues/104344
https://talk.typo3.org/t/empty-href-on-arbitary-pages/5737

Resolves: #1165

Pull Request

Related Issues

  • Closes #
  • Fixes # [bug]
  • Resolves # [feature/question]

Prerequisites

  • Changes have been tested on TYPO3 v11.5 LTS
  • [ x] Changes have been tested on TYPO3 v12.4 LTS
  • Changes have been tested on PHP 7.4
  • Changes have been tested on PHP 8.0
  • Changes have been tested on PHP 8.1
  • [x ] Changes have been tested on PHP 8.2
  • [ x] Changes have been checked for CGL compliance php-cs-fixer fix
  • CSS has been rebuilt (only if there are SCSS changes cd Build; npm ci; npm run build)

Description

[Description of changes proposed in this pull request]

Steps to Validate

See Issue

Since TYPO3 9.5 language settings are handled by the site config.

The linkVars setting causes hard to find bugs where all links on a page are empty, see:

https://forge.typo3.org/issues/104344
https://talk.typo3.org/t/empty-href-on-arbitary-pages/5737

Resolves: benjaminkott#1165
@linawolf
Copy link
Copy Markdown
Contributor Author

phpstan errors are unrelated and have been broken since before

@benjaminkott benjaminkott merged commit d0b8e5b into benjaminkott:master Oct 8, 2024
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.

Current linkVars configuration brakes links on site when ?L= with forged value is added

2 participants