Dolibarr / dolibarr

htdocs/expedition/card.php

Braces +6 added lines, -2 removed lines

@@ -1336,7 +1336,9 @@ 
 											$deliverableQty = min($quantityToBeDelivered, $stock);
 										}
 
-										if ($deliverableQty < 0) $deliverableQty = 0;
+										if ($deliverableQty < 0) {
+											$deliverableQty = 0;
+										}
 
 										$tooltip = '';
 										if (!empty($alreadyQtySetted[$line->fk_product][intval($warehouse_id)])) {
@@ -1435,7 +1437,9 @@ 
 											$deliverableQty = min($quantityToBeDelivered, $batchStock);
 										}
 
-										if ($deliverableQty < 0) $deliverableQty = 0;
+										if ($deliverableQty < 0) {
+											$deliverableQty = 0;
+										}
 
 										$inputName = 'qtyl'.$indiceAsked.'_'.$subj;
 										if (GETPOSTISSET($inputName)) {

Spacing +8 added lines, -8 removed lines

@@ -261,7 +261,7 @@ 
 			$stockLocation = "ent1".$i."_0";
 			$qty = "qtyl".$i;
 
-			$is_batch_or_serial=0;
+			$is_batch_or_serial = 0;
 			if (!empty($objectsrc->lines[$i]->fk_product)) {
 				$resultFetch = $product->fetch($objectsrc->lines[$i]->fk_product, '', '', '', 1, 1, 1);
 				if ($resultFetch < 0) {
@@ -287,9 +287,9 @@ 
 						//var_dump($sub_qty[$j]['id_batch']);
 
 						//var_dump($qty);var_dump($batch);var_dump($sub_qty[$j]['q']);var_dump($sub_qty[$j]['id_batch']);
-						if ($is_batch_or_serial==2 && $sub_qty[$j]['q']>1) {
+						if ($is_batch_or_serial == 2 && $sub_qty[$j]['q'] > 1) {
 							setEventMessages($langs->trans("TooManyQtyForSerialNumber", $product->ref, ''), null, 'errors');
-							$totalqty=0;
+							$totalqty = 0;
 							break 2;
 						}
 						$j++;
@@ -1494,7 +1494,7 @@ 
 							$nbofsuggested = 0;
 							foreach ($product->stock_warehouse as $warehouse_id => $stock_warehouse) {
 								if (($stock_warehouse->real > 0) && (count($stock_warehouse->detail_batch))) {
-									$nbofsuggested+=count($stock_warehouse->detail_batch);
+									$nbofsuggested += count($stock_warehouse->detail_batch);
 								}
 							}
 
@@ -2284,7 +2284,7 @@ 
 								// only show lot numbers from src warehouse when shipping from multiple warehouses
 								$line->fetch($detail_batch->fk_expeditiondet);
 							}
-							$entrepot_id = !empty($detail_batch->entrepot_id)?$detail_batch->entrepot_id:$lines[$i]->entrepot_id;
+							$entrepot_id = !empty($detail_batch->entrepot_id) ? $detail_batch->entrepot_id : $lines[$i]->entrepot_id;
 							print '<td>'.$formproduct->selectLotStock($detail_batch->fk_origin_stock, 'batchl'.$detail_batch->fk_expeditiondet.'_'.$detail_batch->fk_origin_stock, '', 1, 0, $lines[$i]->fk_product, $entrepot_id).'</td>';
 							print '</tr>';
 						}
@@ -2502,7 +2502,7 @@ 
 				 || (!empty($conf->global->MAIN_USE_ADVANCED_PERMS) && !empty($user->rights->expedition->shipping_advance->validate))) {
 					print dolGetButtonAction('', $langs->trans('Validate'), 'default', $_SERVER["PHP_SELF"].'?action=valid&token='.newToken().'&id='.$object->id, '');
 				} else {
-					print dolGetButtonAction($langs->trans('NotAllowed'), $langs->trans('Validate'), 'default', $_SERVER['PHP_SELF']. '#', '', false);
+					print dolGetButtonAction($langs->trans('NotAllowed'), $langs->trans('Validate'), 'default', $_SERVER['PHP_SELF'].'#', '', false);
 				}
 			}
 
