alxarafe / alixar

dolibarr/htdocs/core/lib/contract.lib.php

Indentation +54 added lines, -54 removed lines

@@ -30,25 +30,25 @@ 
  */
 function contract_prepare_head(Contrat $object)
 {
-	global $db, $langs, $conf;
+    global $db, $langs, $conf;
 	
-	$h = 0;
-	$head = array();
-
-	$head[$h][0] = DOL_URL_ROOT.'/contrat/card.php?id='.$object->id;
-	$head[$h][1] = $langs->trans("ContractCard");
-	$head[$h][2] = 'card';
-	$h++;
-
-	if (empty($conf->global->MAIN_DISABLE_CONTACTS_TAB))
-	{
-	    $nbContact = count($object->liste_contact(-1,'internal')) + count($object->liste_contact(-1,'external'));
-	    $head[$h][0] = DOL_URL_ROOT.'/contrat/contact.php?id='.$object->id;
-		$head[$h][1] = $langs->trans("ContactsAddresses");
-		if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
-		$head[$h][2] = 'contact';
-		$h++;
-	}
+    $h = 0;
+    $head = array();
+
+    $head[$h][0] = DOL_URL_ROOT.'/contrat/card.php?id='.$object->id;
+    $head[$h][1] = $langs->trans("ContractCard");
+    $head[$h][2] = 'card';
+    $h++;
+
+    if (empty($conf->global->MAIN_DISABLE_CONTACTS_TAB))
+    {
+        $nbContact = count($object->liste_contact(-1,'internal')) + count($object->liste_contact(-1,'external'));
+        $head[$h][0] = DOL_URL_ROOT.'/contrat/contact.php?id='.$object->id;
+        $head[$h][1] = $langs->trans("ContactsAddresses");
+        if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+        $head[$h][2] = 'contact';
+        $h++;
+    }
 
     // Show more tabs from modules
     // Entries must be declared in modules descriptor with line
@@ -58,35 +58,35 @@ 
 
     if (empty($conf->global->MAIN_DISABLE_NOTES_TAB))
     {
-    	$nbNote = 0;
+        $nbNote = 0;
         if(!empty($object->note_private)) $nbNote++;
-		if(!empty($object->note_public)) $nbNote++;
-    	$head[$h][0] = DOL_URL_ROOT.'/contrat/note.php?id='.$object->id;
-    	$head[$h][1] = $langs->trans("Notes");
-		if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
-    	$head[$h][2] = 'note';
-    	$h++;
+        if(!empty($object->note_public)) $nbNote++;
+        $head[$h][0] = DOL_URL_ROOT.'/contrat/note.php?id='.$object->id;
+        $head[$h][1] = $langs->trans("Notes");
+        if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+        $head[$h][2] = 'note';
+        $h++;
     }
 
-	require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
+    require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
     require_once DOL_DOCUMENT_ROOT.'/core/class/link.class.php';
-	$upload_dir = $conf->contrat->dir_output . "/" . dol_sanitizeFileName($object->ref);
-	$nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
+    $upload_dir = $conf->contrat->dir_output . "/" . dol_sanitizeFileName($object->ref);
+    $nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
     $nbLinks=Link::count($db, $object->element, $object->id);
-	$head[$h][0] = DOL_URL_ROOT.'/contrat/document.php?id='.$object->id;
-	$head[$h][1] = $langs->trans("Documents");
-	if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
-	$head[$h][2] = 'documents';
-	$h++;
+    $head[$h][0] = DOL_URL_ROOT.'/contrat/document.php?id='.$object->id;
+    $head[$h][1] = $langs->trans("Documents");
+    if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+    $head[$h][2] = 'documents';
+    $h++;
 
-	$head[$h][0] = DOL_URL_ROOT.'/contrat/info.php?id='.$object->id;
-	$head[$h][1] = $langs->trans("Info");
-	$head[$h][2] = 'info';
-	$h++;
+    $head[$h][0] = DOL_URL_ROOT.'/contrat/info.php?id='.$object->id;
+    $head[$h][1] = $langs->trans("Info");
+    $head[$h][2] = 'info';
+    $h++;
 
     complete_head_from_modules($conf,$langs,$object,$head,$h,'contract','remove');
 
-	return $head;
+    return $head;
 }
 
 /**
@@ -96,24 +96,24 @@ 
  */
 function contract_admin_prepare_head()
 {
-	global $langs, $conf, $user;
+    global $langs, $conf, $user;
 
-	$h = 0;
-	$head = array();
+    $h = 0;
+    $head = array();
 
-	$head[$h][0] = DOL_URL_ROOT."/admin/contract.php";
-	$head[$h][1] = $langs->trans("Contracts");
-	$head[$h][2] = 'contract';
-	$h++;
+    $head[$h][0] = DOL_URL_ROOT."/admin/contract.php";
+    $head[$h][1] = $langs->trans("Contracts");
+    $head[$h][2] = 'contract';
+    $h++;
 
-	// Show more tabs from modules
-	// Entries must be declared in modules descriptor with line
-	// $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
-	// $this->tabs = array('entity:-tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to remove a tab
-	complete_head_from_modules($conf,$langs,null,$head,$h,'contract_admin');
+    // Show more tabs from modules
+    // Entries must be declared in modules descriptor with line
+    // $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
+    // $this->tabs = array('entity:-tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to remove a tab
+    complete_head_from_modules($conf,$langs,null,$head,$h,'contract_admin');
 
-	$head[$h][0] = DOL_URL_ROOT.'/contrat/admin/contract_extrafields.php';
-	$head[$h][1] = $langs->trans("ExtraFields");
+    $head[$h][0] = DOL_URL_ROOT.'/contrat/admin/contract_extrafields.php';
+    $head[$h][1] = $langs->trans("ExtraFields");
     $head[$h][2] = 'attributes';
     $h++;
 
@@ -124,8 +124,8 @@ 
 
 
 
-	complete_head_from_modules($conf,$langs,null,$head,$h,'contract_admin','remove');
+    complete_head_from_modules($conf,$langs,null,$head,$h,'contract_admin','remove');
 
-		return $head;
+        return $head;
 }
 

Spacing +13 added lines, -13 removed lines

@@ -42,10 +42,10 @@ 
 
 	if (empty($conf->global->MAIN_DISABLE_CONTACTS_TAB))
 	{
-	    $nbContact = count($object->liste_contact(-1,'internal')) + count($object->liste_contact(-1,'external'));
+	    $nbContact = count($object->liste_contact(-1, 'internal')) + count($object->liste_contact(-1, 'external'));
 	    $head[$h][0] = DOL_URL_ROOT.'/contrat/contact.php?id='.$object->id;
 		$head[$h][1] = $langs->trans("ContactsAddresses");
-		if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+		if ($nbContact > 0) $head[$h][1] .= ' <span class="badge">'.$nbContact.'</span>';
 		$head[$h][2] = 'contact';
 		$h++;
 	}
@@ -54,28 +54,28 @@ 
     // Entries must be declared in modules descriptor with line
     // $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
     // $this->tabs = array('entity:-tabname);   												to remove a tab
-    complete_head_from_modules($conf,$langs,$object,$head,$h,'contract');
+    complete_head_from_modules($conf, $langs, $object, $head, $h, 'contract');
 
     if (empty($conf->global->MAIN_DISABLE_NOTES_TAB))
     {
     	$nbNote = 0;
-        if(!empty($object->note_private)) $nbNote++;
-		if(!empty($object->note_public)) $nbNote++;
+        if (!empty($object->note_private)) $nbNote++;
+		if (!empty($object->note_public)) $nbNote++;
     	$head[$h][0] = DOL_URL_ROOT.'/contrat/note.php?id='.$object->id;
     	$head[$h][1] = $langs->trans("Notes");
-		if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+		if ($nbNote > 0) $head[$h][1] .= ' <span class="badge">'.$nbNote.'</span>';
     	$head[$h][2] = 'note';
     	$h++;
     }
 
 	require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
     require_once DOL_DOCUMENT_ROOT.'/core/class/link.class.php';
-	$upload_dir = $conf->contrat->dir_output . "/" . dol_sanitizeFileName($object->ref);
-	$nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
-    $nbLinks=Link::count($db, $object->element, $object->id);
+	$upload_dir = $conf->contrat->dir_output."/".dol_sanitizeFileName($object->ref);
+	$nbFiles = count(dol_dir_list($upload_dir, 'files', 0, '', '(\.meta|_preview.*\.png)$'));
+    $nbLinks = Link::count($db, $object->element, $object->id);
 	$head[$h][0] = DOL_URL_ROOT.'/contrat/document.php?id='.$object->id;
 	$head[$h][1] = $langs->trans("Documents");
-	if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	if (($nbFiles + $nbLinks) > 0) $head[$h][1] .= ' <span class="badge">'.($nbFiles + $nbLinks).'</span>';
 	$head[$h][2] = 'documents';
 	$h++;
 
@@ -84,7 +84,7 @@ 
 	$head[$h][2] = 'info';
 	$h++;
 
-    complete_head_from_modules($conf,$langs,$object,$head,$h,'contract','remove');
+    complete_head_from_modules($conf, $langs, $object, $head, $h, 'contract', 'remove');
 
 	return $head;
 }
@@ -110,7 +110,7 @@ 
 	// Entries must be declared in modules descriptor with line
 	// $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
 	// $this->tabs = array('entity:-tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to remove a tab
-	complete_head_from_modules($conf,$langs,null,$head,$h,'contract_admin');
+	complete_head_from_modules($conf, $langs, null, $head, $h, 'contract_admin');
 
 	$head[$h][0] = DOL_URL_ROOT.'/contrat/admin/contract_extrafields.php';
 	$head[$h][1] = $langs->trans("ExtraFields");
@@ -124,7 +124,7 @@ 
 
 
 
-	complete_head_from_modules($conf,$langs,null,$head,$h,'contract_admin','remove');
+	complete_head_from_modules($conf, $langs, null, $head, $h, 'contract_admin', 'remove');
 
 		return $head;
 }

Braces +15 added lines, -5 removed lines

@@ -45,7 +45,9 @@ 
 	    $nbContact = count($object->liste_contact(-1,'internal')) + count($object->liste_contact(-1,'external'));
 	    $head[$h][0] = DOL_URL_ROOT.'/contrat/contact.php?id='.$object->id;
 		$head[$h][1] = $langs->trans("ContactsAddresses");
-		if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+		if ($nbContact > 0) {
+		    $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+		}
 		$head[$h][2] = 'contact';
 		$h++;
 	}
@@ -59,11 +61,17 @@ 
     if (empty($conf->global->MAIN_DISABLE_NOTES_TAB))
     {
     	$nbNote = 0;
-        if(!empty($object->note_private)) $nbNote++;
-		if(!empty($object->note_public)) $nbNote++;
+        if(!empty($object->note_private)) {
+            $nbNote++;
+        }
+		if(!empty($object->note_public)) {
+		    $nbNote++;
+		}
     	$head[$h][0] = DOL_URL_ROOT.'/contrat/note.php?id='.$object->id;
     	$head[$h][1] = $langs->trans("Notes");
-		if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+		if ($nbNote > 0) {
+		    $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+		}
     	$head[$h][2] = 'note';
     	$h++;
     }
@@ -75,7 +83,9 @@ 
     $nbLinks=Link::count($db, $object->element, $object->id);
 	$head[$h][0] = DOL_URL_ROOT.'/contrat/document.php?id='.$object->id;
 	$head[$h][1] = $langs->trans("Documents");
-	if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	if (($nbFiles+$nbLinks) > 0) {
+	    $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	}
 	$head[$h][2] = 'documents';
 	$h++;
 

dolibarr/htdocs/core/lib/format_cards.lib.php

Indentation +1 added lines, -1 removed lines

@@ -64,6 +64,6 @@
 // We add characteristics to the name
 foreach($_Avery_Labels as $key => $val)
 {
-	$_Avery_Labels[$key]['name'].=' ('.$_Avery_Labels[$key]['paper-size'].' - '.$_Avery_Labels[$key]['NX'].'x'.$_Avery_Labels[$key]['NY'].')';
+    $_Avery_Labels[$key]['name'].=' ('.$_Avery_Labels[$key]['paper-size'].' - '.$_Avery_Labels[$key]['NX'].'x'.$_Avery_Labels[$key]['NY'].')';
 }
 

Spacing +18 added lines, -18 removed lines

@@ -38,22 +38,22 @@ 
 {
     while ($row = $db->fetch_array($resql))
     {
-        $_Avery_Labels[$row['code']]['name']=$row['name'];
-        $_Avery_Labels[$row['code']]['paper-size']=$row['paper_size'];
-        $_Avery_Labels[$row['code']]['orientation']=$row['orientation'];
-        $_Avery_Labels[$row['code']]['metric']=$row['metric'];
-        $_Avery_Labels[$row['code']]['marginLeft']=$row['leftmargin'];
-        $_Avery_Labels[$row['code']]['marginTop']=$row['topmargin'];
-        $_Avery_Labels[$row['code']]['marginTop']=$row['topmargin'];
-        $_Avery_Labels[$row['code']]['NX']=$row['nx'];
-        $_Avery_Labels[$row['code']]['NY']=$row['ny'];
-        $_Avery_Labels[$row['code']]['SpaceX']=$row['spacex'];
-        $_Avery_Labels[$row['code']]['SpaceY']=$row['spacey'];
-        $_Avery_Labels[$row['code']]['width']=$row['width'];
-        $_Avery_Labels[$row['code']]['height']=$row['height'];
-        $_Avery_Labels[$row['code']]['font-size']=$row['font_size'];
-        $_Avery_Labels[$row['code']]['custom_x']=$row['custom_x'];
-        $_Avery_Labels[$row['code']]['custom_y']=$row['custom_y'];
+        $_Avery_Labels[$row['code']]['name'] = $row['name'];
+        $_Avery_Labels[$row['code']]['paper-size'] = $row['paper_size'];
+        $_Avery_Labels[$row['code']]['orientation'] = $row['orientation'];
+        $_Avery_Labels[$row['code']]['metric'] = $row['metric'];
+        $_Avery_Labels[$row['code']]['marginLeft'] = $row['leftmargin'];
+        $_Avery_Labels[$row['code']]['marginTop'] = $row['topmargin'];
+        $_Avery_Labels[$row['code']]['marginTop'] = $row['topmargin'];
+        $_Avery_Labels[$row['code']]['NX'] = $row['nx'];
+        $_Avery_Labels[$row['code']]['NY'] = $row['ny'];
+        $_Avery_Labels[$row['code']]['SpaceX'] = $row['spacex'];
+        $_Avery_Labels[$row['code']]['SpaceY'] = $row['spacey'];
+        $_Avery_Labels[$row['code']]['width'] = $row['width'];
+        $_Avery_Labels[$row['code']]['height'] = $row['height'];
+        $_Avery_Labels[$row['code']]['font-size'] = $row['font_size'];
+        $_Avery_Labels[$row['code']]['custom_x'] = $row['custom_x'];
+        $_Avery_Labels[$row['code']]['custom_y'] = $row['custom_y'];
     }
 }
 else
@@ -62,8 +62,8 @@ 
 }
 
 // We add characteristics to the name
-foreach($_Avery_Labels as $key => $val)
+foreach ($_Avery_Labels as $key => $val)
 {
-	$_Avery_Labels[$key]['name'].=' ('.$_Avery_Labels[$key]['paper-size'].' - '.$_Avery_Labels[$key]['NX'].'x'.$_Avery_Labels[$key]['NY'].')';
+	$_Avery_Labels[$key]['name'] .= ' ('.$_Avery_Labels[$key]['paper-size'].' - '.$_Avery_Labels[$key]['NX'].'x'.$_Avery_Labels[$key]['NY'].')';
 }
 

Braces +1 added lines, -2 removed lines

@@ -55,8 +55,7 @@
         $_Avery_Labels[$row['code']]['custom_x']=$row['custom_x'];
         $_Avery_Labels[$row['code']]['custom_y']=$row['custom_y'];
     }
-}
-else
+} else
 {
     dol_print_error($db);
 }

dolibarr/htdocs/core/lib/contact.lib.php

Indentation +22 added lines, -22 removed lines

@@ -32,31 +32,31 @@ 
  */
 function contact_prepare_head(Contact $object)
 {
-	global $db, $langs, $conf, $user;
+    global $db, $langs, $conf, $user;
 
-	$tab = 0;
-	$head = array();
+    $tab = 0;
+    $head = array();
 
-	$head[$tab][0] = DOL_URL_ROOT.'/contact/card.php?id='.$object->id;
-	$head[$tab][1] = $langs->trans("Card");
-	$head[$tab][2] = 'card';
-	$tab++;
+    $head[$tab][0] = DOL_URL_ROOT.'/contact/card.php?id='.$object->id;
+    $head[$tab][1] = $langs->trans("Card");
+    $head[$tab][2] = 'card';
+    $tab++;
 
-	if ((! empty($conf->ldap->enabled) && ! empty($conf->global->LDAP_CONTACT_ACTIVE))
-		&& (empty($conf->global->MAIN_DISABLE_LDAP_TAB) || ! empty($user->admin)))
-	{
-		$langs->load("ldap");
+    if ((! empty($conf->ldap->enabled) && ! empty($conf->global->LDAP_CONTACT_ACTIVE))
+        && (empty($conf->global->MAIN_DISABLE_LDAP_TAB) || ! empty($user->admin)))
+    {
+        $langs->load("ldap");
 
-		$head[$tab][0] = DOL_URL_ROOT.'/contact/ldap.php?id='.$object->id;
-		$head[$tab][1] = $langs->trans("LDAPCard");
-		$head[$tab][2] = 'ldap';
-		$tab++;
-	}
+        $head[$tab][0] = DOL_URL_ROOT.'/contact/ldap.php?id='.$object->id;
+        $head[$tab][1] = $langs->trans("LDAPCard");
+        $head[$tab][2] = 'ldap';
+        $tab++;
+    }
 
-	$head[$tab][0] = DOL_URL_ROOT.'/contact/perso.php?id='.$object->id;
-	$head[$tab][1] = $langs->trans("PersonalInformations");
-	$head[$tab][2] = 'perso';
-	$tab++;
+    $head[$tab][0] = DOL_URL_ROOT.'/contact/perso.php?id='.$object->id;
+    $head[$tab][1] = $langs->trans("PersonalInformations");
+    $head[$tab][2] = 'perso';
+    $tab++;
 
     // Show more tabs from modules
     // Entries must be declared in modules descriptor with line
@@ -103,8 +103,8 @@ 
 	$head[$tab][2] = 'info';
 	$tab++;*/
 
-	complete_head_from_modules($conf,$langs,$object,$head,$tab,'contact','remove');
+    complete_head_from_modules($conf,$langs,$object,$head,$tab,'contact','remove');
 
-	return $head;
+    return $head;
 }
 

Spacing +14 added lines, -14 removed lines

@@ -42,8 +42,8 @@ 
 	$head[$tab][2] = 'card';
 	$tab++;
 
-	if ((! empty($conf->ldap->enabled) && ! empty($conf->global->LDAP_CONTACT_ACTIVE))
-		&& (empty($conf->global->MAIN_DISABLE_LDAP_TAB) || ! empty($user->admin)))
+	if ((!empty($conf->ldap->enabled) && !empty($conf->global->LDAP_CONTACT_ACTIVE))
+		&& (empty($conf->global->MAIN_DISABLE_LDAP_TAB) || !empty($user->admin)))
 	{
 		$langs->load("ldap");
 
@@ -62,36 +62,36 @@ 
     // Entries must be declared in modules descriptor with line
     // $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
     // $this->tabs = array('entity:-tabname);   												to remove a tab
-    complete_head_from_modules($conf,$langs,$object,$head,$tab,'contact');
+    complete_head_from_modules($conf, $langs, $object, $head, $tab, 'contact');
 
     // Notes
     if (empty($conf->global->MAIN_DISABLE_NOTES_TAB)) {
-        $nbNote = (empty($object->note_private)?0:1)+(empty($object->note_public)?0:1);
+        $nbNote = (empty($object->note_private) ? 0 : 1) + (empty($object->note_public) ? 0 : 1);
         $head[$tab][0] = DOL_URL_ROOT.'/contact/note.php?id='.$object->id;
         $head[$tab][1] = $langs->trans("Note");
-        if($nbNote > 0) $head[$tab][1].= ' <span class="badge">'.$nbNote.'</span>';
+        if ($nbNote > 0) $head[$tab][1] .= ' <span class="badge">'.$nbNote.'</span>';
         $head[$tab][2] = 'note';
         $tab++;
     }
 
     require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
     require_once DOL_DOCUMENT_ROOT.'/core/class/link.class.php';
-    $upload_dir = $conf->societe->dir_output . "/contact/" . dol_sanitizeFileName($object->ref);
-    $nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
-    $nbLinks=Link::count($db, $object->element, $object->id);
+    $upload_dir = $conf->societe->dir_output."/contact/".dol_sanitizeFileName($object->ref);
+    $nbFiles = count(dol_dir_list($upload_dir, 'files', 0, '', '(\.meta|_preview.*\.png)$'));
+    $nbLinks = Link::count($db, $object->element, $object->id);
     $head[$tab][0] = DOL_URL_ROOT.'/contact/document.php?id='.$object->id;
     $head[$tab][1] = $langs->trans("Documents");
-    if (($nbFiles+$nbLinks) > 0) $head[$tab][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+    if (($nbFiles + $nbLinks) > 0) $head[$tab][1] .= ' <span class="badge">'.($nbFiles + $nbLinks).'</span>';
     $head[$tab][2] = 'documents';
     $tab++;
 
     // Agenda / Events
     $head[$tab][0] = DOL_URL_ROOT.'/contact/agenda.php?id='.$object->id;
-    $head[$tab][1].= $langs->trans("Events");
-    if (! empty($conf->agenda->enabled) && (!empty($user->rights->agenda->myactions->read) || !empty($user->rights->agenda->allactions->read) ))
+    $head[$tab][1] .= $langs->trans("Events");
+    if (!empty($conf->agenda->enabled) && (!empty($user->rights->agenda->myactions->read) || !empty($user->rights->agenda->allactions->read)))
     {
-        $head[$tab][1].= '/';
-        $head[$tab][1].= $langs->trans("Agenda");
+        $head[$tab][1] .= '/';
+        $head[$tab][1] .= $langs->trans("Agenda");
     }
     $head[$tab][2] = 'agenda';
     $tab++;
@@ -103,7 +103,7 @@ 
 	$head[$tab][2] = 'info';
 	$tab++;*/
 
-	complete_head_from_modules($conf,$langs,$object,$head,$tab,'contact','remove');
+	complete_head_from_modules($conf, $langs, $object, $head, $tab, 'contact', 'remove');
 
 	return $head;
 }

Braces +6 added lines, -2 removed lines

@@ -69,7 +69,9 @@ 
         $nbNote = (empty($object->note_private)?0:1)+(empty($object->note_public)?0:1);
         $head[$tab][0] = DOL_URL_ROOT.'/contact/note.php?id='.$object->id;
         $head[$tab][1] = $langs->trans("Note");
-        if($nbNote > 0) $head[$tab][1].= ' <span class="badge">'.$nbNote.'</span>';
+        if($nbNote > 0) {
+            $head[$tab][1].= ' <span class="badge">'.$nbNote.'</span>';
+        }
         $head[$tab][2] = 'note';
         $tab++;
     }
@@ -81,7 +83,9 @@ 
     $nbLinks=Link::count($db, $object->element, $object->id);
     $head[$tab][0] = DOL_URL_ROOT.'/contact/document.php?id='.$object->id;
     $head[$tab][1] = $langs->trans("Documents");
