#DC-840 fix and Test Case

Author: estahn

lib/Doctrine/Connection/Mssql.php

@@ -354,7 +354,7 @@ protected function replaceBoundParamsWithInlineValuesInQuery($query, array $para
                 $value = $this->quote($value);
             }
 
-            $re = '/([=,\(][^\\\']*)(\?)/iU';
+            $re = '/([=><,\(][^\\\']*)(\?)/iU';
 
             $query = preg_replace($re, "\\1 {$value}", $query, 1);
 

tests/Ticket/DC840TestCase.php

@@ -0,0 +1,84 @@
+<?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.doctrine-project.org>.
+ */
+
+/**
+ * Doctrine_Ticket_DC840_TestCase
+ *
+ * @package     Doctrine
+ * @author      Enrico Stahn <[email protected]>
+ * @license     http://www.opensource.org/licenses/lgpl-license.php LGPL
+ * @category    Object Relational Mapping
+ * @link        www.doctrine-project.org
+ * @since       1.0
+ * @version     $Revision$
+ */
+class Doctrine_Ticket_DC840_TestCase extends Doctrine_UnitTestCase 
+{
+    private $sqlStackCounter = 0;
+  
+    public function prepareTables()
+    {
+        $this->tables[] = 'Ticket_DC840_Model';
+        parent::prepareTables();
+    }
+
+    public function testInit()
+    {
+        $this->dbh = new Doctrine_Adapter_Mock('mssql');
+        $this->conn = Doctrine_Manager::getInstance()->openConnection($this->dbh, 'DC840');
+    }
+
+    public function testTest()
+    {
+        $q = Doctrine_Query::create()
+            ->select('id')
+            ->from('Ticket_DC840_Model')
+            ->andWhere('password = ?', 'abc')
+            ->andWhere('modified_at > ?', '2010-01-01')
+            ->andWhere('modified_at < ?', '2010-01-01')
+            ->andWhere('modified_at <> ?', '2010-01-01')
+            ->andWhere('modified_at <= ?', '2010-01-01')
+            ->andWhere('modified_at >= ?', '2010-01-01')
+        ;
+        $q->execute();
+         
+        $expected = "SELECT [t].[id] AS [t__id] FROM [ticket__d_c840__model] [t] WHERE ([t].[password] =  'abc' AND [t].[modified_at] >  '2010-01-01' AND [t].[modified_at] <  '2010-01-01' AND [t].[modified_at] <>  '2010-01-01' AND [t].[modified_at] <=  '2010-01-01' AND [t].[modified_at] >=  '2010-01-01')";
+        $sql = current(array_slice($this->dbh->getAll(), $this->sqlStackCounter++, 1));
+
+        $this->assertEqual($expected, $sql);
+    }
+}
+
+class Ticket_DC840_Model extends Doctrine_Record
+{
+    public function setTableDefinition()
+    {
+        $this->hasColumn('id', 'integer', null, array(
+            'type' => 'integer',
+            'unsigned' => false,
+            'primary' => true,
+            'autoincrement' => true,
+        ));
+        $this->hasColumn('modified_at', 'timestamp');
+        $this->hasColumn('username', 'string', 255);
+        $this->hasColumn('password', 'string', 255);
+    }
+}