fix remove html entity in DelDouble (#11)

CHANGELOG.md

@@ -1,12 +1,15 @@
 # Changelog
-## [Unreleased](https://github.com/akhx/typograf/compare/v0.4.8...HEAD)
+## [Unreleased](https://github.com/akhx/typograf/compare/v0.4.9...HEAD)
 
-## [0.4.7](https://github.com/akhx/typograf/compare/v0.4.7...v0.4.8) - 2023-02-15
+## [0.4.9](https://github.com/akhx/typograf/compare/v0.4.8...v0.4.9) - 2023-04-21
+* fix remove html entity in DelDouble
+
+## [0.4.8](https://github.com/akhx/typograf/compare/v0.4.7...v0.4.8) - 2023-02-15
 * fix README
 * debug mode
 * replace quote to html entity
 
-## [0.4.6](https://github.com/akhx/typograf/compare/v0.4.6...v0.4.7) - 2022-12-08
+## [0.4.7](https://github.com/akhx/typograf/compare/v0.4.6...v0.4.7) - 2022-12-08
 * fix izZaPod
 
 ## [0.4.6](https://github.com/akhx/typograf/compare/v0.4.5...v0.4.6) - 2022-06-23

src/Rule/Punctuation/DelDouble.php

@@ -6,7 +6,7 @@
 
 class DelDouble extends AbstractRule
 {
-    public $name = 'Лишние знаки припинания';
+    public $name = 'Лишние знаки препинания';
 
     protected $settings = [
         'maxLenMark' => 3,
@@ -15,16 +15,20 @@ class DelDouble extends AbstractRule
     public function handler(string $text): string
     {
         $pattern = [
-            '#([,:;]){2,}#iu',
-            '#([.]){4,}#iu',
-            '#([!]){' . ($this->settings['maxLenMark'] + 1) . ',}#iu',
-            '#([?]){' . ($this->settings['maxLenMark'] + 1) . ',}#iu',
+            '#(,){2,}#iu',
+            '#(:){2,}#iu',
+            '#(&\w+;;);+|((^|\s)(\w+;));+#iu',
+            '#(\.){4,}#iu',
+            '#(!){' . ($this->settings['maxLenMark'] + 1) . ',}#iu',
+            '#(\?){' . ($this->settings['maxLenMark'] + 1) . ',}#iu',
             '#(^|[^!])!{2}($|[^!])#iu',
             '#(^|[^?])\?{2}($|[^?])#iu',
         ];
 
         $replace = [
             '$1',
+            '$1',
+            '$1$2',
             '$1$1$1',
             str_repeat('$1', $this->settings['maxLenMark']),
             str_repeat('$1', $this->settings['maxLenMark']),

src/Typograf.php

@@ -126,7 +126,7 @@ protected function initRules(): void
 
         foreach ($all as $ruleClass) {
             /**
-             * @var Rule\AbstractRule
+             * @var Rule\AbstractRule $ruleObj
              */
             $ruleObj = new $ruleClass();
             $this->rules[$ruleClass] = $ruleObj;

tests/Rule/Punctuation/DelDoubleTest.php

@@ -23,12 +23,12 @@ public function dataProvider(): array
     {
         return [
             [
-                'У меня была только синяя краска;; но,, несмотря на это,, я затеял нарисовать охоту.',
-                'У меня была только синяя краска; но, несмотря на это, я затеял нарисовать охоту.',
+                'У меня была только синяя краска;; но,, несмотря на это,,, я,,,, затеял нарисовать охоту.',
+                'У меня была только синяя краска; но, несмотря на это, я, затеял нарисовать охоту.',
             ],
             [
-                'Никогда не отказывайся от малого в работе:: из малого строится великое.',
-                'Никогда не отказывайся от малого в работе: из малого строится великое.',
+                'Никогда;;; не отказывайся от малого в работе:: из малого::: строится великое.',
+                'Никогда; не отказывайся от малого в работе: из малого: строится великое.',
             ],
             [
                 'Что??????',
@@ -54,6 +54,26 @@ public function dataProvider(): array
                 'Опа!?????',
                 'Опа!???',
             ],
+            [
+                'Привет;;',
+                'Привет;',
+            ],
+            [
+                'Привет;;;',
+                'Привет;',
+            ],
+            [
+                '«Привет», мир!',
+                '«Привет», мир!',
+            ],
+            [
+                '- «Привет1»;',
+                '- «Привет1»;',
+            ],
+            [
+                '- «Привет2»;;',
+                '- «Привет2»;',
+            ],
         ];
     }
 }