Fix: Removed a lot of checkstyle warnings

Dolibarr · Mar 11, 2012 · b5fc41f · b5fc41f
1 parent aea9707
commit b5fc41f
Show file tree

Hide file tree

Showing 28 changed files with 58 additions and 56 deletions.
diff --git a/htdocs/comm/propal/tpl/linkedobjectblock.tpl.php b/htdocs/comm/propal/tpl/linkedobjectblock.tpl.php
@@ -27,7 +27,7 @@
 echo '<br>';
 print_titre($langs->trans('RelatedCommercialProposals'));
 ?>
-<table class="noborder" width="100%">
+<table class="noborder allwidth">
 <tr class="liste_titre">
 	<td><?php echo $langs->trans("Ref"); ?></td>
 	<td><?php echo $langs->trans('RefCustomer'); ?></td>

diff --git a/htdocs/commande/tpl/linkedobjectblock.tpl.php b/htdocs/commande/tpl/linkedobjectblock.tpl.php
@@ -29,7 +29,7 @@
 print_titre($langs->trans('RelatedOrders'));
 
 ?>
-<table class="noborder" width="100%">
+<table class="noborder allwidth">
 <tr class="liste_titre">
 	<td><?php echo $langs->trans("Ref"); ?></td>
 	<td align="center"><?php echo $langs->trans("Date"); ?></td>

diff --git a/htdocs/compta/facture/tpl/linkedobjectblock.tpl.php b/htdocs/compta/facture/tpl/linkedobjectblock.tpl.php
@@ -29,7 +29,7 @@
 if ($num > 1) print_titre($langs->trans("RelatedBills"));
 else print_titre($langs->trans("RelatedBill"));
 ?>
-<table class="noborder" width="100%">
+<table class="noborder allwidth">
 <tr class="liste_titre">
 	<td><?php echo $langs->trans("Ref"); ?></td>
 	<td align="center"><?php echo $langs->trans("Date"); ?></td>

diff --git a/htdocs/contact/canvas/default/tpl/contactcard_create.tpl.php b/htdocs/contact/canvas/default/tpl/contactcard_create.tpl.php
@@ -36,7 +36,7 @@
 <input type="hidden" name="socid" value="<?php echo $this->control->tpl['company_id']; ?>">
 <?php } ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 	<td width="15%" class="fieldrequired"><?php echo $langs->trans("Lastname").' / '.$langs->trans("Label"); ?></td>

diff --git a/htdocs/contact/canvas/default/tpl/contactcard_edit.tpl.php b/htdocs/contact/canvas/default/tpl/contactcard_edit.tpl.php
@@ -44,7 +44,7 @@
 <input type="hidden" name="socid" value="<?php echo $this->control->tpl['company_id']; ?>">
 <?php } ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 	<td><?php echo $langs->trans("Ref"); ?></td>

diff --git a/htdocs/contact/canvas/default/tpl/contactcard_view.tpl.php b/htdocs/contact/canvas/default/tpl/contactcard_view.tpl.php
@@ -29,7 +29,7 @@
 <?php if ($this->control->tpl['action_create_user']) echo $this->control->tpl['action_create_user']; ?>
 <?php if ($this->control->tpl['action_delete']) echo $this->control->tpl['action_delete']; ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 	<td width="20%"><?php echo $langs->trans("Ref"); ?></td>

diff --git a/htdocs/contrat/tpl/linkedobjectblock.tpl.php b/htdocs/contrat/tpl/linkedobjectblock.tpl.php
@@ -28,7 +28,7 @@
 echo '<br>';
 print_titre($langs->trans('RelatedContracts'));
 ?>
-<table class="noborder" width="100%">
+<table class="noborder allwidth">
 <tr class="liste_titre">
 	<td><?php echo $langs->trans("Ref"); ?></td>
 	<td align="center"><?php echo $langs->trans("Date"); ?></td>

diff --git a/htdocs/fourn/class/fournisseur.class.php b/htdocs/fourn/class/fournisseur.class.php
@@ -89,7 +89,7 @@ function getNbOfOrders()
 	function nbOfProductRefs()
 	{
 		global $conf;
-		
+
 		$sql = "SELECT count(pfp.rowid) as nb";
 		$sql.= " FROM ".MAIN_DB_PREFIX."product_fournisseur_price as pfp";
 		$sql.= " WHERE pfp.entity = ".$conf->entity;
@@ -213,18 +213,6 @@ function ListArray()
 		return $arr;
 	}
 
