|
| 1 | +`8.3.14 (2025-03-04) <https://github.com/neos/flow-development-collection/releases/tag/8.3.14>`_ |
| 2 | +================================================================================================ |
| 3 | + |
| 4 | +Overview of merged pull requests |
| 5 | +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ |
| 6 | + |
| 7 | +`BUGFIX: Avoid type error during `publishFile()` <https://github.com/neos/flow-development-collection/pull/3449>`_ |
| 8 | +------------------------------------------------------------------------------------------------------------------ |
| 9 | + |
| 10 | +This avoids an error when the file cannot be opened or writing. That would result in |
| 11 | +``stream_copy_to_stream(): Argument ``#2 <https://github.com/neos/flow-development-collection/issues/2>``_($to) must be of type resource, bool given`` |
| 12 | +for recent PHP versions. |
| 13 | + |
| 14 | +**Review instructions** |
| 15 | + |
| 16 | +Probably a bit tricky, but you need to create a scenario, where the target file already |
| 17 | +exists and cannot be written to. |
| 18 | + |
| 19 | + |
| 20 | +* Packages: ``Flow`` |
| 21 | + |
| 22 | +`BUGFIX: Drop use of `E_STRICT` to fix PHP 8.4 deprecation <https://github.com/neos/flow-development-collection/pull/3437>`_ |
| 23 | +---------------------------------------------------------------------------------------------------------------------------- |
| 24 | + |
| 25 | +The use of ``E_STRICT`` is deprecated as of PHP 8.4, so this fixes deprecation warnings. Furthermore, the constant is no longer useful… |
| 26 | + |
| 27 | +In PHP 5.4, the functionality of ``E_STRICT`` was incorporated into ``E_ALL``, meaning strict standards notices are included in the ``E_ALL`` error level. As a result, there is no need to use ``E_STRICT`` separately starting with PHP 5.4. This change is documented in the PHP manual under the migration guide for PHP 7.0, which states: |
| 28 | + |
| 29 | +> All of the E_STRICT notices have been reclassified to other levels. |
| 30 | +> The E_STRICT constant is retained, so calls like |
| 31 | +> ``error_reporting(E_ALL|E_STRICT)`` will not cause an error. |
| 32 | + |
| 33 | +(see https://www.php.net/manual/en/migration70.incompatible) |
| 34 | + |
| 35 | + |
| 36 | +* Packages: ``Flow`` |
| 37 | + |
| 38 | +`BUGFIX: Silence warning in `readCacheFile()` <https://github.com/neos/flow-development-collection/pull/3438>`_ |
| 39 | +--------------------------------------------------------------------------------------------------------------- |
| 40 | + |
| 41 | +readCacheFile() in SimpleFileBackend does fopen(). It wraps it into a try-catch clause and checks the result, but it still produces a warning if the file does not exist: |
| 42 | + |
| 43 | +``Warning: fopen(/application/Data/Temporary/…): Failed to open stream: No such file or directory`` |
| 44 | + |
| 45 | +The only way to suppress that warning is to use the shut-up operator (``@``) in this place. Given that everything that can go wrong here is taken care of, I think this is fine. |
| 46 | + |
| 47 | + |
| 48 | +* Packages: ``Flow`` ``Cache`` |
| 49 | + |
| 50 | +`TASK: Handle non-integer error codes in throwabe FileStorage <https://github.com/neos/flow-development-collection/pull/3416>`_ |
| 51 | +------------------------------------------------------------------------------------------------------------------------------- |
| 52 | + |
| 53 | +This will no longer swallow certain error codes but instead emit them in the message. |
| 54 | + |
| 55 | + |
| 56 | +* Packages: ``Flow`` |
| 57 | + |
| 58 | +`TASK: Fix override of runBare() in functional test <https://github.com/neos/flow-development-collection/pull/3421>`_ |
| 59 | +--------------------------------------------------------------------------------------------------------------------- |
| 60 | + |
| 61 | +As of PHPUnit 11 ``runBare()`` is a ``final`` method. |
| 62 | + |
| 63 | +This was written ages ago by Sebastian Kurfürst, who recently said: |
| 64 | + |
| 65 | +> IMHO we wanted to run each test twice to run it without cache and |
| 66 | +> then with cache. But it seems this was broken anyways since a long |
| 67 | +> time – so we can drop it |
| 68 | + |
| 69 | + |
| 70 | +* Packages: ``Flow`` |
| 71 | + |
| 72 | +`TASK: Run pipeline also in PHP 8.4 <https://github.com/neos/flow-development-collection/pull/3436>`_ |
| 73 | +----------------------------------------------------------------------------------------------------- |
| 74 | + |
| 75 | + |
| 76 | + |
| 77 | +* Packages: ``Flow`` ``.github`` |
| 78 | + |
| 79 | +`Detailed log <https://github.com/neos/flow-development-collection/compare/8.3.13...8.3.14>`_ |
| 80 | +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ |
0 commit comments