Bug #83984

Missing AND concatenation in \TYPO3\CMS\Recordlist\RecordList\AbstractDatabaseRecordList::makeQueryArray

Added by Nicole Cordes almost 2 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
Start date:
2018-02-20
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

On testing the pull request from tc_beusers for TYPO3 8.7 (https://github.com/dkd/tc_beuser/pull/8) the author found an sql error while searching for an user. The SQL statement has a missing AND concatenation. As the SQL is produced from core, we need to fix it on our site.


Related issues

Related to TYPO3 Core - Task #76259: Doctrine: migrate EXT:recordlist/Recordlist/RecordList/AbstractDatabaseRecordList Closed 2016-05-22

Associated revisions

Revision 925e83da (diff)
Added by Nicole Cordes over 1 year ago

[BUGFIX] Add missing AND concatenation in generated SQL statement

This patch adds a missing AND concatenation in
\TYPO3\CMS\Recordlist\RecordList\AbstractDatabaseRecordList::makeQueryArray.
As the search string is returned as plain SQL constraint and is added
to an already existing statement, it needs to be prefix with "AND".

Resolves: #83984
Releases: 8.7
Change-Id: I9b62a26f39b0899a1b6e3549eb4a6a488ad1aaac
Reviewed-on: https://review.typo3.org/55840
Tested-by: TYPO3com <>
Reviewed-by: Oliver Hader <>
Reviewed-by: Susanne Moog <>
Tested-by: Susanne Moog <>
Reviewed-by: Andreas Fernandez <>
Tested-by: Andreas Fernandez <>

History

#1 Updated by Gerrit Code Review almost 2 years ago

  • Status changed from New to Under Review

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

#2 Updated by Oliver Hader almost 2 years ago

  • Related to Task #76259: Doctrine: migrate EXT:recordlist/Recordlist/RecordList/AbstractDatabaseRecordList added

#3 Updated by Oliver Hader almost 2 years ago

  • Target version set to 8.7.11

#4 Updated by Nicole Cordes over 1 year ago

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

#5 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF