Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tag slug update wizard causes SQL error #2613

Closed
liayn opened this issue Feb 7, 2025 · 0 comments · Fixed by #2614
Closed

Tag slug update wizard causes SQL error #2613

liayn opened this issue Feb 7, 2025 · 0 comments · Fixed by #2614

Comments

@liayn
Copy link
Contributor

liayn commented Feb 7, 2025

Bug Report

Current Behavior
Running the upgrade wizard on a plain new TYPO3 v12 the upgrade wizard dies with

Unknown column 'sorting' in 'ORDER BY'

Expected behavior/output

No error :-)

Environment

  • TYPO3 version(s): 12.4.36
  • news version: 12.2.0
  • Is your TYPO3 installation set up with Composer (Composer Mode): yes
  • OS: Linux

Possible Solution

Drop the orderby in

news/Classes/Updates/PopulateTagSlugs.php

Line 106 in 5ffc566

->addOrderBy('sorting', 'asc')

As far as I looked into the history, the ? tx_news_domain_model_tagtable never had asorting` field defined in TCA.
liayn added a commit to liayn/news that referenced this issue Feb 7, 2025
@liayn
[BUGFIX] PopulateTagSlugs wizard must not rely on sorting field of ta…
d51466d 
…g-table

The sorting field is not present on newer installations.

Resolves: georgringer#2613
@liayn liayn mentioned this issue Feb 7, 2025
Merged
georgringer pushed a commit that referenced this issue Feb 12, 2025
@liayn
[BUGFIX] PopulateTagSlugs wizard must not rely on sorting field of ta…
8c0bdff 
…g-table (#2614)

The sorting field is not present on newer installations.

Resolves: #2613
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[BUGFIX] PopulateTagSlugs wizard must not rely on sorting field of tag-table liayn/news
1 participant
@liayn