-	/**
-	 *	Return a link on thirdparty (with picto)
-	 *
-	 *	@param		int		$withpicto		Add picto into link (0=No picto, 1=Include picto with link, 2=Picto only)
-	 *	@param		string	$option			Target of link ('', 'customer', 'prospect', 'supplier')
-	 *	@param		int		$maxlen			Max length of text
-	 *	@return		string					String with URL
-	 */
-	function getNomUrl($withpicto=0,$option='supplier',$maxlen=0)
-	{
-		return parent::getNomUrl($withpicto,$option,$maxlen);
-	}
 }
 
 ?>
diff --git a/htdocs/fourn/commande/tpl/linkedobjectblock.tpl.php b/htdocs/fourn/commande/tpl/linkedobjectblock.tpl.php
@@ -28,7 +28,7 @@
 echo '<br>';
 print_titre($langs->trans('RelatedOrders'));
 ?>
-<table class="noborder" width="100%">
+<table class="noborder allwidth">
 <tr class="liste_titre">
 	<td><?php echo $langs->trans("Ref"); ?></td>
 	<td align="center"><?php echo $langs->trans("Date"); ?></td>

diff --git a/htdocs/fourn/facture/tpl/linkedobjectblock.tpl.php b/htdocs/fourn/facture/tpl/linkedobjectblock.tpl.php
@@ -29,7 +29,7 @@
 if ($num > 1) print_titre($langs->trans("RelatedBills"));
 else print_titre($langs->trans("RelatedBill"));
 ?>
-<table class="noborder" width="100%">
+<table class="noborder allwidth">
 <tr class="liste_titre">
 	<td><?php echo $langs->trans("Ref"); ?></td>
 	<td align="center"><?php echo $langs->trans("Date"); ?></td>

diff --git a/htdocs/product/canvas/product/tpl/card_create.tpl.php b/htdocs/product/canvas/product/tpl/card_create.tpl.php
@@ -29,7 +29,7 @@
 <input type="hidden" name="type" value="0">
 <input type="hidden" name="canvas" value="<?php echo $canvas; ?>">
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 <td class="fieldrequired" width="20%"><?php echo $langs->trans("Ref"); ?></td>
@@ -100,7 +100,7 @@
 
 <?php if (! $conf->global->PRODUIT_MULTIPRICES) { ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr><td><?php echo $langs->trans("SellingPrice"); ?></td>
 <td><input name="price" size="10" value="<?php echo $this->control->tpl['price']; ?>">

diff --git a/htdocs/product/canvas/product/tpl/card_edit.tpl.php b/htdocs/product/canvas/product/tpl/card_edit.tpl.php
@@ -32,7 +32,7 @@
 <input type="hidden" name="canvas" value="<?php echo $this->control->tpl['canvas']; ?>">
 
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 <td class="fieldrequired" width="20%"><?php echo $langs->trans("Ref"); ?></td>

diff --git a/htdocs/product/canvas/product/tpl/card_view.tpl.php b/htdocs/product/canvas/product/tpl/card_view.tpl.php
@@ -22,7 +22,7 @@
 
 <?php dol_htmloutput_errors($this->control->tpl['error'],$this->control->tpl['errors']); ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 <td width="15%"><?php echo $langs->trans("Ref"); ?></td>

diff --git a/htdocs/product/canvas/product/tpl/list.tpl.php b/htdocs/product/canvas/product/tpl/list.tpl.php
@@ -19,7 +19,7 @@
 
 <!-- BEGIN PHP TEMPLATE -->
 
-<table width="100%" border="0" class="notopnoleftnoright" style="margin-bottom: 2px;">
+<table class="notopnoleftnoright allwidth" style="margin-bottom: 2px;">
 <tr>
 	<td class="nobordernopadding" width="40" align="left" valign="middle">
 		<?php echo $title_picto; ?>
@@ -38,7 +38,7 @@
 <input type="hidden" name="canvas" value="default">
 <input type="hidden" name="type" value="0">
 
-<table class="liste" width="100%">
+<table class="liste allwidth">
 
 <!-- FIELDS TITLE -->
 
@@ -59,9 +59,9 @@
   				<td class="liste_titre" align="<?php echo $field['align']; ?>"><?php echo $field['title']; ?></td>
 	<?php } } } ?>
 </tr>