@@ -2522,7 +2522,7 @@ 
 					if (empty($conf->global->MAIN_USE_ADVANCED_PERMS) || $user->rights->expedition->shipping_advance->send) {
 						print dolGetButtonAction('', $langs->trans('SendMail'), 'default', $_SERVER["PHP_SELF"].'?action=presend&token='.newToken().'&id='.$object->id.'&mode=init#formmailbeforetitle', '');
 					} else {
-						print dolGetButtonAction('', $langs->trans('SendMail'), 'default', $_SERVER['PHP_SELF']. '#', '', false);
+						print dolGetButtonAction('', $langs->trans('SendMail'), 'default', $_SERVER['PHP_SELF'].'#', '', false);
 					}
 				}
 			}
@@ -2550,7 +2550,7 @@ 
 						$label = "ClassifyBilled";
 						$paramaction = 'classifybilled';
 					}
-					print dolGetButtonAction('', $langs->trans($label), 'default', $_SERVER["PHP_SELF"].'?action='. $paramaction .'&token='.newToken().'&id='.$object->id, '');
+					print dolGetButtonAction('', $langs->trans($label), 'default', $_SERVER["PHP_SELF"].'?action='.$paramaction.'&token='.newToken().'&id='.$object->id, '');
 				}
 			}
 

htdocs/core/modules/modKnowledgeManagement.class.php

Spacing +3 added lines, -3 removed lines

@@ -301,7 +301,7 @@ 
 		*/
 		/* END MODULEBUILDER TOPMENU */
 
