Allow document actions to declare whether they are mutators (ie, they

change an aspect of the document), and whether changes should be available to users in admin mode even if the document is immutable. git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/trunk@5671 c91229c3-7414-0410-bfa2-8a42b809f60b

Allow document actions to declare whether they are mutators (ie, they
change an aspect of the document), and whether changes should be available to users in admin mode even if the document is immutable. git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/trunk@5671 c91229c3-7414-0410-bfa2-8a42b809f60b
Neil Blakey-Milner
1 parent 8626681e
Showing 3 changed files with 23 additions and 2 deletions
lib/actions/documentaction.inc.php
plugins/ktcore/KTDocumentActions.php
plugins/ktcore/document/Rename.php
@@ -42,6 +42,9 @@ class KTDocumentAction extends KTStandardDispatcher {
     var $sSection = "view_details";
+    var $_bMutator = false;
+    var $_bMutationAllowedByAdmin = true;
+
     function KTDocumentAction($oDocument = null, $oUser = null, $oPlugin = null) {
         $this->oDocument =& $oDocument;
         $this->oUser =& $oUser;
@@ -64,7 +67,18 @@ class KTDocumentAction extends KTStandardDispatcher {
     function _show() {
         if (is_null($this->_sShowPermission)) {
             return true;
-        }        
+        }
+        $oFolder = Folder::get($this->oDocument->getFolderId());
+
+        if ($this->_bMutator && $this->oDocument->getImmutable()) {
+            if ($this->_bMutationAllowedByAdmin === true) {
+                if (!KTBrowseUtil::inAdminMode($this->oUser, $oFolder)) {
+                    return false;
+                }
+            } else {
+                return false;
+            }
+        }
         if ($this->_bAdminAlwaysAvailable) {
             if (Permission::userIsSystemAdministrator($this->oUser->getId())) {
@@ -86,7 +100,6 @@ class KTDocumentAction extends KTStandardDispatcher {
 		if (($status == DELETED) || ($status == ARCHIVED)) { return false; } 
         if ($this->bAllowInAdminMode) {
             // check if this user is in admin mode
-            $oFolder = Folder::get($this->oDocument->getFolderId());
             if (KTBrowseUtil::inAdminMode($this->oUser, $oFolder)) {             
                 return true; 
             }		
@@ -96,6 +96,9 @@ class KTDocumentCheckOutAction extends KTDocumentAction {
     var $_sShowPermission = "ktcore.permissions.write";
+    var $_bMutator = true;
+    var $_bMutationAllowedByAdmin = false;
+
     function getDisplayName() {
         return _kt('Checkout');
     }
@@ -381,6 +384,7 @@ class KTDocumentEditAction extends KTDocumentAction {
     var $sName = 'ktcore.actions.document.edit';
     var $_sShowPermission = "ktcore.permissions.write";
+    var $_bMutator = true;
     function getInfo() {
         if ($this->oDocument->getIsCheckedOut()) {
@@ -404,6 +408,7 @@ class KTDocumentDeleteAction extends KTDocumentAction {
     var $sName = 'ktcore.actions.document.delete';
     var $_sShowPermission = "ktcore.permissions.delete";
+    var $_bMutator = true;
     function getDisplayName() {
         return _kt('Delete');
@@ -480,6 +485,7 @@ class KTDocumentMoveAction extends KTDocumentAction {
     var $sName = 'ktcore.actions.document.move';
     var $_sShowPermission = "ktcore.permissions.write";
+    var $_bMutator = true;
     function getDisplayName() {
         return _kt('Move');
@@ -867,6 +873,7 @@ class KTDocumentVersionHistoryAction extends KTDocumentAction {
 class KTDocumentArchiveAction extends KTDocumentAction {
     var $sName = 'ktcore.actions.document.archive';
     var $_sShowPermission = "ktcore.permissions.write";
+    var $_bMutator = true;
     function getDisplayName() {
         return _kt('Archive');
@@ -42,6 +42,7 @@ class KTDocumentRenameAction extends KTDocumentAction {
     var $sName = 'ktcore.actions.document.rename';
     var $_sShowPermission = "ktcore.permissions.write";
+    var $_bMutator = true;
     function getDisplayName() {
         return _kt('Rename');