From: Chuck Hagenbuch <chuck@horde.org>
Date: Sun, 11 Jan 2009 05:30:05 +0000 (-0500)
Subject: rdo bugfixes
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=e039db0a427f87691d87430dc157a527e4f26994;p=horde.git

rdo bugfixes
---

diff --git a/framework/Rdo/lib/Horde/Rdo/Base.php b/framework/Rdo/lib/Horde/Rdo/Base.php
index 305f4c3de..ccc8e7d1b 100644
--- a/framework/Rdo/lib/Horde/Rdo/Base.php
+++ b/framework/Rdo/lib/Horde/Rdo/Base.php
@@ -118,7 +118,7 @@ abstract class Horde_Rdo_Base implements IteratorAggregate {
                 $query = $this->_fillPlaceholders($rel['query']);
                 $this->_fields[$field] = $m->findOne($query);
             } else {
-                $this->_fields[$field] = $m->find($this->{$rel['foreignKey']});
+                $this->_fields[$field] = $m->findOne($this->{$rel['foreignKey']});
             }
             break;
 
diff --git a/framework/Rdo/lib/Horde/Rdo/Mapper.php b/framework/Rdo/lib/Horde/Rdo/Mapper.php
index dd9963aac..942d530da 100644
--- a/framework/Rdo/lib/Horde/Rdo/Mapper.php
+++ b/framework/Rdo/lib/Horde/Rdo/Mapper.php
@@ -508,7 +508,7 @@ abstract class Horde_Rdo_Mapper implements Countable
         if (is_null($arg)) {
             $query = null;
         } elseif (is_scalar($arg)) {
-            $query = array($this->tableDefinition->getPrimaryKey() => $arg);
+            $query = array((string)$this->tableDefinition->getPrimaryKey() => $arg);
         } else {
             $query = $arg;
         }
diff --git a/framework/Rdo/lib/Horde/Rdo/Query.php b/framework/Rdo/lib/Horde/Rdo/Query.php
index 558778c1b..8108a5c55 100644
--- a/framework/Rdo/lib/Horde/Rdo/Query.php
+++ b/framework/Rdo/lib/Horde/Rdo/Query.php
@@ -382,13 +382,11 @@ class Horde_Rdo_Query
             $sql .= ' ORDER BY';
             foreach ($this->sortby as $sort) {
                 if (strpos($sort, '@') !== false) {
-                    /* @TODO parse these placeholders out, or drop them */
-                    list($field, $direction) = $sort;
-                    list($rel, $field) = explode('@', $field);
+                    list($rel, $field) = explode('@', $sort);
                     if (!isset($this->relationships[$rel])) {
                         continue;
                     }
-                    $sql .= ' ' . $this->relationships[$rel]['table'] . '.' . $field . ' ' . $direction . ',';
+                    $sql .= ' ' . $this->relationships[$rel]['table'] . '.' . $field . ',';
                 } else {
                     $sql .= " $sort,";
                 }