Fixed default localizers handling of done callback.

Author: Stevenic

Node/core/src/DefaultLocalizer.ts

@@ -203,7 +203,7 @@ export class DefaultLocalizer implements ILocalizer {
         });
     }
 
-    public load(locale: string, done: ErrorCallback): void {
+    public load(locale: string, done?: ErrorCallback): void {
         if (locale) {
             locale = locale.toLowerCase();
         }
@@ -213,7 +213,9 @@ export class DefaultLocalizer implements ILocalizer {
 
         if (DefaultLocalizer.localeRequests[localeRequestKey]) {
             logger.debug("localizer::already loaded requested locale: %s", localeRequestKey);                                               
-            done(null);
+            if (done) {
+                done(null);
+            }
             return;
         }
 
@@ -276,12 +278,12 @@ export class DefaultLocalizer implements ILocalizer {
 
         ],
         (err, results) => {
-            if (err) {
-                done(err);
-            } else {
+            if (!err) {
                 DefaultLocalizer.localeRequests[localeRequestKey] = true;
                 logger.debug("localizer::loaded requested locale: %s", localeRequestKey);                            
-                done();
+            }
+            if (done) {
+                done(err);
             }
         });
     }

Node/core/src/interfaces.d.ts

@@ -164,7 +164,7 @@ interface ILocationV2 {
 
 interface ILocalizer {
     initialize(localizerSettings?: ILocalizerSettings): void;
-    load(locale: string, callback: ErrorCallback): void;     
+    load(locale: string, callback?: ErrorCallback): void;     
     defaultLocale(locale?: string): string   
     gettext(locale: string, msgid: string, namespace?: string): string;
     trygettext(locale: string, msgid: string, namespace?: string): string;