voku\db\ActiveRecord

Complexity

Total Complexity

107

Size/Duplication

Total Lines	912
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

88.99%

Importance

Changes

0

33 Methods

Rating	Name	Duplication	Size	Complexity
A	getPrimaryKeyName()	0	4	1
A	getPrimaryKey()	0	9	2
A	getTable()	0	4	1
A	reset()	0	7	1
A	resetDirty()	0	6	1
A	setDb()	0	4	1
B	fetch()	0	24	2
A	fetchAll()	0	19	1
A	delete()	0	13	1
A	setPrimaryKeyName()	0	6	1
A	setTable()	0	4	1
B	update()	0	29	4
B	insert()	0	36	4
A	execute()	0	8	2
B	query()	0	27	4
C	getRelation()	0	69	22
C	_buildSqlCallback()	0	32	8
A	_buildSql()	0	9	1
C	__call()	0	38	8
B	wrap()	0	21	5
A	_filterParam()	0	13	4
B	addCondition()	0	23	6
A	join()	0	14	2
A	_addExpression()	0	8	3
A	_addCondition()	0	14	2
A	getDirty()	0	4	1
A	isNewDataAreDirty()	0	4	1
A	setNewDataAreDirty()	0	4	1
B	__set()	0	28	6
A	__unset()	0	14	4
A	group()	0	6	1
A	order()	0	6	1
A	__get()	0	16	4

<?php

namespace voku\db;

use Arrayy\Arrayy;
use voku\db\exceptions\ActiveRecordException;

/**
 * A simple implement of active record via mysqli + php.
 *
 * @method $this select(string $dbProperty)
 * @method $this eq(string $dbProperty, string | null $value = null)
 * @method $this from(string $table)
 * @method $this where(string $where)
 * @method $this having(string $having)
 * @method $this limit(int $start, int | null $end = null)
 *
 * @method $this equal(string $dbProperty, string $value)
 * @method $this notEqual(string $dbProperty, string $value)
 * @method $this ne(string $dbProperty, string $value)
 * @method $this greaterThan(string $dbProperty, int $value)
 * @method $this gt(string $dbProperty, int $value)
 * @method $this lessThan(string $dbProperty, int $value)
 * @method $this lt(string $dbProperty, int $value)
 * @method $this greaterThanOrEqual(string $dbProperty, int $value)
 * @method $this ge(string $dbProperty, int $value)
 * @method $this gte(string $dbProperty, int $value)
 * @method $this lessThanOrEqual(string $dbProperty, int $value)
 * @method $this le(string $dbProperty, int $value)
 * @method $this lte(string $dbProperty, int $value)
 * @method $this between(string $dbProperty, array $value)
 * @method $this like(string $dbProperty, string $value)
 * @method $this in(string $dbProperty, array $value)
 * @method $this notIn(string $dbProperty, array $value)
 * @method $this isnull(string $dbProperty)
 * @method $this isNotNull(string $dbProperty)
 * @method $this notNull(string $dbProperty)
 */
abstract class ActiveRecord extends Arrayy
{
  /**
   * @var DB static property to connect database.
   */
  protected static $db;

  /**
   * @var array maping the function name and the operator, to build Expressions in WHERE condition.
   *
   * user can call it like this:
   * <pre>
   *   $user->isnotnull()->eq('id', 1);
   * </pre>
   *
   * will create Expressions can explain to SQL:
   * <pre>
   *   WHERE user.id IS NOT NULL AND user.id = :ph1
   * </pre>
   */
  protected static $operators = array(
      'equal'              => '=',
      'eq'                 => '=',
      'notequal'           => '<>',
      'ne'                 => '<>',
      'greaterthan'        => '>',
      'gt'                 => '>',
      'lessthan'           => '<',
      'lt'                 => '<',
      'greaterthanorequal' => '>=',
      'ge'                 => '>=',
      'gte'                => '>=',
      'lessthanorequal'    => '<=',
      'le'                 => '<=',
      'lte'                => '<=',
      'between'            => 'BETWEEN',
      'like'               => 'LIKE',
      'in'                 => 'IN',
      'notin'              => 'NOT IN',
      'isnull'             => 'IS NULL',
      'isnotnull'          => 'IS NOT NULL',
      'notnull'            => 'IS NOT NULL',
  );

