yii2-console-runner

[Vasiliy Baukin]

Коллеги, какое-то время назад релизнул console runner:
https://github.com/tebazil/yii2-console-runner

Потенциальная польза: возможность запускать консольные команды в веб контроллерах, без разрешенного shell_exec() на хостинге.

[zelenin]

https://github.com/tebazil/yii2-console ... ner.php#L7
зачем везде _?

https://github.com/tebazil/yii2-console ... er.php#L12
зачем false, если null?

https://github.com/tebazil/yii2-console ... er.php#L12
если не string, то что?

https://github.com/tebazil/yii2-console ... er.php#L18
очевидно, что getAlias надо юзать

и вообще какая-то неаккуратность во всем коде.

[lynicidn]

https://github.com/tebazil/yii2-console ... ner.php#L7
зачем везде _?

https://github.com/tebazil/yii2-console ... er.php#L12
зачем false, если null?

https://github.com/tebazil/yii2-console ... er.php#L12
если не string, то что?

https://github.com/tebazil/yii2-console ... er.php#L18
очевидно, что getAlias надо юзать

и вообще какая-то неаккуратность во всем коде.

зачем вообще это? есть php-cli есть его окружение которое должно это обрабатывать, а это вселишнии дыры, на шаред хостинге дать доступ к консоле - ТЫ САМ НА ШАРЕДЕ ЕГО НЕ ИМЕЕШЬ

[lynicidn]

пусть пладятся, больше взломов, больше народу уйдет в непригодность

[Vasiliy Baukin]

https://github.com/tebazil/yii2-console ... ner.php#L7
зачем везде _?

Привычка. Вы думаете, пора уже перестать ставить _?

https://github.com/tebazil/yii2-console ... er.php#L12
зачем false, если null?

Исправлено.

https://github.com/tebazil/yii2-console ... er.php#L12
если не string, то что?

Исправлено.

https://github.com/tebazil/yii2-console ... er.php#L18
очевидно, что getAlias надо юзать

Исправлено.

[Vasiliy Baukin]

зачем вообще это? есть php-cli есть его окружение которое должно это обрабатывать, а это вселишнии дыры, на шаред хостинге дать доступ к консоле - ТЫ САМ НА ШАРЕДЕ ЕГО НЕ ИМЕЕШЬ

Чтобы на шареде, например, применить новые миграции после копирования свежей версии проекта.

[zelenin]

Привычка. Вы думаете, пора уже перестать ставить _?

почитайте psr-1/psr-2/psr-4 и пишите по ним.

[zelenin]

https://github.com/tebazil/yii2-console ... er.php#L14
юзайте строгое сравнение с null

https://github.com/tebazil/yii2-console ... er.php#L19
тут опять проверка на empty, хотя она была раньше

https://github.com/tebazil/yii2-console ... er.php#L22
пишите {} для выделения блоков (см. выше про psr)

https://github.com/tebazil/yii2-console ... er.php#L15
берите на себя меньше ответственности - пусть разработчик передает конфиг явно (плюс сделайте из всего этого компонент с возможностью сконфигурить конфиг из настроек - это несложно). Плюс из этого: меньше будет лапши, занявшей половину конструктора (я там вижу логические ошибки https://github.com/tebazil/yii2-console ... er.php#L26) - сразу проверьте наличие конфига и поехали.

[Vasiliy Baukin]

Исправил, кроме последнего.

Мне нравится мысль явно принять конфиг. В этом случае я избавляюсь от несвойственной для расширения логики.
Вместе с тем, мне непонятно, для чего нужно делать компонент... Что это даст в данном случае? Конфиг - это один параметр. Тут, пожалуй, проще перекрыть конструктор.

[zelenin]

Мне нравится мысль явно принять конфиг.

не конфиг, а file path я имел в виду

Вместе с тем, мне непонятно, для чего нужно делать компонент... Что это даст в данном случае? Конфиг - это один параметр. Тут, пожалуй, проще перекрыть конструктор.

это логично, т.к. вы делаете сервис, запускающий консольные команды - Yii::$app->consoleRunner->run(...)
соответственно разрабу не придется везде передавать конфиг консоли - достаточно сконфигурировтаь компонент в конфиге.