From f077294c6a5e2ff59951decff6b466dffbc726f5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E5=8D=A1=E8=89=B2?= <cipchk@qq.com>
Date: Wed, 6 Mar 2019 14:12:23 +0700
Subject: [PATCH] fix(module:transfer): fix click checkbox can't trigger
 selection (#3030)

close #3026
---
 components/transfer/nz-transfer-list.component.html | 4 ++--
 components/transfer/transfer.spec.ts                | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/components/transfer/nz-transfer-list.component.html b/components/transfer/nz-transfer-list.component.html
index 993afcbba82..7af09fd138a 100644
--- a/components/transfer/nz-transfer-list.component.html
+++ b/components/transfer/nz-transfer-list.component.html
@@ -19,9 +19,9 @@
   </div>
   <ul class="ant-transfer-list-content">
     <ng-container *ngFor="let item of dataSource">
-      <li *ngIf="!item._hiden" (click)="_handleSelect(item)"
+      <li *ngIf="!item._hiden"
         class="ant-transfer-list-content-item" [ngClass]="{'ant-transfer-list-content-item-disabled': disabled || item.disabled}">
-        <label nz-checkbox [nzChecked]="item.checked" [nzDisabled]="disabled || item.disabled">
+        <label nz-checkbox [nzChecked]="item.checked" (nzCheckedChange)="_handleSelect(item)" [nzDisabled]="disabled || item.disabled">
           <ng-container *ngIf="!render; else renderContainer">{{ item.title }}</ng-container>
           <ng-template #renderContainer [ngTemplateOutlet]="render" [ngTemplateOutletContext]="{ $implicit: item }"></ng-template>
         </label>
diff --git a/components/transfer/transfer.spec.ts b/components/transfer/transfer.spec.ts
index 3b555ce381d..4bcdf2abd53 100644
--- a/components/transfer/transfer.spec.ts
+++ b/components/transfer/transfer.spec.ts
@@ -250,7 +250,7 @@ describe('transfer', () => {
       fixture.detectChanges();
       expect(pageObject.getEl('[data-direction="right"] .ant-transfer-list-header .ant-checkbox').classList).not.toContain('ant-checkbox-checked');
       pageObject.checkItem('right', 1);
-      tick();
+      tick(50);
       fixture.detectChanges();
       expect(pageObject.getEl('[data-direction="right"] .ant-transfer-list-header .ant-checkbox').classList).toContain('ant-checkbox-checked');
     }));
@@ -293,7 +293,7 @@ describe('transfer', () => {
       if (!Array.isArray(index)) {
         index = [ index ];
       }
-      const items = (direction === 'left' ? this.leftList : this.rightList).querySelectorAll('.ant-transfer-list-content-item');
+      const items = (direction === 'left' ? this.leftList : this.rightList).querySelectorAll('.ant-transfer-list-content-item label');
       for (const idx of index) {
         (items[ idx ] as HTMLElement).click();
         fixture.detectChanges();