- 
+
  <!-- FIELDS SEARCH -->
- 
+
 <tr class="liste_titre">
 	<?php
  	$num = count($fieldlist);
@@ -83,7 +83,7 @@
 
 <?php
 $var=true;
-foreach($datas as $line) { 
+foreach($datas as $line) {
 	$var=!$var;	?>
 	<tr <?php echo $bc[$var]; ?>>
    		<?php

diff --git a/htdocs/product/canvas/service/tpl/card_create.tpl.php b/htdocs/product/canvas/service/tpl/card_create.tpl.php
@@ -29,7 +29,7 @@
 <input type="hidden" name="type" value="1">
 <input type="hidden" name="canvas" value="<?php echo $canvas; ?>">
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 <td class="fieldrequired" width="20%"><?php echo $langs->trans("Ref"); ?></td>
@@ -73,7 +73,7 @@
 
 <?php if (! $conf->global->PRODUIT_MULTIPRICES) { ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr><td><?php echo $langs->trans("SellingPrice"); ?></td>
 <td><input name="price" size="10" value="<?php echo $this->object->tpl['price']; ?>">

diff --git a/htdocs/product/canvas/service/tpl/card_edit.tpl.php b/htdocs/product/canvas/service/tpl/card_edit.tpl.php
@@ -32,7 +32,7 @@
 <input type="hidden" name="canvas" value="<?php echo $this->object->tpl['canvas']; ?>">
 
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 <td class="fieldrequired" width="20%"><?php echo $langs->trans("Ref"); ?></td>

diff --git a/htdocs/product/canvas/service/tpl/card_view.tpl.php b/htdocs/product/canvas/service/tpl/card_view.tpl.php
@@ -22,7 +22,7 @@
 
 <?php dol_htmloutput_errors($this->control->tpl['error'],$this->control->tpl['errors']); ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 <td width="15%"><?php echo $langs->trans("Ref"); ?></td>

diff --git a/htdocs/product/canvas/service/tpl/list.tpl.php b/htdocs/product/canvas/service/tpl/list.tpl.php
@@ -19,7 +19,7 @@
 
 <!-- BEGIN PHP TEMPLATE -->
 
-<table width="100%" border="0" class="notopnoleftnoright" style="margin-bottom: 2px;">
+<table class="notopnoleftnoright allwidth" style="margin-bottom: 2px;">
 <tr>
 	<td class="nobordernopadding" width="40" align="left" valign="middle">
 		<?php echo $title_picto; ?>
@@ -38,7 +38,7 @@
 <input type="hidden" name="canvas" value="service">
 <input type="hidden" name="type" value="1">
 
-<table class="liste" width="100%">
+<table class="liste allwidth">
 
 <!-- FIELDS TITLE -->
 
@@ -59,9 +59,9 @@
   				<td class="liste_titre" align="<?php echo $field['align']; ?>"><?php echo $field['title']; ?></td>
 	<?php } } } ?>
 </tr>
- 
+
  <!-- FIELDS SEARCH -->
- 
+
 <tr class="liste_titre">
 	<?php
  	$num = count($fieldlist);
@@ -83,7 +83,7 @@
 
 <?php
 $var=True;
-foreach($datas as $line) { 
+foreach($datas as $line) {
 	$var=!$var;	?>
 	<tr <?php echo $bc[$var]; ?>>
    		<?php

diff --git a/htdocs/societe/canvas/company/tpl/card_create.tpl.php b/htdocs/societe/canvas/company/tpl/card_create.tpl.php
@@ -47,7 +47,7 @@
 <input type="hidden" name="code_auto" value="1">
 <?php } ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 	<td><span class="fieldrequired"><?php echo $langs->trans('ThirdPartyName'); ?></span></td>

diff --git a/htdocs/societe/canvas/company/tpl/card_edit.tpl.php b/htdocs/societe/canvas/company/tpl/card_edit.tpl.php
@@ -40,7 +40,7 @@
 <input type="hidden" name="code_auto" value="1">
 <?php } ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 	<td><span class="fieldrequired"><?php echo $langs->trans('ThirdPartyName'); ?></span></td>

