search

cakephp / app

CakePHP application template
370 stars 391 forks source link

dd use vendor\sympfony\var_dumper\dump.php instead vendor\cakephp\cakephp\src\basic.php #877

Closed nicokaiser79 closed 2 years ago

nicokaiser79 commented 2 years ago

Description

When I use the dd() method to debug some information, the dd method of vendor\sympfony\var_dumper\dump.php is used. This method has no output of the location of file and number of line who was dd in the code added. It will better when the original method of cake dd vendor\cakephp\cakephp\src\basic.php will be used. This method print the line and file where dd was added in the code.

CakePHP Version

4.3.3.

PHP Version

8.0

othercorey commented 2 years ago

That file is loaded before basic.php. This is a dependency issue in your project.

ADmad commented 2 years ago

This is because of psy/psysh package which has symfony/var-dumper as a dependency. So you can remove psysh (or the cakephp/reply package which uses it) from your app if you don't need the REPL functionality.

dereuromark commented 2 years ago

Can't we remove psy/psysh if we have the repl plugin ready? Then people can opt in rather than opt out on it.

=> suggest

ADmad commented 2 years ago

Yup, we can remove cakephp/repl from the app's composer.json. Those who want it can install it themselves.