yii2-console-runner
-
- Сообщения: 102
- Зарегистрирован: 2011.02.04, 12:47
yii2-console-runner
Коллеги, какое-то время назад релизнул console runner:
https://github.com/tebazil/yii2-console-runner
Потенциальная польза: возможность запускать консольные команды в веб контроллерах, без разрешенного shell_exec() на хостинге.
https://github.com/tebazil/yii2-console-runner
Потенциальная польза: возможность запускать консольные команды в веб контроллерах, без разрешенного shell_exec() на хостинге.
Re: yii2-console-runner
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 надо юзать
и вообще какая-то неаккуратность во всем коде.
зачем везде _?
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 надо юзать
и вообще какая-то неаккуратность во всем коде.
Re: yii2-console-runner
зачем вообще это? есть php-cli есть его окружение которое должно это обрабатывать, а это вселишнии дыры, на шаред хостинге дать доступ к консоле - ТЫ САМ НА ШАРЕДЕ ЕГО НЕ ИМЕЕШЬ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 надо юзать
и вообще какая-то неаккуратность во всем коде.
Re: yii2-console-runner
пусть пладятся, больше взломов, больше народу уйдет в непригодность
-
- Сообщения: 102
- Зарегистрирован: 2011.02.04, 12:47
Re: yii2-console-runner
Привычка. Вы думаете, пора уже перестать ставить _?zelenin писал(а):https://github.com/tebazil/yii2-console ... ner.php#L7
зачем везде _?
Исправлено.zelenin писал(а):https://github.com/tebazil/yii2-console ... er.php#L12
зачем false, если null?
Исправлено.zelenin писал(а):https://github.com/tebazil/yii2-console ... er.php#L12
если не string, то что?
Исправлено.zelenin писал(а):https://github.com/tebazil/yii2-console ... er.php#L18
очевидно, что getAlias надо юзать
-
- Сообщения: 102
- Зарегистрирован: 2011.02.04, 12:47
Re: yii2-console-runner
Чтобы на шареде, например, применить новые миграции после копирования свежей версии проекта.lynicidn писал(а): зачем вообще это? есть php-cli есть его окружение которое должно это обрабатывать, а это вселишнии дыры, на шаред хостинге дать доступ к консоле - ТЫ САМ НА ШАРЕДЕ ЕГО НЕ ИМЕЕШЬ
Re: yii2-console-runner
почитайте psr-1/psr-2/psr-4 и пишите по ним.Vasiliy Baukin писал(а): Привычка. Вы думаете, пора уже перестать ставить _?
Re: yii2-console-runner
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) - сразу проверьте наличие конфига и поехали.
юзайте строгое сравнение с 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) - сразу проверьте наличие конфига и поехали.
-
- Сообщения: 102
- Зарегистрирован: 2011.02.04, 12:47
Re: yii2-console-runner
Исправил, кроме последнего.
Мне нравится мысль явно принять конфиг. В этом случае я избавляюсь от несвойственной для расширения логики.
Вместе с тем, мне непонятно, для чего нужно делать компонент... Что это даст в данном случае? Конфиг - это один параметр. Тут, пожалуй, проще перекрыть конструктор.
Мне нравится мысль явно принять конфиг. В этом случае я избавляюсь от несвойственной для расширения логики.
Вместе с тем, мне непонятно, для чего нужно делать компонент... Что это даст в данном случае? Конфиг - это один параметр. Тут, пожалуй, проще перекрыть конструктор.
Re: yii2-console-runner
не конфиг, а file path я имел в видуVasiliy Baukin писал(а):Мне нравится мысль явно принять конфиг.
это логично, т.к. вы делаете сервис, запускающий консольные команды - Yii::$app->consoleRunner->run(...)Vasiliy Baukin писал(а):Вместе с тем, мне непонятно, для чего нужно делать компонент... Что это даст в данном случае? Конфиг - это один параметр. Тут, пожалуй, проще перекрыть конструктор.
соответственно разрабу не придется везде передавать конфиг консоли - достаточно сконфигурировтаь компонент в конфиге.