diff --git a/htdocs/societe/canvas/company/tpl/card_view.tpl.php b/htdocs/societe/canvas/company/tpl/card_view.tpl.php
@@ -33,7 +33,7 @@
 <?php if ($this->control->tpl['action_delete']) echo $this->control->tpl['action_delete']; ?>
 <?php if ($this->control->tpl['js_checkVatPopup']) echo $this->control->tpl['js_checkVatPopup']; ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 	<td width="20%"><?php echo $langs->trans('ThirdPartyName'); ?></td>
@@ -173,7 +173,7 @@
 
 <tr>
 	<td>
-	<table width="100%" class="nobordernopadding">
+	<table class="nobordernopadding allwidth">
 		<tr>
 			<td><?php echo $langs->trans('RIB'); ?></td>
 			<td align="right">
@@ -191,7 +191,7 @@
 
 <tr>
 	<td>
-	<table width="100%" class="nobordernopadding">
+	<table class="nobordernopadding allwidth">
 		<tr>
 			<td><?php echo $langs->trans('ParentCompany'); ?></td>
 			<td align="right">
@@ -209,7 +209,7 @@
 
 <tr>
 	<td>
-	<table width="100%" class="nobordernopadding">
+	<table class="nobordernopadding allwidth">
 		<tr>
 			<td><?php echo $langs->trans('SalesRepresentatives'); ?></td>
 			<td align="right">
@@ -256,8 +256,8 @@
 
 <br>
 
-<table width="100%"><tr><td valign="top" width="50%">
-<a name="builddoc"></a>
+<table class="allwidth"><tr><td valign="top" width="50%">
+<div id="builddoc"></div>
 
 <?php
 /*

diff --git a/htdocs/societe/canvas/individual/tpl/card_create.tpl.php b/htdocs/societe/canvas/individual/tpl/card_create.tpl.php
@@ -47,7 +47,7 @@
 <input type="hidden" name="code_auto" value="1">
 <?php } ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 	<td><span class="fieldrequired"><?php echo $langs->trans('LastName'); ?></span></td>

diff --git a/htdocs/societe/canvas/individual/tpl/card_edit.tpl.php b/htdocs/societe/canvas/individual/tpl/card_edit.tpl.php
@@ -37,7 +37,7 @@
 <input type="hidden" name="code_auto" value="1">
 <?php } ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 	<td><span class="fieldrequired"><?php echo $langs->trans('Name'); ?></span></td>

diff --git a/htdocs/societe/canvas/individual/tpl/card_view.tpl.php b/htdocs/societe/canvas/individual/tpl/card_view.tpl.php
@@ -32,7 +32,7 @@
 <?php if ($this->control->tpl['error']) echo $this->control->tpl['error']; ?>
 <?php if ($this->control->tpl['action_delete']) echo $this->control->tpl['action_delete']; ?>
 
-<table class="border" width="100%">
+<table class="border allwidth">
 
 <tr>
 	<td width="20%"><?php echo $langs->trans('Name'); ?></td>
@@ -132,7 +132,7 @@
 
 <tr>
 	<td>
-	<table width="100%" class="nobordernopadding">
+	<table class="nobordernopadding allwidth">
 		<tr>
 			<td><?php echo $langs->trans('RIB'); ?></td>
 			<td align="right">
@@ -150,7 +150,7 @@
 
 <tr>
 	<td>
-	<table width="100%" class="nobordernopadding">
+	<table class="nobordernopadding allwidth">
 		<tr>
 			<td><?php echo $langs->trans('SalesRepresentatives'); ?></td>
 			<td align="right">
@@ -197,9 +197,8 @@
 
 <br>
 
-<table width="100%"><tr><td valign="top" width="50%">
-<a name="builddoc"></a>
-
+<table class="allwidth"><tr><td valign="top" width="50%">
+<div id="builddoc"></div>
 <?php
 /*
  * Documents generes

diff --git a/htdocs/theme/auguria/style.css.php b/htdocs/theme/auguria/style.css.php
@@ -1065,6 +1065,10 @@
 /* Tables                                                                         */
 /* ============================================================================== */
 
+.allwidth {
+	width: 100%;
+}
+
 #undertopmenu {
     margin-top: 4px;
 }