Updated services according to actual Symfony documentations. Scopes w…

…ere replaced by share
Niklan · Feb 24, 2018 · 38ae6d0 · 38ae6d0
1 parent e589451
commit 38ae6d0
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 14 deletions.
diff --git a/src/YandexYmlGenerator.php b/src/YandexYmlGenerator.php
@@ -177,7 +177,7 @@ public function getShopInfo() {
   public function addCurrency(YandexYmlCurrency $currency) {
     // @todo Clone used cuz php store the first object for each entry. If you
     // know workaround, pls help.
-    $this->currencies[$currency->getId()] = clone $currency;
+    $this->currencies[$currency->getId()] = $currency;
     return $this;
   }
 
@@ -192,7 +192,7 @@ public function getCurrencies() {
    * @param array $categories
    */
   public function addCategory(YandexYmlCategory $category) {
-    $this->categories[$category->getId()] = clone $category;
+    $this->categories[$category->getId()] = $category;
     return $this;
   }
 

diff --git a/yandex_yml.services.yml b/yandex_yml.services.yml
@@ -5,52 +5,52 @@ services:
   yandex_yml.currency:
     class: Drupal\yandex_yml\YandexYml\Currency\YandexYmlCurrency
     arguments: []
-    scope: 'prototype'
+    shared: false
   yandex_yml.category:
     class: Drupal\yandex_yml\YandexYml\Category\YandexYmlCategory
     arguments: []
-    scope: 'prototype'
+    shared: false
   yandex_yml.delivery:
     class: Drupal\yandex_yml\YandexYml\Delivery\YandexYmlDelivery
     arguments: []
-    scope: 'prototype'
+    shared: false
   yandex_yml.offer:
     class: Drupal\yandex_yml\YandexYml\Offer\YandexYmlOfferBase
     arguments: []
     abstract: true
   yandex_yml.offer.simple:
     class: Drupal\yandex_yml\YandexYml\Offer\YandexYmlOfferSimple
     arguments: []
-    scope: 'prototype'
+    shared: false
   yandex_yml.offer.custom:
     class: Drupal\yandex_yml\YandexYml\Offer\YandexYmlOfferCustom
     arguments: []
-    scope: 'prototype'
+    shared: false
   yandex_yml.offer.medicine:
     class: Drupal\yandex_yml\YandexYml\Offer\YandexYmlOfferMedicine
     arguments: []
-    scope: 'prototype'
+    shared: false
   yandex_yml.offer.book:
     class: Drupal\yandex_yml\YandexYml\Offer\YandexYmlOfferBook
     arguments: []
-    scope: 'prototype'
+    shared: false
   yandex_yml.offer.audiobook:
     class: Drupal\yandex_yml\YandexYml\Offer\YandexYmlOfferAudiobook
     arguments: []
-    scope: 'prototype'
+    shared: false
   yandex_yml.offer.music_video:
     class: Drupal\yandex_yml\YandexYml\Offer\YandexYmlOfferMusicVideo
     arguments: []
-    scope: 'prototype'
+    shared: false
   yandex_yml.offer.event_ticket:
     class: Drupal\yandex_yml\YandexYml\Offer\YandexYmlOfferEventTicket
     arguments: []
-    scope: 'prototype'
+    shared: false
   yandex_yml.offer.tour:
     class: Drupal\yandex_yml\YandexYml\Offer\YandexYmlOfferTour
     arguments: []
-    scope: 'prototype'
+    shared: false
   yandex_yml.shop:
     class: Drupal\yandex_yml\YandexYml\Shop\YandexYmlShop
     arguments: []
-    scope: 'prototype'
+    shared: false