  /**
   * @var array Part of SQL, maping the function name and the operator to build SQL Part.
   * <pre>call function like this:
   *      $user->order('id DESC', 'name ASC')->limit(2, 1);
   *  can explain to SQL:
   *      ORDER BY id DESC, name ASC LIMIT 2,1</pre>
   */
  protected $sqlParts = array(
      'select' => 'SELECT',
      'from'   => 'FROM',
      'set'    => 'SET',
      'where'  => 'WHERE',
      'group'  => 'GROUP BY',
      'having' => 'HAVING',
      'order'  => 'ORDER BY',
      'limit'  => 'LIMIT',
      'top'    => 'TOP',
  );

  /**
   * @var array Static property to stored the default Sql Expressions values.
   */
  protected $defaultSqlExpressions = array(
      'expressions' => array(),
      'wrap'        => false,
      'select'      => null,
      'insert'      => null,
      'update'      => null,
      'set'         => null,
      'delete'      => 'DELETE ',
      'join'        => null,
      'from'        => null,
      'values'      => null,
      'where'       => null,
      'having'      => null,
      'limit'       => null,
      'order'       => null,
      'group'       => null,
  );

  /**
   * @var array Stored the Expressions of the SQL.
   */
  protected $sqlExpressions = array();

  /**
   * @var string  The table name in database.
   */
  protected $table;

  /**
   * @var string  The primary key of this ActiveRecord, just suport single primary key.
   */
  protected $primaryKeyName = 'id';

  /**
   * @var array Stored the drity data of this object, when call "insert" or "update" function, will write this data
   *      into database.
   */
  protected $dirty = array();

  /**
   * @var bool
   */
  protected static $new_data_are_dirty = true;

  /**
   * @var array Stored the params will bind to SQL when call DB->query(),
   */
  protected $params = array();

  /**
   * @var ActiveRecordExpressions[] Stored the configure of the relation, or target of the relation.
   */
  protected $relations = array();

  /**
   * @var int The count of bind params, using this count and const "PREFIX" (:ph) to generate place holder in SQL.
   */
  private static $count = 0;

  const BELONGS_TO = 'belongs_to';
  const HAS_MANY   = 'has_many';
  const HAS_ONE    = 'has_one';

  const PREFIX = ':active_record';

  /**
   * @return string
   */
  public function getPrimaryKeyName()
  {
    return $this->primaryKeyName;
  }

  /**
   * @return mixed|null
   */
  public function getPrimaryKey()
  {
    $id = $this->{$this->primaryKeyName};
    if ($id) {
      return $id;
    }

    return null;
  }

  /**
   * @return string
   */
  public function getTable()
  {
    return $this->table;
  }

  /**
   * Function to reset the $params and $sqlExpressions.
   *
   * @return $this
   */
  public function reset()
  {
    $this->params = array();
    $this->sqlExpressions = array();

    return $this;
  }

  /**
   * Reset the dirty data.
   *
   * @return $this
   */
  public function resetDirty()
  {
    $this->array = array();

    return $this;
  }

  /**
   * set the DB connection.
   *
   * @param DB $db
   */
  public static function setDb($db)
  {
    self::$db = $db;
  }

  /**
   * function to find one record and assign in to current object.
   *
   * @param int $id If call this function using this param, will find record by using this id. If not set, just find
   *                the first record in database.
   *
   * @return bool|ActiveRecord if find record, assign in to current object and return it, otherwise return "false".
   */
  public function fetch($id = null)
  {
    if ($id) {

The expression $id of type integer|null is loosely compared to true; this is ambiguous if the integer can be zero. You might want to explicitly use !== null instead.

      $this->reset()->eq($this->primaryKeyName, $id);
    }

    return self::query(
        $this->limit(1)->_buildSql(

The call to limit() misses a required argument $|.

            array(
                'select',
                'from',
                'join',
                'where',
                'group',
                'having',
                'order',
                'limit',
            )
        ),
        $this->params,
        $this->reset(),
        true
    );
  }

  /**
   * Function to find all records in database.
   *
   * @return array return array of ActiveRecord
   */
  public function fetchAll()
  {
    return self::query(
        $this->_buildSql(
            array(
                'select',
                'from',
                'join',
                'where',
                'group',
                'having',
                'order',
                'limit',
            )
        ),
        $this->params,
        $this->reset()
    );
  }

  /**
   * Function to delete current record in database.
   *
   * @return bool
   */
  public function delete()
  {
    return self::execute(
        $this->eq($this->primaryKeyName, $this->{$this->primaryKeyName})->_buildSql(
            array(
                'delete',
                'from',
                'where',
            )
        ),
        $this->params
    );
  }

