@etsoo/react 1.3.55 → 1.3.59

package/lib/app/ReactApp.d.ts

@@ -13,6 +13,7 @@ import { InputDialogProps } from './InputDialogProps';
  * Case 1: undefined, when refresh the whole page
  * Case 2: false, unauthorized
  * Case 3: true, authorized or considered as authorized (maynot, like token expiry)
+ * Case 4: property or properties changed
  * @param props Props
  * @returns Component
  */

package/lib/app/ReactApp.js

@@ -1,5 +1,4 @@
 import { CoreApp, createClient } from '@etsoo/appscript';
-import { Utils } from '@etsoo/shared';
 import React from 'react';
 import { NotifierMU } from '../mu/NotifierMU';
 import { ProgressCount } from '../mu/ProgressCount';
@@ -14,6 +13,7 @@ let globalApp;
  * Case 1: undefined, when refresh the whole page
  * Case 2: false, unauthorized
  * Case 3: true, authorized or considered as authorized (maynot, like token expiry)
+ * Case 4: property or properties changed
  * @param props Props
  * @returns Component
  */
@@ -25,6 +25,7 @@ export function ReactAppStateDetector(props) {
         ? { state: {} }
         : React.useContext(globalApp.userState.context);
     // Match fields
+    const authorized = state.authorized;
     const changedFields = state.lastChangedFields;
     let matchedFields;
     if (targetFields == null || changedFields == null) {
@@ -37,11 +38,12 @@ export function ReactAppStateDetector(props) {
                 matchedFields === null || matchedFields === void 0 ? void 0 : matchedFields.push(targetField);
         });
     }
+    console.log('ReactAppStateDetector', changedFields, targetFields, matchedFields);
     // Ready
     React.useEffect(() => {
         // Callback
-        update(state.authorized, matchedFields);
-    }, [state.authorized, matchedFields]);
+        update(authorized, matchedFields);
+    }, [authorized, matchedFields]);
     // return
     return React.createElement(React.Fragment);
 }
@@ -211,15 +213,10 @@ export class ReactApp extends CoreApp {
      * @param keep Keep in local storage or not
      */
     userLogin(user, refreshToken, keep) {
-        // Changed
-        const dataChanged = !this.authorized ||
-            this.userData == null ||
-            !Utils.objectEqual(this.userData, user, ['seconds', 'token']);
         // Super call, set token
         super.userLogin(user, refreshToken, keep);
         // Dispatch action
-        // Only when unauthorized or with changes except 'token' and 'seconds'
-        if (this.userStateDispatch != null && dataChanged)
+        if (this.userStateDispatch != null)
             this.userStateDispatch({
                 type: UserActionType.Login,
                 user

package/lib/states/UserState.js

@@ -68,6 +68,7 @@ export class UserState {
     }
     getChangedFields(input, init) {
         return Utils.objectUpdated(input, init, [
+            'authorized',
             'seconds',
             'lastChangedFields'
         ]);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@etsoo/react",
-    "version": "1.3.55",
+    "version": "1.3.59",
     "description": "TypeScript ReactJs framework",
     "main": "lib/index.js",
     "types": "lib/index.d.ts",

package/src/app/ReactApp.ts

@@ -7,7 +7,7 @@ import {
     IUserData
 } from '@etsoo/appscript';
 import { NotificationRenderProps } from '@etsoo/notificationbase';
-import { DataTypes, Utils } from '@etsoo/shared';
+import { DataTypes } from '@etsoo/shared';
 import React from 'react';
 import { NotifierMU } from '../mu/NotifierMU';
 import { ProgressCount } from '../mu/ProgressCount';
@@ -37,6 +37,7 @@ let globalApp: IReactApp<IAppSettings, any, IPageData>;
  * Case 1: undefined, when refresh the whole page
  * Case 2: false, unauthorized
  * Case 3: true, authorized or considered as authorized (maynot, like token expiry)
+ * Case 4: property or properties changed
  * @param props Props
  * @returns Component
  */
@@ -53,6 +54,7 @@ export function ReactAppStateDetector(props: IStateProps) {
               );
 
     // Match fields
+    const authorized = state.authorized;
     const changedFields = state.lastChangedFields;
     let matchedFields: string[] | undefined;
     if (targetFields == null || changedFields == null) {
@@ -65,11 +67,18 @@ export function ReactAppStateDetector(props: IStateProps) {
         });
     }
 
+    console.log(
+        'ReactAppStateDetector',
+        changedFields,
+        targetFields,
+        matchedFields
+    );
+
     // Ready
     React.useEffect(() => {
         // Callback
-        update(state.authorized, matchedFields);
-    }, [state.authorized, matchedFields]);
+        update(authorized, matchedFields);
+    }, [authorized, matchedFields]);
 
     // return
     return React.createElement(React.Fragment);
@@ -356,18 +365,11 @@ export class ReactApp<
         refreshToken: string,
         keep?: boolean
     ): void {
-        // Changed
-        const dataChanged =
-            !this.authorized ||
-            this.userData == null ||
-            !Utils.objectEqual(this.userData, user, ['seconds', 'token']);
-
         // Super call, set token
         super.userLogin(user, refreshToken, keep);
 
         // Dispatch action
-        // Only when unauthorized or with changes except 'token' and 'seconds'
-        if (this.userStateDispatch != null && dataChanged)
+        if (this.userStateDispatch != null)
             this.userStateDispatch({
                 type: UserActionType.Login,
                 user

package/src/states/UserState.ts

@@ -124,6 +124,7 @@ export class UserState<D extends IUser> {
 
     private getChangedFields(input: {}, init: {}) {
         return Utils.objectUpdated(input, init, [
+            'authorized',
             'seconds',
             'lastChangedFields'
         ]);