Merge branch 'master' into dev

Исправление багов при отправке сырого JSON для чека коррекции с исправлениями

.github/FUNDING.yml

@@ -1,3 +1,3 @@
 github: # Replace with up to 4 GitHub Sponsors-enabled usernames e.g., [user1, user2]
 patreon: # Replace with a single Patreon username
-custom: ['https://money.yandex.ru/to/41001685237530']
+custom: ['https://yoomoney.ru/to/41001685237530']

.github/workflows/dev.yml

@@ -0,0 +1,25 @@
+name: Dev build
+
+on:
+  push:
+    branches: [ dev ]
+  pull_request:
+    branches: [ dev ]
+
+jobs:
+  Tests:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Validate composer.json and composer.lock
+        uses: php-actions/composer@40-env
+        with:
+          version: 2
+          php_version: 7.4
+          only_args: --prefer-dist --no-progress
+
+      - name: Run phpunit tests
+        uses: php-actions/phpunit@v8
+        with:
+          configuration: ./phpunit.xml

.github/workflows/master.yml

@@ -0,0 +1,25 @@
+name: Master build
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  Tests:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Validate composer.json and composer.lock
+        uses: php-actions/composer@40-env
+        with:
+          version: 2
+          php_version: 7.4
+          only_args: --prefer-dist --no-progress
+
+      - name: Run phpunit tests
+        uses: php-actions/phpunit@v8
+        with:
+          configuration: ./phpunit.xml

.github/workflows/tests.yml

@@ -1,26 +0,0 @@
-name: Build
-
-on:
-  push:
-    branches:
-      - master
-
-jobs:
-  build:
-
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v1
-
-      - name: Validate composer.json and composer.lock
-        run: composer validate
-
-      - name: Install composer dependencies
-        run: composer install --prefer-dist --no-progress --no-suggest
-
-      - name: Run phpunit tests
-        uses: php-actions/phpunit@v1.0.0
-        with:
-          # Configuration file location
-          config: ./phpunit.xml

README.md

@@ -10,7 +10,7 @@
 
 ## Системные требования
 
-* PHP 7.2+
+* PHP 7.4+
 * composer
 * php-json
 
@@ -103,8 +103,6 @@
 
 ## Дополнительные ресурсы
 