  /**
   * @param string $primaryKeyName
   *
   * @return $this
   */
  public function setPrimaryKeyName($primaryKeyName)
  {
    $this->primaryKeyName = $primaryKeyName;

    return $this;
  }

  /**
   * @param string $table
   */
  public function setTable($table)
  {
    $this->table = $table;
  }

  /**
   * Function to build update SQL, and update current record in database, just write the dirty data into database.
   *
   * @return bool|int <p>
   *                  If update was successful, it will return the affected rows as int,
   *                  otherwise it will return false or true (if there are no dirty data).
   *                  </p>
   */
  public function update()
  {
    if (count($this->dirty) == 0) {
      return true;
    }

    foreach ($this->dirty as $field => $value) {
      $this->addCondition($field, '=', $value, ',', 'set');
    }

    $result = self::execute(
        $this->eq($this->primaryKeyName, $this->{$this->primaryKeyName})->_buildSql(
            array(
                'update',
                'set',
                'where',
            )
        ),
        $this->params
    );
    if ($result) {
      $this->resetDirty();
      $this->reset();

      return $result;
    }

    return false;
  }

  /**
   * Function to build insert SQL, and insert current record into database.
   *
   * @return bool|int <p>
   *                  If insert was successful, it will return the new id,
   *                  otherwise it will return false or true (if there are no dirty data).
   *                  </p>
   */
  public function insert()
  {
    if (!self::$db instanceof DB) {
      self::$db = DB::getInstance();
    }

    if (count($this->dirty) === 0) {
      return true;
    }

    $value = $this->_filterParam($this->dirty);
    $this->insert = new ActiveRecordExpressions(

The property insert does not exist on object<voku\db\ActiveRecord>. Since you implemented __set, maybe consider adding a @property annotation.

        array(
            'operator' => 'INSERT INTO ' . $this->table,
            'target'   => new ActiveRecordExpressionsWrap(array('target' => array_keys($this->dirty))),
        )
    );
    $this->values = new ActiveRecordExpressions(

The property values does not exist on object<voku\db\ActiveRecord>. Since you implemented __set, maybe consider adding a @property annotation.

        array(
            'operator' => 'VALUES',
            'target'   => new ActiveRecordExpressionsWrap(array('target' => $value)),
        )
    );

    $result = self::execute($this->_buildSql(array('insert', 'values')), $this->params);
    if ($result) {
      $this->{$this->primaryKeyName} = $result;

      $this->resetDirty();
      $this->reset();

      return $result;
    }

    return false;
  }

  /**
   * Helper function to exec sql.
   *
   * @param string $sql   The SQL need to be execute.
   * @param array  $param The param will be bind to the sql statement.
   *
   * @return bool|int|Result              <p>
   *                                      "Result" by "<b>SELECT</b>"-queries<br />
   *                                      "int" (insert_id) by "<b>INSERT / REPLACE</b>"-queries<br />
   *                                      "int" (affected_rows) by "<b>UPDATE / DELETE</b>"-queries<br />
   *                                      "true" by e.g. "DROP"-queries<br />
   *                                      "false" on error
   *                                      </p>
   */
  public static function execute($sql, array $param = array())
  {
    if (!self::$db instanceof DB) {
      self::$db = DB::getInstance();
    }

    return self::$db->query($sql, $param);
  }

  /**
   * Helper function to query one record by sql and params.
   *
   * @param string            $sql    <p>
   *                                  The SQL query to find the record.
   *                                  </p>
   * @param array             $param  <p>
   *                                  The param will be bind to the $sql query.
   *                                  </p>
   * @param ActiveRecord|null $obj    <p>
   *                                  The object, if find record in database, we will assign the attributes into
   *                                  this object.
   *                                  </p>
   * @param bool              $single <p>
   *                                  If set to true, we will find record and fetch in current object, otherwise
   *                                  will find all records.
   *                                  </p>
   *
   * @return bool|ActiveRecord|array
   */
  public static function query($sql, array $param = array(), $obj = null, $single = false)
  {
    $result = self::execute($sql, $param);

    if (!$result) {
      return false;
    }

    $useObject = is_object($obj);
    if ($useObject === true) {
      $called_class = $obj;
    } else {
      $called_class = get_called_class();
    }

    self::setNewDataAreDirty(false);

    if ($single) {
      $return = $result->fetchObject($called_class, null, true);

It seems like $called_class defined by $obj on line 467 can also be of type null; however, voku\db\Result::fetchObject() does only seem to accept string|object, maybe add an additional type check?

    } else {
      $return = $result->fetchAllObject($called_class, null);

It seems like $called_class defined by $obj on line 467 can also be of type null; however, voku\db\Result::fetchAllObject() does only seem to accept string|object, maybe add an additional type check?

    }

    self::setNewDataAreDirty(true);

    return $return;
  }

