stable
Clone or download
part of story #13463 Split PATCH docman_items_type/id route and add new routes for updating Why this change: PUT metadata will enable users to update all item metadata, the hardcoded one are not returned in representation because we want to handle them separately (description on two line, title, date at the beginning of of metadata...). However our implementation should not impact other people who wants to use REST route and they should be displayed in metadata section. How to test: docman_item/id now returns all metadata in representation check overview of a document: harcoded metadata (title, description ...) are not displayed twice (be careful description is only displayed when it is not empty) Change-Id: I9d748927abe36c913ecec6259d966edf7f78b2f7
Modified Files
| Name | ||||
|---|---|---|---|---|
| M | plugins/docman/include/Docman_MetadataFactory.class.php | +0 | −29 | Go to diff View file |
| M | plugins/docman/include/REST/v1/Metadata/MetadataRepresentation.php | +12 | −3 | Go to diff View file |
| M | plugins/docman/include/REST/v1/Metadata/MetadataRepresentationBuilder.php | +4 | −3 | Go to diff View file |
| M | plugins/docman/phpunit/REST/v1/ItemRepresentationBuilderTest.php | +2 | −1 | Go to diff View file |
| M | plugins/docman/phpunit/REST/v1/ItemRepresentationCollectionBuilderTest.php | +8 | −4 | Go to diff View file |
| M | plugins/docman/phpunit/REST/v1/MetadataRepresentationBuilderTest.php | +5 | −3 | Go to diff View file |
| M | plugins/docman/tests/rest/Docman/DocmanItemsTest.php | +41 | −61 | Go to diff View file |
| M | plugins/document/scripts/document/app.spec.js | +1 | −0 | Go to diff View file |
| A | plugins/document/scripts/document/components/Folder/QuickLook/QuickLookDocumentMetadata.spec.js | +136 | −0 | Go to diff View file |
| M | plugins/document/scripts/document/components/Folder/QuickLook/QuickLookDocumentMetadata.vue | +38 | −8 | Go to diff View file |
| M | plugins/document/scripts/document/helpers/date-formatter.js | +6 | −3 | Go to diff View file |