-    if (($nbFiles+$nbLinks) > 0) $head[$tab][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+    if (($nbFiles+$nbLinks) > 0) {
+        $head[$tab][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+    }
     $head[$tab][2] = 'documents';
     $tab++;
 

dolibarr/htdocs/core/lib/loan.lib.php

Indentation +25 added lines, -25 removed lines

@@ -36,15 +36,15 @@ 
     $tab = 0;
     $head = array();
 
-	$head[$tab][0] = DOL_URL_ROOT.'/loan/card.php?id='.$object->id;
-	$head[$tab][1] = $langs->trans('Card');
-	$head[$tab][2] = 'card';
-	$tab++;
+    $head[$tab][0] = DOL_URL_ROOT.'/loan/card.php?id='.$object->id;
+    $head[$tab][1] = $langs->trans('Card');
+    $head[$tab][2] = 'card';
+    $tab++;
 	
-	$head[$tab][0] = DOL_URL_ROOT.'/loan/schedule.php?loanid='.$object->id;
-	$head[$tab][1] = $langs->trans('FinancialCommitment');
-	$head[$tab][2] = 'FinancialCommitment';
-	$tab++;
+    $head[$tab][0] = DOL_URL_ROOT.'/loan/schedule.php?loanid='.$object->id;
+    $head[$tab][1] = $langs->trans('FinancialCommitment');
+    $head[$tab][2] = 'FinancialCommitment';
+    $tab++;
 
     // Show more tabs from modules
     // Entries must be declared in modules descriptor with line
@@ -52,26 +52,26 @@ 
     // $this->tabs = array('entity:-tabname);   												to remove a tab
     complete_head_from_modules($conf, $langs, $object, $head, $tab,'loan');
 
-	require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
+    require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
     require_once DOL_DOCUMENT_ROOT.'/core/class/link.class.php';
-	$upload_dir = $conf->loan->dir_output . "/" . dol_sanitizeFileName($object->ref);
-	$nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
+    $upload_dir = $conf->loan->dir_output . "/" . dol_sanitizeFileName($object->ref);
+    $nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
     $nbLinks=Link::count($db, $object->element, $object->id);
-	$head[$tab][0] = DOL_URL_ROOT.'/loan/document.php?id='.$object->id;
-	$head[$tab][1] = $langs->trans("Documents");
-	if (($nbFiles+$nbLinks) > 0) $head[$tab][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
-	$head[$tab][2] = 'documents';
-	$tab++;
+    $head[$tab][0] = DOL_URL_ROOT.'/loan/document.php?id='.$object->id;
+    $head[$tab][1] = $langs->trans("Documents");
+    if (($nbFiles+$nbLinks) > 0) $head[$tab][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+    $head[$tab][2] = 'documents';
+    $tab++;
 
-	if (empty($conf->global->MAIN_DISABLE_NOTES_TAB))
-	{
-		$nbNote = (empty($object->note_private)?0:1)+(empty($object->note_public)?0:1);
-		$head[$tab][0] = DOL_URL_ROOT."/loan/note.php?id=".$object->id;
-		$head[$tab][1] = $langs->trans("Notes");
-		if($nbNote > 0) $head[$tab][1].= ' <span class="badge">'.$nbNote.'</span>';
-		$head[$tab][2] = 'note';
-		$tab++;
-	}
+    if (empty($conf->global->MAIN_DISABLE_NOTES_TAB))
+    {
+        $nbNote = (empty($object->note_private)?0:1)+(empty($object->note_public)?0:1);
+        $head[$tab][0] = DOL_URL_ROOT."/loan/note.php?id=".$object->id;
+        $head[$tab][1] = $langs->trans("Notes");
+        if($nbNote > 0) $head[$tab][1].= ' <span class="badge">'.$nbNote.'</span>';
+        $head[$tab][2] = 'note';
+        $tab++;
+    }
 
     $head[$tab][0] = DOL_URL_ROOT.'/loan/info.php?id='.$object->id;
     $head[$tab][1] = $langs->trans("Info");

Spacing +8 added lines, -8 removed lines

@@ -50,25 +50,25 @@ 
     // Entries must be declared in modules descriptor with line
     // $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
     // $this->tabs = array('entity:-tabname);   												to remove a tab
-    complete_head_from_modules($conf, $langs, $object, $head, $tab,'loan');
+    complete_head_from_modules($conf, $langs, $object, $head, $tab, 'loan');
 
 	require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
     require_once DOL_DOCUMENT_ROOT.'/core/class/link.class.php';
-	$upload_dir = $conf->loan->dir_output . "/" . dol_sanitizeFileName($object->ref);
-	$nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
-    $nbLinks=Link::count($db, $object->element, $object->id);
+	$upload_dir = $conf->loan->dir_output."/".dol_sanitizeFileName($object->ref);
+	$nbFiles = count(dol_dir_list($upload_dir, 'files', 0, '', '(\.meta|_preview.*\.png)$'));
+    $nbLinks = Link::count($db, $object->element, $object->id);
 	$head[$tab][0] = DOL_URL_ROOT.'/loan/document.php?id='.$object->id;
 	$head[$tab][1] = $langs->trans("Documents");
-	if (($nbFiles+$nbLinks) > 0) $head[$tab][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	if (($nbFiles + $nbLinks) > 0) $head[$tab][1] .= ' <span class="badge">'.($nbFiles + $nbLinks).'</span>';
 	$head[$tab][2] = 'documents';
 	$tab++;
 
 	if (empty($conf->global->MAIN_DISABLE_NOTES_TAB))
 	{
-		$nbNote = (empty($object->note_private)?0:1)+(empty($object->note_public)?0:1);
+		$nbNote = (empty($object->note_private) ? 0 : 1) + (empty($object->note_public) ? 0 : 1);
 		$head[$tab][0] = DOL_URL_ROOT."/loan/note.php?id=".$object->id;
 		$head[$tab][1] = $langs->trans("Notes");
-		if($nbNote > 0) $head[$tab][1].= ' <span class="badge">'.$nbNote.'</span>';
+		if ($nbNote > 0) $head[$tab][1] .= ' <span class="badge">'.$nbNote.'</span>';
 		$head[$tab][2] = 'note';
 		$tab++;
 	}
@@ -78,7 +78,7 @@ 
     $head[$tab][2] = 'info';
     $tab++;
 
-    complete_head_from_modules($conf,$langs,$object,$head,$tab,'loan','remove');
+    complete_head_from_modules($conf, $langs, $object, $head, $tab, 'loan', 'remove');
 
     return $head;
 }

Braces +6 added lines, -2 removed lines

@@ -59,7 +59,9 @@ 
     $nbLinks=Link::count($db, $object->element, $object->id);
 	$head[$tab][0] = DOL_URL_ROOT.'/loan/document.php?id='.$object->id;
 	$head[$tab][1] = $langs->trans("Documents");
-	if (($nbFiles+$nbLinks) > 0) $head[$tab][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	if (($nbFiles+$nbLinks) > 0) {
+	    $head[$tab][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	}
 	$head[$tab][2] = 'documents';
 	$tab++;
 
@@ -68,7 +70,9 @@ 
 		$nbNote = (empty($object->note_private)?0:1)+(empty($object->note_public)?0:1);
 		$head[$tab][0] = DOL_URL_ROOT."/loan/note.php?id=".$object->id;
 		$head[$tab][1] = $langs->trans("Notes");
-		if($nbNote > 0) $head[$tab][1].= ' <span class="badge">'.$nbNote.'</span>';
+		if($nbNote > 0) {
+		    $head[$tab][1].= ' <span class="badge">'.$nbNote.'</span>';
+		}
 		$head[$tab][2] = 'note';
 		$tab++;
 	}

dolibarr/htdocs/core/lib/images.lib.php

Indentation +394 added lines, -394 removed lines

@@ -74,24 +74,24 @@ 
  */
 function dol_getImageSize($file, $url = false)
 {
-	$ret=array();
+    $ret=array();
 
-	if (image_format_supported($file) < 0) return $ret;
+    if (image_format_supported($file) < 0) return $ret;
 
-	$filetoread = $file;
-	if (!$url)
-	{
-		$filetoread = realpath(dol_osencode($file)); 	// Chemin canonique absolu de l'image
-	}
+    $filetoread = $file;
+    if (!$url)
+    {
+        $filetoread = realpath(dol_osencode($file)); 	// Chemin canonique absolu de l'image
+    }
 
-	if ($filetoread)
-	{
-    	$infoImg = getimagesize($filetoread); // Recuperation des infos de l'image
-    	$ret['width']=$infoImg[0]; // Largeur de l'image
-    	$ret['height']=$infoImg[1]; // Hauteur de l'image
-	}
+    if ($filetoread)
+    {
+        $infoImg = getimagesize($filetoread); // Recuperation des infos de l'image
+        $ret['width']=$infoImg[0]; // Largeur de l'image
+        $ret['height']=$infoImg[1]; // Hauteur de l'image
+    }
 
-	return $ret;
+    return $ret;
 }
 
 
@@ -108,191 +108,191 @@ 
  */
 function dol_imageResizeOrCrop($file, $mode, $newWidth, $newHeight, $src_x=0, $src_y=0)
 {
-	require_once DOL_DOCUMENT_ROOT.'/core/lib/functions2.lib.php';
-
-	global $conf,$langs;
-
-	dol_syslog("dol_imageResizeOrCrop file=".$file." mode=".$mode." newWidth=".$newWidth." newHeight=".$newHeight." src_x=".$src_x." src_y=".$src_y);
-
-	// Clean parameters
-	$file=trim($file);
-
-	// Check parameters
-	if (! $file)
-	{
-		// Si le fichier n'a pas ete indique
-		return 'Bad parameter file';
-	}
-	elseif (! file_exists($file))
-	{
-		// Si le fichier passe en parametre n'existe pas
-		return $langs->trans("ErrorFileNotFound",$file);
-	}
-	elseif(image_format_supported($file) < 0)
-	{
-		return 'This filename '.$file.' does not seem to be an image filename.';
-	}
-	elseif(!is_numeric($newWidth) && !is_numeric($newHeight))
-	{
-		return 'Wrong value for parameter newWidth or newHeight';
-	}
-	elseif ($mode == 0 && $newWidth <= 0 && $newHeight <= 0)
-	{
-		return 'At least newHeight or newWidth must be defined for resizing';
-	}
-	elseif ($mode == 1 && ($newWidth <= 0 || $newHeight <= 0))
-	{
-		return 'Both newHeight or newWidth must be defined for croping';
-	}
-
-	$filetoread = realpath(dol_osencode($file)); 	// Chemin canonique absolu de l'image
-
-	$infoImg = getimagesize($filetoread); // Recuperation des infos de l'image
-	$imgWidth = $infoImg[0]; // Largeur de l'image
-	$imgHeight = $infoImg[1]; // Hauteur de l'image
-
-	if ($mode == 0)	// If resize, we check parameters
-	{
-		if ($newWidth  <= 0)
-		{
-			$newWidth=intval(($newHeight / $imgHeight) * $imgWidth);	// Keep ratio
-		}
-		if ($newHeight <= 0)
-		{
-			$newHeight=intval(($newWidth / $imgWidth) * $imgHeight);	// Keep ratio
-		}
-	}
-
-	$imgfonction='';
-	switch($infoImg[2])
-	{
-		case 1:	// IMG_GIF
-			$imgfonction = 'imagecreatefromgif';
-			break;
-		case 2:	// IMG_JPG
-			$imgfonction = 'imagecreatefromjpeg';
-			break;
-		case 3:	// IMG_PNG
-			$imgfonction = 'imagecreatefrompng';
-			break;
-		case 4:	// IMG_WBMP
-			$imgfonction = 'imagecreatefromwbmp';
-			break;
-	}
-	if ($imgfonction)
-	{
-		if (! function_exists($imgfonction))
-		{
-			// Fonctions de conversion non presente dans ce PHP
-			return 'Resize not possible. This PHP does not support GD functions '.$imgfonction;
-		}
-	}
-
-	// Initialisation des variables selon l'extension de l'image
-	switch($infoImg[2])
-	{
-		case 1:	// Gif
-			$img = imagecreatefromgif($filetoread);
-			$extImg = '.gif';	// File name extension of image
-			$newquality='NU';	// Quality is not used for this format
-			break;
-		case 2:	// Jpg
-			$img = imagecreatefromjpeg($filetoread);
-			$extImg = '.jpg';
-			$newquality=100;	// % quality maximum
-			break;
-		case 3:	// Png
-			$img = imagecreatefrompng($filetoread);
-			$extImg = '.png';
-			$newquality=0;		// No compression (0-9)
-			break;
-		case 4:	// Bmp
-			$img = imagecreatefromwbmp($filetoread);
-			$extImg = '.bmp';
-			$newquality='NU';	// Quality is not used for this format
-			break;
-	}
-
-	// Create empty image
-	if ($infoImg[2] == 1)
-	{
-		// Compatibilite image GIF
-		$imgThumb = imagecreate($newWidth, $newHeight);
-	}
-	else
-	{
-		$imgThumb = imagecreatetruecolor($newWidth, $newHeight);
-	}
-
-	// Activate antialiasing for better quality
-	if (function_exists('imageantialias'))
-	{
-		imageantialias($imgThumb, true);
-	}
-
-	// This is to keep transparent alpha channel if exists (PHP >= 4.2)
-	if (function_exists('imagesavealpha'))
-	{
-		imagesavealpha($imgThumb, true);
-	}
-
-	// Initialisation des variables selon l'extension de l'image
-	switch($infoImg[2])
-	{
-		case 1:	// Gif
-			$trans_colour = imagecolorallocate($imgThumb, 255, 255, 255); // On procede autrement pour le format GIF
-			imagecolortransparent($imgThumb,$trans_colour);
-			break;
-		case 2:	// Jpg
-			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
-			break;
-		case 3:	// Png
-			imagealphablending($imgThumb,false); // Pour compatibilite sur certain systeme
-			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 127);	// Keep transparent channel
-			break;
-		case 4:	// Bmp
-			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
-			break;
-	}
-	if (function_exists("imagefill")) imagefill($imgThumb, 0, 0, $trans_colour);
-
-	dol_syslog("dol_imageResizeOrCrop: convert image from ($imgWidth x $imgHeight) at position ($src_x x $src_y) to ($newWidth x $newHeight) as $extImg, newquality=$newquality");
-	//imagecopyresized($imgThumb, $img, 0, 0, 0, 0, $thumbWidth, $thumbHeight, $imgWidth, $imgHeight); // Insere l'image de base redimensionnee
-	imagecopyresampled($imgThumb, $img, 0, 0, $src_x, $src_y, $newWidth, $newHeight, ($mode==0?$imgWidth:$newWidth), ($mode==0?$imgHeight:$newHeight)); // Insere l'image de base redimensionnee
-
-	$imgThumbName = $file;
-
-	// Check if permission are ok
-	//$fp = fopen($imgThumbName, "w");
-	//fclose($fp);
-
-	// Create image on disk
-	switch($infoImg[2])
-	{
-		case 1:	// Gif
-			imagegif($imgThumb, $imgThumbName);
-			break;
-		case 2:	// Jpg
-			imagejpeg($imgThumb, $imgThumbName, $newquality);
-			break;
-		case 3:	// Png
-			imagepng($imgThumb, $imgThumbName, $newquality);
-			break;
-		case 4:	// Bmp
-			image2wbmp($imgThumb, $imgThumbName);
-			break;
-	}
-
-	// Set permissions on file
-	if (! empty($conf->global->MAIN_UMASK)) @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
-
-	// Free memory. This does not delete image.
+    require_once DOL_DOCUMENT_ROOT.'/core/lib/functions2.lib.php';
+
+    global $conf,$langs;
+
+    dol_syslog("dol_imageResizeOrCrop file=".$file." mode=".$mode." newWidth=".$newWidth." newHeight=".$newHeight." src_x=".$src_x." src_y=".$src_y);
+
+    // Clean parameters
+    $file=trim($file);
+
+    // Check parameters
+    if (! $file)
+    {
+        // Si le fichier n'a pas ete indique
+        return 'Bad parameter file';
+    }
+    elseif (! file_exists($file))
+    {
+        // Si le fichier passe en parametre n'existe pas
+        return $langs->trans("ErrorFileNotFound",$file);
+    }
+    elseif(image_format_supported($file) < 0)
+    {
+        return 'This filename '.$file.' does not seem to be an image filename.';
+    }
+    elseif(!is_numeric($newWidth) && !is_numeric($newHeight))
+    {
+        return 'Wrong value for parameter newWidth or newHeight';
+    }
+    elseif ($mode == 0 && $newWidth <= 0 && $newHeight <= 0)
+    {
+        return 'At least newHeight or newWidth must be defined for resizing';
+    }
+    elseif ($mode == 1 && ($newWidth <= 0 || $newHeight <= 0))
+    {
+        return 'Both newHeight or newWidth must be defined for croping';
+    }
+
+    $filetoread = realpath(dol_osencode($file)); 	// Chemin canonique absolu de l'image
+
+    $infoImg = getimagesize($filetoread); // Recuperation des infos de l'image
+    $imgWidth = $infoImg[0]; // Largeur de l'image
+    $imgHeight = $infoImg[1]; // Hauteur de l'image
+
+    if ($mode == 0)	// If resize, we check parameters
+    {
+        if ($newWidth  <= 0)
+        {
+            $newWidth=intval(($newHeight / $imgHeight) * $imgWidth);	// Keep ratio
+        }
+        if ($newHeight <= 0)
+        {
+            $newHeight=intval(($newWidth / $imgWidth) * $imgHeight);	// Keep ratio
+        }
+    }
+
+    $imgfonction='';
+    switch($infoImg[2])
+    {
+        case 1:	// IMG_GIF
+            $imgfonction = 'imagecreatefromgif';
+            break;
+        case 2:	// IMG_JPG
+            $imgfonction = 'imagecreatefromjpeg';
+            break;
+        case 3:	// IMG_PNG
+            $imgfonction = 'imagecreatefrompng';
+            break;
+        case 4:	// IMG_WBMP
+            $imgfonction = 'imagecreatefromwbmp';
+            break;
+    }
+    if ($imgfonction)
+    {
+        if (! function_exists($imgfonction))
+        {
+            // Fonctions de conversion non presente dans ce PHP
+            return 'Resize not possible. This PHP does not support GD functions '.$imgfonction;
+        }
+    }
+
+    // Initialisation des variables selon l'extension de l'image
+    switch($infoImg[2])
+    {
+        case 1:	// Gif
+            $img = imagecreatefromgif($filetoread);
+            $extImg = '.gif';	// File name extension of image
+            $newquality='NU';	// Quality is not used for this format
+            break;
+        case 2:	// Jpg
+            $img = imagecreatefromjpeg($filetoread);
+            $extImg = '.jpg';
+            $newquality=100;	// % quality maximum
+            break;
+        case 3:	// Png
+            $img = imagecreatefrompng($filetoread);
+            $extImg = '.png';
+            $newquality=0;		// No compression (0-9)
+            break;
+        case 4:	// Bmp
+            $img = imagecreatefromwbmp($filetoread);
+            $extImg = '.bmp';
+            $newquality='NU';	// Quality is not used for this format
+            break;
+    }
+
+    // Create empty image
+    if ($infoImg[2] == 1)
+    {
+        // Compatibilite image GIF
+        $imgThumb = imagecreate($newWidth, $newHeight);
+    }
+    else
+    {
+        $imgThumb = imagecreatetruecolor($newWidth, $newHeight);
+    }
+
+    // Activate antialiasing for better quality
+    if (function_exists('imageantialias'))
+    {
+        imageantialias($imgThumb, true);
+    }
+
+    // This is to keep transparent alpha channel if exists (PHP >= 4.2)
+    if (function_exists('imagesavealpha'))
+    {
+        imagesavealpha($imgThumb, true);
+    }
+
+    // Initialisation des variables selon l'extension de l'image
+    switch($infoImg[2])
+    {
+        case 1:	// Gif
+            $trans_colour = imagecolorallocate($imgThumb, 255, 255, 255); // On procede autrement pour le format GIF
+            imagecolortransparent($imgThumb,$trans_colour);
+            break;
+        case 2:	// Jpg
+            $trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
+            break;
+        case 3:	// Png
+            imagealphablending($imgThumb,false); // Pour compatibilite sur certain systeme
+            $trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 127);	// Keep transparent channel
+            break;
+        case 4:	// Bmp
+            $trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
+            break;
+    }
+    if (function_exists("imagefill")) imagefill($imgThumb, 0, 0, $trans_colour);
+
+    dol_syslog("dol_imageResizeOrCrop: convert image from ($imgWidth x $imgHeight) at position ($src_x x $src_y) to ($newWidth x $newHeight) as $extImg, newquality=$newquality");
+    //imagecopyresized($imgThumb, $img, 0, 0, 0, 0, $thumbWidth, $thumbHeight, $imgWidth, $imgHeight); // Insere l'image de base redimensionnee
+    imagecopyresampled($imgThumb, $img, 0, 0, $src_x, $src_y, $newWidth, $newHeight, ($mode==0?$imgWidth:$newWidth), ($mode==0?$imgHeight:$newHeight)); // Insere l'image de base redimensionnee
+
+    $imgThumbName = $file;
+
+    // Check if permission are ok
+    //$fp = fopen($imgThumbName, "w");
+    //fclose($fp);
+
+    // Create image on disk
+    switch($infoImg[2])
+    {
+        case 1:	// Gif
+            imagegif($imgThumb, $imgThumbName);
+            break;
+        case 2:	// Jpg
+            imagejpeg($imgThumb, $imgThumbName, $newquality);
+            break;
+        case 3:	// Png
+            imagepng($imgThumb, $imgThumbName, $newquality);
+            break;
+        case 4:	// Bmp
+            image2wbmp($imgThumb, $imgThumbName);
+            break;
+    }
+
+    // Set permissions on file
+    if (! empty($conf->global->MAIN_UMASK)) @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
+
+    // Free memory. This does not delete image.
     imagedestroy($img);
-	imagedestroy($imgThumb);
+    imagedestroy($imgThumb);
 
-	clearstatcache();	// File was replaced by a modified one, so we clear file caches.
+    clearstatcache();	// File was replaced by a modified one, so we clear file caches.
 
-	return $imgThumbName;
+    return $imgThumbName;
 }
 
 
@@ -351,247 +351,247 @@ 
  */
 function vignette($file, $maxWidth = 160, $maxHeight = 120, $extName='_small', $quality=50, $outdir='thumbs', $targetformat=0)
 {
-	require_once DOL_DOCUMENT_ROOT.'/core/lib/functions2.lib.php';
+    require_once DOL_DOCUMENT_ROOT.'/core/lib/functions2.lib.php';
 
-	global $conf,$langs;
+    global $conf,$langs;
 
-	dol_syslog("vignette file=".$file." extName=".$extName." maxWidth=".$maxWidth." maxHeight=".$maxHeight." quality=".$quality." outdir=".$outdir." targetformat=".$targetformat);
+    dol_syslog("vignette file=".$file." extName=".$extName." maxWidth=".$maxWidth." maxHeight=".$maxHeight." quality=".$quality." outdir=".$outdir." targetformat=".$targetformat);
 
-	// Clean parameters
-	$file=trim($file);
+    // Clean parameters
+    $file=trim($file);
 
-	// Check parameters
-	if (! $file)
-	{
-		// Si le fichier n'a pas ete indique
-		return 'ErrorBadParameters';
-	}
-	elseif (! file_exists($file))
-	{
-		// Si le fichier passe en parametre n'existe pas
+    // Check parameters
+    if (! $file)
+    {
+        // Si le fichier n'a pas ete indique
+        return 'ErrorBadParameters';
+    }
+    elseif (! file_exists($file))
+    {
+        // Si le fichier passe en parametre n'existe pas
         dol_syslog($langs->trans("ErrorFileNotFound",$file),LOG_ERR);
-	    return $langs->trans("ErrorFileNotFound",$file);
-	}
-	elseif(image_format_supported($file) < 0)
-	{
+        return $langs->trans("ErrorFileNotFound",$file);
+    }
+    elseif(image_format_supported($file) < 0)
+    {
         dol_syslog('This file '.$file.' does not seem to be an image format file name.',LOG_WARNING);
-	    return 'ErrorBadImageFormat';
-	}
-	elseif(!is_numeric($maxWidth) || empty($maxWidth) || $maxWidth < -1){
-		// Si la largeur max est incorrecte (n'est pas numerique, est vide, ou est inferieure a 0)
+        return 'ErrorBadImageFormat';
+    }
+    elseif(!is_numeric($maxWidth) || empty($maxWidth) || $maxWidth < -1){
+        // Si la largeur max est incorrecte (n'est pas numerique, est vide, ou est inferieure a 0)
         dol_syslog('Wrong value for parameter maxWidth',LOG_ERR);
-	    return 'Error: Wrong value for parameter maxWidth';
-	}
-	elseif(!is_numeric($maxHeight) || empty($maxHeight) || $maxHeight < -1){
-		// Si la hauteur max est incorrecte (n'est pas numerique, est vide, ou est inferieure a 0)
+        return 'Error: Wrong value for parameter maxWidth';
+    }
+    elseif(!is_numeric($maxHeight) || empty($maxHeight) || $maxHeight < -1){
+        // Si la hauteur max est incorrecte (n'est pas numerique, est vide, ou est inferieure a 0)
         dol_syslog('Wrong value for parameter maxHeight',LOG_ERR);
-	    return 'Error: Wrong value for parameter maxHeight';
-	}
-
-	$filetoread = realpath(dol_osencode($file)); 	// Chemin canonique absolu de l'image
-
-	$infoImg = getimagesize($filetoread); // Recuperation des infos de l'image
-	$imgWidth = $infoImg[0]; // Largeur de l'image
-	$imgHeight = $infoImg[1]; // Hauteur de l'image
-
-	if ($maxWidth  == -1) $maxWidth=$infoImg[0];	// If size is -1, we keep unchanged
-	if ($maxHeight == -1) $maxHeight=$infoImg[1];	// If size is -1, we keep unchanged
-
-	// Si l'image est plus petite que la largeur et la hauteur max, on ne cree pas de vignette
-	if ($infoImg[0] < $maxWidth && $infoImg[1] < $maxHeight)
-	{
-		// On cree toujours les vignettes
-		dol_syslog("File size is smaller than thumb size",LOG_DEBUG);
-		//return 'Le fichier '.$file.' ne necessite pas de creation de vignette';
-	}
-
-	$imgfonction='';
-	switch($infoImg[2])
-	{
-		case IMAGETYPE_GIF:	    // 1
-			$imgfonction = 'imagecreatefromgif';
-			break;
-		case IMAGETYPE_JPEG:    // 2
-			$imgfonction = 'imagecreatefromjpeg';
-			break;
-		case IMAGETYPE_PNG:	    // 3
-			$imgfonction = 'imagecreatefrompng';
-			break;
-		case IMAGETYPE_BMP:	    // 6
+        return 'Error: Wrong value for parameter maxHeight';
+    }
+
+    $filetoread = realpath(dol_osencode($file)); 	// Chemin canonique absolu de l'image
+
+    $infoImg = getimagesize($filetoread); // Recuperation des infos de l'image
+    $imgWidth = $infoImg[0]; // Largeur de l'image
+    $imgHeight = $infoImg[1]; // Hauteur de l'image
+
+    if ($maxWidth  == -1) $maxWidth=$infoImg[0];	// If size is -1, we keep unchanged
+    if ($maxHeight == -1) $maxHeight=$infoImg[1];	// If size is -1, we keep unchanged
+
+    // Si l'image est plus petite que la largeur et la hauteur max, on ne cree pas de vignette
+    if ($infoImg[0] < $maxWidth && $infoImg[1] < $maxHeight)
+    {
+        // On cree toujours les vignettes
+        dol_syslog("File size is smaller than thumb size",LOG_DEBUG);
+        //return 'Le fichier '.$file.' ne necessite pas de creation de vignette';
+    }
+
+    $imgfonction='';
+    switch($infoImg[2])
+    {
+        case IMAGETYPE_GIF:	    // 1
+            $imgfonction = 'imagecreatefromgif';
+            break;
+        case IMAGETYPE_JPEG:    // 2
+            $imgfonction = 'imagecreatefromjpeg';
+            break;
+        case IMAGETYPE_PNG:	    // 3
+            $imgfonction = 'imagecreatefrompng';
+            break;
+        case IMAGETYPE_BMP:	    // 6
             // Not supported by PHP GD
-			break;
-		case IMAGETYPE_WBMP:	// 15
-			$imgfonction = 'imagecreatefromwbmp';
-			break;
-	}
-	if ($imgfonction)
-	{
-		if (! function_exists($imgfonction))
-		{
-			// Fonctions de conversion non presente dans ce PHP
-			return 'Error: Creation of thumbs not possible. This PHP does not support GD function '.$imgfonction;
-		}
-	}
-
-	// On cree le repertoire contenant les vignettes
-	$dirthumb = dirname($file).($outdir?'/'.$outdir:''); 	// Chemin du dossier contenant les vignettes
-	dol_mkdir($dirthumb);
-
-	// Initialisation des variables selon l'extension de l'image
-	$img=null;
-	switch($infoImg[2])
-	{
-		case IMAGETYPE_GIF:	    // 1
-			$img = imagecreatefromgif($filetoread);
-			$extImg = '.gif'; // Extension de l'image
-			break;
-		case IMAGETYPE_JPEG:    // 2
-			$img = imagecreatefromjpeg($filetoread);
-			$extImg = (preg_match('/\.jpeg$/',$file)?'.jpeg':'.jpg'); // Extension de l'image
-			break;
-		case IMAGETYPE_PNG:	    // 3
-			$img = imagecreatefrompng($filetoread);
-			$extImg = '.png';
-			break;
-		case IMAGETYPE_BMP:	    // 6
+            break;
+        case IMAGETYPE_WBMP:	// 15
+            $imgfonction = 'imagecreatefromwbmp';
+            break;
+    }
+    if ($imgfonction)
+    {
+        if (! function_exists($imgfonction))
+        {
+            // Fonctions de conversion non presente dans ce PHP
+            return 'Error: Creation of thumbs not possible. This PHP does not support GD function '.$imgfonction;
+        }
+    }
+
+    // On cree le repertoire contenant les vignettes
+    $dirthumb = dirname($file).($outdir?'/'.$outdir:''); 	// Chemin du dossier contenant les vignettes
+    dol_mkdir($dirthumb);
+
+    // Initialisation des variables selon l'extension de l'image
+    $img=null;
+    switch($infoImg[2])
+    {
+        case IMAGETYPE_GIF:	    // 1
+            $img = imagecreatefromgif($filetoread);
+            $extImg = '.gif'; // Extension de l'image
+            break;
+        case IMAGETYPE_JPEG:    // 2
+            $img = imagecreatefromjpeg($filetoread);
+            $extImg = (preg_match('/\.jpeg$/',$file)?'.jpeg':'.jpg'); // Extension de l'image
+            break;
+        case IMAGETYPE_PNG:	    // 3
+            $img = imagecreatefrompng($filetoread);
+            $extImg = '.png';
+            break;
+        case IMAGETYPE_BMP:	    // 6
             // Not supported by PHP GD
-			$extImg = '.bmp';
-			break;
-		case IMAGETYPE_WBMP:	// 15
-			$img = imagecreatefromwbmp($filetoread);
-			$extImg = '.bmp';
-			break;
-	}
+            $extImg = '.bmp';
+            break;
+        case IMAGETYPE_WBMP:	// 15
+            $img = imagecreatefromwbmp($filetoread);
+            $extImg = '.bmp';
+            break;
+    }
     if (! is_resource($img))
     {
         dol_syslog('Failed to detect type of image. We found infoImg[2]='.$infoImg[2], LOG_WARNING);
         return 0;
     }
 
-	// Initialisation des dimensions de la vignette si elles sont superieures a l'original
-	if($maxWidth > $imgWidth){ $maxWidth = $imgWidth; }
-	if($maxHeight > $imgHeight){ $maxHeight = $imgHeight; }
-
-	$whFact = $maxWidth/$maxHeight; // Facteur largeur/hauteur des dimensions max de la vignette
-	$imgWhFact = $imgWidth/$imgHeight; // Facteur largeur/hauteur de l'original
-
-	// Fixe les dimensions de la vignette
-	if($whFact < $imgWhFact)
-	{
-		// Si largeur determinante
-		$thumbWidth  = $maxWidth;
-		$thumbHeight = $thumbWidth / $imgWhFact;
-	}
-	else
-	{
-		// Si hauteur determinante
-		$thumbHeight = $maxHeight;
-		$thumbWidth  = $thumbHeight * $imgWhFact;
-	}
-	$thumbHeight=round($thumbHeight);
-	$thumbWidth=round($thumbWidth);
+    // Initialisation des dimensions de la vignette si elles sont superieures a l'original
+    if($maxWidth > $imgWidth){ $maxWidth = $imgWidth; }
+    if($maxHeight > $imgHeight){ $maxHeight = $imgHeight; }
+
+    $whFact = $maxWidth/$maxHeight; // Facteur largeur/hauteur des dimensions max de la vignette
+    $imgWhFact = $imgWidth/$imgHeight; // Facteur largeur/hauteur de l'original
+
+    // Fixe les dimensions de la vignette
+    if($whFact < $imgWhFact)
+    {
+        // Si largeur determinante
+        $thumbWidth  = $maxWidth;
+        $thumbHeight = $thumbWidth / $imgWhFact;
+    }
+    else
+    {
+        // Si hauteur determinante
+        $thumbHeight = $maxHeight;
+        $thumbWidth  = $thumbHeight * $imgWhFact;
+    }
+    $thumbHeight=round($thumbHeight);
+    $thumbWidth=round($thumbWidth);
 
     // Define target format
     if (empty($targetformat)) $targetformat=$infoImg[2];
 
-	// Create empty image
-	if ($targetformat == IMAGETYPE_GIF)
-	{
-		// Compatibilite image GIF
-		$imgThumb = imagecreate($thumbWidth, $thumbHeight);
-	}
-	else
-	{
-		$imgThumb = imagecreatetruecolor($thumbWidth, $thumbHeight);
-	}
-
-	// Activate antialiasing for better quality
-	if (function_exists('imageantialias'))
-	{
-		imageantialias($imgThumb, true);
-	}
-
-	// This is to keep transparent alpha channel if exists (PHP >= 4.2)
-	if (function_exists('imagesavealpha'))
-	{
-		imagesavealpha($imgThumb, true);
-	}
-
-	// Initialisation des variables selon l'extension de l'image
-	// $targetformat is 0 by default, in such case, we keep original extension
-	switch($targetformat)
-	{
-		case IMAGETYPE_GIF:	    // 1
-			$trans_colour = imagecolorallocate($imgThumb, 255, 255, 255); // On procede autrement pour le format GIF
-			imagecolortransparent($imgThumb,$trans_colour);
+    // Create empty image
+    if ($targetformat == IMAGETYPE_GIF)
+    {
+        // Compatibilite image GIF
+        $imgThumb = imagecreate($thumbWidth, $thumbHeight);
+    }
+    else
+    {
+        $imgThumb = imagecreatetruecolor($thumbWidth, $thumbHeight);
+    }
+
+    // Activate antialiasing for better quality
+    if (function_exists('imageantialias'))
+    {
+        imageantialias($imgThumb, true);
+    }
+
+    // This is to keep transparent alpha channel if exists (PHP >= 4.2)
+    if (function_exists('imagesavealpha'))
+    {
+        imagesavealpha($imgThumb, true);
+    }
+
+    // Initialisation des variables selon l'extension de l'image
+    // $targetformat is 0 by default, in such case, we keep original extension
+    switch($targetformat)
+    {
+        case IMAGETYPE_GIF:	    // 1
+            $trans_colour = imagecolorallocate($imgThumb, 255, 255, 255); // On procede autrement pour le format GIF
+            imagecolortransparent($imgThumb,$trans_colour);
             $extImgTarget = '.gif';
             $newquality='NU';
             break;
-		case IMAGETYPE_JPEG:    // 2
+        case IMAGETYPE_JPEG:    // 2
             $trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
             $extImgTarget = (preg_match('/\.jpeg$/i',$file)?'.jpeg':'.jpg');
             $newquality=$quality;
             break;
-		case IMAGETYPE_PNG:	    // 3
-			imagealphablending($imgThumb,false); // Pour compatibilite sur certain systeme
-			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 127);	// Keep transparent channel
+        case IMAGETYPE_PNG:	    // 3
+            imagealphablending($imgThumb,false); // Pour compatibilite sur certain systeme
+            $trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 127);	// Keep transparent channel
             $extImgTarget = '.png';
             $newquality=$quality-100;
             $newquality=round(abs($quality-100)*9/100);
             break;
-		case IMAGETYPE_BMP:	    // 6
+        case IMAGETYPE_BMP:	    // 6
             // Not supported by PHP GD
             $extImgTarget = '.bmp';
             $newquality='NU';
             break;
-		case IMAGETYPE_WBMP:	// 15
-			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
+        case IMAGETYPE_WBMP:	// 15
+            $trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
             $extImgTarget = '.bmp';
             $newquality='NU';
             break;
-	}
-	if (function_exists("imagefill")) imagefill($imgThumb, 0, 0, $trans_colour);
-
-	dol_syslog("vignette: convert image from ($imgWidth x $imgHeight) to ($thumbWidth x $thumbHeight) as $extImg, newquality=$newquality");
-	//imagecopyresized($imgThumb, $img, 0, 0, 0, 0, $thumbWidth, $thumbHeight, $imgWidth, $imgHeight); // Insere l'image de base redimensionnee
-	imagecopyresampled($imgThumb, $img, 0, 0, 0, 0, $thumbWidth, $thumbHeight, $imgWidth, $imgHeight); // Insere l'image de base redimensionnee
-
-	$fileName = preg_replace('/(\.gif|\.jpeg|\.jpg|\.png|\.bmp)$/i','',$file);	// On enleve extension quelquesoit la casse
-	$fileName = basename($fileName);
-	//$imgThumbName = $dirthumb.'/'.getImageFileNameForSize(basename($file), $extName, $extImgTarget);   // Full path of thumb file
-	$imgThumbName = getImageFileNameForSize($file, $extName, $extImgTarget);   // Full path of thumb file
-
-
-	// Check if permission are ok
-	//$fp = fopen($imgThumbName, "w");
-	//fclose($fp);
-
-	// Create image on disk
-	switch($targetformat)
-	{
-		case IMAGETYPE_GIF:	    // 1
-			imagegif($imgThumb, $imgThumbName);
-			break;
-		case IMAGETYPE_JPEG:    // 2
-			imagejpeg($imgThumb, $imgThumbName, $newquality);
-			break;
-		case IMAGETYPE_PNG:	    // 3
-			imagepng($imgThumb, $imgThumbName, $newquality);
-			break;
-		case IMAGETYPE_BMP:	    // 6
+    }
+    if (function_exists("imagefill")) imagefill($imgThumb, 0, 0, $trans_colour);
+
+    dol_syslog("vignette: convert image from ($imgWidth x $imgHeight) to ($thumbWidth x $thumbHeight) as $extImg, newquality=$newquality");
+    //imagecopyresized($imgThumb, $img, 0, 0, 0, 0, $thumbWidth, $thumbHeight, $imgWidth, $imgHeight); // Insere l'image de base redimensionnee
+    imagecopyresampled($imgThumb, $img, 0, 0, 0, 0, $thumbWidth, $thumbHeight, $imgWidth, $imgHeight); // Insere l'image de base redimensionnee
+
+    $fileName = preg_replace('/(\.gif|\.jpeg|\.jpg|\.png|\.bmp)$/i','',$file);	// On enleve extension quelquesoit la casse
+    $fileName = basename($fileName);
+    //$imgThumbName = $dirthumb.'/'.getImageFileNameForSize(basename($file), $extName, $extImgTarget);   // Full path of thumb file
+    $imgThumbName = getImageFileNameForSize($file, $extName, $extImgTarget);   // Full path of thumb file
+
+
+    // Check if permission are ok
+    //$fp = fopen($imgThumbName, "w");
+    //fclose($fp);
+
+    // Create image on disk
+    switch($targetformat)
+    {
+        case IMAGETYPE_GIF:	    // 1
+            imagegif($imgThumb, $imgThumbName);
+            break;
+        case IMAGETYPE_JPEG:    // 2
+            imagejpeg($imgThumb, $imgThumbName, $newquality);
+            break;
+        case IMAGETYPE_PNG:	    // 3
+            imagepng($imgThumb, $imgThumbName, $newquality);
+            break;
+        case IMAGETYPE_BMP:	    // 6
             // Not supported by PHP GD
-			break;
-		case IMAGETYPE_WBMP:    // 15
-			image2wbmp($imgThumb, $imgThumbName);
-			break;
-	}
+            break;
+        case IMAGETYPE_WBMP:    // 15
+            image2wbmp($imgThumb, $imgThumbName);
+            break;
+    }
 
-	// Set permissions on file
-	if (! empty($conf->global->MAIN_UMASK)) @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
+    // Set permissions on file
+    if (! empty($conf->global->MAIN_UMASK)) @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
 
     // Free memory. This does not delete image.
     imagedestroy($img);
     imagedestroy($imgThumb);
 
-	return $imgThumbName;
+    return $imgThumbName;
 }

Spacing +87 added lines, -87 removed lines

@@ -23,8 +23,8 @@ 
  */
 
 // Define size of logo small and mini
-$maxwidthsmall=270;$maxheightsmall=150;
-$maxwidthmini=128;$maxheightmini=72;
+$maxwidthsmall = 270; $maxheightsmall = 150;
+$maxwidthmini = 128; $maxheightmini = 72;
 $quality = 80;
 
 
@@ -37,13 +37,13 @@ 
  */
 function image_format_supported($file)
 {
-    $regeximgext='\.gif|\.jpg|\.jpeg|\.png|\.bmp|\.xpm|\.xbm|\.svg';   // See also into product.class.php
+    $regeximgext = '\.gif|\.jpg|\.jpeg|\.png|\.bmp|\.xpm|\.xbm|\.svg'; // See also into product.class.php
 
     // Case filename is not a format image
-    if (! preg_match('/('.$regeximgext.')$/i',$file,$reg)) return -1;
+    if (!preg_match('/('.$regeximgext.')$/i', $file, $reg)) return -1;
 
     // Case filename is a format image but not supported by this PHP
-    $imgfonction='';
+    $imgfonction = '';
     if (strtolower($reg[1]) == '.gif')  $imgfonction = 'imagecreatefromgif';
     if (strtolower($reg[1]) == '.png')  $imgfonction = 'imagecreatefrompng';
     if (strtolower($reg[1]) == '.jpg')  $imgfonction = 'imagecreatefromjpeg';
@@ -53,7 +53,7 @@ 
     if (strtolower($reg[1]) == '.xbm')  $imgfonction = 'imagecreatefromxbm';
     if ($imgfonction)
     {
-        if (! function_exists($imgfonction))
+        if (!function_exists($imgfonction))
         {
             // Fonctions de conversion non presente dans ce PHP
             return 0;
@@ -74,21 +74,21 @@ 
  */
 function dol_getImageSize($file, $url = false)
 {
-	$ret=array();
+	$ret = array();
 
 	if (image_format_supported($file) < 0) return $ret;
 
 	$filetoread = $file;
 	if (!$url)
 	{
-		$filetoread = realpath(dol_osencode($file)); 	// Chemin canonique absolu de l'image
+		$filetoread = realpath(dol_osencode($file)); // Chemin canonique absolu de l'image
 	}
 
 	if ($filetoread)
 	{
     	$infoImg = getimagesize($filetoread); // Recuperation des infos de l'image
-    	$ret['width']=$infoImg[0]; // Largeur de l'image
-    	$ret['height']=$infoImg[1]; // Hauteur de l'image
+    	$ret['width'] = $infoImg[0]; // Largeur de l'image
+    	$ret['height'] = $infoImg[1]; // Hauteur de l'image
 	}
 
 	return $ret;
@@ -106,33 +106,33 @@ 
  * 		@param	int		$src_y			Position of croping image in source image (not use if mode=0)
  *		@return	int						File name if OK, error message if KO
  */
-function dol_imageResizeOrCrop($file, $mode, $newWidth, $newHeight, $src_x=0, $src_y=0)
+function dol_imageResizeOrCrop($file, $mode, $newWidth, $newHeight, $src_x = 0, $src_y = 0)
 {
 	require_once DOL_DOCUMENT_ROOT.'/core/lib/functions2.lib.php';
 
-	global $conf,$langs;
+	global $conf, $langs;
 
 	dol_syslog("dol_imageResizeOrCrop file=".$file." mode=".$mode." newWidth=".$newWidth." newHeight=".$newHeight." src_x=".$src_x." src_y=".$src_y);
 
 	// Clean parameters
-	$file=trim($file);
+	$file = trim($file);
 
 	// Check parameters
-	if (! $file)
+	if (!$file)
 	{
 		// Si le fichier n'a pas ete indique
 		return 'Bad parameter file';
 	}
-	elseif (! file_exists($file))
+	elseif (!file_exists($file))
 	{
 		// Si le fichier passe en parametre n'existe pas
-		return $langs->trans("ErrorFileNotFound",$file);
+		return $langs->trans("ErrorFileNotFound", $file);
 	}
-	elseif(image_format_supported($file) < 0)
+	elseif (image_format_supported($file) < 0)
 	{
 		return 'This filename '.$file.' does not seem to be an image filename.';
 	}
-	elseif(!is_numeric($newWidth) && !is_numeric($newHeight))
+	elseif (!is_numeric($newWidth) && !is_numeric($newHeight))
 	{
 		return 'Wrong value for parameter newWidth or newHeight';
 	}
@@ -145,7 +145,7 @@ 
 		return 'Both newHeight or newWidth must be defined for croping';
 	}
 
-	$filetoread = realpath(dol_osencode($file)); 	// Chemin canonique absolu de l'image
+	$filetoread = realpath(dol_osencode($file)); // Chemin canonique absolu de l'image
 
 	$infoImg = getimagesize($filetoread); // Recuperation des infos de l'image
 	$imgWidth = $infoImg[0]; // Largeur de l'image
@@ -153,18 +153,18 @@ 
 
 	if ($mode == 0)	// If resize, we check parameters
 	{
-		if ($newWidth  <= 0)
+		if ($newWidth <= 0)
 		{
-			$newWidth=intval(($newHeight / $imgHeight) * $imgWidth);	// Keep ratio
+			$newWidth = intval(($newHeight / $imgHeight) * $imgWidth); // Keep ratio
 		}
 		if ($newHeight <= 0)
 		{
-			$newHeight=intval(($newWidth / $imgWidth) * $imgHeight);	// Keep ratio
+			$newHeight = intval(($newWidth / $imgWidth) * $imgHeight); // Keep ratio
 		}
 	}
 
-	$imgfonction='';
-	switch($infoImg[2])
+	$imgfonction = '';
+	switch ($infoImg[2])
 	{
 		case 1:	// IMG_GIF
 			$imgfonction = 'imagecreatefromgif';
@@ -181,7 +181,7 @@ 
 	}
 	if ($imgfonction)
 	{
-		if (! function_exists($imgfonction))
+		if (!function_exists($imgfonction))
 		{
 			// Fonctions de conversion non presente dans ce PHP
 			return 'Resize not possible. This PHP does not support GD functions '.$imgfonction;
@@ -189,27 +189,27 @@ 
 	}
 
 	// Initialisation des variables selon l'extension de l'image
-	switch($infoImg[2])
+	switch ($infoImg[2])
 	{
 		case 1:	// Gif
 			$img = imagecreatefromgif($filetoread);
-			$extImg = '.gif';	// File name extension of image
-			$newquality='NU';	// Quality is not used for this format
+			$extImg = '.gif'; // File name extension of image
+			$newquality = 'NU'; // Quality is not used for this format
 			break;
 		case 2:	// Jpg
 			$img = imagecreatefromjpeg($filetoread);
 			$extImg = '.jpg';
-			$newquality=100;	// % quality maximum
+			$newquality = 100; // % quality maximum
 			break;
 		case 3:	// Png
 			$img = imagecreatefrompng($filetoread);
 			$extImg = '.png';
-			$newquality=0;		// No compression (0-9)
+			$newquality = 0; // No compression (0-9)
 			break;
 		case 4:	// Bmp
 			$img = imagecreatefromwbmp($filetoread);
 			$extImg = '.bmp';
-			$newquality='NU';	// Quality is not used for this format
+			$newquality = 'NU'; // Quality is not used for this format
 			break;
 	}
 
@@ -237,18 +237,18 @@ 
 	}
 
 	// Initialisation des variables selon l'extension de l'image
-	switch($infoImg[2])
+	switch ($infoImg[2])
 	{
 		case 1:	// Gif
 			$trans_colour = imagecolorallocate($imgThumb, 255, 255, 255); // On procede autrement pour le format GIF
-			imagecolortransparent($imgThumb,$trans_colour);
+			imagecolortransparent($imgThumb, $trans_colour);
 			break;
 		case 2:	// Jpg
 			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
 			break;
 		case 3:	// Png
-			imagealphablending($imgThumb,false); // Pour compatibilite sur certain systeme
-			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 127);	// Keep transparent channel
+			imagealphablending($imgThumb, false); // Pour compatibilite sur certain systeme
+			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 127); // Keep transparent channel
 			break;
 		case 4:	// Bmp
 			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
@@ -258,7 +258,7 @@ 
 
 	dol_syslog("dol_imageResizeOrCrop: convert image from ($imgWidth x $imgHeight) at position ($src_x x $src_y) to ($newWidth x $newHeight) as $extImg, newquality=$newquality");
 	//imagecopyresized($imgThumb, $img, 0, 0, 0, 0, $thumbWidth, $thumbHeight, $imgWidth, $imgHeight); // Insere l'image de base redimensionnee
-	imagecopyresampled($imgThumb, $img, 0, 0, $src_x, $src_y, $newWidth, $newHeight, ($mode==0?$imgWidth:$newWidth), ($mode==0?$imgHeight:$newHeight)); // Insere l'image de base redimensionnee
+	imagecopyresampled($imgThumb, $img, 0, 0, $src_x, $src_y, $newWidth, $newHeight, ($mode == 0 ? $imgWidth : $newWidth), ($mode == 0 ? $imgHeight : $newHeight)); // Insere l'image de base redimensionnee
 
 	$imgThumbName = $file;
 
@@ -267,7 +267,7 @@ 
 	//fclose($fp);
 
 	// Create image on disk
-	switch($infoImg[2])
+	switch ($infoImg[2])
 	{
 		case 1:	// Gif
 			imagegif($imgThumb, $imgThumbName);
@@ -284,13 +284,13 @@ 
 	}
 
 	// Set permissions on file
-	if (! empty($conf->global->MAIN_UMASK)) @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
+	if (!empty($conf->global->MAIN_UMASK)) @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
 
 	// Free memory. This does not delete image.
     imagedestroy($img);
 	imagedestroy($imgThumb);
 
-	clearstatcache();	// File was replaced by a modified one, so we clear file caches.
+	clearstatcache(); // File was replaced by a modified one, so we clear file caches.
 
 	return $imgThumbName;
 }
@@ -349,64 +349,64 @@ 
  *      @param     int		$targetformat     	New format of target (IMAGETYPE_GIF, IMAGETYPE_JPG, IMAGETYPE_PNG, IMAGETYPE_BMP, IMAGETYPE_WBMP ... or 0 to keep old format)
  *    	@return    string						Full path of thumb or '' if it fails or 'Error...' if it fails
  */
-function vignette($file, $maxWidth = 160, $maxHeight = 120, $extName='_small', $quality=50, $outdir='thumbs', $targetformat=0)
+function vignette($file, $maxWidth = 160, $maxHeight = 120, $extName = '_small', $quality = 50, $outdir = 'thumbs', $targetformat = 0)
 {
 	require_once DOL_DOCUMENT_ROOT.'/core/lib/functions2.lib.php';
 
-	global $conf,$langs;
+	global $conf, $langs;
 
 	dol_syslog("vignette file=".$file." extName=".$extName." maxWidth=".$maxWidth." maxHeight=".$maxHeight." quality=".$quality." outdir=".$outdir." targetformat=".$targetformat);
 
 	// Clean parameters
-	$file=trim($file);
+	$file = trim($file);
 
 	// Check parameters
-	if (! $file)
+	if (!$file)
 	{
 		// Si le fichier n'a pas ete indique
 		return 'ErrorBadParameters';
 	}
-	elseif (! file_exists($file))
+	elseif (!file_exists($file))
 	{
 		// Si le fichier passe en parametre n'existe pas
-        dol_syslog($langs->trans("ErrorFileNotFound",$file),LOG_ERR);
-	    return $langs->trans("ErrorFileNotFound",$file);
+        dol_syslog($langs->trans("ErrorFileNotFound", $file), LOG_ERR);
+	    return $langs->trans("ErrorFileNotFound", $file);
 	}
-	elseif(image_format_supported($file) < 0)
+	elseif (image_format_supported($file) < 0)
 	{
-        dol_syslog('This file '.$file.' does not seem to be an image format file name.',LOG_WARNING);
+        dol_syslog('This file '.$file.' does not seem to be an image format file name.', LOG_WARNING);
 	    return 'ErrorBadImageFormat';
 	}
-	elseif(!is_numeric($maxWidth) || empty($maxWidth) || $maxWidth < -1){
+	elseif (!is_numeric($maxWidth) || empty($maxWidth) || $maxWidth < -1) {
 		// Si la largeur max est incorrecte (n'est pas numerique, est vide, ou est inferieure a 0)
-        dol_syslog('Wrong value for parameter maxWidth',LOG_ERR);
+        dol_syslog('Wrong value for parameter maxWidth', LOG_ERR);
 	    return 'Error: Wrong value for parameter maxWidth';
 	}
-	elseif(!is_numeric($maxHeight) || empty($maxHeight) || $maxHeight < -1){
+	elseif (!is_numeric($maxHeight) || empty($maxHeight) || $maxHeight < -1) {
 		// Si la hauteur max est incorrecte (n'est pas numerique, est vide, ou est inferieure a 0)
-        dol_syslog('Wrong value for parameter maxHeight',LOG_ERR);
+        dol_syslog('Wrong value for parameter maxHeight', LOG_ERR);
 	    return 'Error: Wrong value for parameter maxHeight';
 	}
 
-	$filetoread = realpath(dol_osencode($file)); 	// Chemin canonique absolu de l'image
+	$filetoread = realpath(dol_osencode($file)); // Chemin canonique absolu de l'image
 
 	$infoImg = getimagesize($filetoread); // Recuperation des infos de l'image
 	$imgWidth = $infoImg[0]; // Largeur de l'image
 	$imgHeight = $infoImg[1]; // Hauteur de l'image
 
-	if ($maxWidth  == -1) $maxWidth=$infoImg[0];	// If size is -1, we keep unchanged
-	if ($maxHeight == -1) $maxHeight=$infoImg[1];	// If size is -1, we keep unchanged
+	if ($maxWidth == -1) $maxWidth = $infoImg[0]; // If size is -1, we keep unchanged
+	if ($maxHeight == -1) $maxHeight = $infoImg[1]; // If size is -1, we keep unchanged
 
 	// Si l'image est plus petite que la largeur et la hauteur max, on ne cree pas de vignette
 	if ($infoImg[0] < $maxWidth && $infoImg[1] < $maxHeight)
 	{
 		// On cree toujours les vignettes
-		dol_syslog("File size is smaller than thumb size",LOG_DEBUG);
+		dol_syslog("File size is smaller than thumb size", LOG_DEBUG);
 		//return 'Le fichier '.$file.' ne necessite pas de creation de vignette';
 	}
 
-	$imgfonction='';
-	switch($infoImg[2])
+	$imgfonction = '';
+	switch ($infoImg[2])
 	{
 		case IMAGETYPE_GIF:	    // 1
 			$imgfonction = 'imagecreatefromgif';
@@ -426,7 +426,7 @@ 
 	}
 	if ($imgfonction)
 	{
-		if (! function_exists($imgfonction))
+		if (!function_exists($imgfonction))
 		{
 			// Fonctions de conversion non presente dans ce PHP
 			return 'Error: Creation of thumbs not possible. This PHP does not support GD function '.$imgfonction;
@@ -434,12 +434,12 @@ 
 	}
 
 	// On cree le repertoire contenant les vignettes
-	$dirthumb = dirname($file).($outdir?'/'.$outdir:''); 	// Chemin du dossier contenant les vignettes
+	$dirthumb = dirname($file).($outdir ? '/'.$outdir : ''); // Chemin du dossier contenant les vignettes
 	dol_mkdir($dirthumb);
 
 	// Initialisation des variables selon l'extension de l'image
-	$img=null;
-	switch($infoImg[2])
+	$img = null;
+	switch ($infoImg[2])
 	{
 		case IMAGETYPE_GIF:	    // 1
 			$img = imagecreatefromgif($filetoread);
@@ -447,7 +447,7 @@ 
 			break;
 		case IMAGETYPE_JPEG:    // 2
 			$img = imagecreatefromjpeg($filetoread);
-			$extImg = (preg_match('/\.jpeg$/',$file)?'.jpeg':'.jpg'); // Extension de l'image
+			$extImg = (preg_match('/\.jpeg$/', $file) ? '.jpeg' : '.jpg'); // Extension de l'image
 			break;
 		case IMAGETYPE_PNG:	    // 3
 			$img = imagecreatefrompng($filetoread);
@@ -462,21 +462,21 @@ 
 			$extImg = '.bmp';
 			break;
 	}
-    if (! is_resource($img))
+    if (!is_resource($img))
     {
         dol_syslog('Failed to detect type of image. We found infoImg[2]='.$infoImg[2], LOG_WARNING);
         return 0;
     }
 
 	// Initialisation des dimensions de la vignette si elles sont superieures a l'original
-	if($maxWidth > $imgWidth){ $maxWidth = $imgWidth; }
-	if($maxHeight > $imgHeight){ $maxHeight = $imgHeight; }
+	if ($maxWidth > $imgWidth) { $maxWidth = $imgWidth; }
+	if ($maxHeight > $imgHeight) { $maxHeight = $imgHeight; }
 
-	$whFact = $maxWidth/$maxHeight; // Facteur largeur/hauteur des dimensions max de la vignette
-	$imgWhFact = $imgWidth/$imgHeight; // Facteur largeur/hauteur de l'original
+	$whFact = $maxWidth / $maxHeight; // Facteur largeur/hauteur des dimensions max de la vignette
+	$imgWhFact = $imgWidth / $imgHeight; // Facteur largeur/hauteur de l'original
 
 	// Fixe les dimensions de la vignette
-	if($whFact < $imgWhFact)
+	if ($whFact < $imgWhFact)
 	{
 		// Si largeur determinante
 		$thumbWidth  = $maxWidth;
@@ -488,11 +488,11 @@ 
 		$thumbHeight = $maxHeight;
 		$thumbWidth  = $thumbHeight * $imgWhFact;
 	}
-	$thumbHeight=round($thumbHeight);
-	$thumbWidth=round($thumbWidth);
+	$thumbHeight = round($thumbHeight);
+	$thumbWidth = round($thumbWidth);
 
     // Define target format
-    if (empty($targetformat)) $targetformat=$infoImg[2];
+    if (empty($targetformat)) $targetformat = $infoImg[2];
 
 	// Create empty image
 	if ($targetformat == IMAGETYPE_GIF)
@@ -519,35 +519,35 @@ 
 
 	// Initialisation des variables selon l'extension de l'image
 	// $targetformat is 0 by default, in such case, we keep original extension
-	switch($targetformat)
+	switch ($targetformat)
 	{
 		case IMAGETYPE_GIF:	    // 1
 			$trans_colour = imagecolorallocate($imgThumb, 255, 255, 255); // On procede autrement pour le format GIF
-			imagecolortransparent($imgThumb,$trans_colour);
+			imagecolortransparent($imgThumb, $trans_colour);
             $extImgTarget = '.gif';
-            $newquality='NU';
+            $newquality = 'NU';
             break;
 		case IMAGETYPE_JPEG:    // 2
             $trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
-            $extImgTarget = (preg_match('/\.jpeg$/i',$file)?'.jpeg':'.jpg');
-            $newquality=$quality;
+            $extImgTarget = (preg_match('/\.jpeg$/i', $file) ? '.jpeg' : '.jpg');
+            $newquality = $quality;
             break;
 		case IMAGETYPE_PNG:	    // 3
-			imagealphablending($imgThumb,false); // Pour compatibilite sur certain systeme
-			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 127);	// Keep transparent channel
+			imagealphablending($imgThumb, false); // Pour compatibilite sur certain systeme
+			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 127); // Keep transparent channel
             $extImgTarget = '.png';
-            $newquality=$quality-100;
-            $newquality=round(abs($quality-100)*9/100);
+            $newquality = $quality - 100;
+            $newquality = round(abs($quality - 100) * 9 / 100);
             break;
 		case IMAGETYPE_BMP:	    // 6
             // Not supported by PHP GD
             $extImgTarget = '.bmp';
-            $newquality='NU';
+            $newquality = 'NU';
             break;
 		case IMAGETYPE_WBMP:	// 15
 			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
             $extImgTarget = '.bmp';
-            $newquality='NU';
+            $newquality = 'NU';
             break;
 	}
 	if (function_exists("imagefill")) imagefill($imgThumb, 0, 0, $trans_colour);
@@ -556,10 +556,10 @@ 
 	//imagecopyresized($imgThumb, $img, 0, 0, 0, 0, $thumbWidth, $thumbHeight, $imgWidth, $imgHeight); // Insere l'image de base redimensionnee
 	imagecopyresampled($imgThumb, $img, 0, 0, 0, 0, $thumbWidth, $thumbHeight, $imgWidth, $imgHeight); // Insere l'image de base redimensionnee
 
-	$fileName = preg_replace('/(\.gif|\.jpeg|\.jpg|\.png|\.bmp)$/i','',$file);	// On enleve extension quelquesoit la casse
+	$fileName = preg_replace('/(\.gif|\.jpeg|\.jpg|\.png|\.bmp)$/i', '', $file); // On enleve extension quelquesoit la casse
 	$fileName = basename($fileName);
 	//$imgThumbName = $dirthumb.'/'.getImageFileNameForSize(basename($file), $extName, $extImgTarget);   // Full path of thumb file
-	$imgThumbName = getImageFileNameForSize($file, $extName, $extImgTarget);   // Full path of thumb file
+	$imgThumbName = getImageFileNameForSize($file, $extName, $extImgTarget); // Full path of thumb file
 
 
 	// Check if permission are ok
@@ -567,7 +567,7 @@ 
 	//fclose($fp);
 
 	// Create image on disk
-	switch($targetformat)
+	switch ($targetformat)
 	{
 		case IMAGETYPE_GIF:	    // 1
 			imagegif($imgThumb, $imgThumbName);
@@ -587,7 +587,7 @@ 
 	}
 
 	// Set permissions on file
-	if (! empty($conf->global->MAIN_UMASK)) @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
+	if (!empty($conf->global->MAIN_UMASK)) @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
 
     // Free memory. This does not delete image.
     imagedestroy($img);

Braces +67 added lines, -42 removed lines

@@ -40,17 +40,33 @@ 
     $regeximgext='\.gif|\.jpg|\.jpeg|\.png|\.bmp|\.xpm|\.xbm|\.svg';   // See also into product.class.php
 
     // Case filename is not a format image
-    if (! preg_match('/('.$regeximgext.')$/i',$file,$reg)) return -1;
+    if (! preg_match('/('.$regeximgext.')$/i',$file,$reg)) {
+        return -1;
+    }
 
     // Case filename is a format image but not supported by this PHP
     $imgfonction='';
-    if (strtolower($reg[1]) == '.gif')  $imgfonction = 'imagecreatefromgif';
-    if (strtolower($reg[1]) == '.png')  $imgfonction = 'imagecreatefrompng';
-    if (strtolower($reg[1]) == '.jpg')  $imgfonction = 'imagecreatefromjpeg';
-    if (strtolower($reg[1]) == '.jpeg') $imgfonction = 'imagecreatefromjpeg';
-    if (strtolower($reg[1]) == '.bmp')  $imgfonction = 'imagecreatefromwbmp';
-    if (strtolower($reg[1]) == '.xpm')  $imgfonction = 'imagecreatefromxpm';
-    if (strtolower($reg[1]) == '.xbm')  $imgfonction = 'imagecreatefromxbm';
+    if (strtolower($reg[1]) == '.gif') {
+        $imgfonction = 'imagecreatefromgif';
+    }
+    if (strtolower($reg[1]) == '.png') {
+        $imgfonction = 'imagecreatefrompng';
+    }
+    if (strtolower($reg[1]) == '.jpg') {
+        $imgfonction = 'imagecreatefromjpeg';
+    }
+    if (strtolower($reg[1]) == '.jpeg') {
+        $imgfonction = 'imagecreatefromjpeg';
+    }
+    if (strtolower($reg[1]) == '.bmp') {
+        $imgfonction = 'imagecreatefromwbmp';
+    }
+    if (strtolower($reg[1]) == '.xpm') {
+        $imgfonction = 'imagecreatefromxpm';
+    }
+    if (strtolower($reg[1]) == '.xbm') {
+        $imgfonction = 'imagecreatefromxbm';
+    }
     if ($imgfonction)
     {
         if (! function_exists($imgfonction))
@@ -76,7 +92,9 @@ 
 {
 	$ret=array();
 
-	if (image_format_supported($file) < 0) return $ret;
+	if (image_format_supported($file) < 0) {
+	    return $ret;
+	}
 
 	$filetoread = $file;
 	if (!$url)
@@ -122,25 +140,20 @@ 
 	{
 		// Si le fichier n'a pas ete indique
 		return 'Bad parameter file';
-	}
-	elseif (! file_exists($file))
+	} elseif (! file_exists($file))
 	{
 		// Si le fichier passe en parametre n'existe pas
 		return $langs->trans("ErrorFileNotFound",$file);
-	}
-	elseif(image_format_supported($file) < 0)
+	} elseif(image_format_supported($file) < 0)
 	{
 		return 'This filename '.$file.' does not seem to be an image filename.';
-	}
-	elseif(!is_numeric($newWidth) && !is_numeric($newHeight))
+	} elseif(!is_numeric($newWidth) && !is_numeric($newHeight))
 	{
 		return 'Wrong value for parameter newWidth or newHeight';
-	}
-	elseif ($mode == 0 && $newWidth <= 0 && $newHeight <= 0)
+	} elseif ($mode == 0 && $newWidth <= 0 && $newHeight <= 0)
 	{
 		return 'At least newHeight or newWidth must be defined for resizing';
-	}
-	elseif ($mode == 1 && ($newWidth <= 0 || $newHeight <= 0))
+	} elseif ($mode == 1 && ($newWidth <= 0 || $newHeight <= 0))
 	{
 		return 'Both newHeight or newWidth must be defined for croping';
 	}
@@ -151,11 +164,14 @@ 
 	$imgWidth = $infoImg[0]; // Largeur de l'image
 	$imgHeight = $infoImg[1]; // Hauteur de l'image
 
-	if ($mode == 0)	// If resize, we check parameters
+	if ($mode == 0) {
+	    // If resize, we check parameters
 	{
 		if ($newWidth  <= 0)
 		{
-			$newWidth=intval(($newHeight / $imgHeight) * $imgWidth);	// Keep ratio
+			$newWidth=intval(($newHeight / $imgHeight) * $imgWidth);
+	}
+	// Keep ratio
 		}
 		if ($newHeight <= 0)
 		{
@@ -218,8 +234,7 @@ 
 	{
 		// Compatibilite image GIF
 		$imgThumb = imagecreate($newWidth, $newHeight);
-	}
-	else
+	} else
 	{
 		$imgThumb = imagecreatetruecolor($newWidth, $newHeight);
 	}
@@ -254,7 +269,9 @@ 
 			$trans_colour = imagecolorallocatealpha($imgThumb, 255, 255, 255, 0);
 			break;
 	}
-	if (function_exists("imagefill")) imagefill($imgThumb, 0, 0, $trans_colour);
+	if (function_exists("imagefill")) {
+	    imagefill($imgThumb, 0, 0, $trans_colour);
+	}
 
 	dol_syslog("dol_imageResizeOrCrop: convert image from ($imgWidth x $imgHeight) at position ($src_x x $src_y) to ($newWidth x $newHeight) as $extImg, newquality=$newquality");
 	//imagecopyresized($imgThumb, $img, 0, 0, 0, 0, $thumbWidth, $thumbHeight, $imgWidth, $imgHeight); // Insere l'image de base redimensionnee
@@ -284,7 +301,9 @@ 
 	}
 
 	// Set permissions on file
-	if (! empty($conf->global->MAIN_UMASK)) @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
+	if (! empty($conf->global->MAIN_UMASK)) {
+	    @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
+	}
 
 	// Free memory. This does not delete image.
     imagedestroy($img);
@@ -365,24 +384,20 @@ 
 	{
 		// Si le fichier n'a pas ete indique
 		return 'ErrorBadParameters';
-	}
-	elseif (! file_exists($file))
+	} elseif (! file_exists($file))
 	{
 		// Si le fichier passe en parametre n'existe pas
         dol_syslog($langs->trans("ErrorFileNotFound",$file),LOG_ERR);
 	    return $langs->trans("ErrorFileNotFound",$file);
-	}
-	elseif(image_format_supported($file) < 0)
+	} elseif(image_format_supported($file) < 0)
 	{
         dol_syslog('This file '.$file.' does not seem to be an image format file name.',LOG_WARNING);
 	    return 'ErrorBadImageFormat';
-	}
-	elseif(!is_numeric($maxWidth) || empty($maxWidth) || $maxWidth < -1){
+	} elseif(!is_numeric($maxWidth) || empty($maxWidth) || $maxWidth < -1){
 		// Si la largeur max est incorrecte (n'est pas numerique, est vide, ou est inferieure a 0)
         dol_syslog('Wrong value for parameter maxWidth',LOG_ERR);
 	    return 'Error: Wrong value for parameter maxWidth';
-	}
-	elseif(!is_numeric($maxHeight) || empty($maxHeight) || $maxHeight < -1){
+	} elseif(!is_numeric($maxHeight) || empty($maxHeight) || $maxHeight < -1){
 		// Si la hauteur max est incorrecte (n'est pas numerique, est vide, ou est inferieure a 0)
         dol_syslog('Wrong value for parameter maxHeight',LOG_ERR);
 	    return 'Error: Wrong value for parameter maxHeight';
@@ -394,8 +409,14 @@ 
 	$imgWidth = $infoImg[0]; // Largeur de l'image
 	$imgHeight = $infoImg[1]; // Hauteur de l'image
 
-	if ($maxWidth  == -1) $maxWidth=$infoImg[0];	// If size is -1, we keep unchanged
-	if ($maxHeight == -1) $maxHeight=$infoImg[1];	// If size is -1, we keep unchanged
+	if ($maxWidth  == -1) {
+	    $maxWidth=$infoImg[0];
+	}
+	// If size is -1, we keep unchanged
+	if ($maxHeight == -1) {
+	    $maxHeight=$infoImg[1];
+	}
+	// If size is -1, we keep unchanged
 
 	// Si l'image est plus petite que la largeur et la hauteur max, on ne cree pas de vignette
 	if ($infoImg[0] < $maxWidth && $infoImg[1] < $maxHeight)
@@ -481,8 +502,7 @@ 
 		// Si largeur determinante
 		$thumbWidth  = $maxWidth;
 		$thumbHeight = $thumbWidth / $imgWhFact;
-	}
-	else
+	} else
 	{
 		// Si hauteur determinante
 		$thumbHeight = $maxHeight;
@@ -492,15 +512,16 @@ 
 	$thumbWidth=round($thumbWidth);
 
     // Define target format
-    if (empty($targetformat)) $targetformat=$infoImg[2];
+    if (empty($targetformat)) {
+        $targetformat=$infoImg[2];
+    }
 
 	// Create empty image
 	if ($targetformat == IMAGETYPE_GIF)
 	{
 		// Compatibilite image GIF
 		$imgThumb = imagecreate($thumbWidth, $thumbHeight);
-	}
-	else
+	} else
 	{
 		$imgThumb = imagecreatetruecolor($thumbWidth, $thumbHeight);
 	}