  /**
   * Helper function to get relation of this object.
   * There was three types of relations: {BELONGS_TO, HAS_ONE, HAS_MANY}
   *
   * @param string $name The name of the relation, the array key when defind the relation.
   *
   * @return mixed
   *
   * @throws ActiveRecordException <p>If the relation can't be found .</p>
   */
  protected function &getRelation($name)
  {
    $relation = $this->relations[$name];
    if (
        $relation instanceof self
        ||
        (
            is_array($relation)
            &&
            $relation[0] instanceof self
        )
    ) {
      return $relation;
    }

    /* @var $obj ActiveRecord */
    $obj = new $relation[1];

    $this->relations[$name] = $obj;
    if (isset($relation[3]) && is_array($relation[3])) {
      foreach ((array)$relation[3] as $func => $args) {
        call_user_func_array(array($obj, $func), (array)$args);
      }
    }

    $backref = isset($relation[4]) ? $relation[4] : '';
    if (
        (!$relation instanceof self)
        &&
        self::HAS_ONE == $relation[0]
    ) {

      $this->relations[$name] = $obj->eq($relation[2], $this->{$this->primaryKeyName})->fetch();

      if ($backref) {
        $this->relations[$name] && $backref && $obj->__set($backref, $this);
      }

    } elseif (
        is_array($relation)
        &&
        self::HAS_MANY == $relation[0]
    ) {

      $this->relations[$name] = $obj->eq($relation[2], $this->{$this->primaryKeyName})->fetchAll();
      if ($backref) {
        foreach ($this->relations[$name] as $o) {

The expression $this->relations[$name] of type object<voku\db\ActiveRecordExpressions>|object<voku\db\ActiveRecord>|boolean|array is not guaranteed to be traversable. How about adding an additional type check?

          $o->__set($backref, $this);
        }
      }

    } elseif (
        (!$relation instanceof self)
        &&
        self::BELONGS_TO == $relation[0]
    ) {

      $this->relations[$name] = $obj->eq($obj->primaryKeyName, $this->{$relation[2]})->fetch();

      if ($backref) {
        $this->relations[$name] && $backref && $obj->__set($backref, $this);
      }

    } else {
      throw new ActiveRecordException("Relation $name not found.");
    }

    return $this->relations[$name];
  }

  /**
   * Helper function to build SQL with sql parts.
   *
   * @param string       $n The SQL part will be build.
   * @param int          $i The index of $n in $sql array.
   * @param ActiveRecord $o The reference to $this
   */
  private function _buildSqlCallback(&$n, $i, $o)
  {
    if (
        'select' === $n
        &&
        null === $o->$n
    ) {

      $n = strtoupper($n) . ' ' . $o->table . '.*';

    } elseif (
        (
            'update' === $n
            ||
            'from' === $n
        )
        &&
        null === $o->$n
    ) {

      $n = strtoupper($n) . ' ' . $o->table;

    } elseif ('delete' === $n) {

      $n = strtoupper($n) . ' ';

    } else {

      $n = (null !== $o->$n) ? $o->$n . ' ' : '';

    }
  }

  /**
   * Helper function to build SQL with sql parts.
   *
   * @param array $sqls The SQL part will be build.
   *
   * @return string
   */
  protected function _buildSql($sqls = array())
  {
    array_walk($sqls, array($this, '_buildSqlCallback'), $this);

    // DEBUG
    // echo 'SQL: ', implode(' ', $sqls), "\n", 'PARAMS: ', implode(', ', $this->params), "\n";

    return implode(' ', $sqls);
  }

