Project

General

Profile

Actions

Bug #62104

closed

LocalDriver::getFileInfoByIdentifier can be optimised

Added by Michiel Roos about 10 years ago. Updated about 6 years ago.

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

100%

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

Description

\TYPO3\CMS\Core\Resource\Driver\LocalDriver::getFileInfoByIdentifier() fetches the dirPath of the given $fileIdentifier at the start of the method.

This information is not used untill the last line of the method.

The method may fail if the resource does not exist. By that time, some work was performed that was not needed.

I suggest moving the $dirPath calculation to the bottom of the method.

Actions #1

Updated by Gerrit Code Review about 10 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 http://review.typo3.org/34006

Actions #2

Updated by Gerrit Code Review almost 10 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/34006

Actions #3

Updated by Frederic Gaus almost 10 years ago

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

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF