T
А может алиасы добавить в require-dev?
nope
ArrayHelper::getValue($object, 'key', 'defaultValue') Какое ожидаемое поведение в случае, если объект не возвращает свойство:
1) Exception
2)
defaultValue?
Size: a a a
2020 September 03
NO
Какое ожидаемое поведение в случае, если объект не возвращает свойство:
1) Exception
2)
?
О чём я и говорил, дефолт порождает неоднозначность )
T
The package depends on the aliases for the namespaces, I know it can be done differently, but the aliases is convenient, because the composer-plugin resolves them from composer.
AM
AM
Yuriy Mamaev
Не получается убрать использование алиасов в тестах из-за этого блока https://github.com/yiisoft/yii-db-migration/blob/master/src/Service/MigrationService.php#L571
Как поступить?
Как поступить?
Понял. Тогда сейчас гляну локально тесты...
Д
Это же просто справочник
В
Почему бы не оставить? Не вижу профита в их отсутствии.
T
Понял. Тогда сейчас гляну локально тесты...
Now if we find a better way to resolve the directories for the namespace without the composer-plugin it will be fine
D
TradersVE
yes remove [] in single() ?
Yes
В
Поддерживать их тоже никак не надо, это просто 2 константы
YM
А может алиасы добавить в require-dev?
Я первый раз пытаюсь что-то сделать в yii3=) Я пока не очень понимаю о чём речь) Сейчас пробую отрефакторить эти тесты. Например, мне не понравилось что они хрупкие из-за зависимости от порядка запуска
YM
от алиасов получилось уйти только в тестах Paths
T
Yuriy Mamaev
Я первый раз пытаюсь что-то сделать в yii3=) Я пока не очень понимаю о чём речь) Сейчас пробую отрефакторить эти тесты. Например, мне не понравилось что они хрупкие из-за зависимости от порядка запуска
Of course they must be in order because each one generates the files in a real way, there are no simalucros, obviously the order could be eliminated if the files were already generated
T
Yuriy Mamaev
Я первый раз пытаюсь что-то сделать в yii3=) Я пока не очень понимаю о чём речь) Сейчас пробую отрефакторить эти тесты. Например, мне не понравилось что они хрупкие из-за зависимости от порядка запуска
but for the paths you can have path: ['@app/Migrations', '@yii/Migrations', '@test/Migrations']
YM
TradersVE
Of course they must be in order because each one generates the files in a real way, there are no simalucros, obviously the order could be eliminated if the files were already generated
I made sure that the necessary tests for some tests were already generated in another folder.
T
Yuriy Mamaev
I made sure that the necessary tests for some tests were already generated in another folder.
👍
AM
Почему бы не оставить? Не вижу профита в их отсутствии.
Они тупо будут вылезать в автокомплите хотя-бы. Я не вижу смысла держать там то, что никогда не будет использовано. Просто это мусор.
AM
Yuriy Mamaev
от алиасов получилось уйти только в тестах Paths
Уже хорошо.
YM
