seboettg / citeproc-php

src/Rendering/Name/Names.php

Indentation +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
         $this->parent = $parent;
         /**
          * @var SimpleXMLElement $child
-        */
+         */
         foreach ($node->children() as $child) {
             switch ($child->getName()) {
                 case "name":

Spacing +5 added lines, -5 removed lines

@@ -186,7 +186,7 @@ 
                 } else {
                     $arr = [];
                     foreach ($data->editor as $editor) {
-                        $edt = $this->format($editor->family . ", " . $editor->given);
+                        $edt = $this->format($editor->family.", ".$editor->given);
                         $results[] = NameHelper::addExtendedMarkup('editor', $editor, $edt);
                     }
                     $str .= implode($this->delimiter, $arr);
@@ -196,7 +196,7 @@ 
                     $str .= $this->label->render($data);
                 }
                 $vars = $this->variables->toArray();
-                $vars = array_filter($vars, function ($value) {
+                $vars = array_filter($vars, function($value) {
                     return !($value === "editor" || $value === "translator");
                 });
                 $this->variables->setArray($vars);
@@ -220,7 +220,7 @@ 
                     }
                 } else {
                     foreach ($data->{$var} as $name) {
-                        $formatted = $this->format($name->given . " " . $name->family);
+                        $formatted = $this->format($name->given." ".$name->family);
                         $results[] = NameHelper::addExtendedMarkup($var, $name, $formatted);
                     }
                 }
@@ -250,7 +250,7 @@ 
     {
         $this->label->setVariable($var);
         if (in_array($this->label->getForm(), ["verb", "verb-short"])) {
-            $name = $this->label->render($data) . $name;
+            $name = $this->label->render($data).$name;
         } else {
             $name .= $this->label->render($data);
         }
@@ -377,7 +377,7 @@ 
 
     private function filterEmpty(array $results)
     {
-        return array_filter($results, function ($item) {
+        return array_filter($results, function($item) {
             return !empty($item);
         });
     }

src/Rendering/Name/Name.php

Indentation +2 added lines, -2 removed lines

@@ -90,13 +90,13 @@
         $this->nameParts = [];
 
         /**
- * @var SimpleXMLElement $child 
+         * @var SimpleXMLElement $child 
 */
         foreach ($node->children() as $child) {
             switch ($child->getName()) {
             case "name-part":
                 /**
- * @var NamePart $namePart 
+                 * @var NamePart $namePart 
 */
                 $namePart = Factory::create($child, $this);
                 $this->nameParts[$namePart->getName()] = $namePart;

Switch Indentation +64 added lines, -64 removed lines

@@ -94,20 +94,20 @@ 
 */
         foreach ($node->children() as $child) {
             switch ($child->getName()) {
-            case "name-part":
-                /**
+             case "name-part":
+                 /**
  * @var NamePart $namePart 
 */
-                $namePart = Factory::create($child, $this);
-                $this->nameParts[$namePart->getName()] = $namePart;
+                 $namePart = Factory::create($child, $this);
+                 $this->nameParts[$namePart->getName()] = $namePart;
             }
         }
 
         foreach ($node->attributes() as $attribute) {
             switch ($attribute->getName()) {
-            case 'form':
-                $this->form = (string)$attribute;
-                break;
+             case 'form':
+                 $this->form = (string)$attribute;
+                 break;
             }
         }
 
@@ -268,19 +268,19 @@ 
             delimiter-precedes-et-al attribute. */
 
             switch ($this->delimiterPrecedesEtAl) {
-            case 'never':
-                $text = $text . " " . $this->etAl;
-                break;
-            case 'always':
-                $text = $text . $this->delimiter . $this->etAl;
-                break;
-            case 'contextual':
-            default:
-                if (count($resultNames) === 1) {
-                    $text .= " " . $this->etAl;
-                } else {
-                    $text .= $this->delimiter . $this->etAl;
-                }
+             case 'never':
+                 $text = $text . " " . $this->etAl;
+                 break;
+             case 'always':
+                 $text = $text . $this->delimiter . $this->etAl;
+                 break;
+             case 'contextual':
+             default:
+                 if (count($resultNames) === 1) {
+                     $text .= " " . $this->etAl;
+                 } else {
+                     $text .= $this->delimiter . $this->etAl;
+                 }
             }
         }
         return $text;
@@ -352,40 +352,40 @@ 
                 /* “partial-each” - when one or more rendered names in the name variable match those in the preceding
                 bibliographic entry, the value of subsequent-author-substitute substitutes for each matching name.
                 Matching starts with the first name, and continues up to the first mismatch. */
-            case SubsequentAuthorSubstituteRule::PARTIAL_EACH:
-                if (NameHelper::precedingHasAuthor($preceding, $name)) {
-                    $resultNames[] = $subsequentSubstitution;
-                } else {
-                    $resultNames[] = $this->formatName($name, $rank);
-                }
-                break;
-                /* “partial-first” - as “partial-each”, but substitution is limited to the first name of the name
+             case SubsequentAuthorSubstituteRule::PARTIAL_EACH:
+                 if (NameHelper::precedingHasAuthor($preceding, $name)) {
+                     $resultNames[] = $subsequentSubstitution;
+                 } else {
+                     $resultNames[] = $this->formatName($name, $rank);
+                 }
+                 break;
+                 /* “partial-first” - as “partial-each”, but substitution is limited to the first name of the name
                 variable. */
-            case SubsequentAuthorSubstituteRule::PARTIAL_FIRST:
-                if ($rank === 0) {
-                    if ($preceding->author[0]->family === $name->family) {
-                        $resultNames[] = $subsequentSubstitution;
-                    } else {
-                        $resultNames[] = $this->formatName($name, $rank);
-                    }
-                } else {
-                    $resultNames[] = $this->formatName($name, $rank);
-                }
-                break;
-
-                /* “complete-each” - requires a complete match like “complete-all”, but now the value of
+             case SubsequentAuthorSubstituteRule::PARTIAL_FIRST:
+                 if ($rank === 0) {
+                     if ($preceding->author[0]->family === $name->family) {
+                         $resultNames[] = $subsequentSubstitution;
+                     } else {
+                         $resultNames[] = $this->formatName($name, $rank);
+                     }
+                 } else {
+                     $resultNames[] = $this->formatName($name, $rank);
+                 }
+                 break;
+
+                 /* “complete-each” - requires a complete match like “complete-all”, but now the value of
                 subsequent-author-substitute substitutes for each rendered name. */
-            case SubsequentAuthorSubstituteRule::COMPLETE_EACH:
-                try {
-                    if (NameHelper::identicalAuthors($preceding, $data)) {
-                        $resultNames[] = $subsequentSubstitution;
-                    } else {
-                        $resultNames[] = $this->formatName($name, $rank);
-                    }
-                } catch (CiteProcException $e) {
-                    $resultNames[] = $this->formatName($name, $rank);
-                }
-                break;
+             case SubsequentAuthorSubstituteRule::COMPLETE_EACH:
+                 try {
+                     if (NameHelper::identicalAuthors($preceding, $data)) {
+                         $resultNames[] = $subsequentSubstitution;
+                     } else {
+                         $resultNames[] = $this->formatName($name, $rank);
+                     }
+                 } catch (CiteProcException $e) {
+                     $resultNames[] = $this->formatName($name, $rank);
+                 }
+                 break;
             }
         }
         return $resultNames;
@@ -502,18 +502,18 @@ 
         $text = "";
         if (!empty($this->and) && empty($this->etAl)) {
             switch ($this->delimiterPrecedesLast) {
-            case 'after-inverted-name':
-                //TODO: implement
-                break;
-            case 'always':
-                $text = implode($this->delimiter, $resultNames);
-                break;
-            case 'never':
-                $text = $this->renderDelimiterPrecedesLastNever($resultNames);
-                break;
-            case 'contextual':
-            default:
-                $text = $this->renderDelimiterPrecedesLastContextual($resultNames);
+             case 'after-inverted-name':
+                 //TODO: implement
+                 break;
+             case 'always':
+                 $text = implode($this->delimiter, $resultNames);
+                 break;
+             case 'never':
+                 $text = $this->renderDelimiterPrecedesLastNever($resultNames);
+                 break;
+             case 'contextual':
+             default:
+                 $text = $this->renderDelimiterPrecedesLastContextual($resultNames);
             }
         }
         return $text;

Spacing +18 added lines, -18 removed lines

@@ -106,7 +106,7 @@ 
         foreach ($node->attributes() as $attribute) {
             switch ($attribute->getName()) {
             case 'form':
-                $this->form = (string)$attribute;
+                $this->form = (string) $attribute;
                 break;
             }
         }
@@ -169,7 +169,7 @@ 
         cs:names element (taking into account the effects of et-al abbreviation and editor/translator collapsing),
         which allows for advanced sorting. */
         if ($this->form == 'count') {
-            return (int)count($resultNames);
+            return (int) count($resultNames);
         }
 
         return $text;
@@ -269,17 +269,17 @@ 
 
             switch ($this->delimiterPrecedesEtAl) {
             case 'never':
-                $text = $text . " " . $this->etAl;
+                $text = $text." ".$this->etAl;
                 break;
             case 'always':
-                $text = $text . $this->delimiter . $this->etAl;
+                $text = $text.$this->delimiter.$this->etAl;
                 break;
             case 'contextual':
             default:
                 if (count($resultNames) === 1) {
-                    $text .= " " . $this->etAl;
+                    $text .= " ".$this->etAl;
                 } else {
-                    $text .= $this->delimiter . $this->etAl;
+                    $text .= $this->delimiter.$this->etAl;
                 }
             }
         }
@@ -458,7 +458,7 @@ 
                 $text = implode(" ", $resultNames);
             } else { // >2
                 $lastName = array_pop($resultNames);
-                $text = implode($this->delimiter, $resultNames) . " " . $lastName;
+                $text = implode($this->delimiter, $resultNames)." ".$lastName;
             }
         }
         return $text;
