fix(Felamimail/Move): only log non-false iterate results

tine20 · Sep 20, 2021 · 772ab7f · 772ab7f
1 parent 300c534
commit 772ab7f
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/tine20/Felamimail/Controller/Message/Move.php b/tine20/Felamimail/Controller/Message/Move.php
@@ -77,14 +77,16 @@ public function moveMessages($_messages, $_targetFolder, $keepOriginalMessages =
         if ($_messages instanceof Tinebase_Model_Filter_FilterGroup) {
             $iterator = new Tinebase_Record_Iterator(array(
                 'iteratable' => $this,
-                'controller' => $this, 
-                'filter'     => $_messages,
-                'function'   => 'processMoveIteration',
+                'controller' => $this,
+                'filter' => $_messages,
+                'function' => 'processMoveIteration',
             ));
             $iterateResult = $iterator->iterate($targetFolder, $keepOriginalMessages, $checkCopyPreventionConfig);
-
-            if (Tinebase_Core::isLogLevel(Zend_Log::DEBUG)) Tinebase_Core::getLogger()->debug(__METHOD__ . '::' . __LINE__ 
-                . ' Moved/copied ' . $iterateResult['totalcount'] . ' message(s).');
+
+            if ($iterateResult && Tinebase_Core::isLogLevel(Zend_Log::DEBUG)) {
+                Tinebase_Core::getLogger()->debug(__METHOD__ . '::' . __LINE__
+                    . ' Moved/copied ' . $iterateResult['totalcount'] . ' message(s).');
+            }
 
             $result = (! empty($iterateResult['results'])) ? array_pop($iterateResult['results']) : new Tinebase_Record_RecordSet('Felamimail_Model_Folder');
         } else {