andersundsehr / aus_driver_amazon_s3

Provides a TYPO3 FAL driver for the Amazon Web Service S3
GNU Lesser General Public License v3.0
21 stars 40 forks source link

error messages in the backend while using dev-master and TYPO3 v12 #135

Closed magicHatOfTYPO3 closed 1 month ago

magicHatOfTYPO3 commented 1 year ago

Hi,

As far as I can see in #133 the current dev-master should be compatible with TYPO3 v12. Using dev-master also seems to work, but I receive some error messages in the backend while using:

a) PHP Warning: Undefined array key “mtime” in /var/www/html/vendor/typo3/cms-core/Classes/Resource/Folder.php line 557

when accessing the FileStorage Root (the error message disappears as soon as I select a folder in the storage

b) $uid must be positive integer, 0 given

when accessing a file on the storage from the filelist module. The upload to the S3 storage works well and I also can reference the files from a content element, but I can't see the details in the filelist module.

Is this still work in progress or are these unknown bugs?

Can we help or sponsor to get the v12 compatibility?

cweiske commented 11 months ago

The second problem might be solved with #136, but I'm not sure because you did not provide a error file + line number, nor a stacktrace.

cweiske commented 11 months ago

Stack trace for the "mtime" problem:

(1/1) #1476107295 TYPO3\CMS\Core\Error\Exception
PHP Warning: Undefined array key "mtime" in /var/www/typo3/vendor/typo3/cms-core/Classes/Resource/Folder.php line 557

at TYPO3\CMS\Core\Error\ErrorHandler->handleError() in /var/www/typo3/vendor/typo3/cms-core/Classes/Error/ErrorHandler.php line 138
at TYPO3\CMS\Core\Resource\Folder->getModificationTime() in /var/www/typo3/vendor/typo3/cms-core/Classes/Resource/Folder.php line 557
at TYPO3\CMS\Filelist\Dto\ResourceView->getUpdatedAt() in /var/www/typo3/vendor/typo3/cms-filelist/Classes/Dto/ResourceView.php line 189
in /var/www/typo3/vendor/typo3fluid/fluid/src/Core/Variables/StandardVariableProvider.php line 145
at TYPO3Fluid\Fluid\Core\Variables\StandardVariableProvider->getByPath()
at Default_action_Filelist_Tiles_47b410360e836368df00406f30c27aa8779d1cd1->{closure}() in /var/www/typo3/vendor/typo3/cms-fluid/Classes/ViewHelpers/Format/DateViewHelper.php line 169
at TYPO3\CMS\Fluid\ViewHelpers\Format\DateViewHelper::renderStatic() in /var/www/typo3/var/cache/code/fluid_template/Default_action_Filelist_Tiles_47b410360e836368df00406f30c27aa8779d1cd1.php line 419
cweiske commented 11 months ago

Both issues have patches now.

weakbit commented 10 months ago

@magicHatOfTYPO3 we merged #137 into master. There is also a PR open for problem b) https://github.com/andersundsehr/aus_driver_amazon_s3/pull/142

Thank you for contributing

Moongazer commented 3 months ago

Could you please create a release like 1.13.1 where this issue is fixed? Because the file-module for S3 buckets is not usable in v12. Thanks ahead 🙏🏼

weakbit commented 1 month ago

@Moongazer I created a pre-release 1.13.1 hope this fits well for you