@@ -550,7 +571,9 @@ 
             $newquality='NU';
             break;
 	}
-	if (function_exists("imagefill")) imagefill($imgThumb, 0, 0, $trans_colour);
+	if (function_exists("imagefill")) {
+	    imagefill($imgThumb, 0, 0, $trans_colour);
+	}
 
 	dol_syslog("vignette: convert image from ($imgWidth x $imgHeight) to ($thumbWidth x $thumbHeight) as $extImg, newquality=$newquality");
 	//imagecopyresized($imgThumb, $img, 0, 0, 0, 0, $thumbWidth, $thumbHeight, $imgWidth, $imgHeight); // Insere l'image de base redimensionnee
@@ -587,7 +610,9 @@ 
 	}
 
 	// Set permissions on file
-	if (! empty($conf->global->MAIN_UMASK)) @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
+	if (! empty($conf->global->MAIN_UMASK)) {
+	    @chmod($imgThumbName, octdec($conf->global->MAIN_UMASK));
+	}
 
     // Free memory. This does not delete image.
     imagedestroy($img);

dolibarr/htdocs/core/lib/sendings.lib.php

Indentation +316 added lines, -316 removed lines

@@ -34,80 +34,80 @@ 
  */
 function shipping_prepare_head($object)
 {
-	global $db, $langs, $conf, $user;
+    global $db, $langs, $conf, $user;
 
-	// Load translation files required by the page
+    // Load translation files required by the page
     $langs->loadLangs(array("sendings","deliveries"));
 
-	$h = 0;
-	$head = array();
-
-	$head[$h][0] = DOL_URL_ROOT."/expedition/card.php?id=".$object->id;
-	$head[$h][1] = $langs->trans("SendingCard");
-	$head[$h][2] = 'shipping';
-	$h++;
-
-	if ($conf->livraison_bon->enabled && $user->rights->expedition->livraison->lire)
-	{
-		// delivery link
-		$object->fetchObjectLinked($object->id,$object->element);
-		if (count($object->linkedObjectsIds['delivery']) >  0)		// If there is a delivery
-		{
-		    // Take first one element of array
-		    $tmp = reset($object->linkedObjectsIds['delivery']);
-
-			$head[$h][0] = DOL_URL_ROOT."/livraison/card.php?id=".$tmp;
-			$head[$h][1] = $langs->trans("DeliveryCard");
-			$head[$h][2] = 'delivery';
-			$h++;
-		}
-	}
-
-	if (empty($conf->global->MAIN_DISABLE_CONTACTS_TAB))
-	{
-	    $objectsrc = $object;
-	    if ($object->origin == 'commande' && $object->origin_id > 0)
-	    {
-	        $objectsrc = new Commande($db);
-	        $objectsrc->fetch($object->origin_id);
-	    }
-	    $nbContact = count($objectsrc->liste_contact(-1,'internal')) + count($objectsrc->liste_contact(-1,'external'));
-	    $head[$h][0] = DOL_URL_ROOT."/expedition/contact.php?id=".$object->id;
-    	$head[$h][1] = $langs->trans("ContactsAddresses");
-		if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
-    	$head[$h][2] = 'contact';
-    	$h++;
-	}
-
-	require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
+    $h = 0;
+    $head = array();
+
+    $head[$h][0] = DOL_URL_ROOT."/expedition/card.php?id=".$object->id;
+    $head[$h][1] = $langs->trans("SendingCard");
+    $head[$h][2] = 'shipping';
+    $h++;
+
+    if ($conf->livraison_bon->enabled && $user->rights->expedition->livraison->lire)
+    {
+        // delivery link
+        $object->fetchObjectLinked($object->id,$object->element);
+        if (count($object->linkedObjectsIds['delivery']) >  0)		// If there is a delivery
+        {
+            // Take first one element of array
+            $tmp = reset($object->linkedObjectsIds['delivery']);
+
+            $head[$h][0] = DOL_URL_ROOT."/livraison/card.php?id=".$tmp;
+            $head[$h][1] = $langs->trans("DeliveryCard");
+            $head[$h][2] = 'delivery';
+            $h++;
+        }
+    }
+
+    if (empty($conf->global->MAIN_DISABLE_CONTACTS_TAB))
+    {
+        $objectsrc = $object;
+        if ($object->origin == 'commande' && $object->origin_id > 0)
+        {
+            $objectsrc = new Commande($db);
+            $objectsrc->fetch($object->origin_id);
+        }
+        $nbContact = count($objectsrc->liste_contact(-1,'internal')) + count($objectsrc->liste_contact(-1,'external'));
+        $head[$h][0] = DOL_URL_ROOT."/expedition/contact.php?id=".$object->id;
+        $head[$h][1] = $langs->trans("ContactsAddresses");
+        if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+        $head[$h][2] = 'contact';
+        $h++;
+    }
+
+    require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
     require_once DOL_DOCUMENT_ROOT.'/core/class/link.class.php';
-	$upload_dir = $conf->commande->dir_output . "/" . dol_sanitizeFileName($object->ref);
-	$nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
+    $upload_dir = $conf->commande->dir_output . "/" . dol_sanitizeFileName($object->ref);
+    $nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
     $nbLinks=Link::count($db, $object->element, $object->id);
-	$head[$h][0] = DOL_URL_ROOT.'/expedition/document.php?id='.$object->id;
-	$head[$h][1] = $langs->trans('Documents');
-	if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
-	$head[$h][2] = 'documents';
-	$h++;
+    $head[$h][0] = DOL_URL_ROOT.'/expedition/document.php?id='.$object->id;
+    $head[$h][1] = $langs->trans('Documents');
+    if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+    $head[$h][2] = 'documents';
+    $h++;
 
     $nbNote = 0;
     if (!empty($object->note_private)) $nbNote++;
     if (!empty($object->note_public)) $nbNote++;
-	$head[$h][0] = DOL_URL_ROOT."/expedition/note.php?id=".$object->id;
-	$head[$h][1] = $langs->trans("Notes");
-	if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
-	$head[$h][2] = 'note';
-	$h++;
-
-	// Show more tabs from modules
-	// Entries must be declared in modules descriptor with line
+    $head[$h][0] = DOL_URL_ROOT."/expedition/note.php?id=".$object->id;
+    $head[$h][1] = $langs->trans("Notes");
+    if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+    $head[$h][2] = 'note';
+    $h++;
+
+    // Show more tabs from modules
+    // Entries must be declared in modules descriptor with line
     // $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
     // $this->tabs = array('entity:-tabname);   												to remove a tab
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'delivery');
+    complete_head_from_modules($conf,$langs,$object,$head,$h,'delivery');
 
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'delivery','remove');
+    complete_head_from_modules($conf,$langs,$object,$head,$h,'delivery','remove');
 
