Bug #55614

FAL: FileIndexingTask fails to execute in TYPO3 6.2 (current master)

Added by Bjoern Jacob over 1 year ago. Updated over 1 year ago.

Status:Resolved Start date:2014-02-03
Priority:Should have Due date:
Assigned To:- % Done:

100%
Category:File Abstraction Layer (FAL) Spent time: -
Target version:-
TYPO3 Version:6.2 Is Regression:No
PHP Version: Sprint Focus:
Complexity:

Description

I've got 2 TYPO3 6.2 installations which are pretty new and clean. I didn't delete any file mounts . I've uploaded some images via the filelist module (which was a problem discussed here). All uploaded images are listed by the filelist module. When using the page module and editing a tt_content element I don't see the uploaded images (or just one out of four...).

So I remembered the problem discussed in #44105 and created a new FileIndexingTask. When starting the scheduler task I get the following error message:

Die Ausführung von Task "Datei-Abstraktionsschicht: Indizierungs-Auftrag (scheduler)" ist fehlgeschlagen mit folgender Meldung: Task failed to execute successfully. Class: TYPO3\CMS\Scheduler\Task\FileIndexingTask, UID: 2

After that I thought I should provide an error message in English so I switched the TYPO3 backend to English. The error message stays the same but there's something slightly different:

Execution of task "[OBSOLETE] File Abstraction Layer: Indexing job (scheduler)" failed with the following message: Task failed to execute successfully. Class: TYPO3\CMS\Scheduler\Task\FileIndexingTask, UID: 2

As you can see the task is marked as "OBSOLETE". So I've got 2 problems:

  • My uploaded files are not shown when editing tt_content (I didn't test other content tables).
  • The task seems to be obsolete and is not working as expected.

BTW. I'm using the current master as of today.

patch.diff Magnifier (769 Bytes) Ralf Zimmermann, 2014-02-03 13:47

Related issues

related to Core - Bug #55359: Image files with uppercase file extension don't work in e... Resolved 2014-01-27

Associated revisions

Revision b890e468
Added by Ralf Zimmermann over 1 year ago

[BUGFIX] Make FileExtensionFilter case insensitive

Resolves: #55359
Resolves: #55614
Releases: 6.2
Change-Id: I8aceb208ba2f28967f08e9ac458d9a9da3de507b
Reviewed-on: https://review.typo3.org/27359
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

History

#1 Updated by Ralf Zimmermann over 1 year ago

The problem is located the FileExtensionFilter

This filter check a lower case file extension array (jpg, gif...) against the extension of the files located in the given folder (pathinfo($fileName, PATHINFO_EXTENSION)).
If there are files with uppercase extension like JPG, the filter won't match and return FALSE. The files are not listed.

It works with the attached patch, but i am not sure about side effects

#2 Updated by TRITUM GmbH over 1 year ago

This was already a problem and fixed in #50568. The code has changed on 6.2 and now the problem occurs again.

#4 Updated by Frans Saris over 1 year ago

  • Status changed from New to Accepted

Could you push a patch that adds the strtolower again?

Gr. Frans

#5 Updated by Gerrit Code Review over 1 year ago

  • Status changed from Accepted 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/27359

#6 Updated by Gerrit Code Review over 1 year 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/27359

#7 Updated by Gerrit Code Review over 1 year 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/27359

#8 Updated by Ralf Zimmermann over 1 year ago

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

#9 Updated by Johannes Busch over 1 year ago

It appears to me that this bug is nocht 100% resolved. It also throws this exeption when my filenames have capital letters for example My-Uplaod.jpg .
I think the patch just resovles the issus with the file extensions.

#10 Updated by Renzo Bauen over 1 year ago

I do not agree that this Issue is solved! (TYPO3 6.2.0 from download source on typo3.org).

If you read the given error carefully, then you can see, that it states, that the FileIndexingTask has failed.
I looked at typo3/sysext/scheduler/Classes/Task/FileIndexingTask.php

Line 91ff:
/** * Function execute from the Scheduler * * @return boolean TRUE on successful execution, FALSE on error
*/
public function execute() {
return FALSE;
}

If you set "return TRUE" the FileIndexingTask doesn't throw any error anymore!
But, with this code, i am not sure, if it does anything at all??? Is there really any index updated by this scheduler task? Or its just a placebo for the user?

Best Regards
Renzo

#11 Updated by Frans Saris over 1 year ago

I think you are looking at the deprecated scheduler task.

Gr. Frans

#12 Updated by Renzo Bauen over 1 year ago

Ok, in Germen, this task is not marked as obsolet!
Thanks for your hint.
Best Regard, Renzo

#13 Updated by Caspar Wahl-vom Bruch over 1 year ago

I got this error today. But I used this updated script. There seems to be a new reason. The Issue isn't solved, indeed.

#14 Updated by Frans Saris over 1 year ago

Are you dure you run the new scheduler job?

There are now 3 FAL jobs, were 1 is obsolute.

Gr. Frans

Also available in: Atom PDF