seboettg / citeproc-php

vendorPath.php

Indentation +14 added lines, -14 removed lines

@@ -29,19 +29,19 @@
 
 if (!function_exists('vendorPath')) {
 
-	/**
-	 * @return bool|string returns path of composer vendor folder, and false if folder does not exist
-	 */
-	function vendorPath() {
+    /**
+     * @return bool|string returns path of composer vendor folder, and false if folder does not exist
+     */
+    function vendorPath() {
 
-		if (file_exists(__DIR__ . '/vendor/')) {
-			return __DIR__ . '/vendor';
-		} else {
-			if (!file_exists(__DIR__ . '/../../')) {
-				return false;
-			} else {
-				return __DIR__ . '/../..';
-			}
-		}
-	}
+        if (file_exists(__DIR__ . '/vendor/')) {
+            return __DIR__ . '/vendor';
+        } else {
+            if (!file_exists(__DIR__ . '/../../')) {
+                return false;
+            } else {
+                return __DIR__ . '/../..';
+            }
+        }
+    }
 }
\ No newline at end of file

src/CiteProc.php

Doc Comments +4 added lines

@@ -112,6 +112,10 @@
         }
     }
 
+    /**
+     * @param string $csl
+     * @param string $lang
+     */
     private function init($csl, $lang) {
         // define field values appropriate to your data in the csl_mapper class and un-comment the next line.        
         $this->mapper = new Mapper();

Indentation +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
      */
     function __construct($csl = NULL, $lang = 'en') {
         if ($csl) {
-	        $this->init($csl, $lang);
+            $this->init($csl, $lang);
         }
     }
 

Spacing +2 added lines, -2 removed lines

@@ -92,13 +92,13 @@
      * @throws \Exception
      */
     public static function loadStyleSheet($name) {
-        include_once __DIR__.'/../vendorPath.php';
+        include_once __DIR__ . '/../vendorPath.php';
 
         if (!($vendorPath = vendorPath())) {
             throw new \Exception('Error: vendor path not found. Use composer to initialize your project');
         }
 
-        return file_get_contents($vendorPath.'/academicpuma/styles/'.$name.'.csl');
+        return file_get_contents($vendorPath . '/academicpuma/styles/' . $name . '.csl');
     }
 
     /**

src/Layout.php

Doc Comments +3 added lines

@@ -35,6 +35,9 @@
         parent::initFormatting();
     }
 
+    /**
+     * @param string $mode
+     */
     public function render($data, $mode = null)
     {
         //$text = '';

src/Locale.php

Doc Comments +7 added lines

@@ -91,6 +91,9 @@ 
      */
     protected $styleLocale;
 
+    /**
+     * @param string $lang
+     */
     private static function getLocalesFileName($lang) {
 
         include_once __DIR__.'/../vendorPath.php';
@@ -136,6 +139,10 @@ 
         }
     }
 
+    /**
+     * @param string $type
+     * @param string $arg3
+     */
     public function locale($type, $arg1, $arg2 = NULL, $arg3 = NULL) {
 
         switch ($type) {

Indentation +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
     public function __construct($lang = 'en') {
 
         $this->module_path = dirname(__FILE__);
-	    $this->locale = new \SimpleXMLElement(self::getLocalesFileName($lang));
+        $this->locale = new \SimpleXMLElement(self::getLocalesFileName($lang));
 
         if ($this->locale) {
             $this->locale->registerXPathNamespace('cs', 'http://purl.org/net/xbiblio/csl');

Spacing +4 added lines, -4 removed lines

@@ -93,7 +93,7 @@ 
 
     private static function getLocalesFileName($lang) {
 
-        include_once __DIR__.'/../vendorPath.php';
+        include_once __DIR__ . '/../vendorPath.php';
 
         if (!($vendorPath = vendorPath())) {
             throw new \Exception('Error: vendor path not found. Use composer to initialize your project');
@@ -101,10 +101,10 @@ 
 
         $localeFile = null;
 
-        if(isset(self::$langBase[$lang])) {
-            $localeFile = file_get_contents($vendorPath.'/academicpuma/locales/locales-' . self::$langBase[$lang] . '.xml');
+        if (isset(self::$langBase[$lang])) {
+            $localeFile = file_get_contents($vendorPath . '/academicpuma/locales/locales-' . self::$langBase[$lang] . '.xml');
         } else {
-            $localeFile = file_get_contents($vendorPath.'/academicpuma/locales/locales-en-US.xml');
+            $localeFile = file_get_contents($vendorPath . '/academicpuma/locales/locales-en-US.xml');
         }
 
         return $localeFile;

Braces +3 added lines, -2 removed lines

@@ -218,8 +218,9 @@
                     $option = $this->locale->xpath("//cs:style-options[@$arg1]");
                 }
                 foreach ($option[0]->attributes() as $name => $value) {
-                    if ($name == $arg1)
-                        return (string) $value;
+                    if ($name == $arg1) {
+                                            return (string) $value;
+                    }
                 }
                 break;
         }

src/Name.php

Doc Comments +3 added lines

@@ -306,6 +306,9 @@
             $print, $punct, $space, $upper, $word, $patternModifiers);
     }
 
+    /**
+     * @return string
+     */
     protected function format($text, $part = 'base')
     {
 

Spacing +2 added lines, -2 removed lines

@@ -169,9 +169,9 @@
 
         if ($this->form == 'count') {
             if (!$et_al_triggered) {
-                return (int)count($authors);
+                return (int) count($authors);
             } else {
-                return (int)(count($authors) - 1);
+                return (int) (count($authors) - 1);
             }
         }
         // strip out the last delimiter if not required

Braces +21 added lines, -14 removed lines

@@ -58,8 +58,9 @@ 
         $auth_count = 0;
         $et_al_triggered = false;
 
-        if (!$this->attr_init || $this->attr_init != $mode)
-            $this->initAttributes($mode);
+        if (!$this->attr_init || $this->attr_init != $mode) {
+                    $this->initAttributes($mode);
+        }
 
         $initialize_with = $this->{'initialize-with'};
 
@@ -123,8 +124,9 @@ 
                 }
                 $authors[] = trim($this->format($text));
             }
-            if (isset($this->{'et-al-min'}) && $count >= $this->{'et-al-min'})
-                break;
+            if (isset($this->{'et-al-min'}) && $count >= $this->{'et-al-min'}) {
+                            break;
+            }
         }
         if (isset($this->{'et-al-min'}) &&
             $count >= $this->{'et-al-min'} &&
@@ -309,8 +311,9 @@ 
     protected function format($text, $part = 'base')
     {
 
-        if (empty($text) || $this->noOp[$part])
-            return $text;
+        if (empty($text) || $this->noOp[$part]) {
+                    return $text;
+        }
         if (isset($this->{$part}['text-case'])) {
             switch ($this->{$part}['text-case']) {
                 case 'uppercase':
@@ -332,8 +335,9 @@ 
         $close_quote = isset($this->{$part}['close-quote']) ? $this->{$part}['close-quote'] : '';
         $prefix = isset($this->{$part}['prefix']) ? $this->{$part}['prefix'] : '';
         $suffix = isset($this->{$part}['suffix']) ? $this->{$part}['suffix'] : '';
-        if ($text[(strlen($text) - 1)] == $suffix)
-            unset($suffix);
+        if ($text[(strlen($text) - 1)] == $suffix) {
+                    unset($suffix);
+        }
         if (!empty($this->format[$part])) {
             $text = '<span style="' . $this->format[$part] . '">' . $text . '</span>';
         }
@@ -381,10 +385,12 @@ 
             $this->noOp[$part] = false;
         }
 
-        if (isset($attribs['prefix']))
-            $this->{$part}['prefix'] = $attribs['prefix'];
-        if (isset($attribs['suffix']))
-            $this->{$part}['suffix'] = $attribs['suffix'];
+        if (isset($attribs['prefix'])) {
+                    $this->{$part}['prefix'] = $attribs['prefix'];
+        }
+        if (isset($attribs['suffix'])) {
+                    $this->{$part}['suffix'] = $attribs['suffix'];
+        }
 
         $this->format[$part] .= (isset($attribs['font-style'])) ? 'font-style: ' . $attribs['font-style'] . ';' : '';
         $this->format[$part] .= (isset($attribs['font-family'])) ? 'font-family: ' . $attribs['font-family'] . ';' : '';
@@ -397,8 +403,9 @@ 
             $this->noOp[$part] = false;
             $this->{$part}['text-case'] = $attribs['text-case'];
         }
-        if (!empty($this->format[$part]))
-            $this->noOp[$part] = false;
+        if (!empty($this->format[$part])) {
+                    $this->noOp[$part] = false;
+        }
     }
 
 }
\ No newline at end of file

src/HierAttributes.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
         if (empty(HierAttributes::$arr)) {
             $refClass = new \ReflectionClass('\AcademicPuma\CiteProc\HierAttributes');
             $constants = $refClass->getConstants();
-            array_walk($constants, function ($value) {
+            array_walk($constants, function($value) {
                 HierAttributes::$arr[$value] = $value;
             });
         }

src/Date.php

Braces +3 added lines, -2 removed lines

@@ -94,9 +94,10 @@
                 }
                 // now re-order the elements
                 foreach ($parts as $part) {
-                    foreach ($locale_elements as $key => $element)
-                        if ($element->name == $part) {
+                    foreach ($locale_elements as $key => $element) {
+                                            if ($element->name == $part) {
                             $this->elements[] = $element;
+                    }
                             unset($locale_elements[$key]);
                         }
                 }

src/Names.php

Braces +18 added lines, -12 removed lines

@@ -55,8 +55,9 @@ 
         foreach ($domNode->childNodes as $node) {
             if ($node->nodeType == 1) {
                 $element = Factory::create($node, $citeProc);
-                if (($element instanceof Label))
-                    $element->variable = $var;
+                if (($element instanceof Label)) {
+                                    $element->variable = $var;
+                }
                 if (($element instanceof Name) && $etal) {
                     $element->etal = $etal;
                 }
@@ -79,8 +80,9 @@ 
         $variables = explode(' ', $this->variable);
 
         foreach ($variables as $var) {
-            if (in_array($var, $this->citeProc->quash))
-                continue;
+            if (in_array($var, $this->citeProc->quash)) {
+                            continue;
+            }
             if (isset($data->{$var}) && (!empty($data->{$var}))) {
                 $matches[] = $var;
             }
@@ -100,18 +102,21 @@ 
                     } else { // if it's not a "names" element, just render it
                         $text = $element->render($data, $mode);
                         $this->citeProc->quash[] = isset($element->variable) ? $element->variable : $element->var;
-                        if (!empty($text))
-                            $variable_parts[] = $text;
+                        if (!empty($text)) {
+                                                    $variable_parts[] = $text;
+                        }
+                    }
+                    if (!empty($matches)) {
+                                            break;
                     }
-                    if (!empty($matches))
-                        break;
                 }
             }
         }
 
         foreach ($matches as $var) {
-            if (in_array($var, $this->citeProc->quash) && in_array($var, $variables))
-                continue;
+            if (in_array($var, $this->citeProc->quash) && in_array($var, $variables)) {
+                            continue;
+            }
             $text = '';
             if (!empty($data->{$var})) {
                 foreach ($this->elements as $element) {
@@ -123,8 +128,9 @@ 
                     }
                 }
             }
-            if (!empty($text))
-                $variable_parts[] = $text;
+            if (!empty($text)) {
+                            $variable_parts[] = $text;
+            }
         }
 
         if (!empty($variable_parts)) {

src/Group.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
             $element_count++;
             if (($element instanceof Text) &&
                     ($element->source == 'term' ||
-                    $element->source == 'value' )) {
+                    $element->source == 'value')) {
                 $terms++;
             }
             if (($element instanceof Label)) {
@@ -62,7 +62,7 @@ 
                     }
                 }
                 //give the text parts a name
-                if($element instanceof Text) {
+                if ($element instanceof Text) {
                     $text_parts[$element->getVar()] = $text;
                 } else {
                     $text_parts[$element_count] = $text;

Braces +11 added lines, -6 removed lines

@@ -78,12 +78,17 @@
                 }
             }
         }
-        if (empty($text_parts))
-            return;
-        if ($variables && !$have_variables)
-            return; // there has to be at least one other none empty value before the term is output
-        if (count($text_parts) == $terms)
-            return; // there has to be at least one other none empty value before the term is output
+        if (empty($text_parts)) {
+                    return;
+        }
+        if ($variables && !$have_variables) {
+                    return;
+        }
+        // there has to be at least one other none empty value before the term is output
+        if (count($text_parts) == $terms) {
+                    return;
+        }
+        // there has to be at least one other none empty value before the term is output
         $delimiter = $this->delimiter;
         //$text = implode($delimiter, $text_parts); // insert the delimiter if supplied.
         $text = $this->implodeGroup($delimiter, $text_parts);