DbRowSimple::insert() - Code Metrics - Inspection of "DBAL-feat-41a50.74: Revamped Entity class along wi..." - supernova-ws/SuperNova - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — work-fleets ( 7aaf68...e155c0 )

by SuperNova.WS

created 2016-07-27 20:26 UTC

DbRowSimple::insert() B

↳ Parent: DbRowSimple

Complexity

Conditions	4
Paths	6

Size

Total Lines	23
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	4
Bugs	0	Features	0

Metric	Value
cc	4
eloc	12
c	4
b	0
f	0
nc	6
nop	1
dl	0
loc	23
ccs	0
cts	16
cp	0
crap	20
rs	8.7972

<?php

class DbRowSimple {

  /**
   * @param Entity $object
   * @param mixed  $rowId
   *
   * @return Entity
   */
  public function getById($object, $rowId) {
    $stmt = classSupernova::$gc->query

      ->setIdField($object->getIdFieldName())
      ->field('*')
      ->from($object->getTableName())
      ->where($object->getIdFieldName() . ' = "' . $rowId . '"');

    $object->importDbRow($stmt->selectRow());

    return $object;
  }

  /**
   * @param Entity $object
   */
  public function deleteById($object) {
    $db = classSupernova::$gc->db;

    $db->doquery("DELETE FROM `{{" . $object->getTableName() . "}}` WHERE `{$object->getIdFieldName()}` = '{$object->dbId}' LIMIT 1;");
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

    return $db->db_affected_rows();
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
  }

  /**
   * @param Entity $object
   */
  public function insert($object) {
    $db = classSupernova::$gc->db;

    $query = array();
    foreach ($object->exportDbRow(false) as $fieldName => $fieldValue) {
      // TODO: MORE type detection
      if(!is_numeric($fieldValue)) {
        $fieldValue = "'" . $db->db_escape($fieldValue) . "'";
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
      }
      $query[] = "`{$fieldName}` = {$fieldValue}";
    }

    $query = implode(',', $query);
    if (empty($query)) {
      // TODO Exceptiion
      return 0;
    }

    $db->doquery("INSERT INTO `{{" . $object->getTableName() . "}}` SET " . $query);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

    // TODO Exceptiion if db_insert_id() is empty
    return $object->dbId = $db->db_insert_id();
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
  }

}


Push — work-fleets ( 7aaf68...e155c0 )

DbRowSimple::insert() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1			<?php
2
3			class DbRowSimple {
4
5			/**
6			* @param Entity $object
7			* @param mixed $rowId
8			*
9			* @return Entity
10			*/
11			public function getById($object, $rowId) {
12			$stmt = classSupernova::$gc->query
			0 ignored issues – show Bug introduced 2016-07-25 19:14 UTC by Report Bug Copy Issue Report The method `setIdField` cannot be called on `\classSupernova::$gc->query` (of type `callable`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
13			->setIdField($object->getIdFieldName())
14			->field('*')
15			->from($object->getTableName())
16			->where($object->getIdFieldName() . ' = "' . $rowId . '"');
17
18			$object->importDbRow($stmt->selectRow());
19
20			return $object;
21			}
22
23			/**
24			* @param Entity $object
25			*/
26			public function deleteById($object) {
27			$db = classSupernova::$gc->db;
28
29			$db->doquery("DELETE FROM `{{" . $object->getTableName() . "}}` WHERE `{$object->getIdFieldName()}` = '{$object->dbId}' LIMIT 1;");
			0 ignored issues – show Bug introduced 2016-07-26 16:51 UTC by Report Bug Copy Issue Report The method `doquery` does only exist in `db_mysql`, but not in `Closure`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
30
31			return $db->db_affected_rows();
			0 ignored issues – show Bug introduced 2016-07-26 16:51 UTC by Report Bug Copy Issue Report The method `db_affected_rows` does only exist in `db_mysql`, but not in `Closure`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
32			}
33
34			/**
35			* @param Entity $object
36			*/
37			public function insert($object) {
38			$db = classSupernova::$gc->db;
39
40			$query = array();
41			foreach ($object->exportDbRow(false) as $fieldName => $fieldValue) {
42			// TODO: MORE type detection
43			if(!is_numeric($fieldValue)) {
44			$fieldValue = "'" . $db->db_escape($fieldValue) . "'";
			0 ignored issues – show Bug introduced 2016-07-26 16:51 UTC by Report Bug Copy Issue Report The method `db_escape` does only exist in `db_mysql`, but not in `Closure`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
45			}
46			$query[] = "`{$fieldName}` = {$fieldValue}";
47			}
48
49			$query = implode(',', $query);
50			if (empty($query)) {
51			// TODO Exceptiion
52			return 0;
53			}
54
55			$db->doquery("INSERT INTO `{{" . $object->getTableName() . "}}` SET " . $query);
			0 ignored issues – show Bug introduced 2016-07-26 16:51 UTC by Report Bug Copy Issue Report The method `doquery` does only exist in `db_mysql`, but not in `Closure`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
56
57			// TODO Exceptiion if db_insert_id() is empty
58			return $object->dbId = $db->db_insert_id();
			0 ignored issues – show Bug introduced 2016-07-26 16:51 UTC by Report Bug Copy Issue Report The method `db_insert_id` does only exist in `db_mysql`, but not in `Closure`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
59			}
60
61			}
62

supernova-ws / SuperNova

Push — work-fleets ( 7aaf68...e155c0 )

DbRowSimple::insert() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like