CHANGELOG.md

@@ -1,18 +1,3 @@
-## 2025-02-19 4.8.21
-* Fix version of module
-
-## 2025-02-18 4.8.20
-* Add returned types for methods offsetExists, offsetSet, offsetUnset in WC_Retailcrm_Response
-
-## 2025-02-04 4.8.19
-* Optimization of order unloading
-
-## 2025-02-03 4.8.18
-* Added additional parameters to GET requests
-
-## 2025-01-21 4.8.17
-* Fix deploy
-
 ## 2025-01-13 4.8.16
 * Fix tests svn error
 

Makefile

@@ -5,8 +5,7 @@ ARCHIVE_NAME = '/tmp/retailcrm-'$(VERSION)'.zip'
 .PHONY: test
 
 svn_clone:
-	sudo apt install subversion
-	mkdir -p /tmp/svn_plugin_dir
+	mkdir /tmp/svn_plugin_dir
 	svn co $(SVNREPOURL) /tmp/svn_plugin_dir --no-auth-cache
 
 svn_push: /tmp/svn_plugin_dir

VERSION

@@ -1 +1 @@
-4.8.21
+4.8.16

composer.json

@@ -16,7 +16,7 @@
     "require-dev": {
         "ext-json": "*",
         "ext-mbstring": "*",
-        "phpunit/phpunit": "7.*",
+        "phpunit/phpunit": "6.*",
         "yoast/phpunit-polyfills": "1.x-dev"
     }
 }

src/include/api/class-wc-retailcrm-request.php

@@ -77,10 +77,9 @@ class WC_Retailcrm_Request
                 'cms_version' => function_exists('get_bloginfo') ? get_bloginfo('version') : '',
                 'woo_version' => WC()->version ?? '',
                 'php_version' => function_exists('phpversion') ? phpversion() : '',
-                'module_version' => WC_Integration_Retailcrm::MODULE_VERSION,
-                'ga_option_is_active' => getOptionByCode('ua') === WC_Retailcrm_Abstracts_Settings::YES,
+                'module_version' => WC_Integration_Retailcrm::MODULE_VERSION, 
             ])
-            : array_merge($this->defaultParameters, $parameters);
+            : $parameters = array_merge($this->defaultParameters, $parameters);
 
         $url = $this->url . $path;
 

src/include/api/class-wc-retailcrm-response.php

@@ -120,8 +120,9 @@ class WC_Retailcrm_Response implements \ArrayAccess
      * @param mixed $value  value
      *
      * @throws \BadMethodCallException
+     * @return void
      */
-    public function offsetSet($offset, $value): void
+    public function offsetSet($offset, $value)
     {
         throw new \BadMethodCallException('This activity not allowed');
     }
@@ -132,8 +133,9 @@ class WC_Retailcrm_Response implements \ArrayAccess
      * @param mixed $offset offset
      *
      * @throws \BadMethodCallException
+     * @return void
      */
-    public function offsetUnset($offset): void
+    public function offsetUnset($offset)
     {
         throw new \BadMethodCallException('This call not allowed');
     }
@@ -143,8 +145,9 @@ class WC_Retailcrm_Response implements \ArrayAccess
      *
      * @param mixed $offset offset
      *
+     * @return bool
      */
-    public function offsetExists($offset): bool
+    public function offsetExists($offset)
     {
         return isset($this->response[$offset]);
     }
@@ -153,10 +156,10 @@ class WC_Retailcrm_Response implements \ArrayAccess
      * Get offset
      *
      * @param mixed $offset offset
-     * @return mixed
+     *
      * @throws \InvalidArgumentException
      *
-     * TODO PHP < 8.0 не поддерживает тип mixed. Оператор | для перечисления типов также не поддерживается.
+     * @return mixed
      */
     public function offsetGet($offset)
     {

src/include/class-wc-retailcrm-orders.php

@@ -602,32 +602,6 @@ if (!class_exists('WC_Retailcrm_Orders')) :
             );
         }
 
