seboettg / citeproc-php

src/Seboettg/CiteProc/Rendering/Date/Date.php

Doc Comments +5 added lines, -3 removed lines

@@ -254,7 +254,7 @@ 
     }
 
     /**
-     * @param $differentParts
+     * @param integer $differentParts
      * @param DateTime $from
      * @param DateTime $to
      * @param $delim
@@ -406,8 +406,8 @@ 
 
     /**
      * @param $datePart
-     * @param $from
-     * @param $to
+     * @param DateTime $from
+     * @param DateTime $to
      * @param $delim
      * @return string
      */
@@ -421,6 +421,7 @@ 
     }
 
     /**
+     * @param string $format
      * @return bool
      */
     private function hasDatePartsFromLocales($format)
@@ -430,6 +431,7 @@ 
     }
 
     /**
+     * @param string $format
      * @return array
      */
     private function getDatePartsFromLocales($format)

Spacing +10 added lines, -10 removed lines

@@ -142,7 +142,7 @@ 
         if ($this->dateParts->count() < 1 && in_array($form, self::$localizedDateFormats)) {
             if ($this->hasDatePartsFromLocales($form)) {
                 $datePartsFromLocales = $this->getDatePartsFromLocales($form);
-                array_filter($datePartsFromLocales, function (\SimpleXMLElement $item) use ($dateParts) {
+                array_filter($datePartsFromLocales, function(\SimpleXMLElement $item) use ($dateParts) {
                     return in_array($item["name"], $dateParts);
                 });
 
@@ -212,8 +212,8 @@ 
                 $ret = $this->renderDateRange($toRender, $from, $to, $delim);
             }
 
-            if (isset($var->raw) && preg_match("/(\p{L}+)\s?([\-\-\&,])\s?(\p{L}+)/u", $var->raw, $matches)){
-                return $matches[1].$matches[2].$matches[3];
+            if (isset($var->raw) && preg_match("/(\p{L}+)\s?([\-\-\&,])\s?(\p{L}+)/u", $var->raw, $matches)) {
+                return $matches[1] . $matches[2] . $matches[3];
             }
         }
         // fallback:
@@ -242,7 +242,7 @@ 
         $data = [];
         foreach ($dates as $date) {
             if ($date[0] < 1000) {
-                $dateTime = new DateTime(0,0,0);
+                $dateTime = new DateTime(0, 0, 0);
                 $dateTime->setDay(0)->setMonth(0)->setYear(0);
                 $data[] = $dateTime;
             }
@@ -347,7 +347,7 @@ 
                 $dp = $this->dateParts->toArray();
                 $i = 0;
                 $dateParts_ = [];
-                array_walk($dp, function ($datePart, $key) use (&$i, &$dateParts_, $differentParts) {
+                array_walk($dp, function($datePart, $key) use (&$i, &$dateParts_, $differentParts) {
                     if (strpos($key, "year") !== false || strpos($key, "month") !== false) {
                         $dateParts_["yearmonth"][] = $datePart;
                     }
@@ -360,7 +360,7 @@ 
                 $dp = $this->dateParts->toArray();
                 $i = 0;
                 $dateParts_ = [];
-                array_walk($dp, function ($datePart, $key) use (&$i, &$dateParts_, $differentParts) {
+                array_walk($dp, function($datePart, $key) use (&$i, &$dateParts_, $differentParts) {
                     if (strpos($key, "year") !== false || strpos($key, "day") !== false) {
                         $dateParts_["yearday"][] = $datePart;
                     }
@@ -373,7 +373,7 @@ 
                 $dp = $this->dateParts->toArray();
                 $i = 0;
                 $dateParts_ = [];
-                array_walk($dp, function ($datePart, $key) use (&$i, &$dateParts_, $differentParts) {
+                array_walk($dp, function($datePart, $key) use (&$i, &$dateParts_, $differentParts) {
                     //$bit = sprintf("%03d", decbin($differentParts));
                     if (strpos($key, "month") !== false || strpos($key, "day") !== false) {
                         $dateParts_["monthday"][] = $datePart;
@@ -423,7 +423,7 @@ 
         $from = $datePart->renderWithoutAffixes($from, $this);
         $to = $datePart->renderWithoutAffixes($to, $this);
         $suffix = !empty($to) ? $datePart->renderSuffix() : "";
-        return $prefix.$from.$delim.$to.$suffix;
+        return $prefix . $from . $delim . $to . $suffix;
     }
 
     /**
@@ -448,9 +448,9 @@ 
         // no custom date parts within the date element (this)?
         if (!empty($dateFromLocale)) {
 
-            $dateForm = array_filter(is_array($dateFromLocale) ? $dateFromLocale : [$dateFromLocale], function ($element) use ($format) {
+            $dateForm = array_filter(is_array($dateFromLocale) ? $dateFromLocale : [$dateFromLocale], function($element) use ($format) {
                 /** @var \SimpleXMLElement $element */
-                $dateForm = (string)$element->attributes()["form"];
+                $dateForm = (string) $element->attributes()["form"];
                 return $dateForm === $format;
             });
 

src/Seboettg/CiteProc/Rendering/Layout.php

Spacing +2 added lines, -3 removed lines

@@ -73,8 +73,7 @@ 
 
         if (CiteProc::getContext()->isModeBibliography()) {
             if ($data instanceof DataList) {
-                foreach ($data as $citationNumber => $item) {
-                    ++self::$numberOfCitedItems;
+                foreach ($data as $citationNumber => $item) {++self::$numberOfCitedItems;
                     CiteProc::getContext()->getResults()->append($this->wrapBibEntry($this->renderSingle($item, $citationNumber)));
                 }
                 $ret .= implode($this->delimiter, CiteProc::getContext()->getResults()->toArray());
@@ -82,7 +81,7 @@ 
                 $ret .= $this->wrapBibEntry($this->renderSingle($data, $citationNumber));
             }
 
-            return "<div class=\"csl-bib-body\">".$ret."\n</div>";
+            return "<div class=\"csl-bib-body\">" . $ret . "\n</div>";
 
         } else if (CiteProc::getContext()->isModeCitation()) {
             if (is_array($data) || $data instanceof DataList) {

Doc Comments +3 added lines

@@ -127,6 +127,9 @@
         return self::$numberOfCitedItems;
     }
 
+    /**
+     * @param string $value
+     */
     private function wrapBibEntry($value)
     {
         return "\n  <div class=\"csl-entry\">" . $value . "</div>";

src/Seboettg/CiteProc/Rendering/Name/Name.php

Braces +1 added lines, -2 removed lines

@@ -146,8 +146,7 @@
                         $resultNames[] = $this->formatName($name, $rank);
                     }
                 }
-            }
-            else {
+            } else {
                 $resultNames[] = $this->formatName($name, $rank);
             }
         }

Doc Comments +3 added lines

@@ -99,6 +99,9 @@
         $this->initDelimiterAttributes($node);
     }
 
+    /**
+     * @param integer $citationNumber
+     */
     public function render($data, $citationNumber = null)
     {
         if (!$this->attributesInitialized) {

Spacing +4 added lines, -4 removed lines

@@ -113,11 +113,11 @@ 
         $lastName = null;
         $resultNames = [];
         $count = 0;
-        $hasPreceding = CiteProc::getContext()->getCitationItems()->hasKey($citationNumber-1);
+        $hasPreceding = CiteProc::getContext()->getCitationItems()->hasKey($citationNumber - 1);
 
         if ($hasPreceding) {
             /** @var \stdClass $preceding */
-            $preceding = CiteProc::getContext()->getCitationItems()->get($citationNumber-1);
+            $preceding = CiteProc::getContext()->getCitationItems()->get($citationNumber - 1);
         }
 
         $subsequentSubstitution = CiteProc::getContext()->getCitationItems()->getSubsequentAuthorSubstitute();
@@ -215,7 +215,7 @@ 
         which allows for advanced sorting. */
 
         if ($this->form == 'count') {
-            return (int)count($resultNames);
+            return (int) count($resultNames);
         }
 
         return $text;
@@ -234,7 +234,7 @@ 
         if (count($this->nameParts) > 0) {
             /** @var NamePart $namePart */
             foreach ($this->nameParts as $namePart) {
-                $nameObj->{$namePart->getName()} =   $namePart->render($name);
+                $nameObj->{$namePart->getName()} = $namePart->render($name);
             }
         }
 

src/Seboettg/CiteProc/Locale/LocaleXmlParserTrait.php

Spacing +2 added lines, -2 removed lines

@@ -60,7 +60,7 @@ 
     {
         /** @var \SimpleXMLElement $node */
         foreach ($locale as $node) {
-            switch($node->getName()) {
+            switch ($node->getName()) {
                 case 'style-options':
                     $this->optionsXml->add('options', $node);
                     foreach ($node->attributes() as $name => $value) {
@@ -108,7 +108,7 @@ 
                     }
                     break;
                 case 'date':
-                    $form = (string)$node["form"];
+                    $form = (string) $node["form"];
                     $this->dateXml->add($form, $node);
                     foreach ($node->children() as $child) {
                         $date = new \stdClass();

src/Seboettg/CiteProc/Style/InheritableNameAttributesTrait.php

Spacing +11 added lines, -11 removed lines

@@ -214,7 +214,7 @@ 
         $parentStyleElement = null;
         if ($this instanceof  Name) {
             $parentStyleElement = ($context->isModeBibliography() ? $context->getBibliography() : $context->getCitation());
-        } else if ($this instanceof StyleElement){
+        } else if ($this instanceof StyleElement) {
             $parentStyleElement = $context->getRoot();
         }
 
@@ -223,7 +223,7 @@ 
             switch ($nameAttribute) {
                 case 'and':
                     if (!empty($attribute)) {
-                        $this->and = (string)$attribute;
+                        $this->and = (string) $attribute;
 
                     } else if (!empty($parentStyleElement)) { //inherit from parent style
                         $this->and = $parentStyleElement->getAnd();
@@ -231,14 +231,14 @@ 
                     break;
                 case 'delimiter-precedes-et-al':
                     if (!empty($attribute)) {
-                        $this->delimiterPrecedesEtAl = (string)$attribute;
+                        $this->delimiterPrecedesEtAl = (string) $attribute;
                     } else if (!empty($parentStyleElement)) { //inherit from parent style
                         $this->delimiterPrecedesEtAl = $parentStyleElement->getDelimiterPrecedesEtAl();
                     }
                     break;
                 case 'delimiter-precedes-last':
                     if (!empty($attribute)) {
-                        $this->delimiterPrecedesLast = (string)$attribute;
+                        $this->delimiterPrecedesLast = (string) $attribute;
                     } else if (!empty($parentStyleElement)) { //inherit from parent style
                         $this->delimiterPrecedesLast = $parentStyleElement->getDelimiterPrecedesLast();
                     }
@@ -252,14 +252,14 @@ 
                     break;
                 case 'et-al-use-first':
                     if (!empty($attribute)) {
-                        $this->etAlUseFirst = intval((string)$attribute);
+                        $this->etAlUseFirst = intval((string) $attribute);
                     } else if (!empty($parentStyleElement)) {
                         $this->etAlUseFirst = $parentStyleElement->getEtAlUseFirst();
                     }
                     break;
                 case 'et-al-subsequent-min':
                     if (!empty($attribute)) {
-                        $this->etAlSubsequentMin = intval((string)$attribute);
+                        $this->etAlSubsequentMin = intval((string) $attribute);
                     } else if (!empty($parentStyleElement)) {
                         $this->etAlSubsequentMin = $parentStyleElement->getEtAlSubsequentMin();
                     }
@@ -289,7 +289,7 @@ 
                     break;
                 case 'initialize-with':
                     if (!empty($attribute)) {
-                        $this->initializeWith = (string)$attribute;
+                        $this->initializeWith = (string) $attribute;
                     } else if (!empty($parentStyleElement)) {
                         $this->initializeWith = $parentStyleElement->getInitializeWith();
                     }
@@ -311,7 +311,7 @@ 
                 case 'name-form':
                     if ($this instanceof Root || $this instanceof StyleElement) {
                         if (!empty($attribute)) {
-                            $this->nameForm = (string)$attribute;
+                            $this->nameForm = (string) $attribute;
                         } else if (!empty($parentStyleElement)) {
                             $this->nameForm = $parentStyleElement->getNameForm();
                         }
@@ -320,7 +320,7 @@ 
                 case 'form':
                     if ($this instanceof Name) {
                         if (!empty($attribute)) {
-                            $this->form = (string)$attribute;
+                            $this->form = (string) $attribute;
                         } else if (!empty($parentStyleElement)) {
                             $this->form = $parentStyleElement->getNameForm();
                         }
@@ -329,7 +329,7 @@ 
                 case 'name-delimiter':
                     if ($this instanceof Root || $this instanceof StyleElement) {
                         if (!empty($attribute)) {
-                            $this->nameDelimiter = (string)$attribute;
+                            $this->nameDelimiter = (string) $attribute;
                         } else if (!empty($parentStyleElement)) {
                             $this->nameDelimiter = $parentStyleElement->getNameDelimiter();
                         }
@@ -338,7 +338,7 @@ 
                 case 'delimiter':
                     if ($this instanceof Name) {
                         if (!empty($attribute)) {
-                            $this->delimiter = (string)$attribute;
+                            $this->delimiter = (string) $attribute;
                         } else if (!empty($parentStyleElement)) {
                             $this->delimiter = $parentStyleElement->getNameDelimiter();
                         }

src/Seboettg/CiteProc/Util/StringHelper.php

Spacing +5 added lines, -5 removed lines

@@ -64,7 +64,7 @@ 
     {
         $wordArray = explode(" ", $text);
 
-        array_walk($wordArray, function (&$word) {
+        array_walk($wordArray, function(&$word) {
             $word = ucfirst($word);
         });
 
@@ -74,16 +74,16 @@ 
     public static function capitalizeForTitle($titleString)
     {
         if (preg_match('/(.+[^\<\>][\.:\/;\?\!]\s?)([a-z])(.+)/', $titleString, $match)) {
-            $titleString = $match[1].StringHelper::mb_ucfirst($match[2]).$match[3];
+            $titleString = $match[1] . StringHelper::mb_ucfirst($match[2]) . $match[3];
         }
 
         $wordArray = explode(" ", $titleString);
 
-        array_walk($wordArray, function (&$word) {
+        array_walk($wordArray, function(&$word) {
 
             $words = explode("-", $word);
             if (count($words) > 1) {
-                array_walk($words, function (&$w) {
+                array_walk($words, function(&$w) {
                     $w = StringHelper::keepLowerCase($w) ? $w : StringHelper::mb_ucfirst($w);
                 });
                 $word = implode("-", $words);
@@ -96,7 +96,7 @@ 
 
     public static function keepLowerCase($word)
     {
-        $lowerCase =  in_array($word, self::PREPOSITIONS) ||
+        $lowerCase = in_array($word, self::PREPOSITIONS) ||
             in_array($word, self::ARTICLES) ||
             in_array($word, self::CONJUNCTIONS) ||
             in_array($word, self::ADJECTIVES);

src/Seboettg/CiteProc/Rendering/Number.php

Spacing +2 added lines, -2 removed lines

@@ -99,7 +99,7 @@ 
                     $num2 = Util\Number::dec2roman($matches[3]);
                     $text = $this->buildNumberRangeString($num1, $num2, $matches[2]);
                 } else {
-                    $text =  Util\Number::dec2roman($var);
+                    $text = Util\Number::dec2roman($var);
                 }
                 break;
             case 'numeric':
@@ -158,7 +158,7 @@ 
             $numRange = "$num1 " . htmlentities(self::RANGE_DELIMITER_AMPERSAND) . " $num2";
         } else if (self::RANGE_DELIMITER_COMMA === $delim) {
             $numRange = $num1 . htmlentities(self::RANGE_DELIMITER_COMMA) . " $num2";
-        } else  {
+        } else {
             $numRange = $num1 . self::RANGE_DELIMITER_HYPHEN . $num2;
         }
         return $numRange;

src/Seboettg/CiteProc/Rendering/Choose/ChooseIf.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
         foreach ($node->attributes() as $name => $value) {
             if ('match' !== $name) {
-                $this->constraints->append(Factory::createConstraint((string)$name, (string)$value));
+                $this->constraints->append(Factory::createConstraint((string) $name, (string) $value));
             }
         }
 

src/Seboettg/CiteProc/Styles/TextCaseTrait.php

Doc Comments +4 added lines

@@ -79,6 +79,10 @@
     }
 
 
+    /**
+     * @param string $render
+     * @param string $original
+     */
     private function keepNoCase($render, $original)
     {
         if (preg_match('/<span class=\"nocase\">(\p{L}+)<\/span>/i', $original, $match)) {