Bug #36625

PersistenceQueryRewritingAspect does not support count() correctly

Added by Karsten Dambekalns over 3 years ago. Updated over 3 years ago.

Status:Resolved Start date:2012-04-26
Priority:Must have Due date:
Assigned To:Robert Lemke % Done:

100%
Category:Security
Target version:TYPO3 Flow Base Distribution - 1.1
PHP Version: Complexity:
Has patch:No Affected Flow version:Git master

Description

If the aspect intercepts a count() method and the result (because of lacking access rights) would be "none", the aspect returns an EmptyQueryResult object which in turn can not be cast into an integer by the client code.

Associated revisions

Revision 21cf88e5
Added by Robert Lemke over 3 years ago

[BUGFIX] PersistenceQueryRewritingAspect now supports count(0)

If the aspect intercepted a count() method and the result (because
of lacking access rights) would be "none", the aspect returned an
EmptyQueryResult object which in turn could not be cast into an
integer by the client code.

This patch now correctly returns a 0 in that case.

Change-Id: I5cb67715c2cb1ed12df724f45d6e87446750701f
Fixes: #36625
Releases: 1.1

History

#1 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10788

#2 Updated by Gerrit Code Review over 3 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10788

#3 Updated by Gerrit Code Review over 3 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10788

#4 Updated by Gerrit Code Review over 3 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10788

#5 Updated by Gerrit Code Review over 3 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10788

#6 Updated by Robert Lemke over 3 years ago

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

Also available in: Atom PDF