  /**
   * Magic function to make calls witch in function mapping stored in $operators and $sqlPart.
   * also can call function of DB object.
   *
   * @param string $name function name
   * @param array  $args The arguments of the function.
   *
   * @return $this|mixed Return the result of callback or the current object to make chain method calls.
   *
   * @throws ActiveRecordException
   */
  public function __call($name, $args)
  {
    if (!self::$db instanceof DB) {
      self::$db = DB::getInstance();
    }

    $nameTmp = strtolower($name);

    if (array_key_exists($nameTmp, self::$operators)) {

      $this->addCondition(
          $args[0],
          self::$operators[$nameTmp],
          isset($args[1]) ? $args[1] : null,
          (is_string(end($args)) && 'or' === strtolower(end($args))) ? 'OR' : 'AND'
      );

    } elseif (array_key_exists($nameTmp = str_replace('by', '', $nameTmp), $this->sqlParts)) {

      $this->$name = new ActiveRecordExpressions(
          array(
              'operator' => $this->sqlParts[$nameTmp],
              'target'   => implode(', ', $args),
          )
      );

    } elseif (is_callable($callback = array(self::$db, $name))) {

      return call_user_func_array($callback, $args);

    } else {

      throw new ActiveRecordException("Method $name not exist.");

    }

    return $this;
  }

  /**
   * Make wrap when build the SQL expressions of WHERE.
   *
   * @param string $op If give this param will build one WrapExpressions include the stored expressions add into WHERE.
   *                   otherwise wil stored the expressions into array.
   *
   * @return $this
   */
  public function wrap($op = null)
  {
    if (1 === func_num_args()) {
      $this->wrap = false;

The property wrap does not exist on object<voku\db\ActiveRecord>. Since you implemented __set, maybe consider adding a @property annotation.

      if (is_array($this->expressions) && count($this->expressions) > 0) {

The property expressions does not seem to exist. Did you mean defaultSqlExpressions?

        $this->_addCondition(
            new ActiveRecordExpressionsWrap(
                array(
                    'delimiter' => ' ',
                    'target'    => $this->expressions,

The property expressions does not seem to exist. Did you mean defaultSqlExpressions?

                )
            ), 'or' === strtolower($op) ? 'OR' : 'AND'
        );
      }
      $this->expressions = array();

The property expressions does not seem to exist. Did you mean defaultSqlExpressions?

    } else {
      $this->wrap = true;

The property wrap does not exist on object<voku\db\ActiveRecord>. Since you implemented __set, maybe consider adding a @property annotation.

    }

    return $this;
  }

  /**
   * Helper function to build place holder when make SQL expressions.
   *
   * @param mixed $value The value will bind to SQL, just store it in $this->params.
   *
   * @return mixed $value
   */
  protected function _filterParam($value)
  {
    if (is_array($value)) {
      foreach ($value as $key => $val) {
        $this->params[$value[$key] = self::PREFIX . ++self::$count] = $val;
      }
    } elseif (is_string($value)) {
      $this->params[$ph = self::PREFIX . ++self::$count] = $value;
      $value = $ph;
    }

    return $value;
  }

  /**
   * Helper function to add condition into WHERE.
   * create the SQL Expressions.
   *
   * @param string $field The field name, the source of Expressions
   * @param string $operator
   * @param mixed  $value The target of the Expressions
   * @param string $op    The operator to concat this Expressions into WHERE or SET statement.
   * @param string $name  The Expression will contact to.
   */
  public function addCondition($field, $operator, $value, $op = 'AND', $name = 'where')
  {
    $value = $this->_filterParam($value);
    $exp = new ActiveRecordExpressions(
        array(
            'source'   => ('where' == $name ? $this->table . '.' : '') . $field,
            'operator' => $operator,
            'target'   => is_array($value)
                ? new ActiveRecordExpressionsWrap(
                    'between' === strtolower($operator)
                        ? array('target' => $value, 'start' => ' ', 'end' => ' ', 'delimiter' => ' AND ')
                        : array('target' => $value)
                ) : $value,
        )
    );
    if ($exp) {
      if (!$this->wrap) {

The property wrap does not exist on object<voku\db\ActiveRecord>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->_addCondition($exp, $op, $name);
      } else {
        $this->_addExpression($exp, $op);
      }
    }
  }

  /**
   * helper function to add condition into JOIN.
   * create the SQL Expressions.
   *
   * @param string $table The join table name
   * @param string $on    The condition of ON
   * @param string $type  The join type, like "LEFT", "INNER", "OUTER"
   *
   * @return $this
   */
  public function join($table, $on, $type = 'LEFT')
  {
    $this->join = new ActiveRecordExpressions(

The property join does not exist on object<voku\db\ActiveRecord>. Since you implemented __set, maybe consider adding a @property annotation.

        array(
            'source'   => $this->join ?: '',

The property join does not exist on object<voku\db\ActiveRecord>. Since you implemented __get, maybe consider adding a @property annotation.

            'operator' => $type . ' JOIN',
            'target'   => new ActiveRecordExpressions(
                array('source' => $table, 'operator' => 'ON', 'target' => $on)
            ),
        )
    );

    return $this;
  }

