diff --git a/SeedDMS_Core/Core/inc.ClassDocument.php b/SeedDMS_Core/Core/inc.ClassDocument.php index 3859b47eb..5bbd3078f 100644 --- a/SeedDMS_Core/Core/inc.ClassDocument.php +++ b/SeedDMS_Core/Core/inc.ClassDocument.php @@ -5050,7 +5050,14 @@ class SeedDMS_Core_DocumentContent extends SeedDMS_Core_Object { /* {{{ */ } /* If a user is added when the revision has already been startet, then - * put it in S_IN_REVISION otherwise in S_LOG_WAITING. + * put it into S_LOG_WAITING otherwise into S_LOG_SLEEPING. Attention, if a + * document content is in any other status but S_IN_REVISION, then it will + * end up in S_LOG_SLEEPING. As this method is also called by removeFromProcesses() + * when another user takes over the processes, it may happen that revisions + * of document contents in status e.g. S_OBSOLETE, S_EXPIRED will change its + * status from S_LOG_WAITING to S_LOG_SLEEPING. + * This could only be fixed if this method could set an initial revision status + * by possibly passing it as another parameter to the method. */ $st=$this->getStatus(); $queryStr = "INSERT INTO `tblDocumentRevisionLog` (`revisionID`, `status`, `comment`, `date`, `userID`) ".