Bug #58511

addUnionStatement does not respect MM_match_fields

Added by Stefan Froemken about 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Extbase
Target version:
-
Start date:
2014-05-05
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
5.4
Tags:
Complexity:
medium
Is Regression:
No
Sprint Focus:

Description

Hello Core-Team,

it seems that we have another part in persistence of extbase where MM_match_fields was not implemented.
I have added a query like that:

$query->equals('categories.uid', $category);

Because of the dot inside parseOperand() calls method addUnionStatement and here we have to add MM_match_fields, too.

Stefan


Related issues

Related to TYPO3 Core - Bug #27057: Can't query on properties of child objects of the same classClosedAlexander Schnitzler2011-05-26

Actions
#1

Updated by Gerrit Code Review about 7 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/29870

#2

Updated by Gerrit Code Review about 7 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/29870

#3

Updated by Frans Saris about 7 years ago

Why use the .uid? And not just:

$query->equals('categories', $category);

#4

Updated by Stefan Froemken about 7 years ago

Good Morning Frans,

as you can see in the ticket the union statement will only be added if propertypath contains a dot. Sure, I can change it to categories.title, but the result is the same. My test checks, if mm_match_fields is also considered in case of union statements.

Stefan

#5

Updated by Gerrit Code Review about 7 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/29870

#6

Updated by Gerrit Code Review about 7 years ago

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

#7

Updated by Gerrit Code Review almost 7 years ago

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

#8

Updated by Mathias Schreiber over 5 years ago

  • Target version deleted (next-patchlevel)
#9

Updated by Stefan Froemken over 5 years ago

  • Status changed from Under Review to Closed

This issue was solved with: https://forge.typo3.org/issues/27057

Also available in: Atom PDF