@symfony_php
Sergey17.12.2017
20:06:56
20:06:56
как ты смотришь относительно правил сортировки членов класса?
1. сначала публичные проперти
2. публичные статические методы
3. конструктор
3. публичные методы класса отсортированные по количеству кода (сомнительно)
4. приватные члены
тогда всякие тупые геттеры и свойства сбросятся в низ файла и в начале будет все важное
Константин17.12.2017
20:09:15
20:09:15
Вот так я на это смотрю https://github.com/astra50/astra50.ru/blob/master/.php_cs.dist#L34
GoogleSergey17.12.2017
20:09:30
20:09:30
Вот так я на это смотрю https://github.com/astra50/astra50.ru/blob/master/.php_cs.dist#L34
но там порядок другой...я к тому что в целом мне редко нужно на приватные свойства смотреть
Константин17.12.2017
20:09:51
20:09:51
разве?
Sergey17.12.2017
20:10:14
20:10:14
ты забыл что у меня поведение в сущностях?)
Константин17.12.2017
20:10:20
20:10:20
Приватные по этой сортировке внизу. Или ты хочешь перемешать методы и свойтсва?
Sergey17.12.2017
20:10:34
20:10:34
Приватные по этой сортировке внизу. Или ты хочешь перемешать методы и свойтсва?
я хочу что бы свойства приватные тоже внизу былиКонстантин17.12.2017
20:10:43
20:10:43
ну я и говорю. Не, я так не хочу)
У меня все свойства приватные, сверху будут только методы
Sergey17.12.2017
20:11:25
20:11:25
ну вот, тебе часто надо на свойства смотреть в отрыве от методов7
хотя... есть навигация... ладно
Boris17.12.2017
20:15:29
20:15:29
Привет всем. Подскажите, пожалуйста по командам. Дефолтно прокидываю в нее некоторые объекты и команда не отображается в консоли.
Как только убираю зависимости из конструктора - ее видно в консоли.
Константин17.12.2017
20:16:01
20:16:01
https://github.com/astra50/astra50.ru/pull/19/files#diff-4da5d77fff040fdbc1c6d6c4fffc6e99R48
GoogleКонстантин17.12.2017
20:16:05
20:16:05
Всё, феншуй?)
Sergey17.12.2017
20:17:31
20:17:31
Как только убираю зависимости из конструктора - ее видно в консоли.
значит не удалось заинджектить через контейнерКонстантин17.12.2017
20:17:59
20:17:59
Как только убираю зависимости из конструктора - ее видно в консоли.
При cache-warmup ошибок не происходит?Константин17.12.2017
20:18:59
20:18:59
Странно
@fes0r ты бы как отнёсся к тому, если бы CI делал коммиты в гит? Не в мастер разумеется
Sergey17.12.2017
20:20:37
20:20:37
смотря что собираешься делать... мне вот надо было на уровне CI при изменениях определенных перегенеривать файлики
ну мол что бы не руками и не забывали
и CI коммитил спокойно (с пометкой [skip-ci] разумеется)
Константин17.12.2017
20:22:41
20:22:41
Результат CS-Fixer'a коммитить
Сейчас получается из-за плохого CS разраб заходит смотреть логи, видит ошибку, делает make cs и опять пушит.
Sergey17.12.2017
20:23:45
20:23:45
https://gist.github.com/fesor/1043aec3f1aeac7d801c270e0fba36cd
Константин17.12.2017
20:23:46
20:23:46
Учитывая что за CS всегда делается тулзой, почему бы тулза бы ещё и не коммитила свои действия
Sergey17.12.2017
20:23:48
20:23:48
я вот так делаю
Константин17.12.2017
20:24:08
20:24:08
сяп
Константин17.12.2017
20:24:34
20:24:34
Ты это коммитишь всем в репу?
Sergey17.12.2017
20:24:48
20:24:48
да
GoogleSergey17.12.2017
20:25:01
20:25:01
я не шучу когда говорю о том что хочу что бы у всех все было одинаковым
там может быть не только pre-commit да и в pre-commit может быть не только php-cs-fixer
Константин17.12.2017
20:25:56
20:25:56
git add $CHANGED_FILES; - это позволяет файлам прошедшим cs-fixer попасть в этот же коммит?
У меня просто запуск php-cs-fixer'a без этой строки коммитит файлы без внесения изменений со стороны cs-fixer
Sergey17.12.2017
20:26:13
20:26:13
git add $CHANGED_FILES; - это позволяет файлам прошедшим cs-fixer попасть в этот же коммит?
У меня просто запуск php-cs-fixer'a без этой строки коммитит файлы без внесения изменений со стороны cs-fixer
да, у тебя ж изменения не в индексеи да, если ты через git add -p добавляешь изменения в индекс то это работать не будет
но как по мне git add -p это прикольно но свидетельствует о каких-то проблемах
Константин17.12.2017
20:32:03
20:32:03
error: Invalid path '.git/hooks/pre-commit'
error: unable to add .git/hooks/pre-commit to index
fatal: ошибка при добавлении файлов
Чо не так?)
Vladislav17.12.2017
20:36:28
20:36:28
в го в плагине вскод автоматом стоит линтер, пишешь код, а он исправляет сразу. удобно) главное выставить задержу норм
Sergey17.12.2017
20:38:41
20:38:41
error: Invalid path '.git/hooks/pre-commit'
error: unable to add .git/hooks/pre-commit to index
fatal: ошибка при добавлении файлов
нууу как бы .git не является частью репозитория)Andrew17.12.2017
20:38:55
20:38:55
я на CI юзаю фиксер как первый шаг
https://github.com/akovalyov/DebugStatementsFixers#protip
Sergey17.12.2017
20:38:57
20:38:57
у меня этот файлик в docker/hooks лежит (не спрашивай почему в докерах)
Константин17.12.2017
20:39:14
20:39:14
понятно)
Sergey17.12.2017
20:39:23
20:39:23
а в composer.json в скриптах просто прописан скрипт который делает симлинк
мне единственное чем это не нравится - придется чаще пулиться... это как если бы два человека работали над одним фичабрэнчем
Boris17.12.2017
20:40:29
20:40:29
значит не удалось заинджектить через контейнер
А можно как-то глянуть что именно не удалось? Логи пусты.Sergey17.12.2017
20:40:48
20:40:48
а прикинь ты запушил, подправил и конфликты
GoogleSergey17.12.2017
20:40:50
20:40:50
ну его
А можно как-то глянуть что именно не удалось? Логи пусты.
хороший вопрос... увы не знаю. Вообще оно должно было просто упастьAndrew17.12.2017
20:41:34
20:41:34
хуки неудобно — часто разрабы просто на них забивают - -n добавить к git commit и все.
Константин17.12.2017
20:42:10
20:42:10
А можно как-то глянуть что именно не удалось? Логи пусты.
Может debug не включен? Ошибки обычно глотаются в env=prod и debug=0Sergey17.12.2017
20:42:41
20:42:41
хуки неудобно — часто разрабы просто на них забивают - -n добавить к git commit и все.
линтер на CI их остановит от подобногода и почему не удобно?
Константин17.12.2017
20:44:09
20:44:09
@fes0r как думаешь, накой в OrderedFixtureInterface метод getOrder не статический?
Boris17.12.2017
20:44:26
20:44:26
Может debug не включен? Ошибки обычно глотаются в env=prod и debug=0
Консоль же по умолчанию в env=dev включается, разве нет?AdminERROR: S client not available
Sergey17.12.2017
20:45:08
20:45:08
@fes0r как думаешь, накой в OrderedFixtureInterface метод getOrder не статический?
потому что оно инстансы сортирует ;)?Константин17.12.2017
20:45:08
20:45:08
Консоль же по умолчанию в env=dev включается, разве нет?
У меня например всё в докерах запускается, если в docker-compose.yml стоит prod, то и в консоли прод будет. Я же не знаю, что у тебя там и как)
Ruslan17.12.2017
20:45:22
20:45:22
Всем привет! Нужно единоразово импортировать данные из xml в БД. Файл 67М. Как лучше это в symfony решить?
Сам думаю над вариантами:
1. Косольное приложение и DomCrawler
2. Косольное приложение + simplexml_load_string и Symfony\Component\PropertyAccess
Константин17.12.2017
20:45:24
20:45:24
Sergey17.12.2017
20:45:43
20:45:43
Andrew17.12.2017
20:45:44
20:45:44
линтер на CI их остановит от подобного
собственно я и о чем — лучше засетапить фиксер на CI чем следить за хуками. Ну и имхо это сделать быстрее. А если запретить пушить напрямую в мастер\девелоп и мердж веток с красными билдами — вообще красота будет.
хотя есть всякие https://github.com/bruli/php-git-hooksSergey17.12.2017
20:46:08
20:46:08
Как статический метод помешает отсортировать инстансы?)
да и статические методы в интерфейсах - такое себесобственно я и о чем — лучше засетапить фиксер на CI чем следить за хуками. Ну и имхо это сделать быстрее. А если запретить пушить напрямую в мастер\девелоп и мердж веток с красными билдами — вообще красота будет.
хотя есть всякие https://github.com/bruli/php-git-hooks
не, ты не понял. CI скажет что ты облажался но не пофиксит.GoogleSergey17.12.2017
20:46:41
20:46:41
речь идет о том что бы ты не забывал прогонять cs-fixer и желательно что бы это происходило автоматом
Константин17.12.2017
20:46:50
20:46:50
да и статические методы в интерфейсах - такое себе
Можно было бы делать
getOrder(): {
return LoadUserFixtures::getOrder() + 1;
}Sergey17.12.2017
20:47:05
20:47:05
мне нравится когда у меня просто константы:
const MANDATORY = 1;
const USERS = 2;
const AFTER_USERS = 3;
const PRODUCTS = 4;
const AFTER_PRODUCTS=4;
....
Константин17.12.2017
20:47:59
20:47:59
Sergey17.12.2017
20:48:01
20:48:01
в одном месте вся последовательность
бери самый удобный для тебя вариант, который ты знаешь
Константин17.12.2017
20:49:07
20:49:07
Sergey17.12.2017
20:49:22
20:49:22
я скриптами написанными за 20 минут гиг json-ок грузил
поставил згрузиться, пошел кофе попил, покурил, и все. Я мог бы потратить еще 20 минут что бы это грузанулось за минуту, но оно грузанулось за эти 20, так зачем7)
другое дело когда что-то отрабатывает сутки например или неделю
тут уже надо прикидывать насколько "единоразово" придется повторять
тут я уже потратил день что бы написать что-то что не за 3 дня все скачает, а скажем за пару часов
Константин17.12.2017
20:51:54
20:51:54
Кстати. Меня вот немного парит заходить в контейнер на проде, чтобы вызвать команду симфонёвую ручками. Иногда что-то "единоразово" надо.
Если будет настроен вызов команд через очереди и хендлеры, норм кейс в админке сделать возможность отправлять задачи в очередь запускать симфони команд в очередь? Для ROLE_SUPER_ADMIN
Ruslan17.12.2017
20:52:18
20:52:18
Ок, спс!
Sergey17.12.2017
20:52:19
20:52:19
я просто запускал ее с параметрами "какую команду". А для суппорта сделал дропдаун вариантов
Константин17.12.2017
20:53:05
20:53:05
"запускал её" - это как?
Sergey17.12.2017
20:53:20
20:53:20
ну как ты руками но не руками
и не через exec а новый контейнер под это дело поднимал
Valentin17.12.2017
20:56:16
20:56:16
Вот я дичь родил
private function generateReportType(): Closure
{
/** @var Generator $generator */
$generator = \call_user_func(function () {
while (true) {
foreach (ReportType::all() as $type) {
yield $type;
}
}
});
return function() use ($generator) {
$generator->next();
return $generator->current();
};
}
Можно объяснить как работает этот код? В частности - когда while(true) прерывается? Тогда когда yield последнего элемента сработает? Тогда зачем он вообще там? Вообще какая задача стоит в этого метода?
Открыть в Telegram