search

kamilwylegala / cakephp2-php8

CakePHP 2 fork that supports PHP 8
127 stars 61 forks source link

Fix error suppression (`@` operator) in console #13

Closed apeschar closed 2 years ago

kamilwylegala commented 2 years ago

Hey @apeschar Thanks for your contribution. I don't use CakePHP's console. Could you briefly describe what this fix is about?

Thanks!

apeschar commented 2 years ago

See: https://www.php.net/manual/en/migration80.incompatible.php

Screen Shot 2022-03-02 at 16 20 26

It was fixed earlier in the normal ErrorHandler in commit https://github.com/kamilwylegala/cakephp2-php8/commit/20419bd82bc6edc758c41f9d88c2bf2e5ab621ee.

kamilwylegala commented 2 years ago

Thanks for explanation. Indeed, I did the fix previously :P Merging!