@@ -487,7 +487,7 @@ 
     {
         $count = count($resultNames);
         if (!empty($this->and) && $count > 1 && empty($this->etAl)) {
-            $new = $this->and . ' ' . end($resultNames); // add and-prefix of the last name if "and" is defined
+            $new = $this->and.' '.end($resultNames); // add and-prefix of the last name if "and" is defined
             // set prefixed last name at the last position of $resultNames array
             $resultNames[count($resultNames) - 1] = $new;
         }
@@ -535,8 +535,8 @@ 
         if (StringHelper::isLatinString($normalizedName) || StringHelper::isCyrillicString($normalizedName)) {
             if ($this->form === "long"
                 && $nameAsSortOrder
-                && ((string)$demoteNonDroppingParticle === DemoteNonDroppingParticle::NEVER
-                || (string)$demoteNonDroppingParticle === DemoteNonDroppingParticle::SORT_ONLY)
+                && ((string) $demoteNonDroppingParticle === DemoteNonDroppingParticle::NEVER
+                || (string) $demoteNonDroppingParticle === DemoteNonDroppingParticle::SORT_ONLY)
             ) {
                 // [La] [Fontaine], [Jean] [de], [III]
                 NameHelper::prependParticleTo($data, "family", "non-dropping-particle");
@@ -544,25 +544,25 @@ 
 
                 list($family, $given) = $this->renderNameParts($data);
 
-                $text = $family . (!empty($given) ? $this->sortSeparator . $given : "");
-                $text .= !empty($data->suffix) ? $this->sortSeparator . $data->suffix : "";
+                $text = $family.(!empty($given) ? $this->sortSeparator.$given : "");
+                $text .= !empty($data->suffix) ? $this->sortSeparator.$data->suffix : "";
             } elseif ($this->form === "long"
                 && $nameAsSortOrder
                 && (is_null($demoteNonDroppingParticle)
-                || (string)$demoteNonDroppingParticle === DemoteNonDroppingParticle::DISPLAY_AND_SORT)
+                || (string) $demoteNonDroppingParticle === DemoteNonDroppingParticle::DISPLAY_AND_SORT)
             ) {
                 // [Fontaine], [Jean] [de] [La], [III]
                 NameHelper::appendParticleTo($data, "given", "dropping-particle");
                 NameHelper::appendParticleTo($data, "given", "non-dropping-particle");
                 list($family, $given) = $this->renderNameParts($data);
                 $text = $family;
-                $text .= !empty($given) ? $this->sortSeparator . $given : "";
-                $text .= !empty($data->suffix) ? $this->sortSeparator . $data->suffix : "";
+                $text .= !empty($given) ? $this->sortSeparator.$given : "";
+                $text .= !empty($data->suffix) ? $this->sortSeparator.$data->suffix : "";
             } elseif ($this->form === "long" && $nameAsSortOrder && empty($demoteNonDroppingParticle)) {
                 list($family, $given) = $this->renderNameParts($data);
                 $text = $family;
-                $text .= !empty($given) ? $this->delimiter . $given : "";
-                $text .= !empty($data->suffix) ? $this->sortSeparator . $data->suffix : "";
+                $text .= !empty($given) ? $this->delimiter.$given : "";
+                $text .= !empty($data->suffix) ? $this->sortSeparator.$data->suffix : "";
             } elseif ($this->form === "short") {
                 // [La] [Fontaine]
                 NameHelper::prependParticleTo($data, "family", "non-dropping-particle");
@@ -573,7 +573,7 @@ 
                 NameHelper::prependParticleTo($data, "family", "dropping-particle");
                 NameHelper::appendParticleTo($data, "family", "suffix");
                 list($family, $given) = $this->renderNameParts($data);
-                $text = !empty($given) ? $given . " " . $family : $family;
+                $text = !empty($given) ? $given." ".$family : $family;
             }
         } else {
             $text = $this->form === "long" ? $data->family.$data->given : $data->family;

src/Style/Sort/Sort.php

Spacing +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
     public function flatten($array)
     {
         $returnArray = [];
-        array_walk_recursive($array, function ($a) use (&$returnArray) {
+        array_walk_recursive($array, function($a) use (&$returnArray) {
             $returnArray[] = $a;
         });
         return $returnArray;

src/Util/DateHelper.php

Spacing +3 added lines, -3 removed lines

@@ -59,7 +59,7 @@ 
             $dateTime = new DateTime($date->{'raw'});
             $arr = [[$dateTime->format("Y"), $dateTime->format("m"), $dateTime->format("d")]];
         } catch (Exception $e) {
-            throw new CiteProcException("Could not parse date \"" . $date->{'raw'} . "\".", 0, $e);
+            throw new CiteProcException("Could not parse date \"".$date->{'raw'}."\".", 0, $e);
         }
 
         return $arr;
@@ -111,9 +111,9 @@ 
             } elseif ($match === "any" && count($dateParts) === 2) {
                 return true;
             } else {
-                $ret = ($match === "all") ? $ret & true : $ret | true;
+                $ret = ($match === "all") ? $ret&true : $ret|true;
             }
         }
-        return (bool)$ret;
+        return (bool) $ret;
     }
 }

