5.3.15 (2020-05-04)

Overview of merged pull requests

TASK: Trigger GitHub rebase action only on /rebase

Idea is that a /rebase comment rebases the current PR on top of the destination branch, so that we are sure that latest state is included. Current implementation tries the rebase on every comment (see results in https://github.com/neos/flow-development-collection/actions).

In 6.1/6.2 the action was already converted to “v2” style (see https://github.com/neos/flow-development-collection/pull/1765).

So in 5.3 first cherry-picked the conversion from After that changed the workflow definition so it looks exactly like the documented: https://github.com/cirrus-actions/rebase

  • Packages: github

BUGFIX: Make sure to use only a consistent NumbersReaderCache

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.

Same fix was already applied to DatesReader, see https://github.com/neos/flow-development-collection/pull/1899.

Resolves #564

Checklist

  • Packages: Flow

BUGFIX: Objects configured with factory don’t receive factory arguments

Until now a class configured to be constructed by a factory with arguments would end up giving those arguments to both the factory and the class constructor. This can lead to errors when the class constructor expects different arguments. This change fixes that by separating the arguments for the factory from the class constructor arguments. The arguments setting in Objects.yaml will refer to the factory method arguments when factoryObjectName is given, otherwise to the class constructor arguments.

Fixes #1933

  • Packages: Flow

BUGFIX: Add samesite cookie configuration

What I did Adds new cookie setting allow to configure SameSite to ‘strict’, ‘lax’ and ‘none’. Be careful with SameSite=none as there is a webkit bug (https://bugs.webkit.org/show_bug.cgi?id=198181) inadvertently treated this as SameSite=strict This change does not effect the current behavior. When you don’t set the configuration samesite in settings.yaml the cookie attribute is ignored.

  • Packages: Flow