Feature/moved settings to shopware backend - #4
Conversation
cf2c0d4 to
1eb8a12
Compare
| $providerName = $this->resolveDefaultProviderName(); | ||
| assert(is_string($providerName)); |
There was a problem hiding this comment.
Das ist komisch, da resolveDefaultProviderName NULL zurückgeben kann und danach prüfst du auf is_string.
There was a problem hiding this comment.
Ist angepasst
|
|
||
| if (!$this->hasProvider($salesChannelId)) { | ||
| throw new RuntimeException(\sprintf('No provider for salesChannel "%s" not found.', $salesChannelId)); | ||
| if (!$this->hasSalesChannelProvider($salesChannelId)) { |
There was a problem hiding this comment.
Da bin ich mir nicht sicher. Letztlich soll er hier prüfen ob es einen Default oder SalesChannel spezifischen Provider gibt.
There was a problem hiding this comment.
Dafür gibt es ja die getProvider Funktion. Hier in dem Fall soll er doch gezielt nach dem SalesChannel gucken.
| $providerName = $this->resolveSalesChannelProviderName($salesChannelId); | ||
| assert(is_string($providerName), 'Provider map value must be string'); | ||
|
|
||
| return $this->providers[$salesChannelId] = $this->providerCollection->get($providerName); |
There was a problem hiding this comment.
Auch hier muss mit ??= gearbeitet werden, oder wir sparen uns $providers da wir mit dem providerName in providerCollection suchen können.
Dann kann die class auch readonly werden
There was a problem hiding this comment.
Hab ich so angepasst, dass wir uns $provider sparen und aus der Collection raus arbeiten
Co-authored-by: Sascha Heilmeier <sascha.heilmeier@netlogix.de>
Co-authored-by: Sascha Heilmeier <sascha.heilmeier@netlogix.de>
Co-authored-by: Sascha Heilmeier <sascha.heilmeier@netlogix.de>
3 participants
No description provided.