stable
Clone or download
Read-only
The option is not anymore hidden behind a feature flag. In legacy docman UI, user get an error message whenever they tries to access the update properties page (check is done both on Frontend and Backend). In document UI, user don't have access to update properties modal (and REST API guards the update): * In quicklook * In folder content rows * In current folder dropdown * In embedded view dropdown Check is also done in SOAP API and Webdave. For example a file can be updated with a new version in webdav, but we cannot change it's name. Other "update" features are still available: lock, move, delete, new version, … Part of story #24980: forbid modifications of document properties by WRITERS Change-Id: Iadbaa90e33376f77e8e4acd8718c843faa55deb5
Modified Files
| Name | ||||
|---|---|---|---|---|
| M | plugins/docman/include/Docman_Actions.class.php | +8 | −1 | Go to diff View file |
| M | plugins/docman/include/Docman_Controller.class.php | +9 | −2 | Go to diff View file |
| M | plugins/docman/include/Docman_PermissionsManager.class.php | +29 | −14 | Go to diff View file |
| M | plugins/docman/include/REST/v1/DocmanEmbeddedFilesResource.php | +8 | −1 | Go to diff View file |
| M | plugins/docman/include/REST/v1/DocmanEmptyDocumentsResource.php | +9 | −2 | Go to diff View file |
| M | plugins/docman/include/REST/v1/DocmanFilesResource.php | +8 | −1 | Go to diff View file |
| M | plugins/docman/include/REST/v1/DocmanFoldersResource.php | +8 | −1 | Go to diff View file |
| M | plugins/docman/include/REST/v1/DocmanLinksResource.php | +8 | −1 | Go to diff View file |
| M | plugins/docman/include/REST/v1/DocmanWikiResource.php | +8 | −1 | Go to diff View file |
| M | plugins/docman/include/REST/v1/SearchResource.php | +1 | −1 | Go to diff View file |
| A | plugins/docman/include/Settings/ForbidUpdatePropertiesSettings.php | +41 | −0 | Go to diff View file |
| A | plugins/docman/include/Settings/ForbidWritersToUpdateDAOSettings.php | +32 | −0 | Go to diff View file |
| A | plugins/docman/include/Settings/ITellIfWritersAreAllowedToUpdateProperties.php | +28 | −0 | Go to diff View file |
| M | plugins/docman/include/Settings/SettingsDAO.php | +1 | −1 | Go to diff View file |
| M | plugins/docman/include/soap.php | +7 | −1 | Go to diff View file |
| M | plugins/docman/include/view/Docman_View_Admin_Permissions.class.php | +0 | −1 | Go to diff View file |
| M | plugins/docman/templates/admin/permissions-addendum.mustache | +1 | −2 | Go to diff View file |
| M | plugins/docman/tests/soap/DocmanTest.php | +37 | −18 | Go to diff View file |
| M | plugins/docman/tests/unit/Docman_SOAPActionsTest.php | +62 | −1 | Go to diff View file |
| M | plugins/docman/tests/unit/PermissionManager/Docman_PermissionsManagerTest.php | +93 | −50 | Go to diff View file |
| A | plugins/docman/tests/unit/Settings/ForbidUpdatePropertiesSettingsTest.php | +52 | −0 | Go to diff View file |
| M | plugins/document/include/Tree/DocumentTreeController.php | +5 | −0 | Go to diff View file |
| M | plugins/document/include/Tree/DocumentTreePresenter.php | +6 | −0 | Go to diff View file |
| M | plugins/document/include/documentPlugin.php | +4 | −1 | Go to diff View file |
| M | plugins/document/scripts/document/components/Folder/DropDown/DropDownCurrentFolder.test.ts | +43 | −19 | Go to diff View file |
| M | plugins/document/scripts/document/components/Folder/DropDown/DropDownCurrentFolder.vue | +12 | −1 | Go to diff View file |
| M | plugins/document/scripts/document/components/Folder/DropDown/DropDownDisplayedEmbedded.test.ts | +41 | −17 | Go to diff View file |
| M | plugins/document/scripts/document/components/Folder/DropDown/DropDownDisplayedEmbedded.vue | +12 | −1 | Go to diff View file |
| M | plugins/document/scripts/document/components/Folder/DropDown/DropDownMenuTreeView.test.ts | +105 | −37 | Go to diff View file |
| M | plugins/document/scripts/document/components/Folder/DropDown/DropDownMenuTreeView.vue | +31 | −29 | Go to diff View file |
| M | plugins/document/scripts/document/components/Folder/DropDown/DropDownQuickLook.test.ts | +91 | −25 | Go to diff View file |
| M | plugins/document/scripts/document/components/Folder/DropDown/DropDownQuickLook.vue | +31 | −29 | Go to diff View file |
| A | plugins/document/scripts/document/helpers/can-update-properties-helper.test.ts | +76 | −0 | Go to diff View file |
| A | plugins/document/scripts/document/helpers/can-update-properties-helper.ts | +28 | −0 | Go to diff View file |
| M | plugins/document/scripts/document/index.js | +2 | −0 | Go to diff View file |
| M | plugins/document/scripts/document/store/configuration/index.ts | +1 | −0 | Go to diff View file |
| M | plugins/document/templates/document-tree.mustache | +1 | −0 | Go to diff View file |
| M | plugins/webdav/include/ServerBuilder.php | +1 | −0 | Go to diff View file |