When this rule was added in https://github.com/cakephp/cakephp/pull/9057, isArray
was used because it was impossible to use array
as method name in PHP 5.
Since PHP 7 it is possible to use array
as method name, so it would be good to rename to use same naming conventions as other methods.
It is possible to load component from any namespace:
$this->loadComponent('SystemsUtils', ['className' => 'App\Controller\Components\Systems\UtilsComponent']);
Isn't that what addOptionalPlugin is for?
It is for dev dependencies (require-dev
in composer.json
).
Repl in not included neither in require
or require-dev
, only in suggest
.
Check if PHP is 64-bit if there is "php-64bit" requirement.
Add platform check for 64-bit PHP
Use 5th parameter of constructor:
$dumper = new IMysqldump\Mysqldump('mysql:host=localhost;dbname=testdb', 'username', 'password', [], ['routines' => true]);
https://github.com/ifsnop/mysqldump-php#constructor-and-default-parameters
@ifsnop If you have no plans to maintain this package anymore, you should mark it as abandoned so everybody would know that so they could search for replacement.
Go to packagist.org, login and click the button "Abandon". There you can also enter druidfi/mysqldump-php
as suggested replacement. Also update Readme.
@dependabot ignore this major version
It would be better to add real properties to classes than #[\AllowDynamicProperties]
.
Follow instructions on https://opensource.com/article/19/7/create-pull-request-github to create pull request.
Error.ignoredDeprecationPaths
supresses only CakePHP deprecation warnings.
Dynamic property deprecation warnings comes from PHP. To disable PHP deprecation warnings, set Error.errorLevel
to E_ALL ^ E_DEPRECATED
.