-	return $head;
+    return $head;
 }
 
 
@@ -119,39 +119,39 @@ 
  */
 function delivery_prepare_head($object)
 {
-	global $langs, $conf, $user;
+    global $langs, $conf, $user;
 
-	// Load translation files required by the page
+    // Load translation files required by the page
     $langs->loadLangs(array("sendings","deliveries"));
 
-	$h = 0;
-	$head = array();
-
-	if ($conf->expedition_bon->enabled && $user->rights->expedition->lire)
-	{
-		$head[$h][0] = DOL_URL_ROOT."/expedition/card.php?id=".$object->origin_id;
-		$head[$h][1] = $langs->trans("SendingCard");
-		$head[$h][2] = 'shipping';
-		$h++;
-	}
-
-	$head[$h][0] = DOL_URL_ROOT."/livraison/card.php?id=".$object->id;
-	$head[$h][1] = $langs->trans("DeliveryCard");
-	$head[$h][2] = 'delivery';
-	$h++;
-
-	$head[$h][0] = DOL_URL_ROOT."/expedition/contact.php?id=".$object->origin_id;
-	$head[$h][1] = $langs->trans("ContactsAddresses");
-	$head[$h][2] = 'contact';
-	$h++;
-
-	$head[$h][0] = DOL_URL_ROOT."/expedition/note.php?id=".$object->origin_id;
-	$head[$h][1] = $langs->trans("Notes");
-	$head[$h][2] = 'note';
-	$h++;
-
-	// Show more tabs from modules
-	// Entries must be declared in modules descriptor with line
+    $h = 0;
+    $head = array();
+
+    if ($conf->expedition_bon->enabled && $user->rights->expedition->lire)
+    {
+        $head[$h][0] = DOL_URL_ROOT."/expedition/card.php?id=".$object->origin_id;
+        $head[$h][1] = $langs->trans("SendingCard");
+        $head[$h][2] = 'shipping';
+        $h++;
+    }
+
+    $head[$h][0] = DOL_URL_ROOT."/livraison/card.php?id=".$object->id;
+    $head[$h][1] = $langs->trans("DeliveryCard");
+    $head[$h][2] = 'delivery';
+    $h++;
+
+    $head[$h][0] = DOL_URL_ROOT."/expedition/contact.php?id=".$object->origin_id;
+    $head[$h][1] = $langs->trans("ContactsAddresses");
+    $head[$h][2] = 'contact';
+    $h++;
+
+    $head[$h][0] = DOL_URL_ROOT."/expedition/note.php?id=".$object->origin_id;
+    $head[$h][1] = $langs->trans("Notes");
+    $head[$h][2] = 'note';
+    $h++;
+
+    // Show more tabs from modules
+    // Entries must be declared in modules descriptor with line
     // $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
     // $this->tabs = array('entity:-tabname);   				to remove a tab
     // complete_head_from_modules  use $object->id for this link so we temporary change it
@@ -160,10 +160,10 @@ 
 
     complete_head_from_modules($conf,$langs,$object,$head,$h,'delivery');
 
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'delivery','remove');
+    complete_head_from_modules($conf,$langs,$object,$head,$h,'delivery','remove');
 
