intraface / intraface.dk

src/Intraface/modules/cms/element/Filelist.php

Braces +3 added lines, -1 removed lines

@@ -52,7 +52,9 @@
             // @todo this should be avoided

             $filehandler->createInstance();

             // HACK lille hack - til at undg� at vi f�r filer med som ikke har beskrivelser (formentlig slettede filer)

-            if (!$filehandler->get('description')) continue;

+            if (!$filehandler->get('description')) {
+                continue;
+            }
             $this->value['files'][$i] = $filehandler->get();

             $this->value['files'][$i]['append_file_id'] = $append_file_id;

             // $this->value['pictures'][$i]['show_uri'] = $file_uri;

src/Intraface/modules/cms/element/Map.php

Braces +21 added lines, -7 removed lines

@@ -121,13 +121,27 @@
 

     function save_element($var)

     {

-        if (!empty($var['service'])) $this->parameter->save('service', $var['service']);

-        if (!empty($var['text'])) $this->parameter->save('text', $var['text']);

-        if (!empty($var['latitude'])) $this->parameter->save('latitude', $var['latitude']);

-        if (!empty($var['longitude'])) $this->parameter->save('longitude', $var['longitude']);

-        if (!empty($var['height'])) $this->parameter->save('height', $var['height']);

-        if (!empty($var['width'])) $this->parameter->save('width', $var['width']);

-        if (!empty($var['api_key'])) $this->parameter->save('api_key', $var['api_key']);

+        if (!empty($var['service'])) {
+            $this->parameter->save('service', $var['service']);
+        }
+        if (!empty($var['text'])) {
+            $this->parameter->save('text', $var['text']);
+        }
+        if (!empty($var['latitude'])) {
+            $this->parameter->save('latitude', $var['latitude']);
+        }
+        if (!empty($var['longitude'])) {
+            $this->parameter->save('longitude', $var['longitude']);
+        }
+        if (!empty($var['height'])) {
+            $this->parameter->save('height', $var['height']);
+        }
+        if (!empty($var['width'])) {
+            $this->parameter->save('width', $var['width']);
+        }
+        if (!empty($var['api_key'])) {
+            $this->parameter->save('api_key', $var['api_key']);
+        }
         return true;

     }

 }
\ No newline at end of file

src/Intraface/modules/cms/CKEditor.php

Braces +21 added lines, -26 removed lines

@@ -142,10 +142,11 @@ 
         $_config = $this->configSettings($config, $events);
 
         $js = $this->returnGlobalEvents();
-        if (!empty($_config))
-            $js .= "CKEDITOR.replace('".$name."', ".$this->jsEncode($_config).");";
-        else
-            $js .= "CKEDITOR.replace('".$name."');";
+        if (!empty($_config)) {
+                    $js .= "CKEDITOR.replace('".$name."', ".$this->jsEncode($_config).");";
+        } else {
+                    $js .= "CKEDITOR.replace('".$name."');";
+        }
 
         $out .= $this->script($js);
 
@@ -182,8 +183,7 @@ 
         $js = $this->returnGlobalEvents();
         if (!empty($_config)) {
             $js .= "CKEDITOR.replace('".$id."', ".$this->jsEncode($_config).");";
-        }
-        else {
+        } else {
             $js .= "CKEDITOR.replace('".$id."');";
         }
         $out .= $this->script($js);
