seboettg / citeproc-php

src/Seboettg/CiteProc/Styles/DelimiterTrait.php

Spacing +2 added lines, -2 removed lines

@@ -12,8 +12,8 @@
     {
         foreach ($node->attributes() as $attribute) {
             /** @var string $name */
-            $name = (string)$attribute->getName();
-            $value = (string)$attribute;
+            $name = (string) $attribute->getName();
+            $value = (string) $attribute;
 
             switch ($name) {
                 case 'delimiter':

src/Seboettg/CiteProc/Util/Variables.php

Spacing +65 added lines, -65 removed lines

@@ -14,103 +14,103 @@
 {
 
     const NAME_VARIABLES = [
-        'author',               //author
-        'collection-editor',    //editor of the collection holding the item (e.g. the series editor for a book)
-        'composer',             //composer (e.g. of a musical score)
-        'container-author',     //author of the container holding the item (e.g. the book author for a book chapter)
-        'director',             //director (e.g. of a film)
-        'editor',               //editor
-        'editorial-director',   //managing editor (“Directeur de la Publication” in French)
-        'illustrator',          //illustrator (e.g. of a children’s book)
-        'interviewer',          //interviewer (e.g. of an interview)
-        'original-author',      //
-        'recipient',            //recipient (e.g. of a letter)
+        'author', //author
+        'collection-editor', //editor of the collection holding the item (e.g. the series editor for a book)
+        'composer', //composer (e.g. of a musical score)
+        'container-author', //author of the container holding the item (e.g. the book author for a book chapter)
+        'director', //director (e.g. of a film)
+        'editor', //editor
+        'editorial-director', //managing editor (“Directeur de la Publication” in French)
+        'illustrator', //illustrator (e.g. of a children’s book)
+        'interviewer', //interviewer (e.g. of an interview)
+        'original-author', //
+        'recipient', //recipient (e.g. of a letter)
         'reviewed-author'       //author of the item reviewed by the current item
     ];
 
     const NUMBER_VARIABLES = [
-        'chapter-number',       //chapter number
-        'collection-number',    //number identifying the collection holding the item (e.g. the series number for a book)
-        'edition',              //(container) edition holding the item (e.g. “3” when citing a chapter in the third
+        'chapter-number', //chapter number
+        'collection-number', //number identifying the collection holding the item (e.g. the series number for a book)
+        'edition', //(container) edition holding the item (e.g. “3” when citing a chapter in the third
                                 //edition of a book)
-        'issue',                //(container) issue holding the item (e.g. “5” when citing a journal article from
+        'issue', //(container) issue holding the item (e.g. “5” when citing a journal article from
                                 //journal volume 2, issue 5)
-        'number',               //number identifying the item (e.g. a report number)
-        'number-of-pages',      //total number of pages of the cited item
-        'number-of-volumes',    //total number of volumes, usable for citing multi-volume books and such
+        'number', //number identifying the item (e.g. a report number)
+        'number-of-pages', //total number of pages of the cited item
+        'number-of-volumes', //total number of volumes, usable for citing multi-volume books and such
         'volume'                //(container) volume holding the item (e.g. “2” when citing a chapter from book volume 2)
     ];
 
     const DATE_VARIABLES = [
-        'accessed',             //date the item has been accessed
+        'accessed', //date the item has been accessed
         'container',
-        'event-date',           //date the related event took place
-        'issued',               //date the item was issued/published
-        'original-date',        //(issue) date of the original version
+        'event-date', //date the related event took place
+        'issued', //date the item was issued/published
+        'original-date', //(issue) date of the original version
         'submitted'             //date the item (e.g. a manuscript) has been submitted for publication
     ];
 
     const STANDARD_VARIABLE = [
-        'abstract',             //abstract of the item (e.g. the abstract of a journal article)
-        'annote',               //reader’s notes about the item content
-        'archive',              //archive storing the item
-        'archive-location',     //storage location within an archive (e.g. a box and folder number)
-        'archive-place',        //geographic location of the archive
-        'authority',            //issuing or judicial authority (e.g. “USPTO” for a patent, “Fairfax Circuit Court” for
+        'abstract', //abstract of the item (e.g. the abstract of a journal article)
+        'annote', //reader’s notes about the item content
+        'archive', //archive storing the item
+        'archive-location', //storage location within an archive (e.g. a box and folder number)
+        'archive-place', //geographic location of the archive
+        'authority', //issuing or judicial authority (e.g. “USPTO” for a patent, “Fairfax Circuit Court” for
                                 //a legal case)
-        'call-number',          //call number (to locate the item in a library)
-        'citation-label',       //label identifying the item in in-text citations of label styles (e.g. “Ferr78”). May
+        'call-number', //call number (to locate the item in a library)
+        'citation-label', //label identifying the item in in-text citations of label styles (e.g. “Ferr78”). May
                                 //be assigned by the CSL processor based on item metadata.
-        'citation-number',      //index (starting at 1) of the cited reference in the bibliography (generated by the CSL
+        'citation-number', //index (starting at 1) of the cited reference in the bibliography (generated by the CSL
                                 //processor)
-        'collection-title',     //title of the collection holding the item (e.g. the series title for a book)
-        'container-title',      //title of the container holding the item (e.g. the book title for a book chapter, the
+        'collection-title', //title of the collection holding the item (e.g. the series title for a book)
+        'container-title', //title of the container holding the item (e.g. the book title for a book chapter, the
                                 //journal title for a journal article)
-        'container-title-short',//short/abbreviated form of “container-title” (also accessible through the “short” form
+        'container-title-short', //short/abbreviated form of “container-title” (also accessible through the “short” form
                                 //of the “container-title” variable)
-        'dimensions',           //physical (e.g. size) or temporal (e.g. running time) dimensions of the item
-        'DOI',                  //Digital Object Identifier (e.g. “10.1128/AEM.02591-07”)
-        'event',                //name of the related event (e.g. the conference name when citing a conference paper)
-        'event-place',          //geographic location of the related event (e.g. “Amsterdam, the Netherlands”)
+        'dimensions', //physical (e.g. size) or temporal (e.g. running time) dimensions of the item
+        'DOI', //Digital Object Identifier (e.g. “10.1128/AEM.02591-07”)
+        'event', //name of the related event (e.g. the conference name when citing a conference paper)
+        'event-place', //geographic location of the related event (e.g. “Amsterdam, the Netherlands”)
         'first-reference-note-number', //number of a preceding note containing the first reference to the item. Assigned
                                 // by the CSL processor. The variable holds no value for non-note-based styles, or when
                                 // the item hasn’t been cited in any preceding notes.
-        'genre',                //class, type or genre of the item (e.g. “adventure” for an adventure movie,
+        'genre', //class, type or genre of the item (e.g. “adventure” for an adventure movie,
                                 //“PhD dissertation” for a PhD thesis),
-        'ISBN',                 //International Standard Book Number
-        'ISSN',                 //International Standard Serial Number
-        'jurisdiction',         //geographic scope of relevance (e.g. “US” for a US patent)
-        'keyword',              //keyword(s) or tag(s) attached to the item
-        'locator',              //a cite-specific pinpointer within the item (e.g. a page number within a book, or a
+        'ISBN', //International Standard Book Number
+        'ISSN', //International Standard Serial Number
+        'jurisdiction', //geographic scope of relevance (e.g. “US” for a US patent)
+        'keyword', //keyword(s) or tag(s) attached to the item
+        'locator', //a cite-specific pinpointer within the item (e.g. a page number within a book, or a
                                 //volume in a multi-volume work). Must be accompanied in the input data by a label
                                 //indicating the locator type (see the Locators term list), which determines which term
                                 //is rendered by cs:label when the “locator” variable is selected.
-        'medium',               //medium description (e.g. “CD”, “DVD”, etc.)
-        'note',                 //(short) inline note giving additional item details (e.g. a concise summary or commentary)
-        'original-publisher',   //original publisher, for items that have been republished by a different publisher
+        'medium', //medium description (e.g. “CD”, “DVD”, etc.)
+        'note', //(short) inline note giving additional item details (e.g. a concise summary or commentary)
+        'original-publisher', //original publisher, for items that have been republished by a different publisher
         'original-publisher-place', //geographic location of the original publisher (e.g. “London, UK”)
-        'original-title',       //title of the original version (e.g. “Война и мир”, the untranslated Russian title of
+        'original-title', //title of the original version (e.g. “Война и мир”, the untranslated Russian title of
                                 // “War and Peace”)
-        'page',                 //range of pages the item (e.g. a journal article) covers in a container (e.g. a journal
+        'page', //range of pages the item (e.g. a journal article) covers in a container (e.g. a journal
                                 // issue)
-        'page-first',           //first page of the range of pages the item (e.g. a journal article) covers in a
+        'page-first', //first page of the range of pages the item (e.g. a journal article) covers in a
                                 //container (e.g. a journal issue)
-        'PMCID',                //PubMed Central reference number
-        'PMID',                 //PubMed reference number
-        'publisher',            //publisher
-        'publisher-place',      //geographic location of the publisher
-        'references',           //resources related to the procedural history of a legal case
-        'reviewed-title',       //title of the item reviewed by the current item
-        'scale',                //scale of e.g. a map
-        'section',              //container section holding the item (e.g. “politics” for a newspaper article)
-        'source',               //from whence the item originates (e.g. a library catalog or database)
-        'status',               //(publication) status of the item (e.g. “forthcoming”)
-        'title',                //primary title of the item
-        'title-short',          //short/abbreviated form of “title” (also accessible through the “short” form of the
+        'PMCID', //PubMed Central reference number
+        'PMID', //PubMed reference number
+        'publisher', //publisher
+        'publisher-place', //geographic location of the publisher
+        'references', //resources related to the procedural history of a legal case
+        'reviewed-title', //title of the item reviewed by the current item
+        'scale', //scale of e.g. a map
+        'section', //container section holding the item (e.g. “politics” for a newspaper article)
+        'source', //from whence the item originates (e.g. a library catalog or database)
+        'status', //(publication) status of the item (e.g. “forthcoming”)
+        'title', //primary title of the item
+        'title-short', //short/abbreviated form of “title” (also accessible through the “short” form of the
                                 //“title” variable)
-        'URL',                  //Uniform Resource Locator (e.g. “http://aem.asm.org/cgi/content/full/74/9/2766”)
-        'version',              //version of the item (e.g. “2.0.9” for a software program)
-        'year-suffix',          //disambiguating year suffix in author-date styles (e.g. “a” in “Doe, 1999a”)
+        'URL', //Uniform Resource Locator (e.g. “http://aem.asm.org/cgi/content/full/74/9/2766”)
+        'version', //version of the item (e.g. “2.0.9” for a software program)
+        'year-suffix', //disambiguating year suffix in author-date styles (e.g. “a” in “Doe, 1999a”)
 
 
     ];

src/Seboettg/CiteProc/Locale/Term.php

Spacing +2 added lines, -2 removed lines

@@ -30,11 +30,11 @@
     {
         $nameParts = explode("-", $name);
         $attr = "";
-        for ($i = count($nameParts)-1; $i >= 0; --$i) {
+        for ($i = count($nameParts) - 1; $i >= 0; --$i) {
             if ($i > 0) {
                 $attr = ucfirst($nameParts[$i]) . $attr;
             } else {
-                $attr = $nameParts[$i].$attr;
+                $attr = $nameParts[$i] . $attr;
             }
         }
         if (!isset($this->{$attr})) {

src/Seboettg/CiteProc/Util/Number.php

Spacing +3 added lines, -3 removed lines

@@ -57,7 +57,7 @@ 
      */
     public static function getCompareNumber()
     {
-        return function ($numA, $numB, $order) {
+        return function($numA, $numB, $order) {
             if (is_numeric($numA) && is_numeric($numB)) {
                 $ret = $numA - $numB;
             } else {
@@ -138,7 +138,7 @@ 
             "I" => 1
         ];
 
-        if(is_numeric($romanNumber)) {
+        if (is_numeric($romanNumber)) {
             return 0;
         }
 
@@ -152,7 +152,7 @@ 
         }
 
         $sum = 0;
-        while($current = current($values)) {
+        while ($current = current($values)) {
             $next = next($values);
             $next > $current ? $sum += $next - $current + 0 * next($values) : $sum += $current;
         }

src/Seboettg/CiteProc/Constraint/IsNumeric.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
             return $numberFormatter->parse($evalValue) !== false;
         } else if (preg_match(Number::PATTERN_ROMAN, $evalValue)) {
             return Number::roman2Dec($evalValue) !== false;
-        } else if (preg_match(Number::PATTERN_COMMA_AMPERSAND_RANGE, $evalValue)){
+        } else if (preg_match(Number::PATTERN_COMMA_AMPERSAND_RANGE, $evalValue)) {
             return true;
         }
         return false;

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

Spacing +3 added lines, -3 removed lines

@@ -58,13 +58,13 @@
     {
         foreach ($node->attributes() as $attribute) {
             if ("name" === $attribute->getName()) {
-                $this->name = (string)$attribute;
+                $this->name = (string) $attribute;
             }
             if ("form" === $attribute->getName()) {
-                $this->form = (string)$attribute;
+                $this->form = (string) $attribute;
             }
             if ("range-delimiter" === $attribute->getName()) {
-                $this->rangeDelimiter = (string)$attribute;
+                $this->rangeDelimiter = (string) $attribute;
             }
         }
 

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

Spacing +3 added lines, -3 removed lines

@@ -125,9 +125,9 @@
      */
     public function renderNumeric()
     {
-        $ret  =  $this->year;
-        $ret .=  $this->month > 0 && $this->month < 13 ? "-" . sprintf("%02s", $this->month) : "";
-        $ret .=  $this->day > 0 && $this->day < 32 ? "-" . sprintf("%02s", $this->day) : "";
+        $ret  = $this->year;
+        $ret .= $this->month > 0 && $this->month < 13 ? "-" . sprintf("%02s", $this->month) : "";
+        $ret .= $this->day > 0 && $this->day < 32 ? "-" . sprintf("%02s", $this->day) : "";
         return $ret;
     }
 }
\ No newline at end of file

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 +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;
             });