-	$object->id = $tmpObjectId;
-	return $head;
+    $object->id = $tmpObjectId;
+    return $head;
 }
 
 /**
@@ -176,179 +176,179 @@ 
  */
 function show_list_sending_receive($origin,$origin_id,$filter='')
 {
-	global $db, $conf, $langs, $bc;
-	global $form;
-
-	$product_static=new Product($db);
-	$expedition=new Expedition($db);
-	$warehousestatic=new Entrepot($db);
-
-	$sql = "SELECT obj.rowid, obj.fk_product, obj.label, obj.description, obj.product_type as fk_product_type, obj.qty as qty_asked, obj.date_start, obj.date_end,";
-	$sql.= " ed.rowid as edrowid, ed.qty as qty_shipped, ed.fk_expedition as expedition_id, ed.fk_origin_line, ed.fk_entrepot as warehouse_id,";
-	$sql.= " e.rowid as sendingid, e.ref as exp_ref, e.date_creation, e.date_delivery, e.date_expedition,";
-	//if ($conf->livraison_bon->enabled) $sql .= " l.rowid as livraison_id, l.ref as livraison_ref, l.date_delivery, ld.qty as qty_received,";
-	$sql.= ' p.label as product_label, p.ref, p.fk_product_type, p.rowid as prodid, p.tobatch as product_tobatch,';
-	$sql.= ' p.description as product_desc';
-	$sql.= " FROM ".MAIN_DB_PREFIX."expeditiondet as ed";
-	$sql.= ", ".MAIN_DB_PREFIX."expedition as e";
-	$sql.= ", ".MAIN_DB_PREFIX.$origin."det as obj";
-	//if ($conf->livraison_bon->enabled) $sql .= " LEFT JOIN ".MAIN_DB_PREFIX."livraison as l ON l.fk_expedition = e.rowid LEFT JOIN ".MAIN_DB_PREFIX."livraisondet as ld ON ld.fk_livraison = l.rowid  AND obj.rowid = ld.fk_origin_line";
-	$sql.= " LEFT JOIN ".MAIN_DB_PREFIX."product as p ON obj.fk_product = p.rowid";
-	//TODO Add link to expeditiondet_batch
-	$sql.= " WHERE e.entity IN (".getEntity('expedition').")";
-	$sql.= " AND obj.fk_".$origin." = ".$origin_id;
-	$sql.= " AND obj.rowid = ed.fk_origin_line";
-	$sql.= " AND ed.fk_expedition = e.rowid";
-	if ($filter) $sql.= $filter;
-
-	$sql.= " ORDER BY obj.fk_product";
-
-	dol_syslog("show_list_sending_receive", LOG_DEBUG);
-	$resql = $db->query($sql);
-	if ($resql)
-	{
-		$num = $db->num_rows($resql);
-		$i = 0;
-
-		if ($num)
-		{
-			if ($filter) print load_fiche_titre($langs->trans("OtherSendingsForSameOrder"));
-			else print load_fiche_titre($langs->trans("SendingsAndReceivingForSameOrder"));
-
-			print '<table class="liste" width="100%">';
-			print '<tr class="liste_titre">';
-			//print '<td align="left">'.$langs->trans("QtyOrdered").'</td>';
-			print '<td align="left">'.$langs->trans("SendingSheet").'</td>';
-			print '<td align="left">'.$langs->trans("Description").'</td>';
-			print '<td align="center">'.$langs->trans("DateCreation").'</td>';
-			print '<td align="center">'.$langs->trans("DateDeliveryPlanned").'</td>';
-			print '<td align="center">'.$langs->trans("QtyPreparedOrShipped").'</td>';
-			if (! empty($conf->stock->enabled))
-			{
+    global $db, $conf, $langs, $bc;
+    global $form;
+
+    $product_static=new Product($db);
+    $expedition=new Expedition($db);
+    $warehousestatic=new Entrepot($db);
+
+    $sql = "SELECT obj.rowid, obj.fk_product, obj.label, obj.description, obj.product_type as fk_product_type, obj.qty as qty_asked, obj.date_start, obj.date_end,";
+    $sql.= " ed.rowid as edrowid, ed.qty as qty_shipped, ed.fk_expedition as expedition_id, ed.fk_origin_line, ed.fk_entrepot as warehouse_id,";
+    $sql.= " e.rowid as sendingid, e.ref as exp_ref, e.date_creation, e.date_delivery, e.date_expedition,";
+    //if ($conf->livraison_bon->enabled) $sql .= " l.rowid as livraison_id, l.ref as livraison_ref, l.date_delivery, ld.qty as qty_received,";
+    $sql.= ' p.label as product_label, p.ref, p.fk_product_type, p.rowid as prodid, p.tobatch as product_tobatch,';
+    $sql.= ' p.description as product_desc';
+    $sql.= " FROM ".MAIN_DB_PREFIX."expeditiondet as ed";
+    $sql.= ", ".MAIN_DB_PREFIX."expedition as e";
+    $sql.= ", ".MAIN_DB_PREFIX.$origin."det as obj";
+    //if ($conf->livraison_bon->enabled) $sql .= " LEFT JOIN ".MAIN_DB_PREFIX."livraison as l ON l.fk_expedition = e.rowid LEFT JOIN ".MAIN_DB_PREFIX."livraisondet as ld ON ld.fk_livraison = l.rowid  AND obj.rowid = ld.fk_origin_line";
+    $sql.= " LEFT JOIN ".MAIN_DB_PREFIX."product as p ON obj.fk_product = p.rowid";
+    //TODO Add link to expeditiondet_batch
+    $sql.= " WHERE e.entity IN (".getEntity('expedition').")";
+    $sql.= " AND obj.fk_".$origin." = ".$origin_id;
+    $sql.= " AND obj.rowid = ed.fk_origin_line";
+    $sql.= " AND ed.fk_expedition = e.rowid";
+    if ($filter) $sql.= $filter;
+
+    $sql.= " ORDER BY obj.fk_product";
+
+    dol_syslog("show_list_sending_receive", LOG_DEBUG);
+    $resql = $db->query($sql);
+    if ($resql)
+    {
+        $num = $db->num_rows($resql);
+        $i = 0;
+
+        if ($num)
+        {
+            if ($filter) print load_fiche_titre($langs->trans("OtherSendingsForSameOrder"));
+            else print load_fiche_titre($langs->trans("SendingsAndReceivingForSameOrder"));
+
+            print '<table class="liste" width="100%">';
+            print '<tr class="liste_titre">';
+            //print '<td align="left">'.$langs->trans("QtyOrdered").'</td>';
+            print '<td align="left">'.$langs->trans("SendingSheet").'</td>';
+            print '<td align="left">'.$langs->trans("Description").'</td>';
+            print '<td align="center">'.$langs->trans("DateCreation").'</td>';
+            print '<td align="center">'.$langs->trans("DateDeliveryPlanned").'</td>';
+            print '<td align="center">'.$langs->trans("QtyPreparedOrShipped").'</td>';
+            if (! empty($conf->stock->enabled))
+            {
                 print '<td>'.$langs->trans("Warehouse").'</td>';
-			}
-			/*TODO Add link to expeditiondet_batch
+            }
+            /*TODO Add link to expeditiondet_batch
 			if (! empty($conf->productbatch->enabled))
 			{
 			    print '<td>';
 			    print '</td>';
 			}*/
-			if (! empty($conf->livraison_bon->enabled))
-			{
-				print '<td>'.$langs->trans("DeliveryOrder").'</td>';
-				//print '<td align="center">'.$langs->trans("QtyReceived").'</td>';
-				print '<td align="right">'.$langs->trans("DeliveryDate").'</td>';
-			}
-			print "</tr>\n";
-
-			while ($i < $num)
-			{
-				$objp = $db->fetch_object($resql);
-
-				print '<tr class="oddeven">';
-
-				// Sending id
-				print '<td align="left" class="nowrap">';
-				print '<a href="'.DOL_URL_ROOT.'/expedition/card.php?id='.$objp->expedition_id.'">'.img_object($langs->trans("ShowSending"),'sending').' '.$objp->exp_ref.'<a>';
-				print '</td>';
-
-				// Description
-				if ($objp->fk_product > 0)
-				{
-					// Define output language
-					if (! empty($conf->global->MAIN_MULTILANGS) && ! empty($conf->global->PRODUIT_TEXTS_IN_THIRDPARTY_LANGUAGE))
-					{
-						$object = new $origin($db);
-						$object->fetch($origin_id);
-						$object->fetch_thirdparty();
-
-						$prod = new Product($db);
-						$prod->id=$objp->fk_product;
-						$prod->getMultiLangs();
-
-						$outputlangs = $langs;
-						$newlang='';
-						if (empty($newlang) && ! empty($_REQUEST['lang_id'])) $newlang=$_REQUEST['lang_id'];
-						if (empty($newlang)) $newlang=$object->thirdparty->default_lang;
-						if (! empty($newlang))
-						{
-							$outputlangs = new Translate("",$conf);
-							$outputlangs->setDefaultLang($newlang);
-						}
-
-						$label = (! empty($prod->multilangs[$outputlangs->defaultlang]["label"])) ? $prod->multilangs[$outputlangs->defaultlang]["label"] : $objp->product_label;
-					}
-					else
-					{
-						$label = (! empty($objp->label)?$objp->label:$objp->product_label);
-					}
-
-					print '<td>';
-
-					// Show product and description
-					$product_static->type=$objp->fk_product_type;
-					$product_static->id=$objp->fk_product;
-					$product_static->ref=$objp->ref;
-					$product_static->status_batch=$objp->product_tobatch;
-					$text=$product_static->getNomUrl(1);
-					$text.= ' - '.$label;
-					$description=(! empty($conf->global->PRODUIT_DESC_IN_FORM)?'':dol_htmlentitiesbr($objp->description));
-					print $form->textwithtooltip($text,$description,3,'','',$i);
-
-					// Show range
-					print_date_range($objp->date_start,$objp->date_end);
-
-					// Add description in form
-					if (! empty($conf->global->PRODUIT_DESC_IN_FORM))
-					{
-						print (! empty($objp->description) && $objp->description!=$objp->product)?'<br>'.dol_htmlentitiesbr($objp->description):'';
-					}
-
-					print '</td>';
-				}
-				else
-				{
-					print "<td>";
-					if ($objp->fk_product_type==1) $text = img_object($langs->trans('Service'),'service');
-					else $text = img_object($langs->trans('Product'),'product');
-
-					if (! empty($objp->label)) {
-						$text.= ' <strong>'.$objp->label.'</strong>';
-						print $form->textwithtooltip($text,$objp->description,3,'','',$i);
-					} else {
-						print $text.' '.nl2br($objp->description);
-					}
-
-					// Show range
-					print_date_range($objp->date_start,$objp->date_end);
-					print "</td>\n";
-				}
-
-				//print '<td align="center">'.$objp->qty_asked.'</td>';
-
-				// Date creation
-				print '<td align="center" class="nowrap">'.dol_print_date($db->jdate($objp->date_creation),'day').'</td>';
-
-				// Date shipping creation
-				print '<td align="center" class="nowrap">'.dol_print_date($db->jdate($objp->date_delivery),'day').'</td>';
-
-				// Qty shipped
-				print '<td align="center">'.$objp->qty_shipped.'</td>';
-
-				// Warehouse
-				if (! empty($conf->stock->enabled))
-				{
-				    print '<td>';
-    				if ($objp->warehouse_id > 0)
-    				{
-        				$warehousestatic->fetch($objp->warehouse_id);
-        				print $warehousestatic->getNomUrl(1);
-    				}
-    				print '</td>';
-				}
-
-				// Batch number managment
-				/*TODO Add link to expeditiondet_batch
+            if (! empty($conf->livraison_bon->enabled))
+            {
+                print '<td>'.$langs->trans("DeliveryOrder").'</td>';
+                //print '<td align="center">'.$langs->trans("QtyReceived").'</td>';
+                print '<td align="right">'.$langs->trans("DeliveryDate").'</td>';
+            }
+            print "</tr>\n";
+
+            while ($i < $num)
+            {
+                $objp = $db->fetch_object($resql);
+
+                print '<tr class="oddeven">';
+
+                // Sending id
+                print '<td align="left" class="nowrap">';
+                print '<a href="'.DOL_URL_ROOT.'/expedition/card.php?id='.$objp->expedition_id.'">'.img_object($langs->trans("ShowSending"),'sending').' '.$objp->exp_ref.'<a>';
+                print '</td>';
+
+                // Description
+                if ($objp->fk_product > 0)
+                {
+                    // Define output language
+                    if (! empty($conf->global->MAIN_MULTILANGS) && ! empty($conf->global->PRODUIT_TEXTS_IN_THIRDPARTY_LANGUAGE))
+                    {
+                        $object = new $origin($db);
+                        $object->fetch($origin_id);
+                        $object->fetch_thirdparty();
+
+                        $prod = new Product($db);
+                        $prod->id=$objp->fk_product;
+                        $prod->getMultiLangs();
+
+                        $outputlangs = $langs;
+                        $newlang='';
+                        if (empty($newlang) && ! empty($_REQUEST['lang_id'])) $newlang=$_REQUEST['lang_id'];
+                        if (empty($newlang)) $newlang=$object->thirdparty->default_lang;
+                        if (! empty($newlang))
+                        {
+                            $outputlangs = new Translate("",$conf);
+                            $outputlangs->setDefaultLang($newlang);
+                        }
+
+                        $label = (! empty($prod->multilangs[$outputlangs->defaultlang]["label"])) ? $prod->multilangs[$outputlangs->defaultlang]["label"] : $objp->product_label;
+                    }
+                    else
+                    {
+                        $label = (! empty($objp->label)?$objp->label:$objp->product_label);
+                    }
+
+                    print '<td>';
+
+                    // Show product and description
+                    $product_static->type=$objp->fk_product_type;
+                    $product_static->id=$objp->fk_product;
+                    $product_static->ref=$objp->ref;
+                    $product_static->status_batch=$objp->product_tobatch;
+                    $text=$product_static->getNomUrl(1);
+                    $text.= ' - '.$label;
+                    $description=(! empty($conf->global->PRODUIT_DESC_IN_FORM)?'':dol_htmlentitiesbr($objp->description));
+                    print $form->textwithtooltip($text,$description,3,'','',$i);
+
+                    // Show range
+                    print_date_range($objp->date_start,$objp->date_end);
+
+                    // Add description in form
+                    if (! empty($conf->global->PRODUIT_DESC_IN_FORM))
+                    {
+                        print (! empty($objp->description) && $objp->description!=$objp->product)?'<br>'.dol_htmlentitiesbr($objp->description):'';
+                    }
+
+                    print '</td>';
+                }
+                else
+                {
+                    print "<td>";
+                    if ($objp->fk_product_type==1) $text = img_object($langs->trans('Service'),'service');
+                    else $text = img_object($langs->trans('Product'),'product');
+
+                    if (! empty($objp->label)) {
+                        $text.= ' <strong>'.$objp->label.'</strong>';
+                        print $form->textwithtooltip($text,$objp->description,3,'','',$i);
+                    } else {
+                        print $text.' '.nl2br($objp->description);
+                    }
+
+                    // Show range
+                    print_date_range($objp->date_start,$objp->date_end);
+                    print "</td>\n";
+                }
+
+                //print '<td align="center">'.$objp->qty_asked.'</td>';
+
+                // Date creation
+                print '<td align="center" class="nowrap">'.dol_print_date($db->jdate($objp->date_creation),'day').'</td>';
+
+                // Date shipping creation
+                print '<td align="center" class="nowrap">'.dol_print_date($db->jdate($objp->date_delivery),'day').'</td>';
+
+                // Qty shipped
+                print '<td align="center">'.$objp->qty_shipped.'</td>';
+
+                // Warehouse
+                if (! empty($conf->stock->enabled))
+                {
+                    print '<td>';
+                    if ($objp->warehouse_id > 0)
+                    {
+                        $warehousestatic->fetch($objp->warehouse_id);
+                        print $warehousestatic->getNomUrl(1);
+                    }
+                    print '</td>';
+                }
+
+                // Batch number managment
+                /*TODO Add link to expeditiondet_batch
 				if (! empty($conf->productbatch->enabled))
 				{
 				    var_dump($objp->edrowid);
@@ -379,60 +379,60 @@ 
 				    }
 				}*/
 
-				// Informations on receipt
-				if (! empty($conf->livraison_bon->enabled))
-				{
-					include_once DOL_DOCUMENT_ROOT.'/livraison/class/livraison.class.php';
-					$expedition->id=$objp->sendingid;
-					$expedition->fetchObjectLinked($expedition->id,$expedition->element);
-					//var_dump($expedition->linkedObjects);
-
-					$receiving='';
-					if (count($expedition->linkedObjects['delivery']) > 0) $receiving=reset($expedition->linkedObjects['delivery']);   // Take first link
-
-					if (! empty($receiving))
-					{
-						// $expedition->fk_origin_line = id of det line of order
-						// $receiving->fk_origin_line = id of det line of order
-						// $receiving->origin may be 'shipping'
-						// $receiving->origin_id may be id of shipping
-
-						// Ref
-						print '<td>';
-						print $receiving->getNomUrl($db);
-						//print '<a href="'.DOL_URL_ROOT.'/livraison/card.php?id='.$livraison_id.'">'.img_object($langs->trans("ShowReceiving"),'sending').' '.$objp->livraison_ref.'<a>';
-						print '</td>';
-						// Qty received
-						//print '<td align="center">';
-						// TODO No solution for the moment to link a line det of receipt with a line det of shipping,
-						// so no way to know the qty received for this line of shipping.
-						//print $langs->trans("FeatureNotYetAvailable");
-						//print '</td>';
-						// Date shipping real
-						print '<td align="right">';
-						print dol_print_date($receiving->date_delivery,'day');
-						print '</td>';
-					}
-					else
-					{
-						//print '<td>&nbsp;</td>';
-						print '<td>&nbsp;</td>';
-						print '<td>&nbsp;</td>';
-					}
-				}
-				print '</tr>';
-				$i++;
-			}
-
-			print '</table>';
-		}
-		$db->free($resql);
-	}
-	else
-	{
-		dol_print_error($db);
-	}
-
-	return 1;
+                // Informations on receipt
+                if (! empty($conf->livraison_bon->enabled))
+                {
+                    include_once DOL_DOCUMENT_ROOT.'/livraison/class/livraison.class.php';
+                    $expedition->id=$objp->sendingid;
+                    $expedition->fetchObjectLinked($expedition->id,$expedition->element);
+                    //var_dump($expedition->linkedObjects);
+
+                    $receiving='';
+                    if (count($expedition->linkedObjects['delivery']) > 0) $receiving=reset($expedition->linkedObjects['delivery']);   // Take first link
+
+                    if (! empty($receiving))
+                    {
+                        // $expedition->fk_origin_line = id of det line of order
+                        // $receiving->fk_origin_line = id of det line of order
+                        // $receiving->origin may be 'shipping'
+                        // $receiving->origin_id may be id of shipping
+
+                        // Ref
+                        print '<td>';
+                        print $receiving->getNomUrl($db);
+                        //print '<a href="'.DOL_URL_ROOT.'/livraison/card.php?id='.$livraison_id.'">'.img_object($langs->trans("ShowReceiving"),'sending').' '.$objp->livraison_ref.'<a>';
+                        print '</td>';
+                        // Qty received
+                        //print '<td align="center">';
+                        // TODO No solution for the moment to link a line det of receipt with a line det of shipping,
+                        // so no way to know the qty received for this line of shipping.
+                        //print $langs->trans("FeatureNotYetAvailable");
+                        //print '</td>';
+                        // Date shipping real
+                        print '<td align="right">';
+                        print dol_print_date($receiving->date_delivery,'day');
+                        print '</td>';
+                    }
+                    else
+                    {
+                        //print '<td>&nbsp;</td>';
+                        print '<td>&nbsp;</td>';
+                        print '<td>&nbsp;</td>';
+                    }
+                }
+                print '</tr>';
+                $i++;
+            }
+
+            print '</table>';
+        }
+        $db->free($resql);
+    }
+    else
+    {
+        dol_print_error($db);
+    }
+
+    return 1;
 }
 

Spacing +72 added lines, -72 removed lines

@@ -37,7 +37,7 @@ 
 	global $db, $langs, $conf, $user;
 
 	// Load translation files required by the page
-    $langs->loadLangs(array("sendings","deliveries"));
+    $langs->loadLangs(array("sendings", "deliveries"));
 
 	$h = 0;
 	$head = array();
@@ -50,8 +50,8 @@ 
 	if ($conf->livraison_bon->enabled && $user->rights->expedition->livraison->lire)
 	{
 		// delivery link
-		$object->fetchObjectLinked($object->id,$object->element);
-		if (count($object->linkedObjectsIds['delivery']) >  0)		// If there is a delivery
+		$object->fetchObjectLinked($object->id, $object->element);
+		if (count($object->linkedObjectsIds['delivery']) > 0)		// If there is a delivery
 		{
 		    // Take first one element of array
 		    $tmp = reset($object->linkedObjectsIds['delivery']);
@@ -71,22 +71,22 @@ 
 	        $objectsrc = new Commande($db);
 	        $objectsrc->fetch($object->origin_id);
 	    }
-	    $nbContact = count($objectsrc->liste_contact(-1,'internal')) + count($objectsrc->liste_contact(-1,'external'));
+	    $nbContact = count($objectsrc->liste_contact(-1, 'internal')) + count($objectsrc->liste_contact(-1, 'external'));
 	    $head[$h][0] = DOL_URL_ROOT."/expedition/contact.php?id=".$object->id;
     	$head[$h][1] = $langs->trans("ContactsAddresses");
-		if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+		if ($nbContact > 0) $head[$h][1] .= ' <span class="badge">'.$nbContact.'</span>';
     	$head[$h][2] = 'contact';
     	$h++;
 	}
 
 	require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
     require_once DOL_DOCUMENT_ROOT.'/core/class/link.class.php';