src/Util/NameHelper.php

Spacing +4 added lines, -4 removed lines

@@ -70,7 +70,7 @@ 
     public static function appendParticleTo(&$data, $namePart, $particle)
     {
         if (isset($data->{$particle}) && isset($data->{$namePart})) {
-            $data->{$namePart} = $data->{$namePart} . " " . $data->{$particle}; // append $particle to $namePart
+            $data->{$namePart} = $data->{$namePart}." ".$data->{$particle}; // append $particle to $namePart
             unset($data->{$particle}); //remove particle from $data
         }
     }
@@ -84,7 +84,7 @@ 
     public static function prependParticleTo(&$data, $namePart, $particle)
     {
         if (isset($data->{$particle}) && isset($data->{$namePart})) {
-            $data->{$namePart} = $data->{$particle} . " " . $data->{$namePart}; //prepend $particle to $namePart
+            $data->{$namePart} = $data->{$particle}." ".$data->{$namePart}; //prepend $particle to $namePart
             unset($data->{$particle}); //remove particle from $data
         }
     }
@@ -102,7 +102,7 @@ 
             return false;
         }
 
-        array_walk($persons1, function ($name, $key) use ($persons2, &$same) {
+        array_walk($persons1, function($name, $key) use ($persons2, &$same) {
             $family1 = $name->family;
             $family2 = $persons2[$key]->family;
             $same = $same && ($family1 === $family2);
@@ -121,7 +121,7 @@ 
         if (empty($data->family)) {
             throw new CiteProcException("Illegal argument. Name has no family name.");
         }
-        return $data->family . (isset($data->given) ? $data->given : "");
+        return $data->family.(isset($data->given) ? $data->given : "");
     }
 
     public static function addExtendedMarkup($nameVar, $nameItem, $formattedName)

src/Util/StringHelper.php

Spacing +10 added lines, -10 removed lines

@@ -79,7 +79,7 @@ 
     {
         $wordArray = explode(" ", $text);
 
-        array_walk($wordArray, function (&$word) {
+        array_walk($wordArray, function(&$word) {
             $word = ucfirst($word);
         });
 
@@ -93,16 +93,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);
@@ -141,7 +141,7 @@ 
         /** @noinspection PhpInternalEntityUsedInspection */
         $encoding = Mbstring::mb_detect_encoding($firstChar, self::ISO_ENCODINGS, true);
         return in_array($encoding, self::ISO_ENCODINGS) ?
-            Mbstring::mb_strtoupper($firstChar, $encoding) . $then : $firstChar . $then;
+            Mbstring::mb_strtoupper($firstChar, $encoding).$then : $firstChar.$then;
     }
 
     public static function mb_strrev($string)
@@ -190,13 +190,13 @@ 
             foreach ($spaceExploded as $givenPart) {
                 $firstLetter = mb_substr($givenPart, 0, 1, "UTF-8");
                 if (StringHelper::isLatinString($firstLetter)) {
-                    $res .= ctype_upper($firstLetter) ? $firstLetter . $initializeSign : " " . $givenPart . " ";
+                    $res .= ctype_upper($firstLetter) ? $firstLetter.$initializeSign : " ".$givenPart." ";
                 } else {
-                    $res .= $firstLetter . $initializeSign;
+                    $res .= $firstLetter.$initializeSign;
                 }
             }
             if ($i < count($exploded) - 1 && $initializeWithHyphen) {
-                $res = rtrim($res) . "-";
+                $res = rtrim($res)."-";
             }
             ++$i;
         }
@@ -259,7 +259,7 @@ 
         $innerCloseQuote
     ) {
         if (preg_match("/(.*)$outerOpenQuote(.+)$outerCloseQuote(.*)/u", $text, $match)) {
-            return $match[1] . $innerOpenQuote . $match[2] . $innerCloseQuote . $match[3];
+            return $match[1].$innerOpenQuote.$match[2].$innerCloseQuote.$match[3];
         }
         return $text;
     }
@@ -290,6 +290,6 @@ 
      */
     public static function removeBrackets($datePart)
     {
-        return str_replace(["[","]", "(", ")", "{", "}"], "", $datePart);
+        return str_replace(["[", "]", "(", ")", "{", "}"], "", $datePart);
     }
 }

src/Util/Variables.php

Spacing +66 added lines, -66 removed lines

@@ -26,103 +26,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”)
 
 
     ];
