From bd7233a23f44e9636e7884a060bb9805312b936c Mon Sep 17 00:00:00 2001
From: vthinkxie <vthinkxie@users.noreply.github.com>
Date: Tue, 25 Jun 2019 21:09:54 +0800
Subject: [PATCH] fix(module:pagination): fix pagination nzTotal 0 bug (#3651)

close #3648
---
 components/i18n/date-helper.service.ts             | 8 --------
 components/pagination/nz-pagination.component.html | 2 +-
 components/pagination/nz-pagination.spec.ts        | 5 +++++
 3 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/components/i18n/date-helper.service.ts b/components/i18n/date-helper.service.ts
index 0799a087a12..bd3de010e93 100644
--- a/components/i18n/date-helper.service.ts
+++ b/components/i18n/date-helper.service.ts
@@ -12,8 +12,6 @@ import fnsFormat from 'date-fns/format';
 import fnsGetISOWeek from 'date-fns/get_iso_week';
 import fnsParse from 'date-fns/parse';
 
-import { warnDeprecation } from 'ng-zorro-antd/core';
-
 import { mergeDateConfig, NzDateConfig, NZ_DATE_CONFIG } from './date-config';
 import { NzI18nService } from './nz-i18n.service';
 
@@ -87,16 +85,10 @@ export class DateHelperByDateFns extends DateHelperService {
  *
  * @see https://github.com/NG-ZORRO/ng-zorro-antd/issues/2406 - DatePipe may cause non-standard week bug, see:
  *
- * @deprecated 9.0.0 - `DateHelperByDatePipe` would be removed in 9.0.0 due to the serious bug above, unless it's get fixed.
- *   Please use `DateHelperByDateFns` instead.
  */
 export class DateHelperByDatePipe extends DateHelperService {
   constructor(i18n: NzI18nService, @Optional() @Inject(NZ_DATE_CONFIG) config: NzDateConfig) {
     super(i18n, config);
-
-    warnDeprecation(
-      `'DateHelperByDatePipe' would be removed in 9.0.0 due to this serious bug of Angular: https://github.com/NG-ZORRO/ng-zorro-antd/issues/2406. Please use 'DateHelperByDateFns' instead.`
-    );
   }
 
   getISOWeek(date: Date): number {
diff --git a/components/pagination/nz-pagination.component.html b/components/pagination/nz-pagination.component.html
index d9b42f5eb79..b32ea5d0792 100644
--- a/components/pagination/nz-pagination.component.html
+++ b/components/pagination/nz-pagination.component.html
@@ -7,7 +7,7 @@
   </a>
   <a *ngIf="type=='page'">{{ page }}</a>
 </ng-template>
-<ng-container *ngIf="nzHideOnSinglePage && (nzTotal > nzPageSize) || !nzHideOnSinglePage">
+<ng-container *ngIf="(nzHideOnSinglePage && (nzTotal > nzPageSize)) || (nzTotal && !nzHideOnSinglePage)">
   <ul class="ant-pagination"
       [class.ant-table-pagination]="nzInTable"
       [class.ant-pagination-simple]="nzSimple"
diff --git a/components/pagination/nz-pagination.spec.ts b/components/pagination/nz-pagination.spec.ts
index 151fdc2458c..4f88159677f 100644
--- a/components/pagination/nz-pagination.spec.ts
+++ b/components/pagination/nz-pagination.spec.ts
@@ -243,6 +243,11 @@ describe('pagination', () => {
         expect(testComponent.pageIndex).toBe(5);
       });
     });
+    it('should zero total hide all', () => {
+      testComponent.total = 0;
+      fixture.detectChanges();
+      expect(pagination.nativeElement.innerText).toEqual('');
+    });
     it('should be hidden pagination when total is 0 and nzHideOnSinglePage is true', () => {
       (testComponent as NzTestPaginationComponent).total = 0;
       (testComponent as NzTestPaginationComponent).hideOnSinglePage = true;