seboettg / citeproc-php

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

src/Locale.php

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/PIf.php

Braces +45 added lines, -30 removed lines

@@ -36,35 +36,45 @@ 
             $matches = 0;
             foreach ($types as $type) {
                 if (isset($data->type)) {
-                    if ($data->type == $type && $match == 'any')
-                        return true;
-                    if ($data->type != $type && $match == 'all')
-                        return false;
-                    if ($data->type == $type)
-                        $matches++;
+                    if ($data->type == $type && $match == 'any') {
+                                            return true;
+                    }
+                    if ($data->type != $type && $match == 'all') {
+                                            return false;
+                    }
+                    if ($data->type == $type) {
+                                            $matches++;
+                    }
                 }
             }
-            if ($match == 'all' && $matches == count($types))
-                return true;
-            if ($match == 'none' && $matches == 0)
-                return true;
+            if ($match == 'all' && $matches == count($types)) {
+                            return true;
+            }
+            if ($match == 'none' && $matches == 0) {
+                            return true;
+            }
             return false;
         }
         if (($variables = $this->variable)) {
             $variables = explode(' ', $variables);
             $matches = 0;
             foreach ($variables as $var) {
-                if (isset($data->$var) && !empty($data->$var) && $match == 'any')
-                    return true;
-                if ((!isset($data->$var) || empty($data->$var)) && $match == 'all')
-                    return false;
-                if (isset($data->$var) && !empty($data->$var))
-                    $matches++;
+                if (isset($data->$var) && !empty($data->$var) && $match == 'any') {
+                                    return true;
+                }
+                if ((!isset($data->$var) || empty($data->$var)) && $match == 'all') {
+                                    return false;
+                }
+                if (isset($data->$var) && !empty($data->$var)) {
+                                    $matches++;
+                }
+            }
+            if ($match == 'all' && $matches == count($variables)) {
+                            return true;
+            }
+            if ($match == 'none' && $matches == 0) {
+                            return true;
             }
-            if ($match == 'all' && $matches == count($variables))
-                return true;
-            if ($match == 'none' && $matches == 0)
-                return true;
             return false;
         }
         if (($is_numeric = $this->{'is-numeric'})) {
@@ -72,8 +82,9 @@ 
             $matches = 0;
             foreach ($variables as $var) {
                 if (isset($data->$var)) {
-                    if (is_numeric($data->$var) && $match == 'any')
-                        return true;
+                    if (is_numeric($data->$var) && $match == 'any') {
+                                            return true;
+                    }
                     if (!is_numeric($data->$var)) {
                         if (preg_match('/(?:^\d+|\d+$)/', $data->$var)) {
                             $matches++;
@@ -81,18 +92,22 @@ 
                             return false;
                         }
                     }
-                    if (is_numeric($data->$var))
-                        $matches++;
+                    if (is_numeric($data->$var)) {
+                                            $matches++;
+                    }
                 }
             }
-            if ($match == 'all' && $matches == count($variables))
-                return true;
-            if ($match == 'none' && $matches == 0)
-                return true;
+            if ($match == 'all' && $matches == count($variables)) {
+                            return true;
+            }
+            if ($match == 'none' && $matches == 0) {
+                            return true;
+            }
             return false;
         }
-        if (isset($this->locator))
-            $test = explode(' ', $this->type);
+        if (isset($this->locator)) {
+                    $test = explode(' ', $this->type);
+        }
 
         return false;
     }

src/DatePart.php

Braces +3 added lines, -2 removed lines

@@ -46,8 +46,9 @@
                 break;
             case 'month':
                 $text = (isset($date[1])) ? $date[1] : '';
-                if (empty($text) || $text < 1 || $text > 12)
-                    return;
+                if (empty($text) || $text < 1 || $text > 12) {
+                                    return;
+                }
                 // $form = $this->form;
                 switch ($this->form) {
                     case 'numeric': break;

src/Number.php

Braces +3 added lines, -2 removed lines

@@ -30,8 +30,9 @@
     public function render($data, $mode = NULL) {
         $var = $this->variable;
 
-        if (!$var || empty($data->$var))
-            return;
+        if (!$var || empty($data->$var)) {
+                    return;
+        }
 
         //   $form = $this->form;
 

src/CiteProc.php

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');
     }
 
     /**