-	$upload_dir = $conf->commande->dir_output . "/" . dol_sanitizeFileName($object->ref);
-	$nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
-    $nbLinks=Link::count($db, $object->element, $object->id);
+	$upload_dir = $conf->commande->dir_output."/".dol_sanitizeFileName($object->ref);
+	$nbFiles = count(dol_dir_list($upload_dir, 'files', 0, '', '(\.meta|_preview.*\.png)$'));
+    $nbLinks = Link::count($db, $object->element, $object->id);
 	$head[$h][0] = DOL_URL_ROOT.'/expedition/document.php?id='.$object->id;
 	$head[$h][1] = $langs->trans('Documents');
-	if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	if (($nbFiles + $nbLinks) > 0) $head[$h][1] .= ' <span class="badge">'.($nbFiles + $nbLinks).'</span>';
 	$head[$h][2] = 'documents';
 	$h++;
 
@@ -95,7 +95,7 @@ 
     if (!empty($object->note_public)) $nbNote++;
 	$head[$h][0] = DOL_URL_ROOT."/expedition/note.php?id=".$object->id;
 	$head[$h][1] = $langs->trans("Notes");
-	if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+	if ($nbNote > 0) $head[$h][1] .= ' <span class="badge">'.$nbNote.'</span>';
 	$head[$h][2] = 'note';
 	$h++;
 
@@ -103,9 +103,9 @@ 
 	// Entries must be declared in modules descriptor with line
     // $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
     // $this->tabs = array('entity:-tabname);   												to remove a tab
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'delivery');
+	complete_head_from_modules($conf, $langs, $object, $head, $h, 'delivery');
 
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'delivery','remove');
+	complete_head_from_modules($conf, $langs, $object, $head, $h, 'delivery', 'remove');
 
 	return $head;
 }
@@ -122,7 +122,7 @@ 
 	global $langs, $conf, $user;
 
 	// Load translation files required by the page
-    $langs->loadLangs(array("sendings","deliveries"));
+    $langs->loadLangs(array("sendings", "deliveries"));
 
 	$h = 0;
 	$head = array();
@@ -158,9 +158,9 @@ 
     $tmpObjectId = $object->id;
     $object->id = $object->origin_id;
 
-    complete_head_from_modules($conf,$langs,$object,$head,$h,'delivery');
+    complete_head_from_modules($conf, $langs, $object, $head, $h, 'delivery');
 
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'delivery','remove');
+	complete_head_from_modules($conf, $langs, $object, $head, $h, 'delivery', 'remove');
 
 	$object->id = $tmpObjectId;
 	return $head;
@@ -174,34 +174,34 @@ 
  * @param	string		$filter			Filter
  * @return	int							<0 if KO, >0 if OK
  */
-function show_list_sending_receive($origin,$origin_id,$filter='')
+function show_list_sending_receive($origin, $origin_id, $filter = '')
 {
 	global $db, $conf, $langs, $bc;
 	global $form;
 
-	$product_static=new Product($db);
-	$expedition=new Expedition($db);
-	$warehousestatic=new Entrepot($db);
+	$product_static = new Product($db);
+	$expedition = new Expedition($db);
+	$warehousestatic = new Entrepot($db);
 
 	$sql = "SELECT obj.rowid, obj.fk_product, obj.label, obj.description, obj.product_type as fk_product_type, obj.qty as qty_asked, obj.date_start, obj.date_end,";
-	$sql.= " ed.rowid as edrowid, ed.qty as qty_shipped, ed.fk_expedition as expedition_id, ed.fk_origin_line, ed.fk_entrepot as warehouse_id,";
-	$sql.= " e.rowid as sendingid, e.ref as exp_ref, e.date_creation, e.date_delivery, e.date_expedition,";
+	$sql .= " ed.rowid as edrowid, ed.qty as qty_shipped, ed.fk_expedition as expedition_id, ed.fk_origin_line, ed.fk_entrepot as warehouse_id,";
+	$sql .= " e.rowid as sendingid, e.ref as exp_ref, e.date_creation, e.date_delivery, e.date_expedition,";
 	//if ($conf->livraison_bon->enabled) $sql .= " l.rowid as livraison_id, l.ref as livraison_ref, l.date_delivery, ld.qty as qty_received,";
-	$sql.= ' p.label as product_label, p.ref, p.fk_product_type, p.rowid as prodid, p.tobatch as product_tobatch,';
-	$sql.= ' p.description as product_desc';
-	$sql.= " FROM ".MAIN_DB_PREFIX."expeditiondet as ed";
-	$sql.= ", ".MAIN_DB_PREFIX."expedition as e";
-	$sql.= ", ".MAIN_DB_PREFIX.$origin."det as obj";
+	$sql .= ' p.label as product_label, p.ref, p.fk_product_type, p.rowid as prodid, p.tobatch as product_tobatch,';
+	$sql .= ' p.description as product_desc';
+	$sql .= " FROM ".MAIN_DB_PREFIX."expeditiondet as ed";
+	$sql .= ", ".MAIN_DB_PREFIX."expedition as e";
+	$sql .= ", ".MAIN_DB_PREFIX.$origin."det as obj";
 	//if ($conf->livraison_bon->enabled) $sql .= " LEFT JOIN ".MAIN_DB_PREFIX."livraison as l ON l.fk_expedition = e.rowid LEFT JOIN ".MAIN_DB_PREFIX."livraisondet as ld ON ld.fk_livraison = l.rowid  AND obj.rowid = ld.fk_origin_line";
-	$sql.= " LEFT JOIN ".MAIN_DB_PREFIX."product as p ON obj.fk_product = p.rowid";
+	$sql .= " LEFT JOIN ".MAIN_DB_PREFIX."product as p ON obj.fk_product = p.rowid";
 	//TODO Add link to expeditiondet_batch
-	$sql.= " WHERE e.entity IN (".getEntity('expedition').")";
-	$sql.= " AND obj.fk_".$origin." = ".$origin_id;
-	$sql.= " AND obj.rowid = ed.fk_origin_line";
-	$sql.= " AND ed.fk_expedition = e.rowid";
-	if ($filter) $sql.= $filter;
+	$sql .= " WHERE e.entity IN (".getEntity('expedition').")";
+	$sql .= " AND obj.fk_".$origin." = ".$origin_id;
+	$sql .= " AND obj.rowid = ed.fk_origin_line";
+	$sql .= " AND ed.fk_expedition = e.rowid";
+	if ($filter) $sql .= $filter;
 
-	$sql.= " ORDER BY obj.fk_product";
+	$sql .= " ORDER BY obj.fk_product";
 
 	dol_syslog("show_list_sending_receive", LOG_DEBUG);
 	$resql = $db->query($sql);
@@ -223,7 +223,7 @@ 
 			print '<td align="center">'.$langs->trans("DateCreation").'</td>';
 			print '<td align="center">'.$langs->trans("DateDeliveryPlanned").'</td>';
 			print '<td align="center">'.$langs->trans("QtyPreparedOrShipped").'</td>';
-			if (! empty($conf->stock->enabled))
+			if (!empty($conf->stock->enabled))
 			{
                 print '<td>'.$langs->trans("Warehouse").'</td>';
 			}
@@ -233,7 +233,7 @@ 
 			    print '<td>';
 			    print '</td>';
 			}*/
-			if (! empty($conf->livraison_bon->enabled))
+			if (!empty($conf->livraison_bon->enabled))
 			{
 				print '<td>'.$langs->trans("DeliveryOrder").'</td>';
 				//print '<td align="center">'.$langs->trans("QtyReceived").'</td>';
@@ -249,59 +249,59 @@ 
 
 				// Sending id
 				print '<td align="left" class="nowrap">';
-				print '<a href="'.DOL_URL_ROOT.'/expedition/card.php?id='.$objp->expedition_id.'">'.img_object($langs->trans("ShowSending"),'sending').' '.$objp->exp_ref.'<a>';
+				print '<a href="'.DOL_URL_ROOT.'/expedition/card.php?id='.$objp->expedition_id.'">'.img_object($langs->trans("ShowSending"), 'sending').' '.$objp->exp_ref.'<a>';
 				print '</td>';
 
 				// Description
 				if ($objp->fk_product > 0)
 				{
 					// Define output language
-					if (! empty($conf->global->MAIN_MULTILANGS) && ! empty($conf->global->PRODUIT_TEXTS_IN_THIRDPARTY_LANGUAGE))
+					if (!empty($conf->global->MAIN_MULTILANGS) && !empty($conf->global->PRODUIT_TEXTS_IN_THIRDPARTY_LANGUAGE))
 					{
 						$object = new $origin($db);
 						$object->fetch($origin_id);
 						$object->fetch_thirdparty();
 
 						$prod = new Product($db);
-						$prod->id=$objp->fk_product;
+						$prod->id = $objp->fk_product;
 						$prod->getMultiLangs();
 
 						$outputlangs = $langs;
-						$newlang='';
-						if (empty($newlang) && ! empty($_REQUEST['lang_id'])) $newlang=$_REQUEST['lang_id'];
-						if (empty($newlang)) $newlang=$object->thirdparty->default_lang;
-						if (! empty($newlang))
+						$newlang = '';
+						if (empty($newlang) && !empty($_REQUEST['lang_id'])) $newlang = $_REQUEST['lang_id'];
+						if (empty($newlang)) $newlang = $object->thirdparty->default_lang;
+						if (!empty($newlang))
 						{
-							$outputlangs = new Translate("",$conf);
+							$outputlangs = new Translate("", $conf);
 							$outputlangs->setDefaultLang($newlang);
 						}
 
-						$label = (! empty($prod->multilangs[$outputlangs->defaultlang]["label"])) ? $prod->multilangs[$outputlangs->defaultlang]["label"] : $objp->product_label;
+						$label = (!empty($prod->multilangs[$outputlangs->defaultlang]["label"])) ? $prod->multilangs[$outputlangs->defaultlang]["label"] : $objp->product_label;
 					}
 					else
 					{
-						$label = (! empty($objp->label)?$objp->label:$objp->product_label);
+						$label = (!empty($objp->label) ? $objp->label : $objp->product_label);
 					}
 
 					print '<td>';
 
 					// Show product and description
-					$product_static->type=$objp->fk_product_type;
-					$product_static->id=$objp->fk_product;
-					$product_static->ref=$objp->ref;
-					$product_static->status_batch=$objp->product_tobatch;
-					$text=$product_static->getNomUrl(1);
-					$text.= ' - '.$label;
-					$description=(! empty($conf->global->PRODUIT_DESC_IN_FORM)?'':dol_htmlentitiesbr($objp->description));
-					print $form->textwithtooltip($text,$description,3,'','',$i);
+					$product_static->type = $objp->fk_product_type;
+					$product_static->id = $objp->fk_product;
+					$product_static->ref = $objp->ref;
+					$product_static->status_batch = $objp->product_tobatch;
+					$text = $product_static->getNomUrl(1);
+					$text .= ' - '.$label;
+					$description = (!empty($conf->global->PRODUIT_DESC_IN_FORM) ? '' : dol_htmlentitiesbr($objp->description));
+					print $form->textwithtooltip($text, $description, 3, '', '', $i);
 
 					// Show range
-					print_date_range($objp->date_start,$objp->date_end);
+					print_date_range($objp->date_start, $objp->date_end);
 
 					// Add description in form
-					if (! empty($conf->global->PRODUIT_DESC_IN_FORM))
+					if (!empty($conf->global->PRODUIT_DESC_IN_FORM))
 					{
-						print (! empty($objp->description) && $objp->description!=$objp->product)?'<br>'.dol_htmlentitiesbr($objp->description):'';
+						print (!empty($objp->description) && $objp->description != $objp->product) ? '<br>'.dol_htmlentitiesbr($objp->description) : '';
 					}
 
 					print '</td>';
@@ -309,34 +309,34 @@ 
 				else
 				{
 					print "<td>";
-					if ($objp->fk_product_type==1) $text = img_object($langs->trans('Service'),'service');
-					else $text = img_object($langs->trans('Product'),'product');
+					if ($objp->fk_product_type == 1) $text = img_object($langs->trans('Service'), 'service');
+					else $text = img_object($langs->trans('Product'), 'product');
 
-					if (! empty($objp->label)) {
-						$text.= ' <strong>'.$objp->label.'</strong>';
-						print $form->textwithtooltip($text,$objp->description,3,'','',$i);
+					if (!empty($objp->label)) {
+						$text .= ' <strong>'.$objp->label.'</strong>';
+						print $form->textwithtooltip($text, $objp->description, 3, '', '', $i);
 					} else {
 						print $text.' '.nl2br($objp->description);
 					}
 
 					// Show range
-					print_date_range($objp->date_start,$objp->date_end);
+					print_date_range($objp->date_start, $objp->date_end);
 					print "</td>\n";
 				}
 
 				//print '<td align="center">'.$objp->qty_asked.'</td>';
 
 				// Date creation
-				print '<td align="center" class="nowrap">'.dol_print_date($db->jdate($objp->date_creation),'day').'</td>';
+				print '<td align="center" class="nowrap">'.dol_print_date($db->jdate($objp->date_creation), 'day').'</td>';
 
 				// Date shipping creation
-				print '<td align="center" class="nowrap">'.dol_print_date($db->jdate($objp->date_delivery),'day').'</td>';
+				print '<td align="center" class="nowrap">'.dol_print_date($db->jdate($objp->date_delivery), 'day').'</td>';
 
 				// Qty shipped
 				print '<td align="center">'.$objp->qty_shipped.'</td>';
 
 				// Warehouse
-				if (! empty($conf->stock->enabled))
+				if (!empty($conf->stock->enabled))
 				{
 				    print '<td>';
     				if ($objp->warehouse_id > 0)
@@ -380,17 +380,17 @@ 
 				}*/
 
 				// Informations on receipt
-				if (! empty($conf->livraison_bon->enabled))
+				if (!empty($conf->livraison_bon->enabled))
 				{
 					include_once DOL_DOCUMENT_ROOT.'/livraison/class/livraison.class.php';
-					$expedition->id=$objp->sendingid;
-					$expedition->fetchObjectLinked($expedition->id,$expedition->element);
+					$expedition->id = $objp->sendingid;
+					$expedition->fetchObjectLinked($expedition->id, $expedition->element);
 					//var_dump($expedition->linkedObjects);
 
-					$receiving='';
-					if (count($expedition->linkedObjects['delivery']) > 0) $receiving=reset($expedition->linkedObjects['delivery']);   // Take first link
+					$receiving = '';
+					if (count($expedition->linkedObjects['delivery']) > 0) $receiving = reset($expedition->linkedObjects['delivery']); // Take first link
 
-					if (! empty($receiving))
+					if (!empty($receiving))
 					{
 						// $expedition->fk_origin_line = id of det line of order
 						// $receiving->fk_origin_line = id of det line of order
@@ -410,7 +410,7 @@ 
 						//print '</td>';
 						// Date shipping real
 						print '<td align="right">';
-						print dol_print_date($receiving->date_delivery,'day');
+						print dol_print_date($receiving->date_delivery, 'day');
 						print '</td>';
 					}
 					else

Braces +45 added lines, -22 removed lines

@@ -51,10 +51,12 @@ 
 	{
 		// delivery link
 		$object->fetchObjectLinked($object->id,$object->element);
-		if (count($object->linkedObjectsIds['delivery']) >  0)		// If there is a delivery
+		if (count($object->linkedObjectsIds['delivery']) >  0) {
+		    // If there is a delivery
 		{
 		    // Take first one element of array
 		    $tmp = reset($object->linkedObjectsIds['delivery']);
+		}
 
 			$head[$h][0] = DOL_URL_ROOT."/livraison/card.php?id=".$tmp;
 			$head[$h][1] = $langs->trans("DeliveryCard");
@@ -74,7 +76,9 @@ 
 	    $nbContact = count($objectsrc->liste_contact(-1,'internal')) + count($objectsrc->liste_contact(-1,'external'));
 	    $head[$h][0] = DOL_URL_ROOT."/expedition/contact.php?id=".$object->id;
     	$head[$h][1] = $langs->trans("ContactsAddresses");
-		if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+		if ($nbContact > 0) {
+		    $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+		}
     	$head[$h][2] = 'contact';
     	$h++;
 	}
@@ -86,16 +90,24 @@ 
     $nbLinks=Link::count($db, $object->element, $object->id);
 	$head[$h][0] = DOL_URL_ROOT.'/expedition/document.php?id='.$object->id;
 	$head[$h][1] = $langs->trans('Documents');
-	if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	if (($nbFiles+$nbLinks) > 0) {
+	    $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	}
 	$head[$h][2] = 'documents';
 	$h++;
 
     $nbNote = 0;
-    if (!empty($object->note_private)) $nbNote++;
-    if (!empty($object->note_public)) $nbNote++;
+    if (!empty($object->note_private)) {
+        $nbNote++;
+    }
+    if (!empty($object->note_public)) {
+        $nbNote++;
+    }
 	$head[$h][0] = DOL_URL_ROOT."/expedition/note.php?id=".$object->id;
 	$head[$h][1] = $langs->trans("Notes");
-	if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+	if ($nbNote > 0) {
+	    $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+	}
 	$head[$h][2] = 'note';
 	$h++;
 
@@ -199,7 +211,9 @@ 
 	$sql.= " AND obj.fk_".$origin." = ".$origin_id;
 	$sql.= " AND obj.rowid = ed.fk_origin_line";
 	$sql.= " AND ed.fk_expedition = e.rowid";
-	if ($filter) $sql.= $filter;
+	if ($filter) {
+	    $sql.= $filter;
+	}
 
 	$sql.= " ORDER BY obj.fk_product";
 
@@ -212,8 +226,11 @@ 
 
 		if ($num)
 		{
-			if ($filter) print load_fiche_titre($langs->trans("OtherSendingsForSameOrder"));
-			else print load_fiche_titre($langs->trans("SendingsAndReceivingForSameOrder"));
+			if ($filter) {
+			    print load_fiche_titre($langs->trans("OtherSendingsForSameOrder"));
+			} else {
+			    print load_fiche_titre($langs->trans("SendingsAndReceivingForSameOrder"));
+			}
 
 			print '<table class="liste" width="100%">';
 			print '<tr class="liste_titre">';
@@ -268,8 +285,12 @@ 
 
 						$outputlangs = $langs;
 						$newlang='';
-						if (empty($newlang) && ! empty($_REQUEST['lang_id'])) $newlang=$_REQUEST['lang_id'];
-						if (empty($newlang)) $newlang=$object->thirdparty->default_lang;
+						if (empty($newlang) && ! empty($_REQUEST['lang_id'])) {
+						    $newlang=$_REQUEST['lang_id'];
+						}
+						if (empty($newlang)) {
+						    $newlang=$object->thirdparty->default_lang;
+						}
 						if (! empty($newlang))
 						{
 							$outputlangs = new Translate("",$conf);
@@ -277,8 +298,7 @@ 
 						}
 
 						$label = (! empty($prod->multilangs[$outputlangs->defaultlang]["label"])) ? $prod->multilangs[$outputlangs->defaultlang]["label"] : $objp->product_label;
-					}
-					else
+					} else
 					{
 						$label = (! empty($objp->label)?$objp->label:$objp->product_label);
 					}
@@ -305,12 +325,14 @@ 
 					}
 
 					print '</td>';
-				}
-				else
+				} else
 				{
 					print "<td>";
-					if ($objp->fk_product_type==1) $text = img_object($langs->trans('Service'),'service');
-					else $text = img_object($langs->trans('Product'),'product');
+					if ($objp->fk_product_type==1) {
+					    $text = img_object($langs->trans('Service'),'service');
+					} else {
+					    $text = img_object($langs->trans('Product'),'product');
+					}
 
 					if (! empty($objp->label)) {
 						$text.= ' <strong>'.$objp->label.'</strong>';
@@ -388,7 +410,10 @@ 
 					//var_dump($expedition->linkedObjects);
 
 					$receiving='';
-					if (count($expedition->linkedObjects['delivery']) > 0) $receiving=reset($expedition->linkedObjects['delivery']);   // Take first link
+					if (count($expedition->linkedObjects['delivery']) > 0) {
+					    $receiving=reset($expedition->linkedObjects['delivery']);
+					}
+					// Take first link
 
 					if (! empty($receiving))
 					{
@@ -412,8 +437,7 @@ 
 						print '<td align="right">';
 						print dol_print_date($receiving->date_delivery,'day');
 						print '</td>';
-					}
-					else
+					} else
 					{
 						//print '<td>&nbsp;</td>';
 						print '<td>&nbsp;</td>';
@@ -427,8 +451,7 @@ 
 			print '</table>';
 		}
 		$db->free($resql);
-	}
-	else
+	} else
 	{
 		dol_print_error($db);
 	}

dolibarr/htdocs/core/lib/reception.lib.php

Indentation +53 added lines, -53 removed lines

@@ -34,45 +34,45 @@ 
  */
 function reception_prepare_head(Reception $object)
 {
-	global $db, $langs, $conf, $user;
+    global $db, $langs, $conf, $user;
 
-	$langs->load("sendings");
-	$langs->load("deliveries");
+    $langs->load("sendings");
+    $langs->load("deliveries");
 
-	$h = 0;
-	$head = array();
+    $h = 0;
+    $head = array();
 
-	$head[$h][0] = DOL_URL_ROOT."/reception/card.php?id=".$object->id;
-	$head[$h][1] = $langs->trans("ReceptionCard");
-	$head[$h][2] = 'reception';
-	$h++;
+    $head[$h][0] = DOL_URL_ROOT."/reception/card.php?id=".$object->id;
+    $head[$h][1] = $langs->trans("ReceptionCard");
+    $head[$h][2] = 'reception';
+    $h++;
 
 	
 
-	if (empty($conf->global->MAIN_DISABLE_CONTACTS_TAB))
-	{
-	    $objectsrc = $object;
-	    if ($object->origin == 'commande' && $object->origin_id > 0)
-	    {
-	        $objectsrc = new Commande($db);
-	        $objectsrc->fetch($object->origin_id);
-	    }
-	    $nbContact = count($objectsrc->liste_contact(-1,'internal')) + count($objectsrc->liste_contact(-1,'external'));
-	    $head[$h][0] = DOL_URL_ROOT."/reception/contact.php?id=".$object->id;
-    	$head[$h][1] = $langs->trans("ContactsAddresses");
-		if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
-    	$head[$h][2] = 'contact';
-    	$h++;
-	}
+    if (empty($conf->global->MAIN_DISABLE_CONTACTS_TAB))
+    {
+        $objectsrc = $object;
+        if ($object->origin == 'commande' && $object->origin_id > 0)
+        {
+            $objectsrc = new Commande($db);
+            $objectsrc->fetch($object->origin_id);
+        }
+        $nbContact = count($objectsrc->liste_contact(-1,'internal')) + count($objectsrc->liste_contact(-1,'external'));
+        $head[$h][0] = DOL_URL_ROOT."/reception/contact.php?id=".$object->id;
+        $head[$h][1] = $langs->trans("ContactsAddresses");
+        if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+        $head[$h][2] = 'contact';
+        $h++;
+    }
 	
     $nbNote = 0;
     if (!empty($object->note_private)) $nbNote++;
     if (!empty($object->note_public)) $nbNote++;
-	$head[$h][0] = DOL_URL_ROOT."/reception/note.php?id=".$object->id;
-	$head[$h][1] = $langs->trans("Notes");
-	if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
-	$head[$h][2] = 'note';
-	$h++;
+    $head[$h][0] = DOL_URL_ROOT."/reception/note.php?id=".$object->id;
+    $head[$h][1] = $langs->trans("Notes");
+    if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+    $head[$h][2] = 'note';
+    $h++;
 	
 	
 	
@@ -91,39 +91,39 @@ 
  */
 function reception_admin_prepare_head()
 {
-	global $langs, $conf, $user;
-	$langs->load("receptions");
+    global $langs, $conf, $user;
+    $langs->load("receptions");
 
-	$h = 0;
-	$head = array();
+    $h = 0;
+    $head = array();
 	
-	$head[$h][0] = DOL_URL_ROOT."/admin/reception_setup.php";
-	$head[$h][1] = $langs->trans("Reception");
-	$head[$h][2] = 'reception';
-	$h++;
+    $head[$h][0] = DOL_URL_ROOT."/admin/reception_setup.php";
+    $head[$h][1] = $langs->trans("Reception");
+    $head[$h][2] = 'reception';
+    $h++;
 
 	
-	if (! empty($conf->global->MAIN_SUBMODULE_RECEPTION))
-	{
-	$head[$h][0] = DOL_URL_ROOT.'/admin/reception_extrafields.php';
-	$head[$h][1] = $langs->trans("ExtraFields");
-	$head[$h][2] = 'attributes_reception';
-	$h++;
-	}
+    if (! empty($conf->global->MAIN_SUBMODULE_RECEPTION))
+    {
+    $head[$h][0] = DOL_URL_ROOT.'/admin/reception_extrafields.php';
+    $head[$h][1] = $langs->trans("ExtraFields");
+    $head[$h][2] = 'attributes_reception';
+    $h++;
+    }
 	
-	if (! empty($conf->global->MAIN_SUBMODULE_RECEPTION))
-	{
-	$head[$h][0] = DOL_URL_ROOT.'/admin/commande_fournisseur_dispatch_extrafields.php';
-	$head[$h][1] = $langs->trans("ExtraFieldsLines");
-	$head[$h][2] = 'attributeslines_reception';
-	$h++;
-	}
+    if (! empty($conf->global->MAIN_SUBMODULE_RECEPTION))
+    {
+    $head[$h][0] = DOL_URL_ROOT.'/admin/commande_fournisseur_dispatch_extrafields.php';
+    $head[$h][1] = $langs->trans("ExtraFieldsLines");
+    $head[$h][2] = 'attributeslines_reception';
+    $h++;
+    }
 	
 	
 	
-	complete_head_from_modules($conf,$langs,null,$head,$h,'reception_admin','remove');
+    complete_head_from_modules($conf,$langs,null,$head,$h,'reception_admin','remove');
 
-	return $head;
+    return $head;
 }
 
 

Spacing +7 added lines, -7 removed lines

@@ -57,10 +57,10 @@ 
 	        $objectsrc = new Commande($db);
 	        $objectsrc->fetch($object->origin_id);
 	    }
