ACS-1201: Model integrity violation saving properties (#332)

- Fix the name property persistence in ContentModelFormProcessor to only save when the property value is actually changed. This prevents the FilenameFilteringInterceptor to be called when there are no changes to the file name.
This commit is contained in:
Ancuta Morarasu
2021-03-05 12:32:10 +02:00
committed by GitHub
parent 748272bcde
commit 2b03e2bbf0

View File

@@ -2,7 +2,7 @@
* #%L * #%L
* Alfresco Repository * Alfresco Repository
* %% * %%
* Copyright (C) 2005 - 2016 Alfresco Software Limited * Copyright (C) 2005 - 2021 Alfresco Software Limited
* %% * %%
* This file is part of the Alfresco software. * This file is part of the Alfresco software.
* If the software was purchased under a paid Alfresco license, the terms of * If the software was purchased under a paid Alfresco license, the terms of
@@ -634,10 +634,13 @@ public abstract class ContentModelFormProcessor<ItemType, PersistType> extends
{ {
try try
{ {
// if the name property changes the rename method of the file folder if (!fileInfo.getName().equals(fieldData.getValue()))
{
// if the name property changes, the rename method of the file folder
// service should be called rather than updating the property directly // service should be called rather than updating the property directly
this.fileFolderService.rename(nodeRef, (String) fieldData.getValue()); this.fileFolderService.rename(nodeRef, (String) fieldData.getValue());
} }
}
catch (FileExistsException fee) catch (FileExistsException fee)
{ {
// ALF-6739: Notification should be more user friendly on editing with duplicated name. // ALF-6739: Notification should be more user friendly on editing with duplicated name.