-**Discord-сервер для обсуждения этой библиотеки: [discord.gg/mFYTQmp](https://discord.gg/mFYTQmp)**
-
 Функционал, находящийся в разработке: [ROADMAP.md](ROADMAP.md)
 
 Официальные ресурсы АТОЛ Онлайн:

ROADMAP.md

@@ -25,7 +25,8 @@
 - [ ] Тесты для регистрации документа расхода
 - [ ] Тесты для регистрации документа возврата расхода
 - [ ] Тесты для регистрации документа коррекции расхода
-- [ ] Вообще все расчёты вообще везде должны быть строго в копейках. Рубли (дроби) должны быть только в JSON-представлениях
+- [ ] Вообще все расчёты вообще везде должны быть строго в копейках.  
+  Рубли (дроби) должны быть только в JSON-представлениях
 
 ## Поддержка методов API (регистрация документов)
 
@@ -63,18 +64,3 @@
 - [x] Пoддержка `correction.vats` (обязательный)
 - [x] Пoддержка `correction.correction_info` (обязательный)
 - [x] Пoддержка `correction.cashier`
-
-## Не будут реализовываться
-
-### Валидация генерируемых документов согласно актуальной схемы API
-
-- Валидатор схемы для документов прихода, возврата прихода, расхода, возврата расхода
-- Валидатор схемы для документов коррекции прихода, коррекции расхода
-
-1. Отказ обусловлен скоростью выполнения. 
-Базовая реализация, которая была начата, подразумевала синглтон, который кешировал однажды полученную схему.
-Практика показала, что этот единичный запрос может существенно тормозить работу сервера и в течение долгого времени
-не отдавать ответ клиенту.
-
-2. Такая валидация подходит в том случае, если бы при разработке использовалась концепция IoC.
-До версии пакета 2.0.0 таких серьёзных имзенений не планируется.

composer.json

@@ -5,16 +5,21 @@
     "type": "library",
     "keywords": [
         "54-fz",
+        "54-фз",
         "kkt",
+        "ккт",
         "e-commerce",
         "cash",
         "cash register",
         "payment",
         "payment system",
         "atol",
-        "atol-online"
+        "атол",
+        "atol-online",
+        "атол онлайн"
     ],
     "homepage": "https://github.com/anthonyaxenov/atol-online",
+    "readme": "https://github.com/anthonyaxenov/atol-online/blob/master/README.md",
     "authors": [
         {
             "name": "Anthony Axenov",
@@ -25,10 +30,16 @@
     "support": {
         "source": "https://github.com/anthonyaxenov/atol-online",
         "issues": "https://github.com/anthonyaxenov/atol-online/issues",
-        "chat": "https://discord.gg/mFYTQmp"
+        "docs": "https://github.com/anthonyaxenov/atol-online/blob/master/docs/readme.md"
     },
+    "funding": [
+        {
+            "type": "Yoomoney",
+            "url": "https://yoomoney.ru/to/41001685237530"
+        }
+    ],
     "require": {
-        "php": ">=7.2",
+        "php": ">=7.4",
         "ext-json": "*",
         "guzzlehttp/guzzle": "^6.5",
         "psr/log": "^1.1",

src/AtolOnline/Api/Kkt.php

@@ -183,8 +183,8 @@ class Kkt extends Client
     {
         if (valid_strlen($url) > Constraints::MAX_LENGTH_CALLBACK_URL) {
             throw new AtolCallbackUrlTooLongException($url, Constraints::MAX_LENGTH_CALLBACK_URL);
-        } elseif (preg_match(Constraints::PATTERN_CALLBACK_URL, $url)) {
-            throw new AtolInvalidCallbackUrlException();
+        } elseif (!preg_match(Constraints::PATTERN_CALLBACK_URL, $url)) {
+            throw new AtolInvalidCallbackUrlException('Callback URL not matches with pattern');
         }
         $this->kkt_config[$this->isTestMode() ? 'test' : 'prod']['callback_url'] = $url;
         return $this;

src/AtolOnline/Constants/Constraints.php

@@ -72,5 +72,5 @@ class Constraints
     /**
      * Регулярное выражание для валидации строки Callback URL
      */
-    const PATTERN_CALLBACK_URL = "^http(s?)\:\/\/[0-9a-zA-Zа-яА-Я]([-.\w]*[0-9a-zA-Zа-яА-Я])*(:(0-9)*)*(\/?)([a-zAZ0-9а-яА-Я\-\.\?\,\'\/\\\+&=%\$#_]*)?$";
+    const PATTERN_CALLBACK_URL = "/^http(s?)\:\/\/[0-9a-zA-Zа-яА-Я]([-.\w]*[0-9a-zA-Zа-яА-Я])*(:(0-9)*)*(\/?)([a-zAZ0-9а-яА-Я\-\.\?\,\'\/\\\+&=%\$#_]*)?$/";
 }

src/AtolOnline/Entities/CorrectionInfo.php

@@ -68,7 +68,7 @@ class CorrectionInfo extends Entity
      */
     public function getNumber(): ?string
     {
-        return $this->base_name;
+        return $this->base_number;
     }
     
     /**
@@ -165,7 +165,7 @@ class CorrectionInfo extends Entity
             'type' => $this->getType() ?? '', // обязателен
             'base_date' => $this->getDate() ?? '', // обязателен
             'base_number' => $this->getNumber() ?? '', // обязателен
-            'base_name' => $this->getName() ?? '' // обязателен
+            'base_name' => $this->getName() ?? '' // не обязателен
         ];
     }
-}
+}

src/AtolOnline/Entities/Document.php

@@ -362,6 +362,14 @@ class Document extends Entity
                 $array['client']['inn'] ?? null
             ));
         }
+        if (isset($array['correction_info'])) {
+            $doc->setCorrectionInfo(new CorrectionInfo(
+                $array['correction_info']['type'] ?? null,
+                $array['correction_info']['base_date'] ?? null,
+                $array['correction_info']['base_number'] ?? null,
+                $array['correction_info']['base_name'] ?? null,
+            ));
+        }
         if (isset($array['items'])) {
             foreach ($array['items'] as $ar_item) {
                 $item = new Item(
@@ -391,6 +399,18 @@ class Document extends Entity
                 $doc->payments->add($payment);
             }
         }
+        if (isset($array['vats'])) {
+            foreach ($array['vats'] as $vat_payment) {
+                $vat = new Vat();
+                if (isset($vat_payment['type'])) {
+                    $vat->setType($vat_payment['type']);
+                }
+                if (isset($vat_payment['sum'])) {
+                    $vat->setSum($vat_payment['sum']);
+                }
+                $doc->vats->add($vat);
+            }
+        }
         if (isset($array['total']) && $array['total'] != $doc->calcTotal()) {
             throw new AtolException('Real total sum not equals to provided in JSON one');
         }

src/AtolOnline/Entities/Vat.php

@@ -50,7 +50,8 @@ class Vat extends Entity
     }
     
     /**
-     * Устанавливает размер НДС от суммы в копейках
+     * Устанавливает:
+     * размер НДС от суммы в копейках
      *
      * @param string $type   Тип ставки НДС
      * @param int    $kopeks Копейки

src/AtolOnline/Exceptions/AtolAuthFailedException.php

@@ -33,9 +33,9 @@ class AtolAuthFailedException extends Exception
         $message = $last_response->isValid()
             ? $message
             : '['.$last_response->error->code.'] '.$last_response->error->text.
-            '. ERROR_ID: '.$last_response->error->error_ID.
+            '. ERROR_ID: '.$last_response->error->error_id.
             '. TYPE: '.$last_response->error->type;
         $code = $last_response->isValid() ? $code : $last_response->error->code;
         parent::__construct($message, $code, $previous);
     }
-}
+}

src/AtolOnline/Exceptions/AtolException.php

@@ -33,6 +33,7 @@ class AtolException extends Exception
      */
     public function __construct($message = "", $code = 0, Throwable $previous = null)
     {
+        $message = $message ?: $this->message;
         if ($this->getFfdTags()) {
             $message .= ' [FFD tags: '.implode(', ', $this->getFfdTags()).']';
         }