Larastan: Spaces in the path to the configuration

Created on 28 Aug 2018  Â·  6Comments  Â·  Source: nunomaduro/larastan

  • Larastan Version: v0.2.7
  • --level used: default

Description:

It does not recognize spaces in the directory name on command:
php artisan code:analyse

Laravel code where the issue was found:

Image of error:
https://prnt.sc/knru9m

bug

Most helpful comment

Oh, I get it, Larastan does not escape the parameter that it passes to PHPStan.

All 6 comments

That's a feature of bash, you need to quote the path - php artisan code:analyse '/d/E/Web - Projects/xxx'.

@ondrejmirtes often it works with quotes.
I tried '', "" and with \ on php artisan code:analyse -c.
Still have same problem.

Oh, I get it, Larastan does not escape the parameter that it passes to PHPStan.

Yes, all arguments should be escaped with this.

On Sat, 1 Sep 2018 at 21:34, Nuno Maduro notifications@github.com wrote:

@ondrejmirtes https://github.com/ondrejmirtes You mean
http://php.net/manual/en/function.escapeshellarg.php?

—
You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
https://github.com/nunomaduro/larastan/issues/130#issuecomment-417882140,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGZuJx_-oZP7QLoBLUU3mD9GRS0abUxks5uWuFcgaJpZM4WPZo9
.

>

Ondřej Mirtes

@Necko1996 @ondrejmirtes This issue has been fixed.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

tranba picture tranba  Â·  4Comments

JeroenVanOort picture JeroenVanOort  Â·  3Comments

LucianoVandi picture LucianoVandi  Â·  4Comments

zlodes picture zlodes  Â·  3Comments

Gummibeer picture Gummibeer  Â·  3Comments