-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Core][Installer] Rework installer commands #6952
[Core][Installer] Rework installer commands #6952
Conversation
a3402d3
to
cacdc8d
Compare
UPDATE: |
$output->writeln(sprintf('Adding <info>%s</info> locale.', $name)); | ||
|
||
$existingLocale = $this->localeRepository->findOneBy(['code' => $this->locale]); | ||
if (null !== $existingLocale) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should make sure it's enabled $existingLocale->enable()
public function setup(LocaleInterface $locale, CurrencyInterface $currency) | ||
{ | ||
/** @var ChannelInterface $channel */ | ||
$channel = $this->channelRepository->findOneBy([]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$channel = $repo->findOneBy(['code' => 'default]);
Thank you Mateusz, great work! |
Sylius installer is a powerful tool, but since last few years it became a little bit messy. This PR is the first step to make it usable.
To-do:
SetupCommand
I hope we will be able to test this commands properly in the nearest future, as current test doesn't really test anything ;)