fix(pipe): handle non string input (#893)

ngx-translate · Feb 5, 2020 · 9ff74dc · 9ff74dc
1 parent 3e0aebb
commit 9ff74dc
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/projects/ngx-translate/core/src/lib/translate.pipe.ts b/projects/ngx-translate/core/src/lib/translate.pipe.ts
@@ -37,7 +37,7 @@ export class TranslatePipe implements PipeTransform, OnDestroy {
   }
 
   transform(query: string, ...args: any[]): any {
-    if (!query || query.length === 0) {
+    if (!query || !query.length) {
       return query;
     }
 

diff --git a/projects/ngx-translate/core/tests/translate.pipe.spec.ts b/projects/ngx-translate/core/tests/translate.pipe.spec.ts
@@ -161,6 +161,15 @@ describe('TranslatePipe', () => {
     }).toThrowError(`Wrong parameter in TranslatePipe. Expected a valid Object, received: ${param}`);
   });
 
+  it("should return given falsey or non length query", () => {
+    translate.setTranslation('en', {"TEST": "This is a test"});
+    translate.use('en');
+
+    expect(translatePipe.transform(null)).toBeNull();
+    expect(translatePipe.transform(undefined)).toBeUndefined();
+    expect(translatePipe.transform(1234 as any)).toBe(1234);
+  });
+
   describe('should update translations on lang change', () => {
     it('with fake loader', (done) => {
       translate.setTranslation('en', {"TEST": "This is a test"});