SupplierBillSelect::__construct() - Code Metrics - Inspection of "display the count in the table" - laudeco/bbc-dolibarr-flightlog - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — feature/fixing_cost ( ef981e )

by Laurent

created 2020-02-05 13:11 UTC

SupplierBillSelect::__construct() A

↳ Parent: SupplierBillSelect

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	3
dl	0
loc	6
rs	10
c	0
b	0
f	0

<?php


namespace FlightLog\Http\Web\Form;


use flightlog\form\Select;

final class SupplierBillSelect extends Select
{
    /**
     * @var \DoliDB
     */
    private $db;

    /**
     * @param string $name
     * @param \DoliDB $db
     * @param array $options
     */
    public function __construct($name, \DoliDB $db, $options = [])
    {
        parent::__construct($name, $options);
        $this->db = $db;
        $this->buildOptions();
    }

    /**
     * Build the options of the select
     */
    private function buildOptions()
    {

        if ((bool) $this->getOption('show_empty')) {
            $this->addValueOption(-1, ' ');
        }

        $sql = 'SELECT f.rowid, f.ref as ref_supplier, f.total_ttc, society.nom';
        $sql.= ' FROM '.MAIN_DB_PREFIX.'facture_fourn as f';
        $sql.= ' INNER JOIN '.MAIN_DB_PREFIX.'societe as society ON society.rowid = f.fk_soc';
        $sql.= ' ORDER BY f.datec DESC';

        $resql = $this->db->query($sql);
        if ($resql) {
            $num = $this->db->num_rows($resql);
            $i = 0;
            if ($num) {
                while ($i < $num) {
                    $obj = $this->db->fetch_object($resql);

                    $this->addValueOption($obj->rowid, sprintf('(%s) %s (%s€)', $obj->ref_supplier, $obj->nom, $obj->total_ttc));
                    $i++;
                }
            }


        }
    }


}

1			<?php
2
3
4			namespace FlightLog\Http\Web\Form;
5
6
7			use flightlog\form\Select;
8
9			final class SupplierBillSelect extends Select
10			{
11			/**
12			* @var \DoliDB
13			*/
14			private $db;
15
16			/**
17			* @param string $name
18			* @param \DoliDB $db
19			* @param array $options
20			*/
21			public function __construct($name, \DoliDB $db, $options = [])
22			{
23			parent::__construct($name, $options);
24			$this->db = $db;
25			$this->buildOptions();
26			}
27
28			/**
29			* Build the options of the select
30			*/
31			private function buildOptions()
32			{
33
34			if ((bool) $this->getOption('show_empty')) {
35			$this->addValueOption(-1, ' ');
36			}
37
38			$sql = 'SELECT f.rowid, f.ref as ref_supplier, f.total_ttc, society.nom';
39			$sql.= ' FROM '.MAIN_DB_PREFIX.'facture_fourn as f';
40			$sql.= ' INNER JOIN '.MAIN_DB_PREFIX.'societe as society ON society.rowid = f.fk_soc';
41			$sql.= ' ORDER BY f.datec DESC';
42
43			$resql = $this->db->query($sql);
44			if ($resql) {
45			$num = $this->db->num_rows($resql);
46			$i = 0;
47			if ($num) {
48			while ($i < $num) {
49			$obj = $this->db->fetch_object($resql);
50
51			$this->addValueOption($obj->rowid, sprintf('(%s) %s (%s€)', $obj->ref_supplier, $obj->nom, $obj->total_ttc));
52			$i++;
53			}
54			}
55
56
57			}
58			}
59
60
61			}

laudeco / bbc-dolibarr-flightlog

Push — feature/fixing_cost ( ef981e )

SupplierBillSelect::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like