-	    $nbContact = count($objectsrc->liste_contact(-1,'internal')) + count($objectsrc->liste_contact(-1,'external'));
+	    $nbContact = count($objectsrc->liste_contact(-1, 'internal')) + count($objectsrc->liste_contact(-1, 'external'));
 	    $head[$h][0] = DOL_URL_ROOT."/reception/contact.php?id=".$object->id;
     	$head[$h][1] = $langs->trans("ContactsAddresses");
-		if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+		if ($nbContact > 0) $head[$h][1] .= ' <span class="badge">'.$nbContact.'</span>';
     	$head[$h][2] = 'contact';
     	$h++;
 	}
@@ -70,7 +70,7 @@ 
     if (!empty($object->note_public)) $nbNote++;
 	$head[$h][0] = DOL_URL_ROOT."/reception/note.php?id=".$object->id;
 	$head[$h][1] = $langs->trans("Notes");
-	if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+	if ($nbNote > 0) $head[$h][1] .= ' <span class="badge">'.$nbNote.'</span>';
 	$head[$h][2] = 'note';
 	$h++;
 	
@@ -79,7 +79,7 @@ 
 
 	
 
-    complete_head_from_modules($conf,$langs,$object,$head,$h,'order','remove');
+    complete_head_from_modules($conf, $langs, $object, $head, $h, 'order', 'remove');
 
     return $head;
 }
@@ -103,7 +103,7 @@ 
 	$h++;
 
 	
-	if (! empty($conf->global->MAIN_SUBMODULE_RECEPTION))
+	if (!empty($conf->global->MAIN_SUBMODULE_RECEPTION))
 	{
 	$head[$h][0] = DOL_URL_ROOT.'/admin/reception_extrafields.php';
 	$head[$h][1] = $langs->trans("ExtraFields");
@@ -111,7 +111,7 @@ 
 	$h++;
 	}
 	
-	if (! empty($conf->global->MAIN_SUBMODULE_RECEPTION))
+	if (!empty($conf->global->MAIN_SUBMODULE_RECEPTION))
 	{
 	$head[$h][0] = DOL_URL_ROOT.'/admin/commande_fournisseur_dispatch_extrafields.php';
 	$head[$h][1] = $langs->trans("ExtraFieldsLines");
@@ -121,7 +121,7 @@ 
 	
 	
 	
-	complete_head_from_modules($conf,$langs,null,$head,$h,'reception_admin','remove');
+	complete_head_from_modules($conf, $langs, null, $head, $h, 'reception_admin', 'remove');
 
 	return $head;
 }

Braces +12 added lines, -4 removed lines

@@ -60,17 +60,25 @@
 	    $nbContact = count($objectsrc->liste_contact(-1,'internal')) + count($objectsrc->liste_contact(-1,'external'));
 	    $head[$h][0] = DOL_URL_ROOT."/reception/contact.php?id=".$object->id;
     	$head[$h][1] = $langs->trans("ContactsAddresses");
-		if ($nbContact > 0) $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+		if ($nbContact > 0) {
+		    $head[$h][1].= ' <span class="badge">'.$nbContact.'</span>';
+		}
     	$head[$h][2] = 'contact';
     	$h++;
 	}
 	
     $nbNote = 0;
-    if (!empty($object->note_private)) $nbNote++;
-    if (!empty($object->note_public)) $nbNote++;
+    if (!empty($object->note_private)) {
+        $nbNote++;
+    }
+    if (!empty($object->note_public)) {
+        $nbNote++;
+    }
 	$head[$h][0] = DOL_URL_ROOT."/reception/note.php?id=".$object->id;
 	$head[$h][1] = $langs->trans("Notes");
-	if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+	if ($nbNote > 0) {
+	    $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+	}
 	$head[$h][2] = 'note';
 	$h++;
 	

dolibarr/htdocs/core/lib/expensereport.lib.php

Indentation +89 added lines, -89 removed lines

@@ -29,53 +29,53 @@ 
  */
 function expensereport_prepare_head($object)
 {
-	global $db, $langs, $conf;
+    global $db, $langs, $conf;
 
-	$h = 0;
-	$head = array();
+    $h = 0;
+    $head = array();
 
-	$head[$h][0] = DOL_URL_ROOT . '/expensereport/card.php?id=' . $object->id;
-	$head[$h][1] = $langs->trans("Card");
-	$head[$h][2] = 'card';
-	$h++;
+    $head[$h][0] = DOL_URL_ROOT . '/expensereport/card.php?id=' . $object->id;
+    $head[$h][1] = $langs->trans("Card");
+    $head[$h][2] = 'card';
+    $h++;
 
-	// Show more tabs from modules
-	// Entries must be declared in modules descriptor with line
+    // Show more tabs from modules
+    // Entries must be declared in modules descriptor with line
     // $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
     // $this->tabs = array('entity:-tabname);   												to remove a tab
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'expensereport');
+    complete_head_from_modules($conf,$langs,$object,$head,$h,'expensereport');
 
-	require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
+    require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
     require_once DOL_DOCUMENT_ROOT.'/core/class/link.class.php';
-	$upload_dir = $conf->expensereport->dir_output . "/" . dol_sanitizeFileName($object->ref);
-	$nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
+    $upload_dir = $conf->expensereport->dir_output . "/" . dol_sanitizeFileName($object->ref);
+    $nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
     $nbLinks=Link::count($db, $object->element, $object->id);
-	$head[$h][0] = DOL_URL_ROOT.'/expensereport/document.php?id='.$object->id;
-	$head[$h][1] = $langs->trans('Documents');
-	if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
-	$head[$h][2] = 'documents';
-	$h++;
-
-	if (empty($conf->global->MAIN_DISABLE_NOTES_TAB))
-	{
-	    $nbNote = 0;
-	    if(!empty($object->note_private)) $nbNote++;
-	    if(!empty($object->note_public)) $nbNote++;
-	    $head[$h][0] = DOL_URL_ROOT.'/expensereport/note.php?id='.$object->id;
-	    $head[$h][1] = $langs->trans('Notes');
-	    if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
-	    $head[$h][2] = 'note';
-	    $h++;
-	}
-
-	$head[$h][0] = DOL_URL_ROOT . '/expensereport/info.php?id=' . $object->id;
-	$head[$h][1] = $langs->trans("Info");
-	$head[$h][2] = 'info';
-	$h++;
-
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'expensereport','remove');
-
-	return $head;
+    $head[$h][0] = DOL_URL_ROOT.'/expensereport/document.php?id='.$object->id;
+    $head[$h][1] = $langs->trans('Documents');
+    if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+    $head[$h][2] = 'documents';
+    $h++;
+
+    if (empty($conf->global->MAIN_DISABLE_NOTES_TAB))
+    {
+        $nbNote = 0;
+        if(!empty($object->note_private)) $nbNote++;
+        if(!empty($object->note_public)) $nbNote++;
+        $head[$h][0] = DOL_URL_ROOT.'/expensereport/note.php?id='.$object->id;
+        $head[$h][1] = $langs->trans('Notes');
+        if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+        $head[$h][2] = 'note';
+        $h++;
+    }
+
+    $head[$h][0] = DOL_URL_ROOT . '/expensereport/info.php?id=' . $object->id;
+    $head[$h][1] = $langs->trans("Info");
+    $head[$h][2] = 'info';
+    $h++;
+
+    complete_head_from_modules($conf,$langs,$object,$head,$h,'expensereport','remove');
+
+    return $head;
 }
 
 /**
@@ -88,15 +88,15 @@ 
 function payment_expensereport_prepare_head(PaymentExpenseReport $object)
 {
 
-	global $langs, $conf;
+    global $langs, $conf;
 
-	$h = 0;
-	$head = array();
+    $h = 0;
+    $head = array();
 
-	$head[$h][0] = DOL_URL_ROOT.'/expensereport/payment/card.php?id='.$object->id;
-	$head[$h][1] = $langs->trans("Card");
-	$head[$h][2] = 'payment';
-	$h++;
+    $head[$h][0] = DOL_URL_ROOT.'/expensereport/payment/card.php?id='.$object->id;
+    $head[$h][1] = $langs->trans("Card");
+    $head[$h][2] = 'payment';
+    $h++;
 
     // Show more tabs from modules
     // Entries must be declared in modules descriptor with line
@@ -104,14 +104,14 @@ 
     // $this->tabs = array('entity:-tabname);   												to remove a tab
     complete_head_from_modules($conf,$langs,$object,$head,$h,'payment_expensereport');
 
-	$head[$h][0] = DOL_URL_ROOT.'/expensereport/payment/info.php?id='.$object->id;
-	$head[$h][1] = $langs->trans("Info");
-	$head[$h][2] = 'info';
-	$h++;
+    $head[$h][0] = DOL_URL_ROOT.'/expensereport/payment/info.php?id='.$object->id;
+    $head[$h][1] = $langs->trans("Info");
+    $head[$h][2] = 'info';
+    $h++;
 
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'payment_expensereport', 'remove');
+    complete_head_from_modules($conf,$langs,$object,$head,$h,'payment_expensereport', 'remove');
 
-	return $head;
+    return $head;
 }
 
 /**
@@ -121,42 +121,42 @@ 
  */
 function expensereport_admin_prepare_head()
 {
-	global $langs, $conf, $user;
-
-	$h = 0;
-	$head = array();
-
-	$h = 0;
-
-	$head[$h][0] = DOL_URL_ROOT."/admin/expensereport.php";
-	$head[$h][1] = $langs->trans("ExpenseReports");
-	$head[$h][2] = 'expensereport';
-	$h++;
-
-	if (!empty($conf->global->MAIN_USE_EXPENSE_IK))
-	{
-		$head[$h][0] = DOL_URL_ROOT."/admin/expensereport_ik.php";
-		$head[$h][1] = $langs->trans("ExpenseReportsIk");
-		$head[$h][2] = 'expenseik';
-		$h++;
-	}
-
-	if (!empty($conf->global->MAIN_USE_EXPENSE_RULE))
-	{
-		$head[$h][0] = DOL_URL_ROOT."/admin/expensereport_rules.php";
-		$head[$h][1] = $langs->trans("ExpenseReportsRules");
-		$head[$h][2] = 'expenserules';
-		$h++;
-	}
-
-	// Show more tabs from modules
-	// Entries must be declared in modules descriptor with line
-	// $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
-	// $this->tabs = array('entity:-tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to remove a tab
-	complete_head_from_modules($conf,$langs,null,$head,$h,'expensereport_admin');
-
-	$head[$h][0] = DOL_URL_ROOT.'/admin/expensereport_extrafields.php';
-	$head[$h][1] = $langs->trans("ExtraFields");
+    global $langs, $conf, $user;
+
+    $h = 0;
+    $head = array();
+
+    $h = 0;
+
+    $head[$h][0] = DOL_URL_ROOT."/admin/expensereport.php";
+    $head[$h][1] = $langs->trans("ExpenseReports");
+    $head[$h][2] = 'expensereport';
+    $h++;
+
+    if (!empty($conf->global->MAIN_USE_EXPENSE_IK))
+    {
+        $head[$h][0] = DOL_URL_ROOT."/admin/expensereport_ik.php";
+        $head[$h][1] = $langs->trans("ExpenseReportsIk");
+        $head[$h][2] = 'expenseik';
+        $h++;
+    }
+
+    if (!empty($conf->global->MAIN_USE_EXPENSE_RULE))
+    {
+        $head[$h][0] = DOL_URL_ROOT."/admin/expensereport_rules.php";
+        $head[$h][1] = $langs->trans("ExpenseReportsRules");
+        $head[$h][2] = 'expenserules';
+        $h++;
+    }
+
+    // Show more tabs from modules
+    // Entries must be declared in modules descriptor with line
+    // $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
+    // $this->tabs = array('entity:-tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to remove a tab
+    complete_head_from_modules($conf,$langs,null,$head,$h,'expensereport_admin');
+
+    $head[$h][0] = DOL_URL_ROOT.'/admin/expensereport_extrafields.php';
+    $head[$h][1] = $langs->trans("ExtraFields");
     $head[$h][2] = 'attributes';
     $h++;
 
@@ -167,7 +167,7 @@ 
     $h++;
 	*/
 
-	complete_head_from_modules($conf,$langs,null,$head,$h,'expensereport_admin','remove');
+    complete_head_from_modules($conf,$langs,null,$head,$h,'expensereport_admin','remove');
 
     return $head;
 }

Spacing +15 added lines, -15 removed lines

@@ -34,7 +34,7 @@ 
 	$h = 0;
 	$head = array();
 
-	$head[$h][0] = DOL_URL_ROOT . '/expensereport/card.php?id=' . $object->id;
+	$head[$h][0] = DOL_URL_ROOT.'/expensereport/card.php?id='.$object->id;
 	$head[$h][1] = $langs->trans("Card");
 	$head[$h][2] = 'card';
 	$h++;
@@ -43,37 +43,37 @@ 
 	// Entries must be declared in modules descriptor with line
     // $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
     // $this->tabs = array('entity:-tabname);   												to remove a tab
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'expensereport');
+	complete_head_from_modules($conf, $langs, $object, $head, $h, 'expensereport');
 
 	require_once DOL_DOCUMENT_ROOT.'/core/lib/files.lib.php';
     require_once DOL_DOCUMENT_ROOT.'/core/class/link.class.php';
-	$upload_dir = $conf->expensereport->dir_output . "/" . dol_sanitizeFileName($object->ref);
-	$nbFiles = count(dol_dir_list($upload_dir,'files',0,'','(\.meta|_preview.*\.png)$'));
-    $nbLinks=Link::count($db, $object->element, $object->id);
+	$upload_dir = $conf->expensereport->dir_output."/".dol_sanitizeFileName($object->ref);
+	$nbFiles = count(dol_dir_list($upload_dir, 'files', 0, '', '(\.meta|_preview.*\.png)$'));
+    $nbLinks = Link::count($db, $object->element, $object->id);
 	$head[$h][0] = DOL_URL_ROOT.'/expensereport/document.php?id='.$object->id;
 	$head[$h][1] = $langs->trans('Documents');
-	if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	if (($nbFiles + $nbLinks) > 0) $head[$h][1] .= ' <span class="badge">'.($nbFiles + $nbLinks).'</span>';
 	$head[$h][2] = 'documents';
 	$h++;
 
 	if (empty($conf->global->MAIN_DISABLE_NOTES_TAB))
 	{
 	    $nbNote = 0;
-	    if(!empty($object->note_private)) $nbNote++;
-	    if(!empty($object->note_public)) $nbNote++;
+	    if (!empty($object->note_private)) $nbNote++;
+	    if (!empty($object->note_public)) $nbNote++;
 	    $head[$h][0] = DOL_URL_ROOT.'/expensereport/note.php?id='.$object->id;
 	    $head[$h][1] = $langs->trans('Notes');
-	    if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+	    if ($nbNote > 0) $head[$h][1] .= ' <span class="badge">'.$nbNote.'</span>';
 	    $head[$h][2] = 'note';
 	    $h++;
 	}
 
-	$head[$h][0] = DOL_URL_ROOT . '/expensereport/info.php?id=' . $object->id;
+	$head[$h][0] = DOL_URL_ROOT.'/expensereport/info.php?id='.$object->id;
 	$head[$h][1] = $langs->trans("Info");
 	$head[$h][2] = 'info';
 	$h++;
 
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'expensereport','remove');
+	complete_head_from_modules($conf, $langs, $object, $head, $h, 'expensereport', 'remove');
 
 	return $head;
 }
@@ -102,14 +102,14 @@ 
     // Entries must be declared in modules descriptor with line
     // $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
     // $this->tabs = array('entity:-tabname);   												to remove a tab
-    complete_head_from_modules($conf,$langs,$object,$head,$h,'payment_expensereport');
+    complete_head_from_modules($conf, $langs, $object, $head, $h, 'payment_expensereport');
 
 	$head[$h][0] = DOL_URL_ROOT.'/expensereport/payment/info.php?id='.$object->id;
 	$head[$h][1] = $langs->trans("Info");
 	$head[$h][2] = 'info';
 	$h++;
 
-	complete_head_from_modules($conf,$langs,$object,$head,$h,'payment_expensereport', 'remove');
+	complete_head_from_modules($conf, $langs, $object, $head, $h, 'payment_expensereport', 'remove');
 
 	return $head;
 }
@@ -153,7 +153,7 @@ 
 	// Entries must be declared in modules descriptor with line
 	// $this->tabs = array('entity:+tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to add new tab
 	// $this->tabs = array('entity:-tabname:Title:@mymodule:/mymodule/mypage.php?id=__ID__');   to remove a tab
-	complete_head_from_modules($conf,$langs,null,$head,$h,'expensereport_admin');
+	complete_head_from_modules($conf, $langs, null, $head, $h, 'expensereport_admin');
 
 	$head[$h][0] = DOL_URL_ROOT.'/admin/expensereport_extrafields.php';
 	$head[$h][1] = $langs->trans("ExtraFields");
@@ -167,7 +167,7 @@ 
     $h++;
 	*/
 
-	complete_head_from_modules($conf,$langs,null,$head,$h,'expensereport_admin','remove');
+	complete_head_from_modules($conf, $langs, null, $head, $h, 'expensereport_admin', 'remove');
 
     return $head;
 }

Braces +12 added lines, -4 removed lines

@@ -52,18 +52,26 @@
     $nbLinks=Link::count($db, $object->element, $object->id);
 	$head[$h][0] = DOL_URL_ROOT.'/expensereport/document.php?id='.$object->id;
 	$head[$h][1] = $langs->trans('Documents');
-	if (($nbFiles+$nbLinks) > 0) $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	if (($nbFiles+$nbLinks) > 0) {
+	    $head[$h][1].= ' <span class="badge">'.($nbFiles+$nbLinks).'</span>';
+	}
 	$head[$h][2] = 'documents';
 	$h++;
 
 	if (empty($conf->global->MAIN_DISABLE_NOTES_TAB))
 	{
 	    $nbNote = 0;
-	    if(!empty($object->note_private)) $nbNote++;
-	    if(!empty($object->note_public)) $nbNote++;
+	    if(!empty($object->note_private)) {
+	        $nbNote++;
+	    }
+	    if(!empty($object->note_public)) {
+	        $nbNote++;
+	    }
 	    $head[$h][0] = DOL_URL_ROOT.'/expensereport/note.php?id='.$object->id;
 	    $head[$h][1] = $langs->trans('Notes');
-	    if ($nbNote > 0) $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+	    if ($nbNote > 0) {
+	        $head[$h][1].= ' <span class="badge">'.$nbNote.'</span>';
+	    }
 	    $head[$h][2] = 'note';
 	    $h++;
 	}

dolibarr/htdocs/core/lib/treeview.lib.php

Indentation +142 added lines, -142 removed lines

@@ -35,61 +35,61 @@ 
  */
 function tree_showpad(&$fulltree,$key,$silent=0)
 {
-	$pos=1;
-
-	// Loop on each pos, because we will output an img for each pos
-	while ($pos <= $fulltree[$key]['level'] && $fulltree[$key]['level'] > 0)
-	{
-		// Process for column $pos
-
-		$atleastoneofthislevelafter=0;
-		$nbofdirinsub=0;
-		$nbofdocinsub=0;
-		$found=0;
-		//print 'x'.$key;
-		foreach($fulltree as $key2 => $val2)
-		{
+    $pos=1;
+
+    // Loop on each pos, because we will output an img for each pos
+    while ($pos <= $fulltree[$key]['level'] && $fulltree[$key]['level'] > 0)
+    {
+        // Process for column $pos
+
+        $atleastoneofthislevelafter=0;
+        $nbofdirinsub=0;
+        $nbofdocinsub=0;
+        $found=0;
+        //print 'x'.$key;
+        foreach($fulltree as $key2 => $val2)
+        {
             //print "x".$pos." ".$key2." ".$found." ".$fulltree[$key2]['level'];
-			if ($found == 1) // We are after the entry to show
-			{
-				if ($fulltree[$key2]['level'] > $pos)
-				{
-					$nbofdirinsub++;
-					if (isset($fulltree[$key2]['cachenbofdoc']) && $fulltree[$key2]['cachenbofdoc'] > 0) $nbofdocinsub+=$fulltree[$key2]['cachenbofdoc'];
-				}
-				if ($fulltree[$key2]['level'] == $pos)
-				{
-					$atleastoneofthislevelafter=1;
-				}
-				if ($fulltree[$key2]['level'] <= $pos)
-				{
-					break;
-				}
-			}
-			if ($key2 == $key)    // We found ourself, so now every lower level will be counted
-			{
-				$found=1;
-			}
-		}
-		//print $atleastoneofthislevelafter;
-
-		if (! $silent)
-		{
-    		if ($atleastoneofthislevelafter)
-    		{
-    			if ($fulltree[$key]['level'] == $pos) print img_picto_common('','treemenu/branch.gif');
-    			else print img_picto_common('','treemenu/line.gif');
-    		}
-    		else
-    		{
-    			if ($fulltree[$key]['level'] == $pos) print img_picto_common('','treemenu/branchbottom.gif');
-    			else print img_picto_common('','treemenu/linebottom.gif');
-    		}
-		}
-		$pos++;
-	}
-
-	return array($atleastoneofthislevelafter,$nbofdirinsub,$nbofdocinsub);
+            if ($found == 1) // We are after the entry to show
+            {
+                if ($fulltree[$key2]['level'] > $pos)
+                {
+                    $nbofdirinsub++;
+                    if (isset($fulltree[$key2]['cachenbofdoc']) && $fulltree[$key2]['cachenbofdoc'] > 0) $nbofdocinsub+=$fulltree[$key2]['cachenbofdoc'];
+                }
+                if ($fulltree[$key2]['level'] == $pos)
+                {
+                    $atleastoneofthislevelafter=1;
+                }
+                if ($fulltree[$key2]['level'] <= $pos)
+                {
+                    break;
+                }
+            }
+            if ($key2 == $key)    // We found ourself, so now every lower level will be counted
+            {
+                $found=1;
+            }
+        }
+        //print $atleastoneofthislevelafter;
+
+        if (! $silent)
+        {
+            if ($atleastoneofthislevelafter)
+            {
+                if ($fulltree[$key]['level'] == $pos) print img_picto_common('','treemenu/branch.gif');
+                else print img_picto_common('','treemenu/line.gif');
+            }
+            else
+            {
+                if ($fulltree[$key]['level'] == $pos) print img_picto_common('','treemenu/branchbottom.gif');
+                else print img_picto_common('','treemenu/linebottom.gif');
+            }
+        }
+        $pos++;
+    }
+
+    return array($atleastoneofthislevelafter,$nbofdirinsub,$nbofdocinsub);
 }
 
 
@@ -120,9 +120,9 @@ 
     if ($rang == 0 && empty($donoresetalreadyloaded)) $tree_recur_alreadyadded=array();
 
     if ($rang == 0)
