StatsModel::save() - Code Metrics - Inspection of "Optimize the loading of single sources" - joomla/statistics-server - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 17a370...d70e31 )

by Michael

created 2016-03-26 01:12 UTC

StatsModel::save() B

↳ Parent: StatsModel

Complexity

Conditions	2
Paths	2

Size

Total Lines	24
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	14
CRAP Score	2

Importance

Changes	5
Bugs	0	Features	1

Metric	Value
c	5
b	0
f	1
dl	0
loc	24
ccs	14
cts	14
cp	1
rs	8.9713
cc	2
eloc	13
nc	2
nop	1
crap	2

<?php

namespace Stats\Models;

use Joomla\Model\AbstractDatabaseModel;

/**
 * Statistics database model
 *
 * @since  1.0
 */
class StatsModel extends AbstractDatabaseModel
{
	/**
	 * Loads the statistics data from the database.
	 *
	 * @param   string  $column  A single column to filter on
	 *
	 * @return  \stdClass[]  Array of data objects.
	 *
	 * @since   1.0
	 * @throws  \InvalidArgumentException
	 */
	public function getItems($column = null)
	{
		$db = $this->getDb();

		// Validate the requested column is actually in the table
		if ($column !== null)
		{
			$columnList = $db->getTableColumns('#__jstats');

			if (!in_array($column, array_keys($columnList)))
			{
				throw new \InvalidArgumentException('An invalid data source was requested.', 404);
			}
		}
		else
		{
			$column = '*';
		}

		return $db->setQuery(
			$db->getQuery(true)
				->select($column)
				->from('#__jstats')
				->group('unique_id')
		)->loadObjectList();
	}

	/**
	 * Saves the given data.
	 *
	 * @param   \stdClass  $data  Data object to save.
	 *
	 * @return  void
	 *
	 * @since   1.0
	 */
	public function save($data)
	{
		$db = $this->getDb();

		// Set the modified date of the record
		$data->modified = (new \DateTime('now', new \DateTimeZone('UTC')))->format($db->getDateFormat());

		// Check if a row exists for this unique ID and update the existing record if so
		$recordExists = $db->setQuery(
			$db->getQuery(true)
				->select('unique_id')
				->from('#__jstats')
				->where('unique_id = ' . $db->quote($data->unique_id))
		)->loadResult();

		if ($recordExists)
		{
			$db->updateObject('#__jstats', $data, ['unique_id']);
		}
		else
		{
			$db->insertObject('#__jstats', $data, ['unique_id']);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		}
	}
}


1		<?php
2
3		namespace Stats\Models;
4
5		use Joomla\Model\AbstractDatabaseModel;
6
7		/**
8		* Statistics database model
9		*
10		* @since 1.0
11		*/
12		class StatsModel extends AbstractDatabaseModel
13		{
14		/**
15		* Loads the statistics data from the database.
16		*
17		* @param string $column A single column to filter on
18		*
19		* @return \stdClass[] Array of data objects.
20		*
21		* @since 1.0
22		* @throws \InvalidArgumentException
23		*/
24	3	public function getItems($column = null)
25		{
26	3	$db = $this->getDb();
27
28		// Validate the requested column is actually in the table
29	3	if ($column !== null)
30	3	{
31	2	$columnList = $db->getTableColumns('#__jstats');
32
33	2	if (!in_array($column, array_keys($columnList)))
34	2	{
35	1	throw new \InvalidArgumentException('An invalid data source was requested.', 404);
36		}
37	1	}
38		else
39		{
40	1	$column = '*';
41		}
42
43	2	return $db->setQuery(
44	2	$db->getQuery(true)
45	2	->select($column)
46	2	->from('#__jstats')
47	2	->group('unique_id')
48	2	)->loadObjectList();
49		}
50
51		/**
52		* Saves the given data.
53		*
54		* @param \stdClass $data Data object to save.
55		*
56		* @return void
57		*
58		* @since 1.0
59		*/
60	2	public function save($data)
61		{
62	2	$db = $this->getDb();
63
64		// Set the modified date of the record
65	2	$data->modified = (new \DateTime('now', new \DateTimeZone('UTC')))->format($db->getDateFormat());
66
67		// Check if a row exists for this unique ID and update the existing record if so
68	2	$recordExists = $db->setQuery(
69	2	$db->getQuery(true)
70	2	->select('unique_id')
71	2	->from('#__jstats')
72	2	->where('unique_id = ' . $db->quote($data->unique_id))
73	2	)->loadResult();
74
75		if ($recordExists)
76	2	{
77	1	$db->updateObject('#__jstats', $data, ['unique_id']);
78	1	}
79		else
80		{
81	1	$db->insertObject('#__jstats', $data, ['unique_id']);
		0 ignored issues – show Documentation introduced 2016-01-02 17:06 UTC by Report Bug Copy Issue Report `array('unique_id')` is of type `array<integer,string,{"0":"string"}>`, but the function expects a `string\|null`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
82		}
83	2	}
84		}
85

joomla / statistics-server

Push — master ( 17a370...d70e31 )

StatsModel::save() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like