caxy / php-htmldiff

lib/Caxy/HtmlDiff/Table/TableDiff.php

Doc Comments +4 added lines, -4 removed lines

@@ -229,8 +229,8 @@ 
 
     /**
      * @param Operation $operation
-     * @param array     $oldRows
-     * @param array     $newRows
+     * @param TableRow[]     $oldRows
+     * @param TableRow[]     $newRows
      * @param array     $appliedRowSpans
      */
     protected function processEqualOperation(Operation $operation, $oldRows, $newRows, &$appliedRowSpans)
@@ -249,8 +249,8 @@ 
 
     /**
      * @param Operation $operation
-     * @param array     $oldRows
-     * @param array     $newRows
+     * @param TableRow[]     $oldRows
+     * @param TableRow[]     $newRows
      * @param array     $appliedRowSpans
      */
     protected function processReplaceOperation(Operation $operation, $oldRows, $newRows, &$appliedRowSpans)

lib/Caxy/HtmlDiff/HtmlDiffConfig.php

Doc Comments +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
     /**
      * @param int $matchThreshold
      *
-     * @return AbstractDiff
+     * @return HtmlDiffConfig
      */
     public function setMatchThreshold($matchThreshold)
     {

lib/Caxy/HtmlDiff/Preprocessor.php

Doc Comments +10 added lines

@@ -4,6 +4,11 @@ 
 
 class Preprocessor
 {
+    /**
+     * @param string $old
+     * @param string $new
+     * @param Util\MbStringUtil $stringUtil
+     */
     public static function diffCommonPrefix($old, $new, $stringUtil)
     {
         // Quick check for common null cases.
@@ -34,6 +39,11 @@ 
         return $pointerMid;
     }
 
+    /**
+     * @param string $old
+     * @param string $new
+     * @param Util\MbStringUtil $stringUtil
+     */
     public static function diffCommonSuffix($old, $new, $stringUtil)
     {
         // Quick check for common null cases.

lib/Caxy/HtmlDiff/MatchingBlock.php

Spacing +1 added lines, -1 removed lines

@@ -30,6 +30,6 @@
 
     public function count() : int
     {
-        return (int)$this->size;
+        return (int) $this->size;
     }
 }

lib/Caxy/HtmlDiff/LcsService.php

Doc Comments +4 added lines, -4 removed lines

@@ -27,8 +27,8 @@ 
     }
 
     /**
-     * @param array $a
-     * @param array $b
+     * @param string[] $a
+     * @param string[] $b
      *
      * @return array
      */
@@ -70,8 +70,8 @@ 
      * @param $c
      * @param $a
      * @param $b
-     * @param $i
-     * @param $j
+     * @param integer $i
+     * @param integer $j
      * @param $matches
      */
     protected function compileMatches($c, $a, $b, $i, $j, &$matches)

lib/Caxy/HtmlDiff/HtmlDiff.php

Doc Comments +6 added lines

@@ -396,6 +396,12 @@
         return $diff->build();
     }
 
+    /**
+     * @param string $oldText
+     * @param string $newText
+     * @param string $attribute
+     * @param string $element
+     */
     protected function diffElementsByAttribute($oldText, $newText, $attribute, $element)
     {
         $oldAttribute = $this->getAttributeFromTag($oldText, $attribute);

Spacing +8 added lines, -8 removed lines

@@ -545,7 +545,7 @@ 
             } else {
                 $firstOrDefault = false;
                 foreach ($this->config->getSpecialCaseOpeningTags() as $x) {
-                    if (preg_match($x, $words[ 0 ])) {
+                    if (preg_match($x, $words[0])) {
                         $firstOrDefault = $x;
                         break;
                     }
@@ -553,13 +553,13 @@ 
                 if ($firstOrDefault) {
                     $specialCaseTagInjection = '<ins class="mod">';
                     if ($tag === 'del') {
-                        unset($words[ 0 ]);
+                        unset($words[0]);
                     }
-                } elseif (array_search($words[ 0 ], $this->config->getSpecialCaseClosingTags()) !== false) {
+                } elseif (array_search($words[0], $this->config->getSpecialCaseClosingTags()) !== false) {
                     $specialCaseTagInjection = '</ins>';
                     $specialCaseTagInjectionIsBefore = true;
                     if ($tag === 'del') {
-                        unset($words[ 0 ]);
+                        unset($words[0]);
                     }
                 }
             }
@@ -567,7 +567,7 @@ 
                 break;
             }
             if ($specialCaseTagInjectionIsBefore) {
-                $this->content .= $specialCaseTagInjection . implode('', $this->extractConsecutiveWords($words, 'tag'));
+                $this->content .= $specialCaseTagInjection.implode('', $this->extractConsecutiveWords($words, 'tag'));
             } else {
                 $workTag = $this->extractConsecutiveWords($words, 'tag');
 
@@ -589,7 +589,7 @@ 
                     }
                 }
 
