seboettg / citeproc-php

src/Seboettg/CiteProc/CiteProc.php

Doc Comments +2 added lines, -2 removed lines

@@ -115,7 +115,7 @@ 
     }
 
     /**
-     * @param array|DataList $data
+     * @param DataList $data
      * @return string
      */
     protected function bibliography($data)
@@ -125,7 +125,7 @@ 
     }
 
     /**
-     * @param array|DataList $data
+     * @param DataList $data
      * @return string
      */
     protected function citation($data)

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 +8 added lines, -8 removed lines

@@ -140,7 +140,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);
                 });
 
@@ -236,7 +236,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;
             }
@@ -341,7 +341,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;
                     }
@@ -354,7 +354,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;
                     }
@@ -367,7 +367,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;
@@ -417,7 +417,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;
     }
 
     /**
@@ -442,9 +442,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/Name/Name.php

Doc Comments +4 added lines, -1 removed lines

@@ -105,6 +105,9 @@ 
         $this->initDelimiterAttributes($node);
     }
 
+    /**
+     * @param integer|null $citationNumber
+     */
     public function render($data, $citationNumber)
     {
         if (!$this->attributesInitialized) {
@@ -266,7 +269,7 @@ 
     }
 
     /**
-     * @param $name
+     * @param \stdClass $name
      * @return string
      */
     private function getNamesString($name, $rank)

Spacing +7 added lines, -7 removed lines

@@ -114,10 +114,10 @@ 
         $etAl = false;
         $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();
@@ -205,16 +205,16 @@ 
                     if (count($resultNames) === 1) {
                         $text .= " $etAl";
                     } else {
-                        $text .=  $this->delimiter . $etAl;
+                        $text .= $this->delimiter . $etAl;
                     }
 
             }
         }
         if ($this->form == 'count') {
             if ($etAl === false) {
-                return (int)count($resultNames);
+                return (int) count($resultNames);
             } else {
-                return (int)(count($resultNames) - 1);
+                return (int) (count($resultNames) - 1);
             }
         }
         // strip out the last delimiter if not required
@@ -256,7 +256,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);
             }
         }
 
@@ -306,7 +306,7 @@ 
 
                         //remove last comma when no suffix exist.
                         $text = trim($text);
-                        $text = substr($text, -1) === "," ? substr($text, 0, strlen($text)-1) : $text;
+                        $text = substr($text, -1) === "," ? substr($text, 0, strlen($text) - 1) : $text;
                         break;
                     default:
                         /*

Braces +1 added lines, -2 removed lines

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

src/Seboettg/CiteProc/Style/InheritableNameAttributesTrait.php

Spacing +7 added lines, -7 removed lines

@@ -197,7 +197,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();
         }
 
@@ -207,7 +207,7 @@ 
                 case 'and':
                     $and = "";
                     if (!empty($attribute)) {
-                        $and = (string)$attribute;
+                        $and = (string) $attribute;
 
                     } else if (!empty($parentStyleElement)) { //inherit from parent style
                         $and = $parentStyleElement->getAnd();
@@ -220,14 +220,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();
                     }
@@ -241,14 +241,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();
                     }
@@ -278,7 +278,7 @@ 
                     break;
                 case 'initialize-with':
                     if (!empty($attribute)) {
-                        $this->initializeWith = (string)$attribute;
+                        $this->initializeWith = (string) $attribute;
                     } else if (!empty($parentStyleElement)) {
                         $this->initializeWith = $parentStyleElement->getInitializeWith();
                     }

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) {

src/Seboettg/CiteProc/Rendering/Number.php

Spacing +3 added lines, -3 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':
@@ -114,7 +114,7 @@ 
                 if (preg_match("/\s*(\d+)\s*[\-\-\&,]\s*(\d+)\s*/", $var, $matches)) {
                     $text = $this->buildNumberRangeString($matches[1], $matches[3], $matches[2]);
                 } else {
-                    $text =  Util\Number::dec2roman($var);
+                    $text = Util\Number::dec2roman($var);
                 }
                 break;
         }
@@ -158,7 +158,7 @@ 
             $numRange = "$num1 " . 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/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();