Dolibarr / dolibarr

htdocs/api/class/api_documents.class.php

Braces +3 added lines, -1 removed lines

@@ -639,7 +639,9 @@
 								$filearray[$i] = array_merge($filearray[$i], (array) $line);
 							}
 						}
-						if (isset($line->filename)) $filearray[$i]['content-type'] = dol_mimetype($line->filename);
+						if (isset($line->filename)) {
+							$filearray[$i]['content-type'] = dol_mimetype($line->filename);
+						}
 						$arraycontenttype = explode(",", $content_type);
 						if (!empty($content_type) && isset($line->filename) && !in_array(dol_mimetype($line->filename), $arraycontenttype)) {
 							unset($filearray[$i]);

htdocs/contrat/class/contrat.class.php

Braces +4 added lines, -1 removed lines

@@ -1268,7 +1268,10 @@
 		// Delete record into ECM index and physically
 		if (!$error) {
 			$res = $this->deleteEcmFiles(0); // Deleting files physically is done later with the dol_delete_dir_recursive
-			if ($res) $res = $this->deleteEcmFiles(1); // Deleting files physically is done later with the dol_delete_dir_recursive
+			if ($res) {
+				$res = $this->deleteEcmFiles(1);
+			}
+			// Deleting files physically is done later with the dol_delete_dir_recursive
 			if (!$res) {
 				$error++;
 			}

htdocs/accountancy/journal/bankjournal.php

Braces +3 added lines, -1 removed lines

@@ -404,7 +404,9 @@
 			$resultamount = $db->query($sqlamount);
 			if ($resultamount) {
 				$objamount = $db->fetch_object($resultamount);
-				if (!empty($objamount->amount)) $amounttouse = $objamount->amount;
+				if (!empty($objamount->amount)) {
+					$amounttouse = $objamount->amount;
+				}
 			}
 		}
 

htdocs/bom/tpl/objectline_view.tpl.php

Braces +3 added lines, -1 removed lines

@@ -223,7 +223,9 @@
 print '<td id="costline_'.$line->id.'" class="linecolcost nowrap right">';
 
 $line->qty = (float) $line->qty;
-if ($tmpbom->id > 0) $line->qty /= $tmpbom->qty;
+if ($tmpbom->id > 0) {
+	$line->qty /= $tmpbom->qty;
+}
 
 $coldisplay++;
 if (!empty($line->fk_bom_child)) {

htdocs/includes/tcpdi/fpdf_tpl.php

Braces +60 added lines, -40 removed lines

@@ -79,17 +79,22 @@ 
             return;
         }
         
-        if ($this->page <= 0)
-            $this->error("You have to add a page to fpdf first!");
+        if ($this->page <= 0) {
+                    $this->error("You have to add a page to fpdf first!");
+        }
 
-        if ($x == null)
-            $x = 0;
-        if ($y == null)
-            $y = 0;
-        if ($w == null)
-            $w = $this->w;
-        if ($h == null)
-            $h = $this->h;
+        if ($x == null) {
+                    $x = 0;
+        }
+        if ($y == null) {
+                    $y = 0;
+        }
+        if ($w == null) {
+                    $w = $this->w;
+        }
+        if ($h == null) {
+                    $h = $this->h;
+        }
 
         // Save settings
         $this->tpl++;
@@ -165,8 +170,9 @@ 
             $this->FontSize = $tpl['o_FontSize'];
             
             $fontkey = $this->FontFamily . $this->FontStyle;
-            if ($fontkey)
-                $this->CurrentFont =& $this->fonts[$fontkey];
+            if ($fontkey) {
+                            $this->CurrentFont =& $this->fonts[$fontkey];
+            }
             
             return $this->tpl;
         } else {
@@ -192,11 +198,13 @@ 
      * @retrun array The height and width of the template
      */
     function useTemplate($tplidx, $_x = null, $_y = null, $_w = 0, $_h = 0) {
-        if ($this->page <= 0)
-            $this->error('You have to add a page first!');
+        if ($this->page <= 0) {
+                    $this->error('You have to add a page first!');
+        }
         
-        if (!isset($this->tpls[$tplidx]))
-            $this->error('Template does not exist!');
+        if (!isset($this->tpls[$tplidx])) {
+                    $this->error('Template does not exist!');
+        }
             
         if ($this->_intpl) {
             $this->_res['tpl'][$this->tpl]['tpls'][$tplidx] =& $this->tpls[$tplidx];
@@ -206,10 +214,12 @@ 
         $w = $tpl['w'];
         $h = $tpl['h'];
         
-        if ($_x == null)
-            $_x = 0;
-        if ($_y == null)
-            $_y = 0;
+        if ($_x == null) {
+                    $_x = 0;
+        }
+        if ($_y == null) {
+                    $_y = 0;
+        }
             
         $_x += $tpl['x'];
         $_y += $tpl['y'];
@@ -249,8 +259,9 @@ 
      * @return array The height and width of the template
      */
     function getTemplateSize($tplidx, $_w = 0, $_h = 0) {
-        if (!isset($this->tpls[$tplidx]))
-            return false;
+        if (!isset($this->tpls[$tplidx])) {
+                    return false;
+        }
 
         $tpl =& $this->tpls[$tplidx];
         $w = $tpl['w'];
@@ -261,10 +272,12 @@ 
             $_h = $h;
         }
 
-        if($_w == 0)
-            $_w = $_h * $w / $h;
-        if($_h == 0)
-            $_h = $_w * $h / $w;
+        if($_w == 0) {
+                    $_w = $_h * $w / $h;
+        }
+        if($_h == 0) {
+                    $_h = $_w * $h / $w;
+        }
             
         return array("w" => $_w, "h" => $_h);
     }
@@ -323,8 +336,9 @@ 
 			return parent::AddPage(...$args);
         }
         
-        if ($this->_intpl)
-            $this->Error('Adding pages in templates isn\'t possible!');
+        if ($this->_intpl) {
+                    $this->Error('Adding pages in templates isn\'t possible!');
+        }
             
         parent::AddPage($orientation, $format);
     }