-		$this->menu[$r++]=array(
+		$this->menu[$r++] = array(
 			// '' if this is a top menu. For left menu, use 'fk_mainmenu=xxx' or 'fk_mainmenu=xxx,fk_leftmenu=yyy' where xxx is mainmenucode and yyy is a leftmenucode
 			'fk_menu'=>'fk_mainmenu=ticket',
 			// This is a Left menu entry
@@ -322,7 +322,7 @@ 
 			// 0=Menu for internal users, 1=external users, 2=both
 			'user'=>2,
 		);
-		$this->menu[$r++]=array(
+		$this->menu[$r++] = array(
 			// '' if this is a top menu. For left menu, use 'fk_mainmenu=xxx' or 'fk_mainmenu=xxx,fk_leftmenu=yyy' where xxx is mainmenucode and yyy is a leftmenucode
 			'fk_menu'=>'fk_mainmenu=ticket,fk_leftmenu=knowledgemanagement_knowledgerecord',
 			// This is a Left menu entry
@@ -342,7 +342,7 @@ 
 			// 0=Menu for internal users, 1=external users, 2=both
 			'user'=>2,
 		);
-		$this->menu[$r++]=array(
+		$this->menu[$r++] = array(
 			// '' if this is a top menu. For left menu, use 'fk_mainmenu=xxx' or 'fk_mainmenu=xxx,fk_leftmenu=yyy' where xxx is mainmenucode and yyy is a leftmenucode
 			'fk_menu'=>'fk_mainmenu=ticket,fk_leftmenu=knowledgemanagement_knowledgerecord',
 			// This is a Left menu entry

htdocs/compta/paiement/list.php

Spacing +3 added lines, -3 removed lines

@@ -64,7 +64,7 @@ 
 $search_date_endday = GETPOST('search_date_endday', 'int');
 $search_date_endmonth = GETPOST('search_date_endmonth', 'int');
 $search_date_endyear = GETPOST('search_date_endyear', 'int');
-$search_date_start = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear);	// Use tzserver
+$search_date_start = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear); // Use tzserver
 $search_date_end = dol_mktime(23, 59, 59, $search_date_endmonth, $search_date_endday, $search_date_endyear);
 $search_company = GETPOST("search_company", 'alpha');
 $search_paymenttype = GETPOST("search_paymenttype");
@@ -227,10 +227,10 @@ 
 		$sql .= natural_search('p.ref', $search_ref);
 	}
 	if ($search_date_start) {
-		$sql .= " AND p.datep >= '" . $db->idate($search_date_start) . "'";
+		$sql .= " AND p.datep >= '".$db->idate($search_date_start)."'";
 	}
 	if ($search_date_end) {
-		$sql .= " AND p.datep <= '" . $db->idate($search_date_end) . "'";
+		$sql .= " AND p.datep <= '".$db->idate($search_date_end)."'";
 	}
 	if ($search_account > 0) {
 		$sql .= " AND b.fk_account=".((int) $search_account);

Braces +3 added lines, -1 removed lines

@@ -51,7 +51,9 @@
 $userid = GETPOST('userid', 'int');
 
 // Security check
-if ($user->socid) $socid = $user->socid;
+if ($user->socid) {
+	$socid = $user->socid;
+}
 $result = restrictedArea($user, 'facture', $facid, '');
 
 $search_ref = GETPOST("search_ref", "alpha");

htdocs/core/modules/modFournisseur.class.php

Spacing +1 added lines, -1 removed lines

@@ -404,7 +404,7 @@
 		include DOL_DOCUMENT_ROOT.'/core/extrafieldsinexport.inc.php';
 		// End add extra fields object
 		$this->export_sql_start[$r] = 'SELECT DISTINCT ';
-		$this->export_sql_end[$r]  = ' FROM '.MAIN_DB_PREFIX.'societe as s';
+		$this->export_sql_end[$r] = ' FROM '.MAIN_DB_PREFIX.'societe as s';
 		if (is_object($user) && empty($user->rights->societe->client->voir)) {
 			$this->export_sql_end[$r] .= ' LEFT JOIN '.MAIN_DB_PREFIX.'societe_commerciaux as sc ON sc.fk_soc = s.rowid';
 		}

htdocs/core/class/html.formintervention.class.php

Spacing +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
 							$out .= '<option value="'.$obj->rowid.'" selected>'.$labeltoshow.'</option>';
 						} else {
 							$disabled = 0;
-							if (!$obj->fk_statut > 0 && ! $draftonly) {
+							if (!$obj->fk_statut > 0 && !$draftonly) {
 								$disabled = 1;
 								$labeltoshow .= ' ('.$langs->trans("Draft").')';
 							}

Braces +8 added lines, -3 removed lines

@@ -81,7 +81,9 @@ 
 				$sql .= " AND f.fk_soc = ".((int) $socid);
 			}
 		}
-		if ($draftonly) $sql .= " AND f.fk_statut = 0";
+		if ($draftonly) {
+			$sql .= " AND f.fk_statut = 0";
+		}
 
 		dol_syslog(get_class($this)."::select_intervention", LOG_DEBUG);
 		$resql = $this->db->query($sql);
@@ -89,8 +91,11 @@ 
 			$out .= '<select id="interventionid" class="flat" name="'.dol_escape_htmltag($htmlname).'">';
 			if ($showempty) {
 				$out .= '<option value="0">';
-				if (!is_numeric($showempty)) $out .= $showempty;
-				else $out .= '&nbsp;';
+				if (!is_numeric($showempty)) {
+					$out .= $showempty;
+				} else {
+					$out .= '&nbsp;';
+				}
 				$out .= '</option>';
 			}
 			$num = $this->db->num_rows($resql);

htdocs/expensereport/index.php

Spacing +1 added lines, -1 removed lines

@@ -136,7 +136,7 @@
 $dataseries = dol_sort_array($dataseries, 1, 'desc');
 
 // Merge all entrie after the $KEEPNFIRST one into one entry called "Other..." (to avoid to have too much entries in graphic).
-$KEEPNFIRST = 7;	// Keep first $KEEPNFIRST one + 1 with the remain
+$KEEPNFIRST = 7; // Keep first $KEEPNFIRST one + 1 with the remain
 $i = 0;
 if (count($dataseries) > ($KEEPNFIRST + 1)) {
 	foreach ($dataseries as $key => $val) {

htdocs/accountancy/class/accountancyexport.class.php

Indentation +8 added lines, -8 removed lines

@@ -1764,14 +1764,14 @@
 	}
 
 	/**
-	* Export format : iSuite Expert
-	*
-	* by OpenSolus [https://opensolus.fr]
-	*
-	* @param array $objectLines data
-	*
-	* @return void
-	*/
+	 * Export format : iSuite Expert
+	 *
+	 * by OpenSolus [https://opensolus.fr]
+	 *
+	 * @param array $objectLines data
+	 *
+	 * @return void
+	 */
 	public function exportiSuiteExpert($objectLines)
 	{
 		$this->separator = ';';

Spacing +64 added lines, -64 removed lines

@@ -950,63 +950,63 @@ 
 				}
 
 				// FEC:JournalCode
-				print $line->code_journal . $separator;
+				print $line->code_journal.$separator;
 
 				// FEC:JournalLib
-				print dol_string_unaccent($langs->transnoentities($line->journal_label)) . $separator;
+				print dol_string_unaccent($langs->transnoentities($line->journal_label)).$separator;
 
 				// FEC:EcritureNum
-				print $line->piece_num . $separator;
+				print $line->piece_num.$separator;
 
 				// FEC:EcritureDate
-				print $date_document . $separator;
+				print $date_document.$separator;
 
 				// FEC:CompteNum
-				print $line->numero_compte . $separator;
+				print $line->numero_compte.$separator;
 
 				// FEC:CompteLib
-				print dol_string_unaccent($line->label_compte) . $separator;
+				print dol_string_unaccent($line->label_compte).$separator;
 
 				// FEC:CompAuxNum
-				print $line->subledger_account . $separator;
+				print $line->subledger_account.$separator;
 
 				// FEC:CompAuxLib
-				print dol_string_unaccent($line->subledger_label) . $separator;
+				print dol_string_unaccent($line->subledger_label).$separator;
 
 				// FEC:PieceRef
-				print $line->doc_ref . $separator;
+				print $line->doc_ref.$separator;
 
 				// FEC:PieceDate
-				print dol_string_unaccent($date_creation) . $separator;
+				print dol_string_unaccent($date_creation).$separator;
 
 				// FEC:EcritureLib
 				// Clean label operation to prevent problem on export with tab separator & other character
 				$line->label_operation = str_replace(array("\t", "\n", "\r"), " ", $line->label_operation);
-				print dol_string_unaccent($line->label_operation) . $separator;
+				print dol_string_unaccent($line->label_operation).$separator;
 
 				// FEC:Debit
-				print price2fec($line->debit) . $separator;
+				print price2fec($line->debit).$separator;
 
 				// FEC:Credit
-				print price2fec($line->credit) . $separator;
+				print price2fec($line->credit).$separator;
 
 				// FEC:EcritureLet
-				print $line->lettering_code . $separator;
+				print $line->lettering_code.$separator;
 
 				// FEC:DateLet
-				print $date_lettering . $separator;
+				print $date_lettering.$separator;
 
 				// FEC:ValidDate
-				print $date_validation . $separator;
+				print $date_validation.$separator;
 
 				// FEC:Montantdevise
-				print $line->multicurrency_amount . $separator;
+				print $line->multicurrency_amount.$separator;
 
 				// FEC:Idevise
-				print $line->multicurrency_code . $separator;
+				print $line->multicurrency_code.$separator;
 
 				// FEC_suppl:DateLimitReglmt
-				print $date_limit_payment . $separator;
+				print $date_limit_payment.$separator;
 
 				// FEC_suppl:NumFacture
 				// Clean ref invoice to prevent problem on export with tab separator & other character
@@ -1081,63 +1081,63 @@ 
 				}
 
 				// FEC:JournalCode
-				print $line->code_journal . $separator;
+				print $line->code_journal.$separator;
 
 				// FEC:JournalLib
-				print dol_string_unaccent($langs->transnoentities($line->journal_label)) . $separator;
+				print dol_string_unaccent($langs->transnoentities($line->journal_label)).$separator;
 
 				// FEC:EcritureNum
-				print $line->piece_num . $separator;
+				print $line->piece_num.$separator;
 
 				// FEC:EcritureDate
-				print $date_creation . $separator;
+				print $date_creation.$separator;
 
 				// FEC:CompteNum
-				print length_accountg($line->numero_compte) . $separator;
+				print length_accountg($line->numero_compte).$separator;
 
 				// FEC:CompteLib
-				print dol_string_unaccent($line->label_compte) . $separator;
+				print dol_string_unaccent($line->label_compte).$separator;
 
 				// FEC:CompAuxNum
-				print length_accounta($line->subledger_account) . $separator;
+				print length_accounta($line->subledger_account).$separator;
 
 				// FEC:CompAuxLib
-				print dol_string_unaccent($line->subledger_label) . $separator;
+				print dol_string_unaccent($line->subledger_label).$separator;
 
 				// FEC:PieceRef
-				print $line->doc_ref . $separator;
+				print $line->doc_ref.$separator;
 
 				// FEC:PieceDate
-				print $date_document . $separator;
+				print $date_document.$separator;
 
 				// FEC:EcritureLib
 				// Clean label operation to prevent problem on export with tab separator & other character
 				$line->label_operation = str_replace(array("\t", "\n", "\r"), " ", $line->label_operation);
-				print dol_string_unaccent($line->label_operation) . $separator;
+				print dol_string_unaccent($line->label_operation).$separator;
 
 				// FEC:Debit
-				print price2fec($line->debit) . $separator;
+				print price2fec($line->debit).$separator;
 
 				// FEC:Credit
-				print price2fec($line->credit) . $separator;
+				print price2fec($line->credit).$separator;
 
 				// FEC:EcritureLet
-				print $line->lettering_code . $separator;
+				print $line->lettering_code.$separator;
 
 				// FEC:DateLet
-				print $date_lettering . $separator;
+				print $date_lettering.$separator;
 
 				// FEC:ValidDate
-				print $date_validation . $separator;
+				print $date_validation.$separator;
 
 				// FEC:Montantdevise
-				print $line->multicurrency_amount . $separator;
+				print $line->multicurrency_amount.$separator;
 
 				// FEC:Idevise
-				print $line->multicurrency_code . $separator;
+				print $line->multicurrency_code.$separator;
 
 				// FEC_suppl:DateLimitReglmt
-				print $date_limit_payment . $separator;
+				print $date_limit_payment.$separator;
 
 				// FEC_suppl:NumFacture
 				// Clean ref invoice to prevent problem on export with tab separator & other character
@@ -1577,7 +1577,7 @@ 
 				}
 			} elseif ($line->doc_type == 'customer_invoice') {
 				if (($line->amount) < 0) {
-					$nature_piece = 'AC';		// Currently, only the sign of amount allows to know the type of invoice (standard or credit note). Other solution is to analyse debit/credit/role of account. TODO Add column doc_type_long or make amount mandatory with rule on sign.
+					$nature_piece = 'AC'; // Currently, only the sign of amount allows to know the type of invoice (standard or credit note). Other solution is to analyse debit/credit/role of account. TODO Add column doc_type_long or make amount mandatory with rule on sign.
 				} else {
 					$nature_piece = 'FC';
 				}
@@ -1764,8 +1764,8 @@ 
 						($line->doc_type == 'supplier_invoice' && !isset($supplier_invoices_infos[$line->fk_doc]))) {
 						if ($line->doc_type == 'customer_invoice') {
 							// Get new customer invoice ref and company name
-							$sql = 'SELECT f.ref, s.nom FROM ' . MAIN_DB_PREFIX . 'facture as f';
-							$sql .= ' LEFT JOIN ' . MAIN_DB_PREFIX . 'societe AS s ON f.fk_soc = s.rowid';
+							$sql = 'SELECT f.ref, s.nom FROM '.MAIN_DB_PREFIX.'facture as f';
+							$sql .= ' LEFT JOIN '.MAIN_DB_PREFIX.'societe AS s ON f.fk_soc = s.rowid';
 							$sql .= ' WHERE f.rowid = '.((int) $line->fk_doc);
 							$resql = $this->db->query($sql);
 							if ($resql) {
@@ -1778,8 +1778,8 @@ 
 							}
 						} else {
 							// Get new supplier invoice ref and company name
-							$sql = 'SELECT ff.ref, s.nom FROM ' . MAIN_DB_PREFIX . 'facture_fourn as ff';
-							$sql .= ' LEFT JOIN ' . MAIN_DB_PREFIX . 'societe AS s ON ff.fk_soc = s.rowid';
+							$sql = 'SELECT ff.ref, s.nom FROM '.MAIN_DB_PREFIX.'facture_fourn as ff';
+							$sql .= ' LEFT JOIN '.MAIN_DB_PREFIX.'societe AS s ON ff.fk_soc = s.rowid';
 							$sql .= ' WHERE ff.rowid = '.((int) $line->fk_doc);
 							$resql = $this->db->query($sql);
 							if ($resql) {
@@ -1802,28 +1802,28 @@ 
 					}
 				}
 
-				print $line->id . $this->separator;
-				print $date . $this->separator;
-				print substr($line->code_journal, 0, 4) . $this->separator;
+				print $line->id.$this->separator;
+				print $date.$this->separator;
+				print substr($line->code_journal, 0, 4).$this->separator;
 
 				if ((substr($line->numero_compte, 0, 3) == '411') || (substr($line->numero_compte, 0, 3) == '401')) {
-					print length_accountg($line->subledger_account) . $this->separator;
+					print length_accountg($line->subledger_account).$this->separator;
 				} else {
-					print substr(length_accountg($line->numero_compte), 0, 15) . $this->separator;
+					print substr(length_accountg($line->numero_compte), 0, 15).$this->separator;
 				}
 				//Libellé Auto
 				print $this->separator;
 				//print '"'.dol_trunc(str_replace('"', '', $line->label_operation),40,'right','UTF-8',1).'"' . $this->separator;
 				//Libellé manuel
-				print dol_trunc(str_replace('"', '', $invoice_ref . (!empty($company_name) ? ' - ' : '') . $company_name), 40, 'right', 'UTF-8', 1) . $this->separator;
+				print dol_trunc(str_replace('"', '', $invoice_ref.(!empty($company_name) ? ' - ' : '').$company_name), 40, 'right', 'UTF-8', 1).$this->separator;
 				//Numéro de pièce
-				print dol_trunc(str_replace('"', '', $line->piece_num), 10, 'right', 'UTF-8', 1) . $this->separator;
+				print dol_trunc(str_replace('"', '', $line->piece_num), 10, 'right', 'UTF-8', 1).$this->separator;
 				//Devise
-				print 'EUR' . $this->separator;
+				print 'EUR'.$this->separator;
 				//Amount
-				print price2num(abs($line->debit - $line->credit)) . $this->separator;
+				print price2num(abs($line->debit - $line->credit)).$this->separator;
 				//Sens
-				print $line->sens . $this->separator;
+				print $line->sens.$this->separator;
 				//Code lettrage
 				print $this->separator;
 				//Date Echéance
@@ -1851,21 +1851,21 @@ 
 			} else {
 				$date = dol_print_date($line->doc_date, '%d%m%Y');
 
-				print $line->id . $this->separator;
-				print $date . $this->separator;
-				print substr($line->code_journal, 0, 4) . $this->separator;
+				print $line->id.$this->separator;
+				print $date.$this->separator;
+				print substr($line->code_journal, 0, 4).$this->separator;
 				if ((substr($line->numero_compte, 0, 3) == '411') || (substr($line->numero_compte, 0, 3) == '401')) {	// TODO No hard code value
-					print length_accountg($line->subledger_account) . $this->separator;
+					print length_accountg($line->subledger_account).$this->separator;
 				} else {
-					print substr(length_accountg($line->numero_compte), 0, 15) . $this->separator;
+					print substr(length_accountg($line->numero_compte), 0, 15).$this->separator;
 				}
 				print $this->separator;
 				//print '"'.dol_trunc(str_replace('"', '', $line->label_operation),40,'right','UTF-8',1).'"' . $this->separator;
-				print '"' . dol_trunc(str_replace('"', '', $line->doc_ref), 40, 'right', 'UTF-8', 1) . '"' . $this->separator;
-				print '"' . dol_trunc(str_replace('"', '', $line->piece_num), 10, 'right', 'UTF-8', 1) . '"' . $this->separator;
-				print price2num(abs($line->debit - $line->credit)) . $this->separator;
-				print $line->sens . $this->separator;
-				print $date . $this->separator;
+				print '"'.dol_trunc(str_replace('"', '', $line->doc_ref), 40, 'right', 'UTF-8', 1).'"'.$this->separator;
+				print '"'.dol_trunc(str_replace('"', '', $line->piece_num), 10, 'right', 'UTF-8', 1).'"'.$this->separator;
+				print price2num(abs($line->debit - $line->credit)).$this->separator;
+				print $line->sens.$this->separator;
+				print $date.$this->separator;
 				print $this->separator;
 				print $this->separator;
 				print 'EUR';
@@ -1926,7 +1926,7 @@ 
 			$tab[] = $line->code_journal;
 
 			$separator = $this->separator;
-			print implode($separator, $tab) . $this->end_line;
+			print implode($separator, $tab).$this->end_line;
 		}
 	}
 