@@ -226,12 +226,10 @@ 
         if (empty($_config)) {
             if (empty($className)) {
                 $js .= "CKEDITOR.replaceAll();";
-            }
-            else {
+            } else {
                 $js .= "CKEDITOR.replaceAll('".$className."');";
             }
-        }
-        else {
+        } else {
             $classDetection = "";
             $js .= "CKEDITOR.replaceAll( function(textarea, config) {\n";
             if (!empty($className)) {
@@ -289,8 +287,7 @@ 
     {
         if (!empty($event)) {
             $this->events[$event] = array();
-        }
-        else {
+        } else {
             $this->events = array();
         }
     }
@@ -329,8 +326,7 @@ 
     {
         if (!empty($event)) {
             $this->globalEvents[$event] = array();
-        }
-        else {
+        } else {
             $this->globalEvents = array();
         }
     }
@@ -381,11 +377,9 @@ 
             foreach ($_events as $eventName => $handlers) {
                 if (empty($handlers)) {
                     continue;
-                }
-                else if (count($handlers) == 1) {
+                } else if (count($handlers) == 1) {
                     $_config['on'][$eventName] = '@@'.$handlers[0];
-                }
-                else {
+                } else {
                     $_config['on'][$eventName] = '@@function (ev){';
                     foreach ($handlers as $handler => $code) {
                         $_config['on'][$eventName] .= '('.$code.')(ev);';
@@ -488,8 +482,7 @@ 
          */
         if (isset($_SERVER['SCRIPT_FILENAME'])) {
             $realPath = dirname($_SERVER['SCRIPT_FILENAME']);
-        }
-        else {
+        } else {
             /**
              * realpath — Returns canonicalized absolute pathname
              */
@@ -544,8 +537,7 @@ 
             // Use @@ to not use quotes when outputting string value
             if (strpos($val, '@@') === 0) {
                 return substr($val, 2);
-            }
-            else {
+            } else {
                 // All scalars are converted to strings to avoid indeterminism.
                 // PHP's "1" and 1 are equal for all PHP operators, but
                 // JS's "1" and 1 are not. So if we pass "1" or 1 from the PHP backend,
@@ -571,12 +563,15 @@ 
         $result = array();
         if ($isList)
         {
-            foreach ($val as $v) $result[] = $this->jsEncode($v);
+            foreach ($val as $v) {
+                $result[] = $this->jsEncode($v);
+            }
             return '[ ' . join(', ', $result) . ' ]';
-        }
-        else
+        } else
         {
-            foreach ($val as $k => $v) $result[] = $this->jsEncode($k).': '.$this->jsEncode($v);
+            foreach ($val as $k => $v) {
+                $result[] = $this->jsEncode($k).': '.$this->jsEncode($v);
+            }
             return '{ ' . join(', ', $result) . ' }';
         }
     }

src/Intraface/modules/cms/Controller/Index.php

Braces +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
     {
          if (is_numeric($name)) {
              return 'Intraface_modules_cms_Controller_Site';
-         }  elseif ($name == 'create') {
+         } elseif ($name == 'create') {
             return 'Intraface_modules_cms_Controller_SiteEdit';
         }
     }

src/Intraface/modules/accounting/Voucher.php

Braces +3 added lines, -1 removed lines

@@ -491,7 +491,9 @@
 

         foreach ($posts as $p) {

             $post = new Post($this, $p['id']);

-            if ($post->get('stated') == 1) continue;

+            if ($post->get('stated') == 1) {
+                continue;
+            }
 

             if (!$post->setStated()) {

                 $this->error->set('id#' .$p['id'] . ': Det lykkedes ikke at bogføre denne post.');

src/Intraface/modules/accounting/Year.php

Braces +51 added lines, -17 removed lines

@@ -622,58 +622,92 @@
     function setSettings($input)

     {

         if ($this->get('vat') > 0) {

-            if (empty($input['vat_in_account_id'])) $input['vat_in_account_id'] = 0;

+            if (empty($input['vat_in_account_id'])) {
+                $input['vat_in_account_id'] = 0;
+            }
             $this->setSetting('vat_in_account_id', (int)$input['vat_in_account_id']);

 

-            if (empty($input['vat_out_account_id'])) $input['vat_out_account_id'] = 0;

+            if (empty($input['vat_out_account_id'])) {
+                $input['vat_out_account_id'] = 0;
+            }
             $this->setSetting('vat_out_account_id', (int)$input['vat_out_account_id']);

 

-            if (empty($input['vat_abroad_account_id'])) $input['vat_abroad_account_id'] = 0;

+            if (empty($input['vat_abroad_account_id'])) {
+                $input['vat_abroad_account_id'] = 0;
+            }
             $this->setSetting('vat_abroad_account_id', (int)$input['vat_abroad_account_id']);

 

-            if (empty($input['vat_balance_account_id'])) $input['vat_balance_account_id'] = 0;

+            if (empty($input['vat_balance_account_id'])) {
+                $input['vat_balance_account_id'] = 0;
+            }
             $this->setSetting('vat_balance_account_id', (int)$input['vat_balance_account_id']);

 

-            if (empty($input['vat_free_account_id'])) $input['vat_free_account_id'] = 0;

+            if (empty($input['vat_free_account_id'])) {
+                $input['vat_free_account_id'] = 0;
+            }
             $this->setSetting('vat_free_account_id', (int)$input['vat_free_account_id']);

 

-            if (empty($input['eu_sale_account_id'])) $input['eu_sale_account_id'] = 0;

+            if (empty($input['eu_sale_account_id'])) {
+                $input['eu_sale_account_id'] = 0;
+            }
             $this->setSetting('eu_sale_account_id', (int)$input['eu_sale_account_id']);

             //$this->setSetting('eu_buy_account_id', (int)$input['eu_buy_account_id']);

             //$this->setSetting('abroad_buy_account_id', (int)$input['abroad_buy_account_id']);

         }

-        if (empty($input['result_account_id'])) $input['result_account_id'] = 0;

+        if (empty($input['result_account_id'])) {
+            $input['result_account_id'] = 0;
+        }
         $this->setSetting('result_account_id', (int)$input['result_account_id']);

 

-        if (empty($input['debtor_account_id'])) $input['debtor_account_id'] = 0;

+        if (empty($input['debtor_account_id'])) {
+            $input['debtor_account_id'] = 0;
+        }
         $this->setSetting('debtor_account_id', (int)$input['debtor_account_id']);

 

-        if (empty($input['credit_account_id'])) $input['credit_account_id'] = 0;

+        if (empty($input['credit_account_id'])) {
+            $input['credit_account_id'] = 0;
+        }
         $this->setSetting('credit_account_id', (int)$input['credit_account_id']);

 

-        if (empty($input['balance_accounts'])) $input['balance_accounts'] = array();

+        if (empty($input['balance_accounts'])) {
+            $input['balance_accounts'] = array();
+        }
         $this->setSetting('balance_accounts', serialize($input['balance_accounts']));

 

-        if (empty($input['buy_abroad_accounts'])) $input['buy_abroad_accounts'] = array();

+        if (empty($input['buy_abroad_accounts'])) {
+            $input['buy_abroad_accounts'] = array();
+        }
         $this->setSetting('buy_abroad_accounts', serialize($input['buy_abroad_accounts']));

 

-        if (empty($input['buy_eu_accounts'])) $input['buy_eu_accounts'] = array();

+        if (empty($input['buy_eu_accounts'])) {
+            $input['buy_eu_accounts'] = array();
+        }
         $this->setSetting('buy_eu_accounts', serialize($input['buy_eu_accounts']));

 

 

-        if (empty($input['result_account_id_start'])) $input['result_account_id_start'] = 0;

+        if (empty($input['result_account_id_start'])) {
+            $input['result_account_id_start'] = 0;
+        }
         $this->setSetting('result_account_id_start', $input['result_account_id_start']);

 

-        if (empty($input['result_account_id_end'])) $input['result_account_id_end'] = 0;

+        if (empty($input['result_account_id_end'])) {
+            $input['result_account_id_end'] = 0;
+        }
         $this->setSetting('result_account_id_end', $input['result_account_id_end']);

 

-        if (empty($input['balance_account_id_start'])) $input['balance_account_id_start'] = 0;

+        if (empty($input['balance_account_id_start'])) {
+            $input['balance_account_id_start'] = 0;
+        }
         $this->setSetting('balance_account_id_start', $input['balance_account_id_start']);

 

-        if (empty($input['balance_account_id_end'])) $input['balance_account_id_end'] = 0;

+        if (empty($input['balance_account_id_end'])) {
+            $input['balance_account_id_end'] = 0;
+        }
         $this->setSetting('balance_account_id_end', $input['balance_account_id_end']);

 

-        if (empty($input['capital_account_id'])) $input['capital_account_id'] = 0;

+        if (empty($input['capital_account_id'])) {
+            $input['capital_account_id'] = 0;
+        }
         $this->setSetting('capital_account_id', $input['capital_account_id']);

 

         return true;

src/Intraface/modules/accounting/VoucherFile.php

Braces +3 added lines, -1 removed lines

@@ -69,7 +69,9 @@
             $sql_end = " , date_created = NOW()";

         }

 

-        if (empty($var['description'])) $var['description'] = '';

+        if (empty($var['description'])) {
+            $var['description'] = '';
+        }
 

         $sql = $sql_type . " accounting_voucher_file SET

             date_updated = NOW(),

src/Intraface/modules/accounting/Controller/Year/End.php

Braces +8 added lines, -17 removed lines

@@ -59,11 +59,9 @@ 
         		$style = '';
         		if ($account['type'] == 'headline') {
         			$style = $format_bold;
-        		}
-        		elseif ($account['type'] == 'sum') {
+        		} elseif ($account['type'] == 'sum') {
         			$style = $format_italic;
-        		}
-        		else {
+        		} else {
         			$style = $format;
         		}
 
@@ -86,11 +84,9 @@ 
         		$style = '';
         		if ($account['type'] == 'headline') {
         			$style = $format_bold;
-        		}
-        		elseif ($account['type'] == 'sum') {
+        		} elseif ($account['type'] == 'sum') {
         			$style = $format_italic;
-        		}
-        		else {
+        		} else {
         			$style = $format;
         		}
 
@@ -122,8 +118,7 @@ 
         	$year_end = new YearEnd($year);
         	$year_end->setStep($_POST['step'] - 2);
 
-        }
-        elseif (!empty($_POST['next'])) {
+        } elseif (!empty($_POST['next'])) {
         	$year_end = new YearEnd($year);
         	$year_end->setStep($_POST['step']);
         }
@@ -138,9 +133,7 @@ 
         	// @todo save "resultatopgørelsen"
 
         	$year_end->setStep($_POST['step']);
-        }
-
-        elseif (!empty($_POST['step_save_balance'])) {
+        } elseif (!empty($_POST['step_save_balance'])) {
         	$year_end = new YearEnd($year);
 
         	if (!$year_end->saveStatement('balance')) {
@@ -148,8 +141,7 @@ 
         	}
 
         	$year_end->setStep($_POST['step']);
-        }
-        elseif (!empty($_POST['step_transfer_result'])) {
+        } elseif (!empty($_POST['step_transfer_result'])) {
         	$year_end = new YearEnd($year);
 
         	if (!$year_end->resetYearResult()) {
@@ -157,8 +149,7 @@ 
         	}
 
         	$year_end->setStep($_POST['step']);
-        }
-        elseif (!empty($_POST['step_reverse_result_account_reset'])) {
+        } elseif (!empty($_POST['step_reverse_result_account_reset'])) {
         	$year_end = new YearEnd($year);
         	if (!$year_end->resetYearResult('reverse')) {
         		echo $year_end->error->view();

src/Intraface/modules/webshop/Webshop.php

Braces +6 added lines, -2 removed lines

@@ -183,12 +183,16 @@
         $value['message'] = $input['message'];
 
         if (isset($input['customer_coupon']) && $input['customer_coupon'] != '') {
-            if ($value['message'] != '') $value['message'] .= "\n\n";
+            if ($value['message'] != '') {
+                $value['message'] .= "\n\n";
+            }
             $value['message'] .= "Kundekupon:". $input['customer_coupon'];
         }
 
         if (isset($input['customer_comment']) && $input['customer_comment'] != '') {
-            if ($value['message'] != '') $value['message'] .= "\n\n";
+            if ($value['message'] != '') {
+                $value['message'] .= "\n\n";
+            }
             $value['message'] .= "Kommentar:\n". $input['customer_comment'];
         }