@@ -338,8 +352,9 @@ 
 			return parent::Link(...$args);
         }
         
-        if ($this->_intpl)
-            $this->Error('Using links in templates aren\'t possible!');
+        if ($this->_intpl) {
+                    $this->Error('Using links in templates aren\'t possible!');
+        }
             
         parent::Link($x, $y, $w, $h, $link);
     }
@@ -351,8 +366,9 @@ 
 
         }
         
-        if ($this->_intpl)
-            $this->Error('Adding links in templates aren\'t possible!');
+        if ($this->_intpl) {
+                    $this->Error('Adding links in templates aren\'t possible!');
+        }
         return parent::AddLink();
     }
     
@@ -362,8 +378,9 @@ 
 			return parent::SetLink(...$args);
         }
         
-        if ($this->_intpl)
-            $this->Error('Setting links in templates aren\'t possible!');
+        if ($this->_intpl) {
+                    $this->Error('Setting links in templates aren\'t possible!');
+        }
         parent::SetLink($link, $y, $page);
     }
     
@@ -403,8 +420,9 @@ 
             $this->_out('<</ProcSet [/PDF /Text /ImageB /ImageC /ImageI]');
             if (isset($this->_res['tpl'][$tplidx]['fonts']) && count($this->_res['tpl'][$tplidx]['fonts'])) {
                 $this->_out('/Font <<');
-                foreach($this->_res['tpl'][$tplidx]['fonts'] as $font)
-                    $this->_out('/F' . $font['i'] . ' ' . $font['n'] . ' 0 R');
+                foreach($this->_res['tpl'][$tplidx]['fonts'] as $font) {
+                                    $this->_out('/F' . $font['i'] . ' ' . $font['n'] . ' 0 R');
+                }
                 $this->_out('>>');
             }
             if(isset($this->_res['tpl'][$tplidx]['images']) && count($this->_res['tpl'][$tplidx]['images']) || 
@@ -412,12 +430,14 @@ 
             {
                 $this->_out('/XObject <<');
                 if (isset($this->_res['tpl'][$tplidx]['images']) && count($this->_res['tpl'][$tplidx]['images'])) {
-                    foreach($this->_res['tpl'][$tplidx]['images'] as $image)
-                          $this->_out('/I' . $image['i'] . ' ' . $image['n'] . ' 0 R');
+                    foreach($this->_res['tpl'][$tplidx]['images'] as $image) {
+                                              $this->_out('/I' . $image['i'] . ' ' . $image['n'] . ' 0 R');
+                    }
                 }
                 if (isset($this->_res['tpl'][$tplidx]['tpls']) && count($this->_res['tpl'][$tplidx]['tpls'])) {
-                    foreach($this->_res['tpl'][$tplidx]['tpls'] as $i => $tpl)
-                        $this->_out($this->tplprefix . $i . ' ' . $tpl['n'] . ' 0 R');
+                    foreach($this->_res['tpl'][$tplidx]['tpls'] as $i => $tpl) {
+                                            $this->_out($this->tplprefix . $i . ' ' . $tpl['n'] . ' 0 R');
+                    }
                 }
                 $this->_out('>>');
             }

htdocs/core/lib/accounting.lib.php

Braces +4 added lines, -1 removed lines

@@ -175,7 +175,10 @@
 {
 	global $conf;
 
-	if (empty($auxiliary_account)) return true; // No check needed if no auxiliary account used
+	if (empty($auxiliary_account)) {
+		return true;
+	}
+	// No check needed if no auxiliary account used
 
 	// Build SQL to get general account info based on rowid or account number
 	$sql = "SELECT rowid, account_number, centralized";

htdocs/core/class/html.formmail.class.php

Braces +4 added lines, -1 removed lines

@@ -657,7 +657,10 @@
 					if (in_array($key, array('__NEWREF__', '__REFCLIENT__', '__REFSUPPLIER__', '__SUPPLIER_ORDER_DATE_DELIVERY__', '__SUPPLIER_ORDER_DELAY_DELIVERY__'))) {
 						continue;
 					}
-					if (is_array($val)) $val = implode(', ', $val); // key __MULTICURRENCY_CODE__ is an array and crashes dolGetFirstLineOfText function which accept only text
+					if (is_array($val)) {
+						$val = implode(', ', $val);
+					}
+					// key __MULTICURRENCY_CODE__ is an array and crashes dolGetFirstLineOfText function which accept only text
 					$helpforsubstitution .= $key.' -> '.$langs->trans(dol_string_nohtmltag(dolGetFirstLineOfText($val))).'<br>';
 					$helpforsubstitution .= '</span>';
 				}

htdocs/accountancy/class/accountingjournal.class.php

Braces +50 added lines, -18 removed lines

@@ -378,15 +378,21 @@ 
 				case 1: // Various Journal
 					if (isModEnabled('asset') && !getDolGlobalInt('ACCOUNTING_DISABLE_TRANSFER_ON_ASSETS')) {
 						$tmp = $this->getAssetData($user, $type, $date_start, $date_end, $in_bookkeeping);
-						if (is_array($tmp)) $data = array_merge($data, $tmp);
+						if (is_array($tmp)) {
+							$data = array_merge($data, $tmp);
+						}
 					}
 					if (isModEnabled('invoice') && !getDolGlobalInt('ACCOUNTING_DISABLE_TRANSFER_ON_DISCOUNTS')) {
 						$tmp = $this->getDiscountCustomer($user, $type, $date_start, $date_end, $in_bookkeeping);
-						if (is_array($tmp)) $data = array_merge($data, $tmp);
+						if (is_array($tmp)) {
+							$data = array_merge($data, $tmp);
+						}
 					}
 					if (isModEnabled('supplier_invoice') && !getDolGlobalInt('ACCOUNTING_DISABLE_TRANSFER_ON_DISCOUNTS')) {
 						$tmp = $this->getDiscountSupplier($user, $type, $date_start, $date_end, $in_bookkeeping);
-						if (is_array($tmp)) $data = array_merge($data, $tmp);
+						if (is_array($tmp)) {
+							$data = array_merge($data, $tmp);
+						}
 					}
 					break;
 					//              case 2: // Sells Journal
@@ -745,8 +751,12 @@ 
 		$langs->loadLangs(array('bills'));
 
 		// Clean parameters
-		if (empty($type)) $type = 'view';
-		if (empty($in_bookkeeping)) $in_bookkeeping = 'notyet';
+		if (empty($type)) {
+			$type = 'view';
+		}
+		if (empty($in_bookkeeping)) {
+			$in_bookkeeping = 'notyet';
+		}
 
 		// Build SQL - Customer invoices closed by discount
 		$sql = "SELECT f.rowid, f.ref, f.datef, f.fk_soc, f.total_ttc";
@@ -794,7 +804,9 @@ 
 		$acc_vat_coll_def = getDolGlobalString('ACCOUNTING_VAT_BUY_ACCOUNT');			// Normal to apply vat default account for buy with customer's discount
 
 		while ($obj = $this->db->fetch_object($resql)) {
-			if ($invoice_static->fetch((int) $obj->rowid) <= 0) continue;
+			if ($invoice_static->fetch((int) $obj->rowid) <= 0) {
+				continue;
+			}
 
 			$customer_static->fetch($invoice_static->socid);
 			$account_customer_general = !empty($customer_static->accountancy_code_customer_general) ? $customer_static->accountancy_code_customer_general : getDolGlobalString('ACCOUNTING_ACCOUNT_CUSTOMER');
@@ -808,7 +820,9 @@ 
 			$useddep = (float) price2num($invoice_static->getSumDepositsUsed(), 'MT');
 			$ttc_inv = (float) price2num($invoice_static->total_ttc, 'MT');
 			$escompte_ttc = (float) price2num(max(0, $ttc_inv - $paid - $usedcn - $useddep), 'MT');
-			if ($escompte_ttc <= 0) continue;
+			if ($escompte_ttc <= 0) {
+				continue;
+			}
 
 			$bookkeeping_static = new BookKeeping($this->db);
 			$label_discount = $bookkeeping_static->accountingLabelForOperation($customer_static->getNomUrl(1, 'customer'), $invoice_static->ref, $langs->trans('DiscountGranted'));
@@ -817,9 +831,13 @@ 
 			$ttcByRate = array(); $totalTTC = 0.0;
 			foreach ((array) $invoice_static->lines as $li) {
 				$ttc = (float) $li->total_ttc;
-				if (!$ttc) continue;
+				if (!$ttc) {
+					continue;
+				}
 				$key = number_format((float) $li->tva_tx, 3, '.', '');
-				if (!isset($ttcByRate[$key])) $ttcByRate[$key] = 0.0;
+				if (!isset($ttcByRate[$key])) {
+					$ttcByRate[$key] = 0.0;
+				}
 				$ttcByRate[$key] += $ttc;
 				$totalTTC += $ttc;
 			}
@@ -1035,8 +1053,12 @@ 
 		$langs->loadLangs(array('suppliers'));
 
 		// Clean parameters
-		if (empty($type)) $type = 'view';
-		if (empty($in_bookkeeping)) $in_bookkeeping = 'notyet';
+		if (empty($type)) {
+			$type = 'view';
+		}
+		if (empty($in_bookkeeping)) {
+			$in_bookkeeping = 'notyet';
+		}
 
 		// SQL - Supplier invoices closed by discount
 		$sql = "SELECT ff.rowid, ff.ref, ff.datef, ff.fk_soc, ff.total_ttc";
@@ -1084,7 +1106,9 @@ 
 		$acc_vat_ded_def  = getDolGlobalString('ACCOUNTING_VAT_SOLD_ACCOUNT');			// Normal to apply vat default account for sold with supplier's discount
 
 		while ($obj = $this->db->fetch_object($resql)) {
-			if ($invoicesupplier_static->fetch((int) $obj->rowid) <= 0) continue;
+			if ($invoicesupplier_static->fetch((int) $obj->rowid) <= 0) {
+				continue;
+			}
 
 			$supplier_static->fetch($invoicesupplier_static->socid);
 			$account_supplier_general = !empty($supplier_static->accountancy_code_supplier_general) ? $supplier_static->accountancy_code_supplier_general : getDolGlobalString('ACCOUNTING_ACCOUNT_SUPPLIER');
@@ -1098,7 +1122,9 @@ 
 			$useddep = (float) price2num($invoicesupplier_static->getSumDepositsUsed(), 'MT');
 			$ttc_inv = (float) price2num($invoicesupplier_static->total_ttc, 'MT');
 			$escompte_ttc = (float) price2num(max(0, $ttc_inv - $paid - $usedcn - $useddep), 'MT');
-			if ($escompte_ttc <= 0) continue;
+			if ($escompte_ttc <= 0) {
+				continue;
+			}
 
 			$bookkeeping_static = new BookKeeping($this->db);
 			$label_discount = $bookkeeping_static->accountingLabelForOperation($supplier_static->getNomUrl(1, 'supplier'), $invoicesupplier_static->ref, $langs->trans('DiscountReceived'));
@@ -1107,9 +1133,13 @@ 
 			$ttcByRate = array(); $totalTTC = 0.0;
 			foreach ((array) $invoicesupplier_static->lines as $li) {
 				$ttc = (float) $li->total_ttc;
-				if (!$ttc) continue;
+				if (!$ttc) {
+					continue;
+				}
 				$key = number_format((float) $li->tva_tx, 3, '.', '');
-				if (!isset($ttcByRate[$key])) $ttcByRate[$key] = 0.0;
+				if (!isset($ttcByRate[$key])) {
+					$ttcByRate[$key] = 0.0;
+				}
 				$ttcByRate[$key] += $ttc;
 				$totalTTC += $ttc;
 			}
@@ -1273,9 +1303,11 @@ 
 					}
 				} else {
 					// si TVA = 0, pousser les 2 lignes view/bookkeeping déjà constituées
-					if ($type == 'view') $element['blocks'][] = $lines_view;
-					elseif ($type == 'bookkeeping') $element['blocks'][] = $lines_book;
-					else { // csv
+					if ($type == 'view') {
+						$element['blocks'][] = $lines_view;
+					} elseif ($type == 'bookkeeping') {
+						$element['blocks'][] = $lines_book;
+					} else { // csv
 						$element['blocks'][] = array($docdate, $invoicesupplier_static->ref, length_accountg($account_supplier_general), $label_discount.' - '.$langs->transnoentitiesnoconv('Supplier'), price($ttc_part), '');
 						$element['blocks'][] = array($docdate, $invoicesupplier_static->ref, length_accountg($acc_disc_recv), $label_discount.' ('.$rateStr.'%)', '', price($ht_part));
 					}

htdocs/projet/class/api_tasks.class.php

Braces +3 added lines, -2 removed lines

@@ -608,8 +608,9 @@
 		$this->task->timespent_duration = $duration;
 		$this->task->timespent_fk_user  = $uid;
 		$this->task->timespent_note     = $note;
-		if (!empty($this->task->progress))
-			$this->task->progress  		= $progress;
+		if (!empty($this->task->progress)) {
+					$this->task->progress  		= $progress;
+		}
 
 		$result = $this->task->addTimeSpent(DolibarrApiAccess::$user, 0);
 		if ($result == 0) {