-                $appendContent = implode('', $workTag) . $specialCaseTagInjection;
+                $appendContent = implode('', $workTag).$specialCaseTagInjection;
 
                 if (isset($workTag[0]) === true && $this->stringUtil->stripos($workTag[0], '<img') !== false) {
                     $appendContent = $this->wrapText($appendContent, $tag, $cssClass);
@@ -765,7 +765,7 @@ 
         $space = $this->stringUtil->strpos($word, ' ', 1);
 
         if ($space > 0) {
-            return '<' . $this->stringUtil->substr($word, 1, $space) . '>';
+            return '<'.$this->stringUtil->substr($word, 1, $space).'>';
         }
 
         return trim($word, '<>');
@@ -782,7 +782,7 @@ 
         for ($indexInOld = $startInOld; $indexInOld < $endInOld; ++$indexInOld) {
             $newMatchLengthAt = [];
 
-            $index = $this->oldWords[ $indexInOld ];
+            $index = $this->oldWords[$indexInOld];
 
             if ($this->isTag($index) === true) {
                 $index = $this->stripTagAttributes($index);

lib/Caxy/HtmlDiff/ListDiffLines.php

Spacing +2 added lines, -2 removed lines

@@ -98,7 +98,7 @@ 
 
     protected function findListNode(DOMDocument $dom) : DOMNode
     {
-        $xPathQuery = '//' . implode('|//', self::LIST_TAG_NAMES);
+        $xPathQuery = '//'.implode('|//', self::LIST_TAG_NAMES);
         $xPath      = new DOMXPath($dom);
         $listNodes  = $xPath->query($xPathQuery);
 
@@ -385,7 +385,7 @@ 
     {
         $bufferDom = new DOMDocument('1.0', 'UTF-8');
 
-        foreach($node->childNodes as $childNode)
+        foreach ($node->childNodes as $childNode)
         {
             $bufferDom->appendChild($bufferDom->importNode($childNode, true));
         }

lib/Caxy/HtmlDiff/AbstractDiff.php

Doc Comments +1 added lines, -1 removed lines

@@ -421,8 +421,8 @@
     }
 
     /**
-     * @param array $characterString
      *
+     * @param string $text
      * @return array
      */
     protected function convertHtmlToListOfWords($text)

Spacing +6 added lines, -6 removed lines

@@ -129,7 +129,7 @@ 
     public function initPurifier($defaultPurifierSerializerCache = null)
     {
         if (null !== $this->purifierConfig) {
-            $HTMLPurifierConfig  = $this->purifierConfig;
+            $HTMLPurifierConfig = $this->purifierConfig;
         } else {
             $HTMLPurifierConfig = HTMLPurifier_Config::createDefault();
         }
@@ -433,7 +433,7 @@ 
         $specialCharacters = '';
 
         foreach ($this->config->getSpecialCaseChars() as $char) {
-            $specialCharacters .= '\\' . $char;
+            $specialCharacters .= '\\'.$char;
         }
 
         // Normalize no-break-spaces to regular spaces
@@ -459,11 +459,11 @@ 
             $lastCharacter  = $sentenceOrHtmlTag[$this->stringUtil->strlen($sentenceOrHtmlTag) - 1];
 
             if ($firstCharacter === ' ' || $firstCharacter === "\r" || $firstCharacter === "\n") {
-                $sentenceOrHtmlTag = ' ' . ltrim($sentenceOrHtmlTag);
+                $sentenceOrHtmlTag = ' '.ltrim($sentenceOrHtmlTag);
             }
 
             if ($lastCharacter === ' ' || $lastCharacter === "\r" || $lastCharacter === "\n") {
-                $sentenceOrHtmlTag = rtrim($sentenceOrHtmlTag) . ' ';
+                $sentenceOrHtmlTag = rtrim($sentenceOrHtmlTag).' ';
             }
 
             $sentenceSplitIntoWords = [];
@@ -477,8 +477,8 @@ 
             // Third group is alpha numeric words including special characters
             // Fourth group is a catch all of everything else.
             preg_match_all(
-                '/\s|[a-zA-Z0-9' . $specialCharacters . '\pL]+(?=[' . $specialCharacters . '][\s\r\n])|[a-zA-Z0-9' . $specialCharacters . '\pL]+|[^\s]/mu',
-                $sentenceOrHtmlTag . ' ', // Inject a space at the end to make sure the second regex always hits.
+                '/\s|[a-zA-Z0-9'.$specialCharacters.'\pL]+(?=['.$specialCharacters.'][\s\r\n])|[a-zA-Z0-9'.$specialCharacters.'\pL]+|[^\s]/mu',
+                $sentenceOrHtmlTag.' ', // Inject a space at the end to make sure the second regex always hits.
                 $sentenceSplitIntoWords,
                 PREG_SPLIT_NO_EMPTY
             );