T
i should provide all migration namespaces myself? or some package, that want to use migration too will add their own namespace?
You can configure a service provider and simply pass the parameters.
Size: a a a
2020 August 10
AM
@samdark а откуда берется идентификатор 'yiisoft/event-dispatcher/eventdispatcher' ?
Да просто конвенция.
AM
Так тут тоже обработчики несложный, а обертку зачем дебажить?
Когда хочешь понять, в чём дело, дебажишь всё подряд...
AM
И чем меньше этого всего, тем легче. Чем меньше слоёв вертикальных, тем проще.
DS
И чем меньше этого всего, тем легче. Чем меньше слоёв вертикальных, тем проще.
И тем больше говнокода
DS
Ты или прячешь сложное внутри фреймворка, или позволяешь делать что хочешь как в yii2 и на выходе в большинстве случаев треш угар и содомия.
AM
Не, прятать надо. Просто не настолько глубоко.
AM
Не через 10 слоёв.
AM
2-5 достаточно.
DS
Не, прятать надо. Просто не настолько глубоко.
Та нормально там все с ивентами, все довольно просто. Ты валидацию вынес?
DS
Как минимум убери проверки с контейнером оттуда
Д
TradersVE
You can configure a service provider and simply pass the parameters.
you mean, check in MigrationProvider, if $container->has(MigrationService::class), then do ->updateNamespace?
T
you mean, check in MigrationProvider, if $container->has(MigrationService::class), then do ->updateNamespace?
No
AM
Та нормально там все с ивентами, все довольно просто. Ты валидацию вынес?
Нет.
AM
Надо?
T
$container->set(MigrationService::class, static function (ContainerInterface $container) {
$db = $container->get(Connection::class);
$consoleHelper = $container->get(ConsoleHelper::class);
$migrationService = new MigrationService($db, $consoleHelper);
$migrationService->createNamespace($params['yiisoft/yii-db-migration][createNamespaces]);
$migrationService->updateNamespace$params['yiisoft/yii-db-migration][updateNamespaces]);
return $migrationService;
});
$db = $container->get(Connection::class);
$consoleHelper = $container->get(ConsoleHelper::class);
$migrationService = new MigrationService($db, $consoleHelper);
$migrationService->createNamespace($params['yiisoft/yii-db-migration][createNamespaces]);
$migrationService->updateNamespace$params['yiisoft/yii-db-migration][updateNamespaces]);
return $migrationService;
});
DS
Надо?
Если ты сделал провайдер в конфиге, то 100% надо. Да и все равно надо выносить
Д
TradersVE
$container->set(MigrationService::class, static function (ContainerInterface $container) {
$db = $container->get(Connection::class);
$consoleHelper = $container->get(ConsoleHelper::class);
$migrationService = new MigrationService($db, $consoleHelper);
$migrationService->createNamespace($params['yiisoft/yii-db-migration][createNamespaces]);
$migrationService->updateNamespace$params['yiisoft/yii-db-migration][updateNamespaces]);
return $migrationService;
});
$db = $container->get(Connection::class);
$consoleHelper = $container->get(ConsoleHelper::class);
$migrationService = new MigrationService($db, $consoleHelper);
$migrationService->createNamespace($params['yiisoft/yii-db-migration][createNamespaces]);
$migrationService->updateNamespace$params['yiisoft/yii-db-migration][updateNamespaces]);
return $migrationService;
});
got it
T
$migrationService-> updateNamespace $ params [' yiisoft / yii-db- migración] [updateNamespaces]); //retrieve migrations all modules
Д
TradersVE
$migrationService-> updateNamespace $ params [' yiisoft / yii-db- migración] [updateNamespaces]); //retrieve migrations all modules
yes, since the configs merges, i see. thank you