Bug report: for Yii connector

Sorry couldn’t find where to report bugs. Google kept directing me to forum posts starting “Bug report” so here you are. In db_phpyii.php

	public function select($sql){
		$res = $this->connection->findAll();
		if (sizeof($res)){
			$name = get_class($this->connection);
			$temp = array();
			foreach ($res as $obj)
				$temp[]=&$obj->getAttributes();
		}
		return new ArrayQueryWrapper($temp);
	}

The initialisation of $temp should be moved to before the “if” statement. When the if evaluates to zero (false) the PHP reports:

Also passing by reference in PHP is deprecated and you get another warning for that. Method should be changed to:

Code should be

	public function select($sql){
		$res = $this->connection->findAll();
		$temp = array();
		if (sizeof($res)){
			$name = get_class($this->connection);
			foreach ($res as $obj)
				$temp[]=$obj->getAttributes();
		}
		return new ArrayQueryWrapper($temp);
	}

Yep, you are right.
The same fix will be added into the next build
( you can grab the update code from github - github.com/dhtmlx/connector-php )