API improvement & bug fix:

- subscribed global event doesnt unsubcribed when process is killed
- process killall API doesnt work as expected
- improve core API

d.ts/antos.d.ts

@@ -3225,12 +3225,11 @@ declare namespace OS {
              * Update the application local from the system
              * locale or application specific locale configuration
              *
-             * @private
              * @param {string} name locale name e.g. `en_GB`
              * @returns {void}
              * @memberof BaseApplication
              */
-            protected updateLocale(name: string): void;
+            updateLocale(name: string): void;
             /**
              * Execute the callback subscribed to a
              * keyboard shortcut
@@ -4559,11 +4558,10 @@ declare namespace OS {
         /**
          * Update the model locale
          *
-         * @protected
          * @param {string} name
          * @memberof BaseModel
          */
-        protected updateLocale(name: string): void;
+        updateLocale(name: string): void;
         /**
          * Render the model's UI
          *

src/core/BaseApplication.ts

@@ -83,11 +83,6 @@ namespace OS {
                 }
                 this.setting = setting.applications[this.name];
                 this.keycomb = {};
-                this.subscribe("appregistry", (m) => {
-                    if (m.name === this.name) {
-                        this.applySetting(m.message as string);
-                    }
-                });
             }
 
             /**
@@ -135,6 +130,11 @@ namespace OS {
                     }
                 });
                 this.on("apptitlechange", () => this.sysdock.update(this));
+                this.subscribe("appregistry", (m) => {
+                    if (m.name === this.name) {
+                        this.applySetting(m.message as string);
+                    }
+                });
                 this.updateLocale(this.systemsetting.system.locale);
                 return this.loadScheme();
             }
@@ -225,12 +225,11 @@ namespace OS {
              * Update the application local from the system
              * locale or application specific locale configuration
              *
-             * @private
              * @param {string} name locale name e.g. `en_GB`
              * @returns {void}
              * @memberof BaseApplication
              */
-            protected updateLocale(name: string): void {
+            updateLocale(name: string): void {
                 const meta = this.meta();
                 if (!meta || !meta.locales) {
                     return;

src/core/BaseModel.ts

@@ -296,10 +296,6 @@ namespace OS {
             this.on("exit", () => this.quit(false));
             this.host = this._gui.workspace;
             this.dialog = undefined;
-            this.subscribe("systemlocalechange", (d) => {
-                this.updateLocale(d.message as string);
-                return this.update();
-            });
         }
 
         /**
@@ -315,11 +311,10 @@ namespace OS {
         /**
          * Update the model locale
          *
-         * @protected
          * @param {string} name
          * @memberof BaseModel
          */
-        protected updateLocale(name: string) {}
+        updateLocale(name: string) {}
         /**
          * Render the model's UI
          *
@@ -676,7 +671,11 @@ namespace OS {
          */
         update(): void {
             if (this.scheme) {
-                return this.scheme.update();
+                this.scheme.update();
+            }
+            if(this.dialog)
+            {
+                this.dialog.update();
             }
         }
 

src/core/gui.ts

@@ -405,7 +405,6 @@ namespace OS {
                 const js = path + "/main.js";
                 try {
                     const d = await js.asFileHandle().read("script");
-                    try {
                     const data: API.PackageMetaType = await `${path}/package.json`
                         .asFileHandle()
                         .read("json");
@@ -420,7 +419,6 @@ namespace OS {
                     }
                     //load css file
                     const css = `${path}/main.css`;
-                        try {
                     const d_1 = await css.asFileHandle().onready();
                     const stamp = new Date().timestamp();
                     const el = $("<link>", {
@@ -433,13 +431,7 @@ namespace OS {
                     }
                     return resolve(app);
                 } catch (e) {
-                            return resolve(app);
+                    return reject(__e(e));
-                        }
-                    } catch (e_1) {
-                        return reject(__e(e_1));
-                    }
-                } catch (e_2) {
-                    return reject(__e(e_2));
                 }
             });
         }
@@ -459,36 +451,29 @@ namespace OS {
                 const arr = ph.split("/");
                 const srv = arr[1];
                 const app = arr[0];
-                if (application[srv]) {
                 try {
+                    if (application[srv]) {
                         const d = await OS.PM.createProcess(
                             srv,
                             application[srv]
                         );
                         return resolve(d);
-                    } catch (e) {
-                        return reject(__e(e));
-                    }
                     } else {
-                    try {
                         await loadApp(app);
                         if (!application[srv]) {
                             return reject(
                                 API.throwe(__("Service not found: {0}", ph))
                             );
                         }
-                        try {
                         const d_1 = await PM.createProcess(
                             srv,
                             application[srv]
                         );
                         return resolve(d_1);
-                        } catch (e_1) {
-                            return reject(__e(e_1));
                     }
-                    } catch (e_2) {
-                        return reject(__e(e_2));
                 }
+                catch (e) {
+                    return reject(__e(e));
                 }
             });
         }
@@ -529,11 +514,11 @@ namespace OS {
          * @param {AppArgumentsType[]} args application arguments
          */
         export function launch(app: string, args: AppArgumentsType[]): Promise<OS.PM.ProcessType> {
-            return new Promise((resolve, reject) => {
+            return new Promise(async (resolve, reject) => {
+                try {
                     if (!application[app]) {
                         // first load it
-                    loadApp(app)
+                        await loadApp(app);
-                        .then((a) => {
                         if (!application[app]) {
                             const e = API.throwe(__("Application not found"));
                             announcer.oserror(
@@ -541,47 +526,30 @@ namespace OS {
                                 e);
                             return reject(e);
                         }
-                            PM.createProcess(
+                        const p = await PM.createProcess(
                             app,
                             application[app],
                             args
-                            ).catch((e) => {
+                        );
+                        resolve(p);
+                    } else {
+                        // now launch it
+                        const p = await PM.createProcess(
+                            app,
+                            application[app],
+                            args
+                        );
+                        resolve(p);
+                    }
+                }
+                catch (e) {
                     announcer.osfail(
                         __("Unable to launch: {0}", app),
                         e
                     );
-                                return reject(e);
+                    return reject(__e(e));
-                            }
-                            ).then((p: PM.ProcessType) => resolve(p));
-                        }
-                        )
-                        .catch((e) => {
-                            announcer.osfail(__("Unable to launch: {0}", app), e);
-                            reject(e);
-                        }
-                        );
-                } else {
-                    // now launch it
-                    if (application[app]) {
-                        PM.createProcess(
-                            app,
-                            application[app],
-                            args
-                        ).catch((e: Error) => {
-                            announcer.osfail(__("Unable to launch: {0}", app), e);
-                            return reject(e);
                 }
 
-                        );
-                    } else {
-                        const e = API.throwe(__("Application not found"));
-                        announcer.osfail(
-                            __("Unable to find: {0}", app),
-                            e
-                        );
-                        return reject(e);
-                    }
-                }
             });
         }
 
@@ -990,8 +958,7 @@ namespace OS {
 
                 $(desktop).on("click", function (e) {
                     let el: any = $(e.target).closest("afx-app-window")[0];
-                    if(el)
+                    if (el) {
-                    {
                         return;
                     }
                     el = $(e.target).parent();

src/core/pm.ts

@@ -63,6 +63,10 @@ namespace OS {
                         obj.birth = new Date().getTime();
                         PM.pidalloc++;
                         obj.pid = PM.pidalloc;
+                        obj.subscribe("systemlocalechange", (d) => {
+                            obj.updateLocale(d.message as string);
+                            return obj.update();
+                        });
                         PM.processes[app].push(obj);
                         if (metaclass.type === ModelType.Application) {
                             GUI.dock(
@@ -149,7 +153,11 @@ namespace OS {
             if (!PM.processes[app]) {
                 return;
             }
-            PM.processes[app].map((a) => a.quit(force));
+            const arr = PM.processes[app].map( e => e);
+            for(const p of arr)
+            {
+                p.quit(force);
+            }
         }
     }
 }