From 29cf87d85d01292af3ea24fcd752a33582497192 Mon Sep 17 00:00:00 2001 From: Uwe Steinmann Date: Tue, 16 Mar 2021 19:13:37 +0100 Subject: [PATCH] fix SeedDMS_Core_Document::remove when version had its own attachments --- SeedDMS_Core/Core/inc.ClassDocument.php | 5 +++++ SeedDMS_Core/package.xml | 1 + 2 files changed, 6 insertions(+) diff --git a/SeedDMS_Core/Core/inc.ClassDocument.php b/SeedDMS_Core/Core/inc.ClassDocument.php index 4128d076b..fe5d46a5e 100644 --- a/SeedDMS_Core/Core/inc.ClassDocument.php +++ b/SeedDMS_Core/Core/inc.ClassDocument.php @@ -2362,6 +2362,11 @@ class SeedDMS_Core_Document extends SeedDMS_Core_Object { /* {{{ */ } // remove all document files + /* make sure to get all attachments because _removeContent() did remove + * the attachments of a version already and the list of attachments + * is still stored in _documentFiles + * */ + unset($this->_documentFiles); $res = $this->getDocumentFiles(); if (is_bool($res) && !$res) { $db->rollbackTransaction(); diff --git a/SeedDMS_Core/package.xml b/SeedDMS_Core/package.xml index fb059f999..d1e6826b3 100644 --- a/SeedDMS_Core/package.xml +++ b/SeedDMS_Core/package.xml @@ -31,6 +31,7 @@ - add parameter $incdisabled to SeedDMS_Core_Folder::getNotifyList() - do not validate value in SeedDMS_Core_Attribute::setValue(), it should have been done before - SeedDMS_Core_DMS::search() can search for last date of document status change +- fix SeedDMS_Core_Document::remove when version had its own attachments