@@ -169,7 +169,7 @@ 
         $parent = null;
         $names = new Names(
             new SimpleXMLElement(
-                "<names variable=\"$variable\" delimiter=\"-\">" .
+                "<names variable=\"$variable\" delimiter=\"-\">".
                 "<name form=\"long\" sort-separator=\",\" name-as-sort-order=\"all\"/></names>"
             ),
             $parent

src/Util/PageHelper.php

Spacing +3 added lines, -3 removed lines

@@ -67,7 +67,7 @@ 
 
                 $digitFrom = $from[$i];
                 if ($digitTo !== $digitFrom) {
-                    $resTo = $digitTo . $resTo;
+                    $resTo = $digitTo.$resTo;
                 }
             }
             return $resTo;
@@ -78,9 +78,9 @@ 
     private static function renderChicago($from, $to)
     {
         if ($from > 100 && ($from % 100 > 0) && intval(($from / 100), 10) === intval(($to / 100), 10)) {
-            return "" . ($to % 100);
+            return "".($to % 100);
         } elseif ($from >= 10000) {
-            return "" . ($to % 1000);
+            return "".($to % 1000);
         }
         return $to;
     }

src/Util/Factory.php

Spacing +3 added lines, -3 removed lines

@@ -54,10 +54,10 @@
      */
     public static function create($node, $param = null)
     {
-        $nodeClass = self::CITE_PROC_NODE_NAMESPACE . self::$nodes[$node->getName()];
+        $nodeClass = self::CITE_PROC_NODE_NAMESPACE.self::$nodes[$node->getName()];
         if (!class_exists($nodeClass)) {
-            throw new InvalidStylesheetException("For node {$node->getName()} " .
-                "does not exist any counterpart class \"" . $nodeClass .
+            throw new InvalidStylesheetException("For node {$node->getName()} ".
+                "does not exist any counterpart class \"".$nodeClass.
                 "\". The given stylesheet seems to be invalid.");
         }
         if ($param != null) {