ACS-8824 [ACA] View Details / The user cannot edit node properties af… (

#10246) * ACS-8824 [ACA] View Details / The user cannot edit node properties after failing to change node name with special characters * ACS-8824 [ACA] View Details / The user cannot edit node properties after failing to change node name with special characters
Alfresco · Sep 27, 2024 · da44b4c · da44b4c
1 parent 602460c
commit da44b4c
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/lib/content-services/src/lib/content-metadata/services/basic-properties.service.ts b/lib/content-services/src/lib/content-metadata/services/basic-properties.service.ts
@@ -17,7 +17,13 @@
 
 import { inject, Injectable } from '@angular/core';
 import { Node } from '@alfresco/js-api';
-import { CardViewDateItemModel, CardViewTextItemModel, FileSizePipe, TranslationService } from '@alfresco/adf-core';
+import {
+    CardViewDateItemModel,
+    CardViewItemMatchValidator,
+    CardViewTextItemModel,
+    FileSizePipe,
+    TranslationService
+} from '@alfresco/adf-core';
 
 @Injectable({
     providedIn: 'root'
@@ -37,7 +43,10 @@ export class BasicPropertiesService {
                 label: 'CORE.METADATA.BASIC.NAME',
                 value: node.name,
                 key: 'properties.cm:name',
-                editable: true
+                editable: true,
+                validators: [
+                    new CardViewItemMatchValidator('[\\/\\*\\\\"\\\\]')
+                ]
             }),
             new CardViewTextItemModel({
                 label: 'CORE.METADATA.BASIC.TITLE',

diff --git a/...core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.spec.ts b/...core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.spec.ts
@@ -581,15 +581,15 @@ describe('CardViewTextItemComponent', () => {
             expect(cardViewUpdateService.update).toHaveBeenCalledWith(property, 'updated-value');
         });
 
-        it('should trigger the update event if the editedValue is NOT valid', async () => {
+        it('should NOT trigger the update event if the editedValue is invalid', async () => {
             const cardViewUpdateService = TestBed.inject(CardViewUpdateService);
             spyOn(cardViewUpdateService, 'update');
             component.property.isValid = () => false;
 
             updateTextField(component.property.key, '@invalid-value');
             await fixture.whenStable();
 
-            expect(cardViewUpdateService.update).toHaveBeenCalled();
+            expect(cardViewUpdateService.update).not.toHaveBeenCalled();
         });
 
         it('should trigger the update event if the editedValue is valid', async () => {

diff --git a/lib/core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.ts b/lib/core/src/lib/card-view/components/card-view-textitem/card-view-textitem.component.ts
@@ -149,7 +149,6 @@ export class CardViewTextItemComponent extends BaseCardView<CardViewTextItemMode
                 this.cardViewUpdateService.update({ ...this.property } as CardViewTextItemModel, this.property.value);
             } else {
                 this.errors = this.property.getValidationErrors(this.editedValue);
-                this.cardViewUpdateService.update({ ...this.property } as CardViewTextItemModel, this.editedValue);
             }
         }
     }