Checkstyle

This commit is contained in:
Laurent Destailleur 2012-05-06 00:39:59 +02:00
parent 6599f2d755
commit ed3dfe2756
8 changed files with 30 additions and 34 deletions

View File

@ -83,7 +83,7 @@
<!-- Tweaks to metrics -->
<rule ref="Generic.Metrics.CyclomaticComplexity">
<properties>
<property name="complexity" value="40" />
<property name="complexity" value="80" />
<property name="absoluteComplexity" value="100" />
</properties>
</rule>

View File

@ -83,7 +83,7 @@
<!-- Tweaks to metrics -->
<rule ref="Generic.Metrics.CyclomaticComplexity">
<properties>
<property name="complexity" value="40" />
<property name="complexity" value="80" />
<property name="absoluteComplexity" value="100" />
</properties>
</rule>

View File

@ -27,8 +27,7 @@ require_once(DOL_DOCUMENT_ROOT ."/core/class/commonobject.class.php");
/**
* \class RemiseCheque
* \brief Classe permettant la gestion des remises de cheque
* Classe permettant la gestion des remises de cheque
*/
class RemiseCheque extends CommonObject
{
@ -119,7 +118,7 @@ class RemiseCheque extends CommonObject
* @param array $toRemise array with cheques to remise
* @return int <0 if KO, >0 if OK
*/
function create($user, $account_id, $limit=40, $toRemise)
function create($user, $account_id, $limit, $toRemise)
{
global $conf;

View File

@ -25,8 +25,7 @@
/**
* \class HookManager
* \brief Class to manage hooks
* Class to manage hooks
*/
class HookManager
{
@ -54,7 +53,6 @@ class HookManager
/**
* Init array this->hooks with instantiated action controlers.
*
* First, a hook is declared by a module by adding a constant MAIN_MODULE_MYMODULENAME_HOOKS
* with value 'nameofcontext1:nameofcontext2:...' into $this->const of module descriptor file.
* This make conf->hooks_modules loaded with an entry ('modulename'=>array(nameofcontext1,nameofcontext2,...))

View File

@ -203,16 +203,16 @@ class Notify
$mailfile = new CMailFile(
$subject,
$sendto,
$replyto,
$message,
array($file),
array($mimefile),
array($filename[count($filename)-1]),
'',
'',
0,
$msgishtml
$sendto,
$replyto,
$message,
array($file),
array($mimefile),
array($filename[count($filename)-1]),
'',
'',
0,
$msgishtml
);
if ($mailfile->sendfile())

View File

@ -34,8 +34,7 @@ require_once(DOL_DOCUMENT_ROOT.'/core/lib/pdf.lib.php');
/**
* \class pdf_oursin
* \brief Classe permettant de generer les factures au modele oursin
* Classe permettant de generer les factures au modele oursin
*/
class pdf_oursin extends ModelePDFFactures
{
@ -392,7 +391,7 @@ class pdf_oursin extends ModelePDFFactures
$invoice->fetch($obj->fk_facture_source);
$pdf->SetXY($tab3_posx, $tab3_top+$y );
$pdf->SetXY($tab3_posx, $tab3_top+$y);
$pdf->MultiCell(20, 3, dol_print_date($obj->datef,'day',false,$outputlangs,true), 0, 'L', 0);
$pdf->SetXY($tab3_posx+21, $tab3_top+$y);
$pdf->MultiCell(20, 3, price($obj->amount_ttc), 0, 'L', 0);
@ -401,7 +400,7 @@ class pdf_oursin extends ModelePDFFactures
$pdf->SetXY($tab3_posx+58, $tab3_top+$y);
$pdf->MultiCell(20, 3, $invoice->ref, 0, 'L', 0);
$pdf->line($tab3_posx, $tab3_top+$y+3, $tab3_posx+$tab3_width, $tab3_top+$y+3 );
$pdf->line($tab3_posx, $tab3_top+$y+3, $tab3_posx+$tab3_width, $tab3_top+$y+3);
$i++;
}
@ -429,7 +428,7 @@ class pdf_oursin extends ModelePDFFactures
$y+=3;
$row = $this->db->fetch_object($resql);
$pdf->SetXY($tab3_posx, $tab3_top+$y );
$pdf->SetXY($tab3_posx, $tab3_top+$y);
$pdf->MultiCell(20, 3, dol_print_date($this->db->jdate($row->date),'day',false,$outputlangs,true), 0, 'L', 0);
$pdf->SetXY($tab3_posx+21, $tab3_top+$y);
$pdf->MultiCell(20, 3, price($row->amount), 0, 'L', 0);
@ -440,7 +439,7 @@ class pdf_oursin extends ModelePDFFactures
$pdf->SetXY($tab3_posx+58, $tab3_top+$y);
$pdf->MultiCell(30, 3, $row->num, 0, 'L', 0);
$pdf->line($tab3_posx, $tab3_top+$y+3, $tab3_posx+$tab3_width, $tab3_top+$y+3 );
$pdf->line($tab3_posx, $tab3_top+$y+3, $tab3_posx+$tab3_width, $tab3_top+$y+3);
$i++;
}

View File

@ -31,8 +31,7 @@ require_once(DOL_DOCUMENT_ROOT."/core/class/commondocgenerator.class.php");
/**
* \class ModelePDFDeliveryOrder
* \brief Classe mere des modeles de bon de livraison
* Classe mere des modeles de bon de livraison
*/
abstract class ModelePDFDeliveryOrder extends CommonDocGenerator
{
@ -151,7 +150,7 @@ abstract class ModeleNumRefDeliveryOrder
*
* @param DoliDB $db objet base de donnee
* @param Object $object object delivery
* @param string $model force le modele a utiliser ('' to not force)
* @param string $modele force le modele a utiliser ('' to not force)
* @param Translate $outputlangs objet lang a utiliser pour traduction
* @return int 0 if KO, 1 if OK
*/

View File

@ -260,13 +260,14 @@ class pdf_standardlabel
/**
* Print dot line
*
* @param PDF &$pdf PDF
* @param int $x1 X1
* @param int $y1 Y1
* @param int $x2 X2
* @param int $y2 Y2
* @param int $epaisseur Epaisseur
* @param int $nbPointilles Nb pointilles
* @param PDF &$pdf PDF
* @param int $x1 X1
* @param int $y1 Y1
* @param int $x2 X2
* @param int $y2 Y2
* @param int $epaisseur Epaisseur
* @param int $nbPointilles Nb pointilles
* @return void
*/
function _Pointille(&$pdf,$x1=0,$y1=0,$x2=210,$y2=297,$epaisseur=1,$nbPointilles=15)
{