Drop unused hooks and translate russian comments

arrilot · arrilot · commit ce7c9f15b244 · 2021-07-22T01:39:11.000+03:00
diff --git a/.git_hooks/post-checkout/clear-cache.sh b/.git_hooks/post-checkout/clear-cache.sh
diff --git a/.git_hooks/post-checkout/install-dependecies.sh b/.git_hooks/post-checkout/install-dependecies.sh
@@ -1,9 +1,5 @@
 #!/bin/bash
 
-# В данном хуке выполняется 
-# - composer install если был изменен composer.lock
-# - npm install если был изменен package-lock.json
-
 ESC_SEQ="\x1b["
 COL_RESET=$ESC_SEQ"39;49;00m"
 COL_RED=$ESC_SEQ"0;31m"
diff --git a/.git_hooks/post-merge/clear-cache.sh b/.git_hooks/post-merge/clear-cache.sh
diff --git a/.git_hooks/post-merge/install-dependecies.sh b/.git_hooks/post-merge/install-dependecies.sh
@@ -1,9 +1,5 @@
 #!/bin/bash
 
-# В данном хуке выполняется 
-# - composer install если был изменен composer.lock
-# - npm install если был изменен package-lock.json
-
 ESC_SEQ="\x1b["
 COL_RESET=$ESC_SEQ"39;49;00m"
 COL_RED=$ESC_SEQ"0;31m"
diff --git a/.git_hooks/pre-commit/lint-openapi.sh b/.git_hooks/pre-commit/lint-openapi.sh
diff --git a/.git_hooks/pre-commit/lint-php.sh b/.git_hooks/pre-commit/lint-php.sh
@@ -1,7 +1,5 @@
 #!/bin/bash
 
-# В данном хуке выполняется линтинг всех добавленных в коммит php файлов через php -l
-
 ROOT_DIR="$(pwd)/"
 LIST=$(git diff-index --cached --name-only --diff-filter=ACMR HEAD)
 ERRORS_BUFFER=""
diff --git a/.git_hooks/pre-commit/php-cs-fixer.sh b/.git_hooks/pre-commit/php-cs-fixer.sh
@@ -1,7 +1,5 @@
 #!/bin/bash
 
-# В данном хуке выполняется приведение код-стайла в соответствие с конфигом .php_cs
-
 EXECUTABLE_NAME=php-cs-fixer
 EXECUTABLE_COMMAND=fix
 CONFIG_FILE=.php_cs
diff --git a/.git_hooks/pre-push/composer-validate.sh b/.git_hooks/pre-push/composer-validate.sh
@@ -1,6 +1,6 @@
 #!/bin/bash
 
-# В данном хуке выполняется валидация composer.json перед коммитом
+# validate composer 
 
 ESC_SEQ="\x1b["
 COL_RESET=$ESC_SEQ"39;49;00m"
diff --git a/.git_hooks/pre-push/var-dump-checker.sh b/.git_hooks/pre-push/var-dump-checker.sh
@@ -1,7 +1,5 @@
 #!/bin/bash
 
-# В данном хуке выполняется проверка файлов проекта на dd, var_dump и прочие подобные отладочные функции через var-dump-check
-
 ESC_SEQ="\x1b["
 COL_RESET=$ESC_SEQ"39;49;00m"
 COL_RED=$ESC_SEQ"0;31m"
diff --git a/.huskyrc.json b/.huskyrc.json
@@ -1,8 +1,8 @@
 {
     "hooks": {
-        "post-checkout": ".git_hooks/post-checkout/install-dependecies.sh && .git_hooks/post-checkout/clear-cache.sh",
-        "post-merge": ".git_hooks/post-merge/install-dependecies.sh && .git_hooks/post-merge/clear-cache.sh",
-        "pre-commit": ".git_hooks/pre-commit/lint-php.sh && .git_hooks/pre-commit/php-cs-fixer.sh && .git_hooks/pre-commit/lint-openapi.sh",
+        "post-checkout": ".git_hooks/post-checkout/install-dependecies.sh",
+        "post-merge": ".git_hooks/post-merge/install-dependecies.sh",
+        "pre-commit": ".git_hooks/pre-commit/lint-php.sh && .git_hooks/pre-commit/php-cs-fixer.sh",
         "pre-push": ".git_hooks/pre-push/composer-validate.sh && .git_hooks/pre-push/var-dump-checker.sh && composer test"
     }
 }