-	{
-		// Test also done with jstree and dynatree (not able to have <a> inside label)
-		print '<script type="text/javascript" language="javascript">
+    {
+        // Test also done with jstree and dynatree (not able to have <a> inside label)
+        print '<script type="text/javascript" language="javascript">
 		$(document).ready(function(){
 			$("#'.$iddivjstree.'").treeview({
 				collapsed: true,
@@ -136,94 +136,94 @@ 
 		})
 		</script>';
 
-		print '<ul id="'.$iddivjstree.'">';
-	}
-
-	if ($rang > 50)
-	{
-	    return;	// Protect against infinite loop. Max 50 depth
-	}
-
-	//ballayage du tableau
-	$sizeoftab=count($tab);
-	$ulprinted=0;
-	for ($x=0; $x < $sizeoftab; $x++)
-	{
-		//var_dump($tab[$x]);exit;
-		// If an element has $pere for parent
-		if ($tab[$x]['fk_menu'] != -1 && $tab[$x]['fk_menu'] == $pere['rowid'])
-		{
-		    //print 'rang='.$rang.'-x='.$x." rowid=".$tab[$x]['rowid']." tab[x]['fk_leftmenu'] = ".$tab[$x]['fk_leftmenu']." leftmenu pere = ".$pere['leftmenu']."<br>\n";
-			if (empty($ulprinted) && ! empty($pere['rowid']))
-			{
-    		    if (! empty($tree_recur_alreadyadded[$tab[$x]['rowid']]))
-    		    {
-    		          dol_syslog('Error, record with id '.$tab[$x]['rowid'].' seems to be a child of record with id '.$pere['rowid'].' but it was already output. Complete field "leftmenu" and "mainmenu" on ALL records to avoid ambiguity.', LOG_WARNING);
-    		          continue;
-    		    }
+        print '<ul id="'.$iddivjstree.'">';
+    }
+
+    if ($rang > 50)
+    {
+        return;	// Protect against infinite loop. Max 50 depth
+    }
+
+    //ballayage du tableau
+    $sizeoftab=count($tab);
+    $ulprinted=0;
+    for ($x=0; $x < $sizeoftab; $x++)
+    {
+        //var_dump($tab[$x]);exit;
+        // If an element has $pere for parent
+        if ($tab[$x]['fk_menu'] != -1 && $tab[$x]['fk_menu'] == $pere['rowid'])
+        {
+            //print 'rang='.$rang.'-x='.$x." rowid=".$tab[$x]['rowid']." tab[x]['fk_leftmenu'] = ".$tab[$x]['fk_leftmenu']." leftmenu pere = ".$pere['leftmenu']."<br>\n";
+            if (empty($ulprinted) && ! empty($pere['rowid']))
+            {
+                if (! empty($tree_recur_alreadyadded[$tab[$x]['rowid']]))
+                {
+                        dol_syslog('Error, record with id '.$tab[$x]['rowid'].' seems to be a child of record with id '.$pere['rowid'].' but it was already output. Complete field "leftmenu" and "mainmenu" on ALL records to avoid ambiguity.', LOG_WARNING);
+                        continue;
+                }
 
                 print '<ul'.(empty($pere['rowid'])?' id="treeData"':'').'>'; $ulprinted++;
-			}
-			print "\n".'<li '.($tab[$x]['statut']?' class="liuseractive"':'class="liuserdisabled"').'>';
-			if ($showfk)
-			{
-			    print '<table class="nobordernopadding centpercent"><tr><td>';
-			    print '<strong> &nbsp; ';
-			    print $tab[$x]['title'];
-			    print '&nbsp; (fk_mainmenu='.$tab[$x]['fk_mainmenu'].' fk_leftmenu='.$tab[$x]['fk_leftmenu'].')';
-			    print '</td><td align="right">';
-			    print $tab[$x]['buttons'];
-			    print '</td></tr></table>';
-			}
-			else
-			{
-			    print $tab[$x]['entry'];
-			}
-			//print ' -> A '.$tab[$x]['rowid'].' mainmenu='.$tab[$x]['mainmenu'].' leftmenu='.$tab[$x]['leftmenu'].' fk_mainmenu='.$tab[$x]['fk_mainmenu'].' fk_leftmenu='.$tab[$x]['fk_leftmenu'].'<br>'."\n";
-		    $tree_recur_alreadyadded[$tab[$x]['rowid']]=($rang + 1);
-			// And now we search all its sons of lower level
-			tree_recur($tab, $tab[$x], $rang+1, 'iddivjstree', 0, $showfk);
-			print '</li>';
-		}
-		elseif (! empty($tab[$x]['rowid']) && $tab[$x]['fk_menu'] == -1 && $tab[$x]['fk_mainmenu'] == $pere['mainmenu'] && $tab[$x]['fk_leftmenu'] == $pere['leftmenu'])
-		{
-		    //print 'rang='.$rang.'-x='.$x." rowid=".$tab[$x]['rowid']." tab[x]['fk_leftmenu'] = ".$tab[$x]['fk_leftmenu']." leftmenu pere = ".$pere['leftmenu']."<br>\n";
-		    if (empty($ulprinted) && ! empty($pere['rowid']))
-		    {
-		        if (! empty($tree_recur_alreadyadded[$tab[$x]['rowid']]))
-		        {
-		            dol_syslog('Error, record with id '.$tab[$x]['rowid'].' seems to be a child of record with id '.$pere['rowid'].' but it was already output. Complete field "leftmenu" and "mainmenu" on ALL records to avoid ambiguity.', LOG_WARNING);
-		            //print 'Error, record with id '.$tab[$x]['rowid'].' seems to be a child of record with id '.$pere['rowid'].' but it was already output. Complete field "leftmenu" and "mainmenu" on ALL records to avoid ambiguity.';
+            }
+            print "\n".'<li '.($tab[$x]['statut']?' class="liuseractive"':'class="liuserdisabled"').'>';
+            if ($showfk)
+            {
+                print '<table class="nobordernopadding centpercent"><tr><td>';
+                print '<strong> &nbsp; ';
+                print $tab[$x]['title'];
+                print '&nbsp; (fk_mainmenu='.$tab[$x]['fk_mainmenu'].' fk_leftmenu='.$tab[$x]['fk_leftmenu'].')';
+                print '</td><td align="right">';
+                print $tab[$x]['buttons'];
+                print '</td></tr></table>';
+            }
+            else
+            {
+                print $tab[$x]['entry'];
+            }
+            //print ' -> A '.$tab[$x]['rowid'].' mainmenu='.$tab[$x]['mainmenu'].' leftmenu='.$tab[$x]['leftmenu'].' fk_mainmenu='.$tab[$x]['fk_mainmenu'].' fk_leftmenu='.$tab[$x]['fk_leftmenu'].'<br>'."\n";
+            $tree_recur_alreadyadded[$tab[$x]['rowid']]=($rang + 1);
+            // And now we search all its sons of lower level
+            tree_recur($tab, $tab[$x], $rang+1, 'iddivjstree', 0, $showfk);
+            print '</li>';
+        }
+        elseif (! empty($tab[$x]['rowid']) && $tab[$x]['fk_menu'] == -1 && $tab[$x]['fk_mainmenu'] == $pere['mainmenu'] && $tab[$x]['fk_leftmenu'] == $pere['leftmenu'])
+        {
+            //print 'rang='.$rang.'-x='.$x." rowid=".$tab[$x]['rowid']." tab[x]['fk_leftmenu'] = ".$tab[$x]['fk_leftmenu']." leftmenu pere = ".$pere['leftmenu']."<br>\n";
+            if (empty($ulprinted) && ! empty($pere['rowid']))
+            {
+                if (! empty($tree_recur_alreadyadded[$tab[$x]['rowid']]))
+                {
+                    dol_syslog('Error, record with id '.$tab[$x]['rowid'].' seems to be a child of record with id '.$pere['rowid'].' but it was already output. Complete field "leftmenu" and "mainmenu" on ALL records to avoid ambiguity.', LOG_WARNING);
+                    //print 'Error, record with id '.$tab[$x]['rowid'].' seems to be a child of record with id '.$pere['rowid'].' but it was already output. Complete field "leftmenu" and "mainmenu" on ALL records to avoid ambiguity.';
                     continue;
-		        }
-
-		        print '<ul'.(empty($pere['rowid'])?' id="treeData"':'').'>'; $ulprinted++;
-		    }
-			print "\n".'<li '.($tab[$x]['statut']?' class="liuseractive"':'class="liuserdisabled"').'>';
-			if ($showfk)
-			{
-			    print '<table class="nobordernopadding centpercent"><tr><td>';
-			    print '<strong> &nbsp; <a href="edit.php?menu_handler='.$menu_handler_to_search.'&action=edit&menuId='.$menu['rowid'].'">';
-			    print $tab[$x]['title'];
-			    print '</a></strong>';
-			    print '&nbsp; (mainmenu='.$tab[$x]['mainmenu'].' leftmenu='.$tab[$x]['leftmenu'].' - fk_mainmenu='.$tab[$x]['fk_mainmenu'].' fk_leftmenu='.$tab[$x]['fk_leftmenu'].')';
-			    print '</td><td align="right">';
-			    print $tab[$x]['buttons'];
-			    print '</td></tr></table>';
-			}
-			else
-			{
-			    print $tab[$x]['entry'];
-			}
-			//print ' -> B '.$tab[$x]['rowid'].' mainmenu='.$tab[$x]['mainmenu'].' leftmenu='.$tab[$x]['leftmenu'].' fk_mainmenu='.$tab[$x]['fk_mainmenu'].' fk_leftmenu='.$tab[$x]['fk_leftmenu'].'<br>'."\n";
-			$tree_recur_alreadyadded[$tab[$x]['rowid']]=($rang + 1);
-			// And now we search all its sons of lower level
-			//print 'Call tree_recur for x='.$x.' rowid='.$tab[$x]['rowid']." fk_mainmenu pere = ".$tab[$x]['fk_mainmenu']." fk_leftmenu pere = ".$tab[$x]['fk_leftmenu']."<br>\n";
-		    tree_recur($tab, $tab[$x], $rang+1, 'iddivjstree', 0, $showfk);
-			print '</li>';
-		}
-	}
-	if (! empty($ulprinted) && ! empty($pere['rowid'])) { print '</ul>'."\n"; }
+                }
+
+                print '<ul'.(empty($pere['rowid'])?' id="treeData"':'').'>'; $ulprinted++;
+            }
+            print "\n".'<li '.($tab[$x]['statut']?' class="liuseractive"':'class="liuserdisabled"').'>';
+            if ($showfk)
+            {
+                print '<table class="nobordernopadding centpercent"><tr><td>';
+                print '<strong> &nbsp; <a href="edit.php?menu_handler='.$menu_handler_to_search.'&action=edit&menuId='.$menu['rowid'].'">';
+                print $tab[$x]['title'];
+                print '</a></strong>';
+                print '&nbsp; (mainmenu='.$tab[$x]['mainmenu'].' leftmenu='.$tab[$x]['leftmenu'].' - fk_mainmenu='.$tab[$x]['fk_mainmenu'].' fk_leftmenu='.$tab[$x]['fk_leftmenu'].')';
+                print '</td><td align="right">';
+                print $tab[$x]['buttons'];
+                print '</td></tr></table>';
+            }
+            else
+            {
+                print $tab[$x]['entry'];
+            }
+            //print ' -> B '.$tab[$x]['rowid'].' mainmenu='.$tab[$x]['mainmenu'].' leftmenu='.$tab[$x]['leftmenu'].' fk_mainmenu='.$tab[$x]['fk_mainmenu'].' fk_leftmenu='.$tab[$x]['fk_leftmenu'].'<br>'."\n";
+            $tree_recur_alreadyadded[$tab[$x]['rowid']]=($rang + 1);
+            // And now we search all its sons of lower level
+            //print 'Call tree_recur for x='.$x.' rowid='.$tab[$x]['rowid']." fk_mainmenu pere = ".$tab[$x]['fk_mainmenu']." fk_leftmenu pere = ".$tab[$x]['fk_leftmenu']."<br>\n";
+            tree_recur($tab, $tab[$x], $rang+1, 'iddivjstree', 0, $showfk);
+            print '</li>';
+        }
+    }
+    if (! empty($ulprinted) && ! empty($pere['rowid'])) { print '</ul>'."\n"; }
 
     if ($rang == 0) print '</ul>';
 }

Spacing +36 added lines, -36 removed lines

@@ -33,21 +33,21 @@ 
  * @param	int		$silent			Do not output indent and picto, returns only value
  * @return	integer[]					array(0 or 1 if at least one of this level after, 0 or 1 if at least one of higher level after, nbofdirinsub, nbofdocinsub)
  */
-function tree_showpad(&$fulltree,$key,$silent=0)
+function tree_showpad(&$fulltree, $key, $silent = 0)
 {
-	$pos=1;
+	$pos = 1;
 
 	// Loop on each pos, because we will output an img for each pos
 	while ($pos <= $fulltree[$key]['level'] && $fulltree[$key]['level'] > 0)
 	{
 		// Process for column $pos
 
-		$atleastoneofthislevelafter=0;
-		$nbofdirinsub=0;
-		$nbofdocinsub=0;
-		$found=0;
+		$atleastoneofthislevelafter = 0;
+		$nbofdirinsub = 0;
+		$nbofdocinsub = 0;
+		$found = 0;
 		//print 'x'.$key;
-		foreach($fulltree as $key2 => $val2)
+		foreach ($fulltree as $key2 => $val2)
 		{
             //print "x".$pos." ".$key2." ".$found." ".$fulltree[$key2]['level'];
 			if ($found == 1) // We are after the entry to show
@@ -55,11 +55,11 @@ 
 				if ($fulltree[$key2]['level'] > $pos)
 				{
 					$nbofdirinsub++;
-					if (isset($fulltree[$key2]['cachenbofdoc']) && $fulltree[$key2]['cachenbofdoc'] > 0) $nbofdocinsub+=$fulltree[$key2]['cachenbofdoc'];
+					if (isset($fulltree[$key2]['cachenbofdoc']) && $fulltree[$key2]['cachenbofdoc'] > 0) $nbofdocinsub += $fulltree[$key2]['cachenbofdoc'];
 				}
 				if ($fulltree[$key2]['level'] == $pos)
 				{
-					$atleastoneofthislevelafter=1;
+					$atleastoneofthislevelafter = 1;
 				}
 				if ($fulltree[$key2]['level'] <= $pos)
 				{
@@ -68,28 +68,28 @@ 
 			}
 			if ($key2 == $key)    // We found ourself, so now every lower level will be counted
 			{
-				$found=1;
+				$found = 1;
 			}
 		}
 		//print $atleastoneofthislevelafter;
 
-		if (! $silent)
+		if (!$silent)
 		{
     		if ($atleastoneofthislevelafter)
     		{
-    			if ($fulltree[$key]['level'] == $pos) print img_picto_common('','treemenu/branch.gif');
-    			else print img_picto_common('','treemenu/line.gif');
+    			if ($fulltree[$key]['level'] == $pos) print img_picto_common('', 'treemenu/branch.gif');
+    			else print img_picto_common('', 'treemenu/line.gif');
     		}
     		else
     		{
-    			if ($fulltree[$key]['level'] == $pos) print img_picto_common('','treemenu/branchbottom.gif');
-    			else print img_picto_common('','treemenu/linebottom.gif');
+    			if ($fulltree[$key]['level'] == $pos) print img_picto_common('', 'treemenu/branchbottom.gif');
+    			else print img_picto_common('', 'treemenu/linebottom.gif');
     		}
 		}
 		$pos++;
 	}
 
-	return array($atleastoneofthislevelafter,$nbofdirinsub,$nbofdocinsub);
+	return array($atleastoneofthislevelafter, $nbofdirinsub, $nbofdocinsub);
 }
 
 
@@ -113,11 +113,11 @@ 
  *  @param  int     $showfk         1=show fk_links to parent into label  (used by menu editor only)
  *  @return	void
  */
-function tree_recur($tab, $pere, $rang, $iddivjstree='iddivjstree', $donoresetalreadyloaded=0, $showfk=0)
+function tree_recur($tab, $pere, $rang, $iddivjstree = 'iddivjstree', $donoresetalreadyloaded = 0, $showfk = 0)
 {
     global $tree_recur_alreadyadded;
 
-    if ($rang == 0 && empty($donoresetalreadyloaded)) $tree_recur_alreadyadded=array();
+    if ($rang == 0 && empty($donoresetalreadyloaded)) $tree_recur_alreadyadded = array();
 
     if ($rang == 0)
 	{
@@ -141,30 +141,30 @@ 
 
 	if ($rang > 50)
 	{
-	    return;	// Protect against infinite loop. Max 50 depth
+	    return; // Protect against infinite loop. Max 50 depth
 	}
 
 	//ballayage du tableau
-	$sizeoftab=count($tab);
-	$ulprinted=0;
-	for ($x=0; $x < $sizeoftab; $x++)
+	$sizeoftab = count($tab);
+	$ulprinted = 0;
+	for ($x = 0; $x < $sizeoftab; $x++)
 	{
 		//var_dump($tab[$x]);exit;
 		// If an element has $pere for parent
 		if ($tab[$x]['fk_menu'] != -1 && $tab[$x]['fk_menu'] == $pere['rowid'])
 		{
 		    //print 'rang='.$rang.'-x='.$x." rowid=".$tab[$x]['rowid']." tab[x]['fk_leftmenu'] = ".$tab[$x]['fk_leftmenu']." leftmenu pere = ".$pere['leftmenu']."<br>\n";
-			if (empty($ulprinted) && ! empty($pere['rowid']))
+			if (empty($ulprinted) && !empty($pere['rowid']))
 			{
-    		    if (! empty($tree_recur_alreadyadded[$tab[$x]['rowid']]))
+    		    if (!empty($tree_recur_alreadyadded[$tab[$x]['rowid']]))
     		    {
     		          dol_syslog('Error, record with id '.$tab[$x]['rowid'].' seems to be a child of record with id '.$pere['rowid'].' but it was already output. Complete field "leftmenu" and "mainmenu" on ALL records to avoid ambiguity.', LOG_WARNING);
     		          continue;
     		    }
 
-                print '<ul'.(empty($pere['rowid'])?' id="treeData"':'').'>'; $ulprinted++;
+                print '<ul'.(empty($pere['rowid']) ? ' id="treeData"' : '').'>'; $ulprinted++;
 			}
-			print "\n".'<li '.($tab[$x]['statut']?' class="liuseractive"':'class="liuserdisabled"').'>';
+			print "\n".'<li '.($tab[$x]['statut'] ? ' class="liuseractive"' : 'class="liuserdisabled"').'>';
 			if ($showfk)
 			{
 			    print '<table class="nobordernopadding centpercent"><tr><td>';
@@ -180,26 +180,26 @@ 
 			    print $tab[$x]['entry'];
 			}
 			//print ' -> A '.$tab[$x]['rowid'].' mainmenu='.$tab[$x]['mainmenu'].' leftmenu='.$tab[$x]['leftmenu'].' fk_mainmenu='.$tab[$x]['fk_mainmenu'].' fk_leftmenu='.$tab[$x]['fk_leftmenu'].'<br>'."\n";
-		    $tree_recur_alreadyadded[$tab[$x]['rowid']]=($rang + 1);
+		    $tree_recur_alreadyadded[$tab[$x]['rowid']] = ($rang + 1);
 			// And now we search all its sons of lower level
-			tree_recur($tab, $tab[$x], $rang+1, 'iddivjstree', 0, $showfk);
+			tree_recur($tab, $tab[$x], $rang + 1, 'iddivjstree', 0, $showfk);
 			print '</li>';
 		}
-		elseif (! empty($tab[$x]['rowid']) && $tab[$x]['fk_menu'] == -1 && $tab[$x]['fk_mainmenu'] == $pere['mainmenu'] && $tab[$x]['fk_leftmenu'] == $pere['leftmenu'])
+		elseif (!empty($tab[$x]['rowid']) && $tab[$x]['fk_menu'] == -1 && $tab[$x]['fk_mainmenu'] == $pere['mainmenu'] && $tab[$x]['fk_leftmenu'] == $pere['leftmenu'])
 		{
 		    //print 'rang='.$rang.'-x='.$x." rowid=".$tab[$x]['rowid']." tab[x]['fk_leftmenu'] = ".$tab[$x]['fk_leftmenu']." leftmenu pere = ".$pere['leftmenu']."<br>\n";
-		    if (empty($ulprinted) && ! empty($pere['rowid']))
+		    if (empty($ulprinted) && !empty($pere['rowid']))
 		    {
-		        if (! empty($tree_recur_alreadyadded[$tab[$x]['rowid']]))
+		        if (!empty($tree_recur_alreadyadded[$tab[$x]['rowid']]))
 		        {
 		            dol_syslog('Error, record with id '.$tab[$x]['rowid'].' seems to be a child of record with id '.$pere['rowid'].' but it was already output. Complete field "leftmenu" and "mainmenu" on ALL records to avoid ambiguity.', LOG_WARNING);
 		            //print 'Error, record with id '.$tab[$x]['rowid'].' seems to be a child of record with id '.$pere['rowid'].' but it was already output. Complete field "leftmenu" and "mainmenu" on ALL records to avoid ambiguity.';
                     continue;
 		        }
 
-		        print '<ul'.(empty($pere['rowid'])?' id="treeData"':'').'>'; $ulprinted++;
+		        print '<ul'.(empty($pere['rowid']) ? ' id="treeData"' : '').'>'; $ulprinted++;
 		    }
-			print "\n".'<li '.($tab[$x]['statut']?' class="liuseractive"':'class="liuserdisabled"').'>';
+			print "\n".'<li '.($tab[$x]['statut'] ? ' class="liuseractive"' : 'class="liuserdisabled"').'>';
 			if ($showfk)
 			{
 			    print '<table class="nobordernopadding centpercent"><tr><td>';
@@ -216,14 +216,14 @@ 
 			    print $tab[$x]['entry'];
 			}
 			//print ' -> B '.$tab[$x]['rowid'].' mainmenu='.$tab[$x]['mainmenu'].' leftmenu='.$tab[$x]['leftmenu'].' fk_mainmenu='.$tab[$x]['fk_mainmenu'].' fk_leftmenu='.$tab[$x]['fk_leftmenu'].'<br>'."\n";
-			$tree_recur_alreadyadded[$tab[$x]['rowid']]=($rang + 1);
+			$tree_recur_alreadyadded[$tab[$x]['rowid']] = ($rang + 1);
 			// And now we search all its sons of lower level
 			//print 'Call tree_recur for x='.$x.' rowid='.$tab[$x]['rowid']." fk_mainmenu pere = ".$tab[$x]['fk_mainmenu']." fk_leftmenu pere = ".$tab[$x]['fk_leftmenu']."<br>\n";
-		    tree_recur($tab, $tab[$x], $rang+1, 'iddivjstree', 0, $showfk);
+		    tree_recur($tab, $tab[$x], $rang + 1, 'iddivjstree', 0, $showfk);
 			print '</li>';
 		}
 	}
-	if (! empty($ulprinted) && ! empty($pere['rowid'])) { print '</ul>'."\n"; }
+	if (!empty($ulprinted) && !empty($pere['rowid'])) { print '</ul>'."\n"; }
 
     if ($rang == 0) print '</ul>';
 }

Braces +30 added lines, -18 removed lines

@@ -50,12 +50,16 @@ 
 		foreach($fulltree as $key2 => $val2)
 		{
             //print "x".$pos." ".$key2." ".$found." ".$fulltree[$key2]['level'];
-			if ($found == 1) // We are after the entry to show
+			if ($found == 1) {
+			    // We are after the entry to show
 			{
 				if ($fulltree[$key2]['level'] > $pos)
 				{
 					$nbofdirinsub++;
-					if (isset($fulltree[$key2]['cachenbofdoc']) && $fulltree[$key2]['cachenbofdoc'] > 0) $nbofdocinsub+=$fulltree[$key2]['cachenbofdoc'];
+			}
+					if (isset($fulltree[$key2]['cachenbofdoc']) && $fulltree[$key2]['cachenbofdoc'] > 0) {
+					    $nbofdocinsub+=$fulltree[$key2]['cachenbofdoc'];
+					}
 				}
 				if ($fulltree[$key2]['level'] == $pos)
 				{
@@ -66,10 +70,12 @@ 
 					break;
 				}
 			}
-			if ($key2 == $key)    // We found ourself, so now every lower level will be counted
+			if ($key2 == $key) {
+			    // We found ourself, so now every lower level will be counted
 			{
 				$found=1;
 			}
+			}
 		}
 		//print $atleastoneofthislevelafter;
 
@@ -77,13 +83,18 @@ 
 		{
     		if ($atleastoneofthislevelafter)
     		{
-    			if ($fulltree[$key]['level'] == $pos) print img_picto_common('','treemenu/branch.gif');
-    			else print img_picto_common('','treemenu/line.gif');
-    		}
-    		else
+    			if ($fulltree[$key]['level'] == $pos) {
+    			    print img_picto_common('','treemenu/branch.gif');
+    			} else {
+    			    print img_picto_common('','treemenu/line.gif');
+    			}
+    		} else
     		{
-    			if ($fulltree[$key]['level'] == $pos) print img_picto_common('','treemenu/branchbottom.gif');
-    			else print img_picto_common('','treemenu/linebottom.gif');
+    			if ($fulltree[$key]['level'] == $pos) {
+    			    print img_picto_common('','treemenu/branchbottom.gif');
+    			} else {
+    			    print img_picto_common('','treemenu/linebottom.gif');
+    			}
     		}
 		}
 		$pos++;
@@ -117,7 +128,9 @@ 
 {
     global $tree_recur_alreadyadded;
 
-    if ($rang == 0 && empty($donoresetalreadyloaded)) $tree_recur_alreadyadded=array();
+    if ($rang == 0 && empty($donoresetalreadyloaded)) {
+        $tree_recur_alreadyadded=array();
+    }
 
     if ($rang == 0)
 	{
@@ -174,8 +187,7 @@ 
 			    print '</td><td align="right">';
 			    print $tab[$x]['buttons'];
 			    print '</td></tr></table>';
-			}
-			else
+			} else
 			{
 			    print $tab[$x]['entry'];
 			}
@@ -184,8 +196,7 @@ 
 			// And now we search all its sons of lower level
 			tree_recur($tab, $tab[$x], $rang+1, 'iddivjstree', 0, $showfk);
 			print '</li>';
-		}
-		elseif (! empty($tab[$x]['rowid']) && $tab[$x]['fk_menu'] == -1 && $tab[$x]['fk_mainmenu'] == $pere['mainmenu'] && $tab[$x]['fk_leftmenu'] == $pere['leftmenu'])
+		} elseif (! empty($tab[$x]['rowid']) && $tab[$x]['fk_menu'] == -1 && $tab[$x]['fk_mainmenu'] == $pere['mainmenu'] && $tab[$x]['fk_leftmenu'] == $pere['leftmenu'])
 		{
 		    //print 'rang='.$rang.'-x='.$x." rowid=".$tab[$x]['rowid']." tab[x]['fk_leftmenu'] = ".$tab[$x]['fk_leftmenu']." leftmenu pere = ".$pere['leftmenu']."<br>\n";
 		    if (empty($ulprinted) && ! empty($pere['rowid']))
@@ -210,8 +221,7 @@ 
 			    print '</td><td align="right">';
 			    print $tab[$x]['buttons'];
 			    print '</td></tr></table>';
-			}
-			else
+			} else
 			{
 			    print $tab[$x]['entry'];
 			}
@@ -225,6 +235,8 @@ 
 	}
 	if (! empty($ulprinted) && ! empty($pere['rowid'])) { print '</ul>'."\n"; }
 
-    if ($rang == 0) print '</ul>';
-}
+    if ($rang == 0) {
+        print '</ul>';
+    }
+    }