Google Groups Home
Help | Sign in
r4366 - in branches/0.11/lib/Doctrine: . Query
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
  1 message - Collapse all
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
no-re...@phpdoctrine.org  
View profile
 More options 16 May, 21:40
From: no-re...@phpdoctrine.org
Date: Fri, 16 May 2008 21:40:56 +0100
Local: Fri 16 May 2008 21:40
Subject: r4366 - in branches/0.11/lib/Doctrine: . Query
Author: romanb
Date: 2008-05-16 21:40:54 +0100 (Fri, 16 May 2008)
New Revision: 4366

Modified:
   branches/0.11/lib/Doctrine/Connection.php
   branches/0.11/lib/Doctrine/Hydrator.php
   branches/0.11/lib/Doctrine/Query/JoinCondition.php
Log:
Fixed #917.

Modified: branches/0.11/lib/Doctrine/Connection.php
===================================================================
--- branches/0.11/lib/Doctrine/Connection.php   2008-05-16 19:09:48 UTC (rev 4365)
+++ branches/0.11/lib/Doctrine/Connection.php   2008-05-16 20:40:54 UTC (rev 4366)
@@ -452,13 +452,13 @@

         if (extension_loaded('pdo')) {
             if (in_array($e[0], PDO::getAvailableDrivers())) {
-               try {
-                    $this->dbh = new PDO($this->options['dsn'], $this->options['username'],
-                                     $this->options['password'], $this->options['other']);
-                                    
-                    $this->dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
-               } catch (PDOException $e) {
-                       throw new Doctrine_Connection_Exception('PDO Connection Error: ' . $e->getMessage());
+               try {
+                    $this->dbh = new PDO($this->options['dsn'], $this->options['username'],
+                                     $this->options['password'], $this->options['other']);
+                                    
+                    $this->dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
+               } catch (PDOException $e) {
+                       throw new Doctrine_Connection_Exception('PDO Connection Error: ' . $e->getMessage());
                }
                 $found = true;
             }

Modified: branches/0.11/lib/Doctrine/Hydrator.php
===================================================================
--- branches/0.11/lib/Doctrine/Hydrator.php     2008-05-16 19:09:48 UTC (rev 4365)
+++ branches/0.11/lib/Doctrine/Hydrator.php     2008-05-16 20:40:54 UTC (rev 4366)
@@ -283,6 +283,9 @@
         foreach ($data as $key => $value) {
             // Parse each column name only once. Cache the results.
             if ( ! isset($cache[$key])) {
+                // check ignored names. fastest solution for now. if we get more we'll start
+                // to introduce a list.
+                if ($key == 'DOCTRINE_ROWNUM') continue;
                 $e = explode('__', $key);
                 $last = strtolower(array_pop($e));
                 $cache[$key]['dqlAlias'] = $this->_tableAliases[strtolower(implode('__', $e))];

Modified: branches/0.11/lib/Doctrine/Query/JoinCondition.php
===================================================================
--- branches/0.11/lib/Doctrine/Query/JoinCondition.php  2008-05-16 19:09:48 UTC (rev 4365)
+++ branches/0.11/lib/Doctrine/Query/JoinCondition.php  2008-05-16 20:40:54 UTC (rev 4366)
@@ -35,7 +35,7 @@
     public function load($condition)
     {
         $condition = trim($condition);
-
+        
         $e = $this->_tokenizer->sqlExplode($condition);

         if (count($e) > 2) {


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »

Create a group - Google Groups - Google Home - Terms of Service - Privacy Policy
©2008 Google