Bug #64044

Revert fix for #63275 until concept is worked out and agreed upon

Added by Helmut Hummel over 4 years ago. Updated 7 months ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
-
Target version:
-
Start date:
2014-12-26
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

It may be a useful feature, but then it must be documented as such.
Even more important, it needs to be evaluated whether the comments given by Sebastian are valid or not: https://forge.typo3.org/issues/7848#note-5
How is the decision here? What is the reasoning on why the comments are not valid?
How does "contains" compare to "in" now? What are the differences and how should these be documented?
I prepare a revert until these things are worked out.


Related issues

Related to TYPO3 Core - Feature #7848: Support array / multiple values in $query->contains Closed 2010-05-21 2010-06-04
Related to TYPO3 Core - Bug #63275: Typo3DbQueryParser generates broken SQL for array in CONTAINS-comparison Closed 2014-11-23

Associated revisions

Revision e73f204d (diff)
Added by Helmut Hummel over 4 years ago

Revert "[BUGFIX] Typo3DbQueryParser: Use IN with array-operand2"

Reverts: #63275
Resolves: #64044
Related: #7848

This reverts commit e235c9ad8b32267ece98d843a43daeab430db235.

Change-Id: I71a134469b9b44f892ad13be52d828ca653c03f1
Reviewed-on: http://review.typo3.org/35648
Reviewed-by: Helmut Hummel <>
Tested-by: Helmut Hummel <>

History

#1 Updated by Gerrit Code Review over 4 years ago

  • Status changed from Accepted 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 http://review.typo3.org/35648

#2 Updated by Gerrit Code Review over 4 years ago

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

#3 Updated by Helmut Hummel over 4 years ago

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

#4 Updated by Benni Mack 7 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF