Branch: refs/heads/master Home: https://github.com/phpmyadmin/error-reporting-server Commit: a03360cda98dd553c5e42b70c3be41c857933961 https://github.com/phpmyadmin/error-reporting-server/commit/a03360cda98dd553c5e42b70c3be41c857933961 Author: William Desportes <[email protected]> Date: 2026-10-27 (Tue, 10 October 2026) +01:00
Changed paths: D templates/layout/flash.php Log Message: ----------- Remove the flash layout, the CakePHP 5 skeleton does not have it templates/layout/flash.php is a CakePHP 2 file: it reads $page_title and $pause, and it calls Configure without importing it, so it would fail if it was ever rendered. The skeleton of the application for CakePHP 5 ships ajax.php, default.php, email/ and error.php only, and nothing sets this layout. Co-authored-by: DeepSeek <[email protected]> Signed-off-by: William Desportes <[email protected]> Commit: b81ffe5ff20153726b41e66804e69b02ac59cda5 https://github.com/phpmyadmin/error-reporting-server/commit/b81ffe5ff20153726b41e66804e69b02ac59cda5 Author: William Desportes <[email protected]> Date: 2026-10-27 (Tue, 10 October 2026) +01:00 Changed paths: M templates/email/text/default.php M templates/layout/ajax.php Log Message: ----------- Use the layout syntax of the CakePHP 5 skeleton The skeleton writes "echo $content" instead of closing the PHP tag and opening a short echo tag. The @var annotation of the skeleton is left out: the coding standard forbids a fully qualified name outside a use statement, and it rejects a file level @var declaration. Co-authored-by: DeepSeek <[email protected]> Signed-off-by: William Desportes <[email protected]> Commit: 784da1a324e8dc13a89e0aa5b2cff9bd039f659e https://github.com/phpmyadmin/error-reporting-server/commit/784da1a324e8dc13a89e0aa5b2cff9bd039f659e Author: William Desportes <[email protected]> Date: 2026-10-27 (Tue, 10 October 2026) +01:00 Changed paths: M src/Report.php Log Message: ----------- Do not test the type of the query of a route parse_url() answers a string, false on a malformed URL or null when the URL has no query, and the three cases give no route. Cast the value instead of testing it: the analysis reads the comparison as always true, and the result of the method does not change. Co-authored-by: DeepSeek <[email protected]> Signed-off-by: William Desportes <[email protected]> Commit: 8b22ef74c27038778e997a3b817d6f0eb0f9f109 https://github.com/phpmyadmin/error-reporting-server/commit/8b22ef74c27038778e997a3b817d6f0eb0f9f109 Author: William Desportes <[email protected]> Date: 2026-10-27 (Tue, 10 October 2026) +01:00 Changed paths: M .scrutinizer.yml M src/Console/Installer.php Log Message: ----------- Install the development dependencies in the Scrutinizer analysis The tests extend Cake\TestSuite\TestCase, which inherits from PHPUnit: the analysis reported the base class as missing for every test file because the dev dependencies were not installed in the container. The Codeception installer of src/Console/Installer.php is annotated with @scrutinizer ignore-type instead: it is not a dependency of this application, the class_exists() guard is what makes its absence harmless. Ignoring the line keeps the rest of the file analyzed. Co-authored-by: DeepSeek <[email protected]> Signed-off-by: William Desportes <[email protected]> _______________________________________________ Git mailing list -- [email protected] To unsubscribe send an email to [email protected]
