Commit fecce947 authored by zYne's avatar zYne

fixed the handling of hooked parameters

parent 84acea5d
......@@ -49,10 +49,6 @@ class Doctrine_Hook
'limit',
'offset'
);
/**
* @var array $params query parameters
*/
protected $params = array();
/**
* @var array $fieldParsers custom field parsers array
* keys as field names in the format componentAlias.FieldName
......@@ -143,12 +139,10 @@ class Doctrine_Hook
$parser->parse($alias, $column, $value);
$this->query->addWhere($parser->getCondition());
$this->params += $parser->getParams();
$this->query->addWhere($parser->getCondition(), $parser->getParams());
}
}
}
$this->params += $params;
return true;
}
......
......@@ -94,6 +94,16 @@ class Doctrine_Import_Mysql extends Doctrine_Import
}
return $result;
}
/**
* lists table foreign keys
*
* @param string $table database table name
* @return array
*/
public function listTableForeignKeys($table)
{
$sql = 'SHOW CREATE TABLE ' . $this->conn->quoteIdentifier($table, true);
}
/**
* lists table constraints
*
......@@ -102,7 +112,7 @@ class Doctrine_Import_Mysql extends Doctrine_Import
*/
public function listTableColumns($table)
{
$sql = 'DESCRIBE ' . $table;
$sql = 'DESCRIBE ' . $this->conn->quoteIdentifier($table, true);
$result = $this->conn->fetchAssoc($sql);
$description = array();
......
<?php
/*
* $Id$
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
* A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
* OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
* LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
* DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
* THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*
* This software consists of voluntary contributions made by many individuals
* and is licensed under the LGPL. For more information, see
* <http://www.phpdoctrine.com>.
*/
/**
* Doctrine_Hook_TestCase
*
* @package Doctrine
* @author Konsta Vesterinen <kvesteri@cc.hut.fi>
* @license http://www.opensource.org/licenses/lgpl-license.php LGPL
* @category Object Relational Mapping
* @link www.phpdoctrine.com
* @since 1.0
* @version $Revision$
*/
class Doctrine_Hook_TestCase extends Doctrine_UnitTestCase {
public function testWordLikeParserSupportsHyphens() {
$parser = new Doctrine_Hook_WordLike();
......@@ -51,8 +82,18 @@ class Doctrine_Hook_TestCase extends Doctrine_UnitTestCase {
$hook->hookWhere($a['where']);
$this->assertEqual($hook->getQuery()->getQuery(), 'SELECT e.id AS e__id, e.name AS e__name FROM entity e LEFT JOIN phonenumber p ON e.id = p.entity_id WHERE (e.name LIKE ? OR e.name LIKE ?) AND e.loginname LIKE ? AND (e.type = 0)');
$this->assertEqual($hook->getQuery()->getParams(), array('Jack%', 'Daniels%', 'TheMan%'));
}
public function testHookWhereSupportsIntegerTypes() {
$hook = new Doctrine_Hook('SELECT u.name FROM User u LEFT JOIN u.Phonenumber p');
$a['where'] = array('u.id' => 10000);
$hook->hookWhere($a['where']);
$this->assertEqual($hook->getQuery()->getQuery(), 'SELECT e.id AS e__id, e.name AS e__name FROM entity e LEFT JOIN phonenumber p ON e.id = p.entity_id WHERE e.id = ? AND (e.type = 0)');
$this->assertEqual($hook->getQuery()->getParams(), array(10000));
}
public function testHookWhereDoesntAcceptUnknownColumn() {
$hook = new Doctrine_Hook('SELECT u.name FROM User u LEFT JOIN u.Phonenumber p');
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment