Reorder these checks, cleaner.

horde · Dec 3, 2016 · aee9494 · aee9494
1 parent 975730c
commit aee9494
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/framework/ActiveSync/lib/Horde/ActiveSync/Imap/Adapter.php b/framework/ActiveSync/lib/Horde/ActiveSync/Imap/Adapter.php
@@ -348,9 +348,8 @@ public function getMessageChanges(
             $this->_procid,
             serialize($status))
         );
-        $current_modseq = $status[Horde_ActiveSync_Folder_Imap::HIGHESTMODSEQ];
 
-        // Sanity checks.
+        // UIDVALIDITY
         if ($folder->uidnext() != 0) {
             $folder->checkValidity($status);
         }
@@ -363,6 +362,7 @@ public function getMessageChanges(
         // otherwise corrupted. Normally this would be handled as a loss of
         // server state and handled by a complete resync, but a majority of
         // EAS clients do not properly handle the status codes that report this.
+        $current_modseq = $status[Horde_ActiveSync_Folder_Imap::HIGHESTMODSEQ];
         if ($modseq_corrupted = $folder->modseq() > $current_modseq) {
             $this->_logger->err(sprintf(
                 '[%s] IMAP Server error: Current HIGHESTMODSEQ is lower than previously reported.',