diff --git a/config/config.php b/config/config.php
@@ -2,7 +2,6 @@
 
 return [
     'connection' => [
-
         /*
          * Elasticsearch hosts in format http[s]://[user][:pass]@hostname[:9200]
          */
diff --git a/src/Raw/Contracts/Criteria.php b/src/Raw/Contracts/Criteria.php
@@ -2,9 +2,6 @@
 
 namespace Greensight\LaravelElasticQuery\Raw\Contracts;
 
-/**
- * Интерфейс-маркер для условий отбора.
- */
 interface Criteria extends DSLAware
 {
 }
diff --git a/src/Raw/Contracts/SearchIndex.php b/src/Raw/Contracts/SearchIndex.php
@@ -5,14 +5,14 @@
 interface SearchIndex
 {
     /**
-     * Возвращает имя поля, содержащего уникальные значения в пределах индекса.
+     * Returns the name of attribute with unique values in index scope.
      *
      * @return string
      */
     public function tiebreaker(): string;
 
     /**
-     * Выполняет поисковый запрос.
+     * Perform search query.
      *
      * @param array $dsl
      * @return array
diff --git a/src/Raw/Debug/QueryLog.php b/src/Raw/Debug/QueryLog.php
@@ -16,7 +16,7 @@ public function __construct()
 
     public function log(string $indexName, array $query): void
     {
-        // Используем метку времени, чтобы исключить влияние setTestNow
+        // Here we use timestamp to disable influence of setTestNow
         $timestamp = Date::createFromTimestamp(microtime());
 
         $record = new QueryLogRecord($indexName, $query, $timestamp);
diff --git a/src/Raw/Search/SearchQuery.php b/src/Raw/Search/SearchQuery.php
@@ -67,7 +67,7 @@ public function cursorPaginate(int $size, ?string $cursor = null): CursorPage
         $current = Cursor::decode($cursor) ?? Cursor::BOF();
 
         if (!$sorts->matchCursor($current)) {
-            throw new InvalidArgumentException('Курсор не относится к текущей сортировке');
+            throw new InvalidArgumentException('Cursor is not suitable for current sort');
         }
 
         $response = $this->execute($sorts, $size, cursor: $current);
diff --git a/src/Raw/Search/Sorting/SortCollection.php b/src/Raw/Search/Sorting/SortCollection.php
@@ -28,7 +28,7 @@ public function toDSL(): array
     public function add(Sort $sort): void
     {
         $field = $sort->field();
-        Assert::false($this->items->has($field), "По полю \"$field\" уже задана сортировка");
+        Assert::false($this->items->has($field), "Field \"$field\" is already sorted");
 
         $this->items->put($field, $sort);
     }
@@ -80,7 +80,7 @@ public function createCursor(?array $hit): ?Cursor
         $values = $hit['sort'] ?? [];
 
         if (count($values) !== $this->items->count()) {
-            throw new InvalidArgumentException('Количество полей сортировки не совпадает с курсором');
+            throw new InvalidArgumentException('Sort fields count do not match cursor');
         }
 
         return new Cursor(
diff --git a/tests/Functional/Raw/Aggregating/AggregationQueryTest.php b/tests/Functional/Raw/Aggregating/AggregationQueryTest.php
@@ -24,7 +24,7 @@ protected function setUp(): void
     public function testGet(): void
     {
         $this->testing
-            ->where('package', 'бутылка')
+            ->where('package', 'bottle')
             ->terms('codes', 'code')
             ->nested(
                 'offers',
@@ -43,7 +43,7 @@ public function testGet(): void
     public function testComposite(): void
     {
         $this->testing->composite(function (AggregationsBuilder $builder) {
-            $builder->where('package', 'бутылка')
+            $builder->where('package', 'bottle')
                 ->terms('codes', 'code');
         });
 
diff --git a/tests/Seeds/fixtures/products_default.json b/tests/Seeds/fixtures/products_default.json
@@ -2,12 +2,12 @@
   "1": {
     "product_id": 1,
     "active": true,
-    "name": "Телевизор",
+    "name": "TV station",
     "code": "tv",
     "rating": 7,
     "package": null,
-    "description": "Изображение на экране телевизора поражает своей реалистичностью, высокой детализацией и глубоким цветом.",
-    "tags": ["видео"],
+    "description": "TV station description",
+    "tags": ["video"],
     "offers": [
       {"seller_id": 10, "active": true, "price": 20000.0},
       {"seller_id": 15, "active": true, "price": 21000.0},
@@ -17,12 +17,12 @@
   "150": {
     "product_id": 150,
     "active": true,
-    "name": "Вода природная TASSAY негазированная, 2 шт. по 5 л",
+    "name": "TASSAY water",
     "code": "water",
     "rating": 10,
-    "package": "бутылка",
-    "description": "TASSAY - природная питьевая вода из глубинного подземного источника в предгорьях Тянь-Шаня, защищенного от техногенного воздействия самой природой.",
-    "tags": ["напитки", "вода"],
+    "package": "bottle",
+    "description": "TASSAY description water",
+    "tags": ["drinks", "watter"],
     "offers": [
       {"seller_id": 10, "active": false, "price": 168.0},
       {"seller_id": 15, "active": true, "price": 210.0},
@@ -32,11 +32,11 @@
   "319": {
     "product_id": 319,
     "active": false,
-    "name": "Перчатки кожаные",
+    "name": "Leather gloes",
     "code": "gloves",
     "rating": 0,
-    "description": "Ткань Marmot MemBrain обеспечивает водонепроницаемость, воздухопроницаемость и защиту от ветра. Легкий и компактный утеплитель.",
-    "tags": ["одежда", "перчатки"],
+    "description": "L gloves descriptions",
+    "tags": ["clothes", "gloves"],
     "offers": [
       {"seller_id": 10, "active": false, "price": 980.0},
       {"seller_id": 15, "active": false, "price": 980.0},
@@ -46,12 +46,12 @@
   "328": {
     "product_id": 328,
     "active": true,
-    "name": "4K UHD Телевизор Philips 43PUS7805/60 43",
+    "name": "4K UHD TV Philips 43PUS7805/60 43",
     "code": "4k-uhd-televizor-philips-43pus7805-60-43-196797693",
     "rating": 5,
     "package": null,
-    "description": "С Philips Ambilight каждый момент становится ближе. Интеллектуальные светодиоды по краям телевизора реагируют на происходящее на экране и изменяют цвет для еще большего эффекта погружения.",
-    "tags": ["видео"],
+    "description": "Nice tv",
+    "tags": ["video"],
     "offers": [
       {"seller_id": 90, "active": true, "price": 41999.0},
       {"seller_id": 20, "active": true, "price": 28990.0}
@@ -60,12 +60,12 @@
   "405": {
     "product_id": 405,
     "active": true,
-    "name": "Минеральная вода San Pellegrino, газированная, 1 л х 6 шт",
+    "name": "San Pellegrino, 1 l x 6 pieces",
     "code": "voda-san-pellegrino-mineralnaya-gazirovannaya",
     "rating": 8,
-    "package": "бутылка",
-    "description": "S.Pellegrino - минеральная вода натуральной газации, которая добывается из термальных источников, расположенных а предгорье Альп на севере Италии.",
-    "tags": ["напитки", "вода"],
+    "package": "bottle",
+    "description": "S.Pellegrino from Dolomites",
+    "tags": ["drinks", "water"],
     "offers": [
       {"seller_id": 10, "active": false, "price": 611.0},
       {"seller_id": 15, "active": false, "price": 620.0}
@@ -74,11 +74,11 @@
   "471": {
     "product_id": 471,
     "active": true,
-    "name": "Перчатки VITTOVAR",
+    "name": "Gloves VITTOVAR",
     "code": "perchatki-s-zashchitoy",
     "rating": 3,
-    "description": "Поверхность с уникальной дышащей технологией. Защита-амортизация на костяшках пальцев.",
-    "tags": ["одежда", "перчатки"],
+    "description": "VITTOVAR description",
+    "tags": ["clothes", "gloves"],
     "offers": [
     ]
   }

Original file line number	Diff line number	Diff line change
`@@ -1,8 +1,8 @@`
`1`	`1`	`{`
`2`	`2`	`"hooks": {`
`3`		`- "post-checkout": ".git_hooks/post-checkout/install-dependecies.sh && .git_hooks/post-checkout/clear-cache.sh",`
`4`		`- "post-merge": ".git_hooks/post-merge/install-dependecies.sh && .git_hooks/post-merge/clear-cache.sh",`
`5`		`- "pre-commit": ".git_hooks/pre-commit/lint-php.sh && .git_hooks/pre-commit/php-cs-fixer.sh && .git_hooks/pre-commit/lint-openapi.sh",`
	`3`	`+ "post-checkout": ".git_hooks/post-checkout/install-dependecies.sh",`
	`4`	`+ "post-merge": ".git_hooks/post-merge/install-dependecies.sh",`
	`5`	`+ "pre-commit": ".git_hooks/pre-commit/lint-php.sh && .git_hooks/pre-commit/php-cs-fixer.sh",`
`6`	`6`	`"pre-push": ".git_hooks/pre-push/composer-validate.sh && .git_hooks/pre-push/var-dump-checker.sh && composer test"`
`7`	`7`	`}`
`8`	`8`	`}`