absences_StatisticsXls::getRows() - Code Metrics - cantico/absences - Measure and Improve Code Quality continuously with Scrutinizer

absences_StatisticsXls::getRows() B
last analyzed 2017-12-22 09:39 UTC

↳ Parent: absences_StatisticsXls

Complexity

Conditions	3
Paths	2

Size

Total Lines	32
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	10
c	0
b	0
f	0
nc	2
nop	0
dl	0
loc	32
rs	8.8571

<?php
/************************************************************************
 * OVIDENTIA http://www.ovidentia.org                                   *
 ************************************************************************
 * Copyright (c) 2003 by CANTICO ( http://www.cantico.fr )              *
 *                                                                      *
 * This file is part of Ovidentia.                                      *
 *                                                                      *
 * Ovidentia is free software; you can redistribute it and/or modify    *
 * it under the terms of the GNU General Public License as published by *
 * the Free Software Foundation; either version 2, or (at your option)  *
 * any later version.													*
 *																		*
 * This program is distributed in the hope that it will be useful, but  *
 * WITHOUT ANY WARRANTY; without even the implied warranty of			*
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.					*
 * See the  GNU General Public License for more details.				*
 *																		*
 * You should have received a copy of the GNU General Public License	*
 * along with this program; if not, write to the Free Software			*
 * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307,*
 * USA.																	*
************************************************************************/


require_once dirname(__FILE__).'/xls.class.php';


/**
 * days by year and by right types
 *
 */
class absences_StatisticsXls extends absences_Xls
{
	/**
	 *
	 * @var array
	 */
	private $type_rows = null;

	/**
	 *
	 * @var Worksheet
	 */
	private $type_worksheet = null;


	/**
	 *
	 * @var array
	 */
	private $type_years = array();


	/**
	 *
	 * @var array
	 */
	private $filter = array();



	public function __construct(Array $filter)
	{
		parent::__construct(absences_translate('Statistics'));

		$this->filter = $filter;

		$this->year_format = $this->workbook->addFormat();
class MyClass { }

$x = new MyClass();
$x->foo = true;
		$this->year_format->setBold();

		$this->year_format->setPattern(1);

		$this->year_format->setFgColor(15);

		$this->year_format->setAlign('center');


		$this->addTypesWorksheet();


		$this->workbook->close();
	}


