refactor(*): use nullish coalescing instead of || (#1940)

Author: cipchk

packages/abc/date-picker/range.directive.ts

@@ -49,7 +49,7 @@ export class RangePickerDirective implements OnDestroy, AfterViewInit {
     if (typeof val !== 'object') {
       item.enabled = val !== false;
     }
-    (item.list || []).forEach(i => {
+    (item.list ?? []).forEach(i => {
       i._text = this.dom.bypassSecurityTrustHtml(i.text);
     });
     this._shortcut = item;

packages/abc/down-file/down-file.directive.ts

@@ -27,7 +27,7 @@ export class DownFileDirective {
   @Output() readonly error = new EventEmitter<NzSafeAny>();
 
   private getDisposition(data: string | null): NzSafeAny {
-    const arr: Array<Record<string, string>> = (data || '')
+    const arr: Array<Record<string, string>> = (data ?? '')
       .split(';')
       .filter(i => i.includes('='))
       .map(v => {
@@ -65,7 +65,7 @@ export class DownFileDirective {
     this.setDisabled(true);
     this._http
       .request(this.httpMethod, this.httpUrl, {
-        params: this.httpData || {},
+        params: this.httpData ?? {},
         responseType: 'blob',
         observe: 'response',
         body: this.httpBody

packages/abc/ellipsis/ellipsis.component.ts

@@ -61,7 +61,7 @@ export class EllipsisComponent implements AfterViewInit, OnChanges {
   }
 
   private get win(): NzSafeAny {
-    return this.doc.defaultView || window;
+    return this.doc.defaultView ?? window;
   }
 
   private getStrFullLength(str: string): number {
@@ -169,7 +169,7 @@ export class EllipsisComponent implements AfterViewInit, OnChanges {
     } else if (type === 'line') {
       const { shadowOrgEl, shadowTextEl } = this;
       const orgNode = shadowOrgEl.nativeElement as HTMLElement;
-      const lineText = orgNode.innerText || orgNode.textContent!;
+      const lineText = orgNode.innerText ?? orgNode.textContent!;
       const lineHeight = parseInt(this.win.getComputedStyle(this.getEl('.ellipsis')).lineHeight!, 10);
       const targetHeight = lines! * lineHeight;
       const handleEl = this.getEl('.ellipsis__handle');

packages/abc/error-collect/error-collect.component.ts

@@ -70,7 +70,7 @@ export class ErrorCollectComponent implements OnInit {
     if (this.count() === 0) return false;
     // nz-form-control
     const els = this.errEls;
-    const formItemEl = this.findParent(els[0], '[nz-form-control]') || els[0];
+    const formItemEl = this.findParent(els[0], '[nz-form-control]') ?? els[0];
     formItemEl.scrollIntoView(true);
     // fix header height
     this.doc.documentElement.scrollTop -= this.offsetTop();

packages/abc/loading/loading.service.ts

@@ -55,7 +55,7 @@ export class LoadingService implements OnDestroy {
       backdropClass: 'loading-backdrop'
     });
     this.compRef = this._overlayRef.attach(new ComponentPortal(LoadingDefaultComponent));
-    const dir = this.configSrv.get('loading')!.direction || this.directionality.value;
+    const dir = this.configSrv.get('loading')!.direction ?? this.directionality.value;
     if (this.instance != null) {
       this.instance!.options = this.opt;
       this.instance!.dir = dir;

packages/abc/lodop/lodop.service.ts

@@ -168,7 +168,7 @@ export class LodopService implements OnDestroy {
         if (arr != null && Array.isArray(arr) && contextObj) {
           for (let i = 0; i < arr.length; i++) {
             if (typeof arr[i] === 'string') {
-              arr[i] = (arr[i] as string).replace(/{{(.*?)}}/g, (_match, key) => contextObj[key.trim()] || '');
+              arr[i] = (arr[i] as string).replace(/{{(.*?)}}/g, (_match, key) => contextObj[key.trim()] ?? '');
             }
           }
         }

packages/abc/notice-icon/notice-icon-tab.component.html

@@ -9,7 +9,7 @@
     }
     <p>
       <ng-container *nzStringTemplateOutlet="d.emptyText">
-        {{ d.emptyText || locale().emptyText }}
+        {{ d.emptyText ?? locale().emptyText }}
       </ng-container>
     </p>
   </div>
@@ -45,5 +45,5 @@
       </nz-list-item>
     </ng-template>
   </nz-list>
-  <div class="notice-icon__clear" (click)="onClear()">{{ d.clearText || locale().clearText }}</div>
+  <div class="notice-icon__clear" (click)="onClear()">{{ d.clearText ?? locale().clearText }}</div>
 </ng-template>

packages/abc/onboarding/onboarding.component.ts

@@ -74,7 +74,7 @@ export class OnboardingComponent implements OnDestroy, AfterViewInit {
   }
 
   private _getWin(): Window {
-    return this._getDoc().defaultView || window;
+    return this._getDoc().defaultView ?? window;
   }
 
   private getLightData(): OnboardingLightData | null {
@@ -85,8 +85,8 @@ export class OnboardingComponent implements OnDestroy, AfterViewInit {
       return null;
     }
 
-    const scrollTop = win.scrollY || doc.documentElement.scrollTop || doc.body.scrollTop;
-    const scrollLeft = win.scrollX || doc.documentElement.scrollLeft || doc.body.scrollLeft;
+    const scrollTop = win.scrollY ?? doc.documentElement.scrollTop ?? doc.body.scrollTop;
+    const scrollLeft = win.scrollX ?? doc.documentElement.scrollLeft ?? doc.body.scrollLeft;
     const rect = el.getBoundingClientRect();
     const top = rect.top + scrollTop;
     const left = rect.left + scrollLeft;

packages/abc/onboarding/onboarding.service.ts

@@ -116,7 +116,7 @@ export class OnboardingService implements OnDestroy {
       ...this.locale(),
       ...items[this.active]
     } as OnboardingItem;
-    const dir = this.configSrv.get('onboarding')!.direction || this.directionality.value;
+    const dir = this.configSrv.get('onboarding')!.direction ?? this.directionality.value;
     Object.assign(this.compRef.instance, { item, config: this.config, active: this.active, max: items.length, dir });
     const pipes = [
       switchMap(() => (item.url ? this.router.navigateByUrl(item.url) : of(true))),

packages/abc/page-header/page-header.component.ts

@@ -167,7 +167,7 @@ export class PageHeaderComponent implements OnInit, OnChanges, AfterViewInit {
     // add home
     if (this.home) {
       paths.splice(0, 0, {
-        title: (this.homeI18n && this.i18nSrv.fanyi(this.homeI18n)) || this.home,
+        title: (this.homeI18n && this.i18nSrv.fanyi(this.homeI18n)) ?? this.home,
         link: [this.homeLink!]
       });
     }