Project

General

Profile

Actions

Bug #85228

closed

Invalid count query in SuggestWizardDefaultReceiver

Added by Alexander Stehlik over 6 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2018-06-11
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
easy
Is Regression:
Yes
Sprint Focus:

Description

This is a regression from #78129:

The count query in the queryTable() method in the SuggestWizardDefaultReceiver does not respect the constraints set in the constructor and it is invalid because the table name is missing.

As far as I can see the count query is not needed and can be removed.


Files

screenshot.png (626 KB) screenshot.png Oliver Hader, 2018-06-12 10:23

Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #78129: Statement::rowCount not reliable for SELECT queriesClosed2016-10-02

Actions
Related to TYPO3 Core - Bug #101246: Invalid count query in SuggestWizardDefaultReceiverClosed2023-07-05

Actions
Actions #1

Updated by Alexander Stehlik over 6 years ago

  • Related to Bug #78129: Statement::rowCount not reliable for SELECT queries added
Actions #2

Updated by Gerrit Code Review over 6 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57192

Actions #3

Updated by Gerrit Code Review over 6 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57192

Actions #4

Updated by Oliver Hader over 6 years ago

  • Is Regression set to Yes

Confirmed exception

An exception occurred while executing 'SELECT COUNT(`uid`)':

Unknown column 'uid' in 'field list'
Actions #5

Updated by Oliver Hader over 6 years ago

How to reproduce:
  • use TYPO3 v9 (master)
  • edit some element making use of the suggest wizard in form engine, e.g. pages, "appearance" tab, field "replace content"
  • type in some name that shall be auto completed, e.g. "media"
  • see the HTTP 500 error in the browser's console pointing out the SQL error

Actions #6

Updated by Gerrit Code Review over 6 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57192

Actions #7

Updated by Alexander Stehlik over 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #8

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions #9

Updated by Christoph Lehmann over 1 year ago

  • Related to Bug #101246: Invalid count query in SuggestWizardDefaultReceiver added
Actions

Also available in: Atom PDF