	/**
	 *
	 */
	private function getRows()
	{

		if (!isset($this->type_rows))
		{

			global $babDB;


			$res = $babDB->db_query('
				SELECT
					id,
					name
				FROM
					absences_types
				ORDER BY name
					');

			$row = 2;
			while ($arr = $babDB->db_fetch_assoc($res))
			{
				$this->type_rows[$arr['id']] = array(
					'name' 	=> $arr['name'],
					'row' 	=> $row++
				);
			}

		}


		return $this->type_rows;
	}


	/**
	 * Get row number by id type
	 */
	private function getRow($id_type)
	{
		$rows = $this->getRows();
		return $rows[$id_type]['row'];
	}




	private function getTypeWorksheet()
	{
		if (!isset($this->type_worksheet))
		{
			$this->type_worksheet = $this->workbook->addWorksheet(absences_translate('Vacation rights'));


			// create first column
			$this->type_worksheet->setColumn(0, 0, 30);

			$this->type_worksheet->writeString(1, 0, absences_translate('Vacation type')		, $this->header);
class MyClass { }

$x = new MyClass();
$x->foo = true;

			foreach($this->getRows() as $arr)
$collection = json_decode($data, true);
if ( ! is_array($collection)) {
    throw new \RuntimeException('$collection must be an array.');
}

foreach ($collection as $item) { /** ... */ }
			{
				$this->type_worksheet->writeString($arr['row'], 0, $arr['name']);

			}
		}

		return $this->type_worksheet;
	}



	private function prepareTypeData()
	{
		global $babDB;

		$years = array();
		$types = array();

		$organization = '';
		if(isset($this->filter['organization']) && $this->filter['organization']){
			$organization = 'AND p.id_organization='.$babDB->quote($this->filter['organization']);
		}

		$res = $babDB->db_query('
			SELECT
				t.id,
				YEAR(e.date_begin) year,
				SUM(ee.quantity) total,
				quantity_unit
			FROM
				absences_types t,
				absences_rights r,
				absences_entries_elem ee,
				absences_entries e,
				absences_personnel p
			WHERE r.id_type=t.id
			AND ee.id_right=r.id
			AND ee.id_entry=e.id 
		    AND e.status=\'Y\' 
 			AND p.id_user=e.id_user
			'.$organization.'
			GROUP BY year, t.id, r.quantity_unit
			ORDER BY year
				');

		while ($arr = $babDB->db_fetch_assoc($res))
		{
			$years[$arr['year']] = $arr['year'];
			$types[$arr['id']][$arr['year']][$arr['quantity_unit']] = $arr['total'];
		}


		return array($years, $types);
	}


	protected function addTypesWorksheet()
	{
		$worksheet = $this->getTypeWorksheet();
		list($years, $types) = $this->prepareTypeData();

		$col_index = array();

		// first an second row
		$col = 1;
		foreach($years as $y)
		{
			$worksheet->mergeCells(0, $col, 0, 1+$col);


			$worksheet->setColumn(0, 0, 30);

			$worksheet->writeString(0, $col, $y		, $this->year_format);

			$col_index[$y]['D'] = $col;
			$worksheet->writeString(1, $col++, absences_translate('days')		, $this->header);

			$col_index[$y]['H'] = $col;
			$worksheet->writeString(1, $col++, absences_translate('hours')		, $this->header);

		}


		foreach($types as $id_type => $yearly)
		{
			$row = $this->getRow($id_type);
			foreach($yearly as $year => $quantities)
			{
				foreach($quantities as $quantity_unit => $total)
				{
					$col = $col_index[$year][$quantity_unit];
					$worksheet->writeNumber($row, $col, $total, $this->quantity);
class MyClass { }

$x = new MyClass();
$x->foo = true;
				}
			}
		}
	}
}

1			<?php
2			/************************************************************************
3			* OVIDENTIA http://www.ovidentia.org *
4			************************************************************************
5			* Copyright (c) 2003 by CANTICO ( http://www.cantico.fr ) *
6			* *
7			* This file is part of Ovidentia. *
8			* *
9			* Ovidentia is free software; you can redistribute it and/or modify *
10			* it under the terms of the GNU General Public License as published by *
11			* the Free Software Foundation; either version 2, or (at your option) *
12			* any later version. *
13			* *
14			* This program is distributed in the hope that it will be useful, but *
15			* WITHOUT ANY WARRANTY; without even the implied warranty of *
16			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. *
17			* See the GNU General Public License for more details. *
18			* *
19			* You should have received a copy of the GNU General Public License *
20			* along with this program; if not, write to the Free Software *
21			* Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307,*
22			* USA. *
23			************************************************************************/
24
25
26			require_once dirname(__FILE__).'/xls.class.php';
27
28
29			/**
30			* days by year and by right types
31			*
32			*/
33			class absences_StatisticsXls extends absences_Xls
34			{
35			/**
36			*
37			* @var array
38			*/
39			private $type_rows = null;
40
41			/**
42			*
43			* @var Worksheet
44			*/
45			private $type_worksheet = null;
46
47
48			/**
49			*
50			* @var array
51			*/
52			private $type_years = array();
53
54
55			/**
56			*
57			* @var array
58			*/
59			private $filter = array();
60
61
62
63			public function __construct(Array $filter)
64			{
65			parent::__construct(absences_translate('Statistics'));
66
67			$this->filter = $filter;
68
69			$this->year_format = $this->workbook->addFormat();
			0 ignored issues – show Bug introduced 2015-11-30 13:10 UTC by Report Bug Copy Issue Report The property `year_format` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history... Deprecated Code introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `Workbook::addformat()` has been deprecated with message: Use add_format instead This method has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead. Loading history...
70			$this->year_format->setBold();
			0 ignored issues – show Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `setBold()` does not seem to exist on `object<Format>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
71			$this->year_format->setPattern(1);
			0 ignored issues – show Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `setPattern()` does not seem to exist on `object<Format>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
72			$this->year_format->setFgColor(15);
			0 ignored issues – show Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `setFgColor()` does not seem to exist on `object<Format>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
73			$this->year_format->setAlign('center');
			0 ignored issues – show Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `setAlign()` does not seem to exist on `object<Format>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
74
75			$this->addTypesWorksheet();
76
77
78			$this->workbook->close();
79			}
80
81
82			/**
83			*
84			*/
85			private function getRows()
86			{
87
88			if (!isset($this->type_rows))
89			{
90
91			global $babDB;
92
93
94			$res = $babDB->db_query('
95			SELECT
96			id,
97			name
98			FROM
99			absences_types
100			ORDER BY name
101			');
102
103			$row = 2;
104			while ($arr = $babDB->db_fetch_assoc($res))
105			{
106			$this->type_rows[$arr['id']] = array(
107			'name' => $arr['name'],
108			'row' => $row++
109			);
110			}
111
112			}
113
114
115			return $this->type_rows;
116			}
117
118
119			/**
120			* Get row number by id type
121			*/
122			private function getRow($id_type)
123			{
124			$rows = $this->getRows();
125			return $rows[$id_type]['row'];
126			}
127
128
129
130
131			private function getTypeWorksheet()
132			{
133			if (!isset($this->type_worksheet))
134			{
135			$this->type_worksheet = $this->workbook->addWorksheet(absences_translate('Vacation rights'));
			0 ignored issues – show Deprecated Code introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `Workbook::addworksheet()` has been deprecated with message: Use add_worksheet instead This method has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead. Loading history...
136
137			// create first column
138			$this->type_worksheet->setColumn(0, 0, 30);
			0 ignored issues – show Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `setColumn()` does not seem to exist on `object<Worksheet>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
139			$this->type_worksheet->writeString(1, 0, absences_translate('Vacation type') , $this->header);
			0 ignored issues – show Bug introduced 2015-11-30 13:10 UTC by Report Bug Copy Issue Report The property `header` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history... Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `writeString()` does not seem to exist on `object<Worksheet>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
140
141			foreach($this->getRows() as $arr)
			0 ignored issues – show Bug introduced 2015-11-30 13:10 UTC by Report Bug Copy Issue Report The expression `$this->getRows()` of type `null\|array` is not guaranteed to be traversable. How about adding an additional type check? There are different options of fixing this problem. If you want to be on the safe side, you can add an additional type-check: $collection = json_decode($data, true); if ( ! is_array($collection)) { throw new \RuntimeException('$collection must be an array.'); } foreach ($collection as $item) { /** ... / } If you are sure that the expression is traversable, you might want to add a doc comment cast* to improve IDE auto-completion and static analysis: /** @var array $collection / $collection = json_decode($data, true); foreach ($collection as $item) { /* .. / } Mark the issue as a false-positive*: Just hover the remove button, in the top-right corner of this issue for more options. Loading history...
142			{
143			$this->type_worksheet->writeString($arr['row'], 0, $arr['name']);
			0 ignored issues – show Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `writeString()` does not seem to exist on `object<Worksheet>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
144			}
145			}
146
147			return $this->type_worksheet;
148			}
149
150
151
152			private function prepareTypeData()
153			{
154			global $babDB;
155
156			$years = array();
157			$types = array();
158
159			$organization = '';
160			if(isset($this->filter['organization']) && $this->filter['organization']){
161			$organization = 'AND p.id_organization='.$babDB->quote($this->filter['organization']);
162			}
163
164			$res = $babDB->db_query('
165			SELECT
166			t.id,
167			YEAR(e.date_begin) year,
168			SUM(ee.quantity) total,
169			quantity_unit
170			FROM
171			absences_types t,
172			absences_rights r,
173			absences_entries_elem ee,
174			absences_entries e,
175			absences_personnel p
176			WHERE r.id_type=t.id
177			AND ee.id_right=r.id
178			AND ee.id_entry=e.id
179			AND e.status=\'Y\'
180			AND p.id_user=e.id_user
181			'.$organization.'
182			GROUP BY year, t.id, r.quantity_unit
183			ORDER BY year
184			');
185
186			while ($arr = $babDB->db_fetch_assoc($res))
187			{
188			$years[$arr['year']] = $arr['year'];
189			$types[$arr['id']][$arr['year']][$arr['quantity_unit']] = $arr['total'];
190			}
191
192
193			return array($years, $types);
194			}
195
196
197			protected function addTypesWorksheet()
198			{
199			$worksheet = $this->getTypeWorksheet();
200			list($years, $types) = $this->prepareTypeData();
201
202			$col_index = array();
203
204			// first an second row
205			$col = 1;
206			foreach($years as $y)
207			{
208			$worksheet->mergeCells(0, $col, 0, 1+$col);
			0 ignored issues – show Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `mergeCells()` does not seem to exist on `object<Worksheet>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
209
210			$worksheet->setColumn(0, 0, 30);
			0 ignored issues – show Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `setColumn()` does not seem to exist on `object<Worksheet>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
211			$worksheet->writeString(0, $col, $y , $this->year_format);
			0 ignored issues – show Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `writeString()` does not seem to exist on `object<Worksheet>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
212			$col_index[$y]['D'] = $col;
213			$worksheet->writeString(1, $col++, absences_translate('days') , $this->header);
			0 ignored issues – show Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `writeString()` does not seem to exist on `object<Worksheet>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
214			$col_index[$y]['H'] = $col;
215			$worksheet->writeString(1, $col++, absences_translate('hours') , $this->header);
			0 ignored issues – show Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `writeString()` does not seem to exist on `object<Worksheet>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
216			}
217
218
219			foreach($types as $id_type => $yearly)
220			{
221			$row = $this->getRow($id_type);
222			foreach($yearly as $year => $quantities)
223			{
224			foreach($quantities as $quantity_unit => $total)
225			{
226			$col = $col_index[$year][$quantity_unit];
227			$worksheet->writeNumber($row, $col, $total, $this->quantity);
			0 ignored issues – show Bug introduced 2015-11-30 13:10 UTC by Report Bug Copy Issue Report The property `quantity` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history... Bug introduced 2017-08-01 09:30 UTC by Report Bug Copy Issue Report The method `writeNumber()` does not seem to exist on `object<Worksheet>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
228			}
229			}
230			}
231			}
232			}

cantico / absences

absences_StatisticsXls::getRows() B last analyzed 2017-12-22 09:39 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

absences_StatisticsXls::getRows() B
last analyzed 2017-12-22 09:39 UTC