fix(module:cascader): should change check state trigger ngModelChange (#8941)

Author: Laffery

components/cascader/cascader.component.ts

@@ -535,7 +535,6 @@ export class NzCascaderComponent
         if (shouldClose) {
           this.delaySetMenuVisible(false);
         }
-        this.emitValue(this.cascaderService.values);
         this.nzSelectionChange.emit(this.getAncestorOptionList(node));
         this.cdr.markForCheck();
       }
@@ -947,7 +946,7 @@ export class NzCascaderComponent
       this.cascaderService.setNodeActivated(node, columnIndex, true, true);
     } else {
       // only update selected nodes and not set node activated by default
-      this.updateSelectedNodes();
+      this.cascaderService.setNodeSelected(node, columnIndex, true);
     }
   }
 

components/cascader/cascader.spec.ts

@@ -1842,6 +1842,21 @@ describe('cascader', () => {
       expect(leaf.classList).toContain('ant-cascader-menu-item-active');
       expect(checkbox.classList).not.toContain('ant-cascader-checkbox-checked');
     }));
+
+    it('should change check state trigger ngModelChange', fakeAsync(() => {
+      spyOn(testComponent, 'onChanges');
+      expect(testComponent.onChanges).not.toHaveBeenCalled();
+      cascader.componentInstance.setMenuVisible(true);
+      fixture.detectChanges();
+      tick(600);
+      fixture.detectChanges();
+      expect(testComponent.onChanges).not.toHaveBeenCalled();
+
+      const checkbox = getCheckboxAtColumnAndRow(1, 1)!;
+      checkbox.click();
+      fixture.detectChanges();
+      expect(testComponent.onChanges).toHaveBeenCalledWith([['light']]);
+    }));
   });
 
   describe('load data lazily', () => {

components/cascader/demo/multiple.ts

@@ -4,7 +4,7 @@ import { FormsModule } from '@angular/forms';
 import { NzCascaderModule, NzCascaderOption } from 'ng-zorro-antd/cascader';
 import { NzSafeAny } from 'ng-zorro-antd/core/types';
 
-const options: NzCascaderOption[] = [
+const getOptions = (): NzCascaderOption[] => [
   {
     label: 'Light',
     value: 'light',
@@ -55,7 +55,7 @@ const options: NzCascaderOption[] = [
   `
 })
 export class NzDemoCascaderMultipleComponent {
-  nzOptions: NzCascaderOption[] = options;
+  nzOptions: NzCascaderOption[] = getOptions();
   values: NzSafeAny[][] | null = null;
 
   onChanges(values: NzSafeAny[][]): void {