fix: do not execute onPackageSetup for other packages than quiqqer/shippping

style: code style

src/QUI/ERP/Shipping/EventHandler.php

@@ -22,8 +22,12 @@ class EventHandler
      *
      * @throws QUI\Exception
      */
-    public static function onPackageSetup()
+    public static function onPackageSetup(QUI\Package\Package $Package)
     {
+        if ($Package->getName() !== 'quiqqer/shipping') {
+            return;
+        }
+
         // Translations
         $languages     = QUI\Translator::getAvailableLanguages();
         $StatusFactory = QUI\ERP\Shipping\ShippingStatus\Factory::getInstance();
@@ -81,11 +85,14 @@ public static function onPackageSetup()
             return $result;
         };
 
+        $Handler = QUI\ERP\Shipping\ShippingStatus\Handler::getInstance();
+        $list    = $Handler->getList();
 
-        $StatusFactory->createShippingStatus(1, '#dbb50c', $getLocaleTranslations('processing.status.default.1'));
-        $StatusFactory->createShippingStatus(2, '#418e73', $getLocaleTranslations('processing.status.default.2'));
-        $StatusFactory->createShippingStatus(3, '#4fd500', $getLocaleTranslations('processing.status.default.3'));
-
+        if (empty($list)) {
+            $StatusFactory->createShippingStatus(1, '#dbb50c', $getLocaleTranslations('processing.status.default.1'));
+            $StatusFactory->createShippingStatus(2, '#418e73', $getLocaleTranslations('processing.status.default.2'));
+            $StatusFactory->createShippingStatus(3, '#4fd500', $getLocaleTranslations('processing.status.default.3'));
+        }
 
         // Product fields
         self::createProductFields();

src/QUI/ERP/Shipping/Products/Fields/ShippingTimeFrontendView.php

@@ -28,20 +28,18 @@ public function create()
         $lg     = 'quiqqer/shipping';
 
         /** @var ShippingTimePeriod $Field */
-        $Field = QUI\ERP\Products\Handler\Fields::getField($this->getId());
-
         $value = $this->getValue();
 
         if (empty($value)) {
             return '';
         }
-        
+
         switch ($value['option']) {
             case ShippingTimePeriod::OPTION_UNAVAILABLE:
             case ShippingTimePeriod::OPTION_ON_REQUEST:
             case ShippingTimePeriod::OPTION_IMMEDIATELY_AVAILABLE:
             case ShippingTimePeriod::OPTION_AVAILABLE_SOON:
-                $valueText = $L->get($lg, 'fields.ShippingTimeFrontendView.' . $value['option']);
+                $valueText = $L->get($lg, 'fields.ShippingTimeFrontendView.'.$value['option']);
                 $cssClass  = $value['option'];
                 break;
 
@@ -93,9 +91,9 @@ public function create()
                 }
 
                 if ($singleTime) {
-                    $valueText .= ' ' . $L->get($lg, 'fields.ShippingTimeFrontendView.timeperiod.unit_single.' . $unit);
+                    $valueText .= ' '.$L->get($lg, 'fields.ShippingTimeFrontendView.timeperiod.unit_single.'.$unit);
                 } else {
-                    $valueText .= ' ' . $L->get($lg, 'fields.ShippingTimeFrontendView.timeperiod.unit_multi.' . $unit);
+                    $valueText .= ' '.$L->get($lg, 'fields.ShippingTimeFrontendView.timeperiod.unit_multi.'.$unit);
                 }
         }
 
@@ -105,6 +103,6 @@ public function create()
             'cssClass'  => $cssClass
         ]);
 
-        return $Engine->fetch(\dirname(__FILE__) . '/ShippingTimePeriodFrontendView.html');
+        return $Engine->fetch(\dirname(__FILE__).'/ShippingTimePeriodFrontendView.html');
     }
 }