-        public function processOrderForUpload($orderIds)
-        {
-            $ordersForUpload = [];
-            $errorOrders = [];
-
-            foreach ($orderIds as $orderId) {
-                try {
-                    $this->order = [];
-                    $this->processOrder(wc_get_order($orderId));
-
-                    if ($this->order === []) {
-                        throw new \RuntimeException(sprintf('Order %s is not uploaded', $orderId));
-                    }
-
-                    $ordersForUpload[] = $this->order;
-                } catch (Throwable $exception) {
-                    $errorOrders[$orderId] = sprintf(
-                        'Exception for Order [%s]: %s. Trace: %s',
-                        $orderId, $exception->getMessage(), $exception->getTraceAsString()
-                    );
-                }
-            }
-
-            return [$ordersForUpload, $errorOrders];
-        }
-
         /**
          * Send payment in CRM
          *

src/include/class-wc-retailcrm-uploader.php

@@ -83,11 +83,14 @@ if (class_exists('WC_Retailcrm_Uploader') === false) {
          */
         public function uploadArchiveOrders(?int $page, array $ids = [])
         {
+            WC_Retailcrm_Logger::info(__METHOD__, 'Archive order IDs: ' . implode(', ', $ids));
+
             if (!$this->retailcrm instanceof WC_Retailcrm_Proxy) {
                 return null;
             }
 
             $orderIds = [];
+            $uploadErrors = [];
 
             if (null !== $page) {
                 $orderIds = $this->getCmsOrders($page);
@@ -95,40 +98,14 @@ if (class_exists('WC_Retailcrm_Uploader') === false) {
                 $orderIds = $ids;
             }
 
-            if ($orderIds === []) {
-                return null;
-            }
+            if ($orderIds !== []) {
+                foreach ($orderIds as $orderId) {
+                    $errorMessage = $this->orders->orderCreate($orderId);
 
-            WC_Retailcrm_Logger::info(__METHOD__, 'Archive order IDs: ' . implode(', ', $ids));
-
-            [$ordersForUpload, $uploadErrors] = $this->orders->processOrderForUpload($orderIds);
-
-            try {
-                $response = $this->retailcrm->ordersUpload($ordersForUpload);
-
-                if (!$response->isSuccessful()) {
-                    throw new RuntimeException(
-                        sprintf(
-                            'Failure to upload orders: %s. Status code: %s',
-                            $response->getErrorString(),
-                            $response->getStatusCode()
-                        )
-                    );
+                    if (is_string($errorMessage)) {
+                        $uploadErrors[$orderId] = $errorMessage;
+                    }
                 }
-            } catch (Exception $exception) {
-                WC_Retailcrm_Logger::error(
-                    __METHOD__,
-                    sprintf("Error while uploading orders: %s", $exception->getMessage())
-                );
-
-                return null;
-            }
-
-            /** WP version >= 6 */
-            if (function_exists('wp_cache_flush_runtime')) {
-                wp_cache_flush_runtime();
-            } else {
-                wp_cache_flush();
             }
 
             $this->logOrdersUploadErrors($uploadErrors);
@@ -248,7 +225,7 @@ if (class_exists('WC_Retailcrm_Uploader') === false) {
          */
         private function logOrdersUploadErrors($errors)
         {
-            if ($errors === []) {
+            if (empty($errors) === true) {
                 return;
             }
 

src/include/functions.php

@@ -207,8 +207,3 @@ function isCorporateOrder($wcCustomer, $wcOrder)
 {
     return !empty($wcCustomer->get_billing_company()) || !empty($wcOrder->get_billing_company());
 }
-
-function getOptionByCode($optionName)
-{
-    return get_option(WC_Retailcrm_Base::$option_key)[$optionName] ?? null;
-}

src/readme.txt

@@ -5,7 +5,7 @@ Tags: Интеграция, Simla.com, simla
 Requires PHP: 7.1
 Requires at least: 5.3
 Tested up to: 6.5
-Stable tag: 4.8.21
+Stable tag: 4.8.16
 License: GPLv1 or later
 License URI: http://www.gnu.org/licenses/gpl-1.0.html
 
@@ -82,21 +82,6 @@ Asegúrate de tener una clave API específica para cada tienda. Las siguientes i
 
 
 == Changelog ==
-= 4.8.21 =
-* Fix version of module
-
-= 4.8.20 =
-* Add returned types for methods offsetExists, offsetSet, offsetUnset in WC_Retailcrm_Response
-
-= 4.8.19 =
-* Optimization of order unloading
-
-= 4.8.18 =
-* Added additional parameters to GET requests
-
-= 4.8.17 =
-* Fix deploy
-
 = 4.8.16 =
 * Fix tests svn error
 

src/retailcrm.php

@@ -5,7 +5,7 @@
  * Description: Integration plugin for WooCommerce & Simla.com
  * Author: RetailDriver LLC
  * Author URI: http://retailcrm.pro/
- * Version: 4.8.21
+ * Version: 4.8.16
  * Tested up to: 6.5
  * Requires Plugins: woocommerce
  * WC requires at least: 5.4
@@ -27,7 +27,7 @@ if (!class_exists( 'WC_Integration_Retailcrm')) :
     class WC_Integration_Retailcrm {
         const WOOCOMMERCE_SLUG = 'woocommerce';
         const WOOCOMMERCE_PLUGIN_PATH = 'woocommerce/woocommerce.php';
-        const MODULE_VERSION = '4.8.21';
+        const MODULE_VERSION = '4.8.16';
 
         private static $instance;
 

src/uninstall.php

@@ -16,7 +16,7 @@
  *
  * @link       https://wordpress.org/plugins/woo-retailcrm/
  *
- * @version    4.8.21
+ * @version    4.8.16
  *
  * @package    RetailCRM
  */

tests/test-wc-retailcrm-uploader.php

@@ -36,8 +36,7 @@ class WC_Retailcrm_Uploader_Test extends WC_Retailcrm_Test_Case_Helper
                                   'getCountOrders',
                                   'customersGet',
                                   'customersList',
-                                  'ordersCreate',
-                                  'ordersUpload',
+                                  'ordersCreate'
                               ))
                               ->getMock();
 
@@ -97,10 +96,10 @@ class WC_Retailcrm_Uploader_Test extends WC_Retailcrm_Test_Case_Helper
      * @param $retailcrm
      * @dataProvider dataProviderApiClient
      */
-    public function test_upload_selected_orders($retailcrm)
+    public function test_upload_selected_orders()
     {
         $_GET['order_ids_retailcrm'] = '123, 345, 456';
-        $retailcrm_uploader = $this->getRetailcrmUploader($retailcrm);
+        $retailcrm_uploader = $this->getRetailcrmUploader($this->apiMock);
         $uploadSelectedOrders = $retailcrm_uploader->uploadSelectedOrders();
 
         $this->assertEquals(null, $uploadSelectedOrders);
@@ -124,11 +123,6 @@ class WC_Retailcrm_Uploader_Test extends WC_Retailcrm_Test_Case_Helper
     public function dataProviderApiClient()
     {
         $this->setUp();
-        $this->apiMock
-            ->expects($this->any())
-            ->method('ordersUpload')
-            ->willReturn(new WC_Retailcrm_Response(200, ''))
-        ;
 
         return array(
             array(