Unnecessary/wrong code to remove in FileDelivery v5.0.6

[crealistiques]

In my opinion, the code (line 114-126) in the FileDelivery class should be removed to get the intended functionality!

typo3-secure-downloads/Classes/Resource/FileDelivery.php

Line 114 in 7ef9d9b

$fileObject = $this->resourceFactory->retrieveFileOrFolderObject($this->token->getFile());

The code part (lines 114-126) should simply be removed, because otherwise the use of the event in line 111 makes no sense and has no use. The event is very good for manipulating the file to be delivered, which is very handy if I want to deliver a different version of the original file, for example. But if I manipulate $file in an event listener of AfterFileRetrievedEvent, the new/modified $file will not be reflected/used in the returned response/stream!

So please remove the code if that makes sense to you.

Thank you very much for your good work and support!

[balasch]

Many thanks for the tip. We will take a closer look at it.