From 69b7803f25c35c2b1c0d48cd47ada4e214a0528e Mon Sep 17 00:00:00 2001 From: Thomas Bruederli Date: Thu, 6 Feb 2014 17:30:40 +0100 Subject: [PATCH] Adapt to kolab_storage_folder optimizations: don't cast resultset into array (#2828) --- plugins/calendar/drivers/kolab/kolab_calendar.php | 2 +- plugins/kolab_config/kolab_config.php | 2 +- plugins/tasklist/drivers/kolab/tasklist_kolab_driver.php | 8 ++++---- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/plugins/calendar/drivers/kolab/kolab_calendar.php b/plugins/calendar/drivers/kolab/kolab_calendar.php index 9427b1c7..2fe072ab 100644 --- a/plugins/calendar/drivers/kolab/kolab_calendar.php +++ b/plugins/calendar/drivers/kolab/kolab_calendar.php @@ -233,7 +233,7 @@ class kolab_calendar } $events = array(); - foreach ((array)$this->storage->select($query) as $record) { + foreach ($this->storage->select($query) as $record) { $event = $this->_to_rcube_event($record); $this->events[$event['id']] = $event; diff --git a/plugins/kolab_config/kolab_config.php b/plugins/kolab_config/kolab_config.php index e5f07ad5..d4a4753a 100644 --- a/plugins/kolab_config/kolab_config.php +++ b/plugins/kolab_config/kolab_config.php @@ -167,7 +167,7 @@ class kolab_config extends rcube_plugin if ($default && !$folder->default) continue; - foreach ((array)$folder->select($query) as $object) { + foreach ($folder->select($query) as $object) { if ($object['type'] == 'dictionary' && ($object['language'] == $lang || $object['language'] == 'XX')) { if (is_array($this->dicts[$lang])) $this->dicts[$lang]['e'] = array_merge((array)$this->dicts[$lang]['e'], $object['e']); diff --git a/plugins/tasklist/drivers/kolab/tasklist_kolab_driver.php b/plugins/tasklist/drivers/kolab/tasklist_kolab_driver.php index edb60e6d..99971f01 100644 --- a/plugins/tasklist/drivers/kolab/tasklist_kolab_driver.php +++ b/plugins/tasklist/drivers/kolab/tasklist_kolab_driver.php @@ -294,7 +294,7 @@ class tasklist_kolab_driver extends tasklist_driver $counts = array('all' => 0, 'flagged' => 0, 'today' => 0, 'tomorrow' => 0, 'overdue' => 0, 'nodate' => 0); foreach ($lists as $list_id) { $folder = $this->folders[$list_id]; - foreach ((array)$folder->select(array(array('tags','!~','x-complete'))) as $record) { + foreach ($folder->select(array(array('tags','!~','x-complete'))) as $record) { $rec = $this->_to_rcube_task($record); if ($rec['complete'] >= 1.0) // don't count complete tasks @@ -358,7 +358,7 @@ class tasklist_kolab_driver extends tasklist_driver foreach ($lists as $list_id) { $folder = $this->folders[$list_id]; - foreach ((array)$folder->select($query) as $record) { + foreach ($folder->select($query) as $record) { $task = $this->_to_rcube_task($record); $task['list'] = $list_id; @@ -421,7 +421,7 @@ class tasklist_kolab_driver extends tasklist_driver $query_ids = array(); foreach ($task_ids as $task_id) { $query = array(array('tags','=','x-parent:' . $task_id)); - foreach ((array)$folder->select($query) as $record) { + foreach ($folder->select($query) as $record) { // don't rely on kolab_storage_folder filtering if ($record['parent_id'] == $task_id) { $childs[] = $record['uid']; @@ -475,7 +475,7 @@ class tasklist_kolab_driver extends tasklist_driver continue; $folder = $this->folders[$lid]; - foreach ((array)$folder->select($query) as $record) { + foreach ($folder->select($query) as $record) { if (!$record['alarms']) // don't trust query :-) continue;