5htp-core 0.4.1-2 → 0.4.1-4

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "5htp-core",
 	"description": "Convenient TypeScript framework designed for Performance and Productivity.",
-	"version": "0.4.1-2",
+	"version": "0.4.1-4",
 	"author": "Gaetan Le Gac (https://github.com/gaetanlegac)",
 	"repository": "git://github.com/gaetanlegac/5htp-core.git",
 	"license": "MIT",

package/src/client/app/index.ts

@@ -88,13 +88,12 @@ export default abstract class Application {
     public bindErrorHandlers() {
 
         // Impossible de recup le stacktrace ...
-        /*window.addEventListener("unhandledrejection", (e) => {
-            clientBug(JSON.stringify(e))
-            console.log("unhandledrejection", e.stack);
-            
-        });*/
+        window.addEventListener("unhandledrejection", (e) => {
+            console.error(`Exception catched by method A`, e);
+        });
         
-        window.onerror = (message, file, line, col, stacktrace) =>
+        window.onerror = (message, file, line, col, stacktrace) => {
+            console.error(`Exception catched by method B`, message);
             this.reportBug({
                 stacktrace: stacktrace?.stack || JSON.stringify({ message, file, line, col })
             }).then(() => {
@@ -106,9 +105,10 @@ export default abstract class Application {
                 { autohide: false });*/
 
             })
+        }
     }
 
-    public abstract handleError( error: CoreError | Error, httpCode?: number );
+    public abstract handleError( error: CoreError | Error );
 
     // TODO: move on app side
     public reportBug = (infos: TBugReportInfos) => fetch('/feedback/bug/ui', {

package/src/client/assets/css/components/table.less

@@ -9,7 +9,7 @@ table {
     width: 100%;
     padding: 0;
     border-collapse: separate; 
-    border-spacing: 0 0.5em;
+    border-spacing: 0 0; // must be fully managed by padding
 
     // By default, chrome disables text inherits
     line-height: inherit;
@@ -48,6 +48,14 @@ table {
         }
     }
 
+    // Size
+    &.s {
+        td, th {
+            padding: 0.5em 1em;
+            font-size: 0.9rem;
+        }
+    }
+
     // Tableua responsive
     //  Utilisation déconseillée (lauvaise utilisation de l'espace, ergonomie horrible
     /*@media (max-width: @responsive2) {

package/src/client/components/Form.ts

@@ -225,7 +225,13 @@ export default function useForm<TFormData extends {}>(
     const form = {
         fields: fields.current,
         data,
-        set: setData,
+        set: data => {
+            setState(current => ({
+                ...current,
+                hasChanged: true
+            }));
+            setData(data);
+        },
         validate,
         submit,
         options,

package/src/client/components/Select/index.tsx

@@ -168,7 +168,7 @@ export default ({
 
     return dropdown ? (
         <Popover content={(
-            <div class="card col">
+            <div class="card col al-top">
 
                 <div class="col">
 

package/src/client/services/router/index.tsx

@@ -448,7 +448,8 @@ export default class ClientRouter<
         // Nor page configurated for this error
         if (route === undefined) {
             console.error(`Error page for http error code ${code} not found.`, this.errors, this.routes);
-            this.app.handleError(e, 404);
+            e.http = 404;
+            this.app.handleError(e);
             throw new Error(`Error page for http error code ${code} not found.`);
         }
 

package/src/client/services/router/request/api.ts

@@ -117,11 +117,28 @@ export default class ApiClient implements ApiClientService {
         const [method, path, data, options] = args;
         return {
             method, path, data, options,
+
             // For async calls: api.post(...).then((data) => ...)
-            then: (callback: (data: any) => void) => this.fetchAsync<TData>(...args).then(callback),
-            catch: (callback: (data: any) => void) => this.fetchAsync<TData>(...args).catch(callback),
-            finally: (callback: () => void) => this.fetchAsync<TData>(...args).finally(callback),
+            then: (callback: (data: any) => void) => this.fetchAsync<TData>(...args)
+                .then(callback)
+                .catch( e => this.app.handleError(e)),
+
+            // Default error behavior only if not handled before by the app
+            catch: (callback: (data: any) => void) => this.fetchAsync<TData>(...args)
+                .catch((e) => {
+                    try {
+                        callback(e);
+                    } catch (error) {
+                        this.app.handleError(e)
+                    }
+                }),
+
+            finally: (callback: () => void) => this.fetchAsync<TData>(...args)
+                .finally(callback)
+                .catch( e => this.app.handleError(e)),
+
             run: () => this.fetchAsync<TData>(...args)
+                .catch( e => this.app.handleError(e)),
         };
     }
 
@@ -132,10 +149,7 @@ export default class ApiClient implements ApiClientService {
         /*if (options?.captcha !== undefined)
             await this.gui.captcha.check(options?.captcha);*/
 
-        return await this.execute<TData>(method, path, data, options).catch((e) => {
-            this.app.handleError(e);
-            throw e; // Throw to break the loop
-        })
+        return await this.execute<TData>(method, path, data, options);
     }
 
     public async fetchSync(fetchers: TFetcherList, alreadyLoadedData: {}): Promise<TObjetDonnees> {
@@ -163,6 +177,12 @@ export default class ApiClient implements ApiClientService {
 
                 return data;
 
+            }).catch(e => {
+
+                // API Error hook
+                this.app.handleError(e);
+
+                throw e;
             })
 
         // Errors will be catched in the caller
@@ -239,9 +259,6 @@ export default class ApiClient implements ApiClientService {
                         e.response.data
                     );
 
-                    // API Error hook
-                    this.app.handleError(error, e.response.status);
-
                     throw error;
     
                 // Erreur réseau: l'utilisateur n'ets probablement plus connecté à internet

package/src/server/services/router/response/page/document.tsx

@@ -139,7 +139,7 @@ export default class DocumentRenderer<TRouter extends Router> {
 
     private styles( page: Page ) {
         return <>
-            <link rel="stylesheet" type="text/css" href="/public/icons.css" />
+            <link rel="stylesheet" type="text/css" href={"/public/icons.css?" + BUILD_ID} />
             <link rel="preload" href="/public/client.css" as="style" />
             <link rel="stylesheet" type="text/css" href="/public/client.css" />