seboettg / citeproc-php

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

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

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

Braces +6 added lines, -4 removed lines

@@ -64,10 +64,12 @@
             }
         }
 
-        if (empty($text))
-            return;
-        if ($this->{'strip-periods'})
-            $text = str_replace('.', '', $text);
+        if (empty($text)) {
+                    return;
+        }
+        if ($this->{'strip-periods'}) {
+                    $text = str_replace('.', '', $text);
+        }
         return $this->format($text);
     }
 

src/Name.php

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