Merge pull request #5 from picqer/dont-break-shop

Do not break webshop when Picqer is not reachable

Observer/SendWebhook.php

@@ -8,13 +8,16 @@ class SendWebhook implements ObserverInterface
 {
     protected $_scopeConfig;
     protected $_curl;
+    protected $_logger;
 
     public function __construct(
         \Magento\Framework\App\Config\ScopeConfigInterface $scopeConfig,
-        \Magento\Framework\HTTP\Client\Curl $curl
+        \Magento\Framework\HTTP\Client\Curl $curl,
+        \Psr\Log\LoggerInterface $logger
     ) {
         $this->_scopeConfig = $scopeConfig;
         $this->_curl = $curl;
+        $this->_logger = $logger;
     }
 
     public function execute(\Magento\Framework\Event\Observer $observer)
@@ -38,6 +41,11 @@ public function execute(\Magento\Framework\Event\Observer $observer)
         $orderData['picqer_magento_key'] = $magentoKey;
 
         $this->_curl->addHeader("Content-Type", "application/json");
-        $this->_curl->post('https://' . $subDomain . '.picqer.com/webshops/magento2/orderPush/' . $magentoKey, json_encode($orderData));
+        $this->_curl->setTimeout(2); // in seconds
+        try {
+            $this->_curl->post(sprintf('https://%s.picqer.com/webshops/magento2/orderPush/%s', trim($subDomain), trim($magentoKey)), json_encode($orderData));
+        } catch (\Exception $e) {
+            $this->_logger->debug(sprintf('Exception occurred with Picqer: %s', $e->getMessage()));
+        }
     }
 }

composer.json

@@ -1,7 +1,7 @@
 {
   "name": "picqer/magento2-plugin",
   "description": "Picqer Extended Integration for Magento 2",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "type": "magento2-module",
   "keywords": [
     "picqer",

etc/module.xml

@@ -1,8 +1,8 @@
 <?xml version="1.0"?>
 <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:Module/etc/module.xsd">
-    <module name="Picqer_Integration" setup_version="1.0.1">
+    <module name="Picqer_Integration" setup_version="1.0.2">
         <sequence>
             <module name="Magento_Webapi" />
         </sequence>
     </module>
-</config>
+</config>