[Svn] r2627 - in trunk/roundcubemail/program: include steps/mail

[trac at roundcube.net]

Author: thomasb
Date: 2009-06-09 03:18:06 -0500 (Tue, 09 Jun 2009)
New Revision: 2627

Modified:
   trunk/roundcubemail/program/include/rcube_user.php
   trunk/roundcubemail/program/steps/mail/compose.inc
Log:
Change rcube_user::list_identities() to return an array instead of a sql-result

Modified: trunk/roundcubemail/program/include/rcube_user.php
===================================================================
--- trunk/roundcubemail/program/include/rcube_user.php	2009-06-09 08:08:00 UTC (rev 2626)
+++ trunk/roundcubemail/program/include/rcube_user.php	2009-06-09 08:18:06 UTC (rev 2627)
@@ -135,12 +135,12 @@
    * Get default identity of this user
    *
    * @param int  Identity ID. If empty, the default identity is returned
-   * @return array Hash array with all cols of the 
+   * @return array Hash array with all cols of the identity record
    */
   function get_identity($id = null)
   {
-    $sql_result = $this->list_identities($id ? sprintf('AND identity_id=%d', $id) : '');
-    return $this->db->fetch_assoc($sql_result);
+    $result = $this->list_identities($id ? sprintf('AND identity_id=%d', $id) : '');
+    return $result[0];
   }
   
   
@@ -160,7 +160,12 @@
        ORDER BY ".$this->db->quoteIdentifier('standard')." DESC, name ASC, identity_id ASC",
       $this->ID);
     
-    return $sql_result;
+    $result = array();
+    while ($sql_arr = $this->db->fetch_assoc($sql_result)) {
+      $result[] = $sql_arr;
+    }
+    
+    return $result;
   }
   
   

Modified: trunk/roundcubemail/program/steps/mail/compose.inc
===================================================================
--- trunk/roundcubemail/program/steps/mail/compose.inc	2009-06-09 08:08:00 UTC (rev 2626)
+++ trunk/roundcubemail/program/steps/mail/compose.inc	2009-06-09 08:18:06 UTC (rev 2627)
@@ -278,9 +278,9 @@
   }
 
   // get this user's identities
-  $sql_result = $USER->list_identities();
+  $user_identities = $USER->list_identities();
 
-  if ($DB->num_rows($sql_result))
+  if (count($user_identities))
   {
     $from_id = 0;
     $a_signatures = array();
@@ -288,7 +288,7 @@
     $field_attrib['onchange'] = JS_OBJECT_NAME.".change_identity(this)";
     $select_from = new html_select($field_attrib);
 
-    while ($sql_arr = $DB->fetch_assoc($sql_result))
+    foreach ($user_identities as $sql_arr)
     {
       $identity_id = $sql_arr['identity_id'];
       $select_from->add(format_email_recipient($sql_arr['email'], $sql_arr['name']), $identity_id);

_______________________________________________
http://lists.roundcube.net/mailman/listinfo/svn