Bug #42593

Failure in AbstractRepository::findByUid on deleted record

Added by Oliver Hader almost 7 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
File Abstraction Layer (FAL)
Target version:
Start date:
2012-11-01
Due date:
% Done:

100%

TYPO3 Version:
6.0
PHP Version:
5.3
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Resource\AbstractRepository::findByUid fails if the record to be fetched has been deleted or simply does not exist. The condition whether something was fetched from database is wrong.


Related issues

Related to Media Management - Bug #42595: Failure if accessing invalid records Resolved 2012-11-01

Associated revisions

Revision d40ddbc9 (diff)
Added by Oliver Hader almost 7 years ago

[BUGFIX] Failure in AbstractRepository::findByUid on deleted record

Resource\AbstractRepository::findByUid fails if the record to
be fetched has been deleted or simply does not exist. The
condition whether something was fetched from database is wrong.

count(FALSE) is TRUE, thus it needs to be ensure that there is
a non-empty array.

Change-Id: I9fbdb2ed79ceef9ed0782eeed3db15af73636731
Releases: 6.0
Fixes: #42593
Reviewed-on: http://review.typo3.org/16102
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader

History

#1 Updated by Gerrit Code Review almost 7 years ago

  • Status changed from New to Under Review

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

#2 Updated by Gerrit Code Review almost 7 years ago

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

#3 Updated by Gerrit Code Review almost 7 years ago

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

#4 Updated by Oliver Hader almost 7 years ago

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

#5 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF