5.3.12 (2020-02-25)

Overview of merged pull requests

TASK: Fix validation warnings/errors on .travis.yml

Fixes build config validation complaints:

  • W notifications.slack: unknown key on_start (never)

  • I root: missing os, using the default linux

  • I root: key matrix is an alias for jobs, using jobs

BUGFIX: Make sure to use only a consistent DatesReaderCache

It might happen that there is a discrepancy between parsedFormatsIndices and parsedFormats cache. We need to make sure we are accessing the consistent information instead of relying that both are there, to avoid a PHP exception.

Resolves #564

The problem described in #564 is not easy to reproduce, as it only happens “once in a while” in production. See issue comments for further information.

Checklist

  • Packages: Flow

BUGFIX: Fix error handling for importing resources

<!– Thanks for your contribution, we appreciate it!

Please read through our pull request guidelines, there are some interesting things there: https://discuss.neos.io/t/creating-a-pull-request/506

And one more thing… Don’t forget about the tests! –>

While working with ResourceManager I noticed that importing non-existent resources would fail later in the process than expected: They would only fail when trying to copy the temporary file to it’s persistent location instead of failing when fetching the original file.

What I did This PR fixes the incorrect error handling of fetching files when importing resources.

How I did it The code assumed that copy would throw an exception if it failed - however, it returns false in that case.

How to verify it Try to import a non-existent resource (e.g. http://example.com/this-file-does-not-exist). It should now correctly throw the Could not copy the file from “…” to temporary file “…” exception from WritableFileSystemStorage::importResource instead of the much later The temporary file of the file import could not be moved to the final target “…” from WritableFileSystemStorage::moveTemporaryFileToFinalDestination

Checklist

  • Packages: Flow

Detailed log