Project

General

Profile

Actions

Bug #55511

closed

retrieveFileOrFolderObject does not support delted files.

Added by Nikolay Orlenko almost 11 years ago. Updated over 5 years ago.

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

0%

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

Description

related to #45748

When i am trying to pass not existed file to function retrieveFileOrFolderObject (ResourceFactory.php) by absolute or relative path it return error about not existed file.

The problem that after checking line #442

} elseif (@is_file(PATH_site . $input)) {

it tries to make directory object.

The method should check file_exists before going to directory initialization somehow at line #L447.

Actions #1

Updated by Mathias Schreiber almost 10 years ago

  • Sprint Focus set to On Location Sprint
Actions #2

Updated by Ingo Schmitt almost 10 years ago

  • TYPO3 Version changed from 6.1 to 7
  • Complexity set to medium

Solution is:
add is_directoy in the else part

Actions #3

Updated by Ingo Schmitt almost 10 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Gerrit Code Review almost 10 years ago

  • Status changed from In Progress 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/36518

Actions #5

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/36518

Actions #6

Updated by Gerrit Code Review almost 10 years ago

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

Actions #7

Updated by Gerrit Code Review almost 10 years ago

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

Actions #8

Updated by Ingo Schmitt almost 10 years ago

  • Assignee set to Frans Saris
  • Sprint Focus deleted (On Location Sprint)
Actions #9

Updated by Mathias Schreiber over 8 years ago

  • Status changed from Under Review to New
Actions #10

Updated by Susanne Moog over 5 years ago

  • Status changed from New to Closed

Benni wanted to close this issue years ago, so I'm closing it now - see linked review.

Actions

Also available in: Atom PDF