htdocs/don/admin/donation.php

Spacing +4 added lines, -4 removed lines

@@ -116,10 +116,10 @@ 
 
 	if (!$error) {
 		setEventMessages($langs->trans("SetupSaved"), null, 'mesgs');
-		$action = '';	// To avoid to execute next actions
+		$action = ''; // To avoid to execute next actions
 	} else {
 		setEventMessages($langs->trans("Error"), null, 'errors');
-		$action = '';	// To avoid to execute next actions
+		$action = ''; // To avoid to execute next actions
 	}
 }
 
@@ -134,10 +134,10 @@ 
 
 	if (!$error) {
 		setEventMessages($langs->trans("SetupSaved"), null, 'mesgs');
-		$action = '';	// To avoid to execute next actions
+		$action = ''; // To avoid to execute next actions
 	} else {
 		setEventMessages($langs->trans("Error"), null, 'errors');
-		$action = '';	// To avoid to execute next actions
+		$action = ''; // To avoid to execute next actions
 	}
 }
 

htdocs/core/modules/product_batch/mod_sn_standard.php

Spacing +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
 		$sql .= " AND entity = ".$conf->entity;
 
 		$resql = $db->query($sql);
-		if ($resql)	{
+		if ($resql) {
 			$obj = $db->fetch_object($resql);
 			if ($obj) {
 				$max = intval($obj->max);
@@ -127,7 +127,7 @@ 
 		$sql .= " AND entity = ".$conf->entity;
 
 		$resql = $db->query($sql);
-		if ($resql)	{
+		if ($resql) {
 			$obj = $db->fetch_object($resql);
 			if ($obj) {
 				$max = intval($obj->max);

Braces +7 added lines, -2 removed lines

@@ -143,8 +143,13 @@
 		$date = dol_now();
 		$yymm = strftime("%y%m", $date);
 
-		if ($max >= (pow(10, 4) - 1)) $num = $max + 1; // If counter > 9999, we do not format on 4 chars, we take number as it is
-		else $num = sprintf("%04s", $max + 1);
+		if ($max >= (pow(10, 4) - 1)) {
+			$num = $max + 1;
+		}
+		// If counter > 9999, we do not format on 4 chars, we take number as it is
+		else {
+			$num = sprintf("%04s", $max + 1);
+		}
 
 		dol_syslog("mod_sn_standard::getNextValue return ".$this->prefix.$yymm."-".$num);
 		return $this->prefix.$yymm."-".$num;