From 40219b2f0d2ed65a0880e6634faf0ff0b183ef2c Mon Sep 17 00:00:00 2001 From: Aleksander Machniak Date: Sun, 26 Feb 2017 10:32:28 +0100 Subject: [PATCH] Performance: Don't resolve tag members on every mail message preview This process might be expensive when using tags heavily and we already do this when we list messages. --- plugins/kolab_tags/lib/kolab_tags_engine.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/plugins/kolab_tags/lib/kolab_tags_engine.php b/plugins/kolab_tags/lib/kolab_tags_engine.php index b1158fc7..e55d6c8a 100644 --- a/plugins/kolab_tags/lib/kolab_tags_engine.php +++ b/plugins/kolab_tags/lib/kolab_tags_engine.php @@ -335,7 +335,7 @@ class kolab_tags_engine $tags = array(); foreach ($taglist as $tag) { - $tag = $this->parse_tag($tag, true); + $tag = $this->parse_tag($tag, true, false); if (in_array($uid, (array)$tag['uids'][$folder])) { unset($tag['uids']); $tags[] = $tag; @@ -434,7 +434,7 @@ class kolab_tags_engine /** * "Convert" tag object to simple array for use in javascript */ - private function parse_tag($tag, $list = false) + private function parse_tag($tag, $list = false, $force = true) { $result = array( 'uid' => $tag['uid'], @@ -443,7 +443,7 @@ class kolab_tags_engine ); if ($list) { - $result['uids'] = $this->get_tag_messages($tag); + $result['uids'] = $this->get_tag_messages($tag, $force); } return $result;