  /**
   * helper function to make wrapper. Stored the expression in to array.
   *
   * @param ActiveRecordExpressions $exp      The expression will be stored.
   * @param string                  $operator The operator to concat this Expressions into WHERE statment.
   */
  protected function _addExpression($exp, $operator)
  {
    if (!is_array($this->expressions) || count($this->expressions) == 0) {

The property expressions does not seem to exist. Did you mean defaultSqlExpressions?

      $this->expressions = array($exp);

The property expressions does not seem to exist. Did you mean defaultSqlExpressions?

    } else {
      $this->expressions[] = new ActiveRecordExpressions(array('operator' => $operator, 'target' => $exp));

The property expressions does not seem to exist. Did you mean defaultSqlExpressions?

    }
  }

  /**
   * helper function to add condition into WHERE.
   *
   * @param ActiveRecordExpressions $exp      The expression will be concat into WHERE or SET statment.
   * @param string                  $operator the operator to concat this Expressions into WHERE or SET statment.
   * @param string                  $name     The Expression will contact to.
   */
  protected function _addCondition($exp, $operator, $name = 'where')
  {
    if (!$this->$name) {
      $this->$name = new ActiveRecordExpressions(array('operator' => strtoupper($name), 'target' => $exp));
    } else {
      $this->$name->target = new ActiveRecordExpressions(
          array(
              'source'   => $this->$name->target,
              'operator' => $operator,
              'target'   => $exp,
          )
      );
    }
  }

  /**
   * @return array
   */
  public function getDirty()
  {
    return $this->dirty;
  }

  /**
   * @return bool
   */
  public static function isNewDataAreDirty()
  {
    return self::$new_data_are_dirty;
  }

  /**
   * @param bool $bool
   */
  public static function setNewDataAreDirty($bool)
  {
    self::$new_data_are_dirty = (bool)$bool;
  }

  /**
   * Magic function to SET values of the current object.
   *
   * @param mixed $var
   * @param mixed $val
   */
  public function __set($var, $val)
  {
    if (
        array_key_exists($var, $this->sqlExpressions)
        ||
        array_key_exists($var, $this->defaultSqlExpressions)
    ) {

      $this->sqlExpressions[$var] = $val;

    } elseif (
        array_key_exists($var, $this->relations)
        &&
        $val instanceof self
    ) {

      $this->relations[$var] = $val;

    } else {

      $this->array[$var] = $val;

      if (self::$new_data_are_dirty === true) {
        $this->dirty[$var] = $val;
      }

    }
  }

  /**
   * Magic function to UNSET values of the current object.
   *
   * @param mixed $var
   */
  public function __unset($var)
  {
    if (array_key_exists($var, $this->sqlExpressions)) {
      unset($this->sqlExpressions[$var]);
    }

    if (isset($this->array[$var])) {
      unset($this->array[$var]);
    }

    if (isset($this->dirty[$var])) {
      unset($this->dirty[$var]);
    }
  }

  /**
   * Helper function for "GROUP BY".
   *
   * @param array $args
   * @param null  $dummy <p>only needed for API compatibility with Arrayy</p>
   *
   * @return $this
   */
  public function group($args, $dummy = null)
  {
    $this->__call('group', func_get_args());

    return $this;
  }

  /**
   * Helper function for "ORDER BY".
   *
   * @param $args ...
   *
   * @return $this
   */
  public function order($args)

The parameter $args is not used and could be removed.

  {
    $this->__call('order', func_get_args());

    return $this;
  }

  /**
   * Magic function to GET the values of current object.
   *
   * @param $var
   *
   * @return mixed
   */
  public function &__get($var)
  {
    if (isset($this->dirty[$var])) {
      return $this->dirty[$var];
    }

    if (array_key_exists($var, $this->sqlExpressions)) {
      return $this->sqlExpressions[$var];
    }

    if (array_key_exists($var, $this->relations)) {
      return $this->getRelation($var);

The expression $this->getRelation($var); of type voku\db\ActiveRecordExpressions|voku\db\ActiveRecord|array|boolean adds the type array to the return on line 945 which is incompatible with the return type of the parent method Arrayy\Arrayy::__get of type object|integer|double|string|null|boolean.

    }

    return parent::__get($var);
  }
}