@zeniai/client-epic-state 5.2.39-betaRD0 → 5.2.39

package/lib/esm/view/financeStatement/financeStatementReducer.js

@@ -17,7 +17,6 @@ export const initialFinanceStatementState = {
     displayOrder: 'ascending_date',
     reportFormat: 'more_detailed',
     lastFetchRequestTime: undefined,
-    isPeriodSelectionExplicit: false,
 };
 const financeStatement = createSlice({
     name: 'financeStatement',
@@ -34,26 +33,14 @@ const financeStatement = createSlice({
             },
         },
         updateFinanceStatementTimeframe(draft, action) {
-            // A month anchor is meaningless once the timeframe becomes quarter/year, so
-            // a real timeframe change has to drop it. Callers also re-dispatch the
-            // *current* timeframe as a side effect of unrelated UI (report format, view
-            // switches); dropping the anchor there silently resets the user's period to
-            // the latest one, so leave the selection alone when nothing changed.
-            if (draft.timeframe === action.payload) {
-                return;
-            }
             draft.timeframe = action.payload;
             draft.selectedCOABalancesRange = {
                 numberOfPeriods: draft.selectedCOABalancesRange.numberOfPeriods,
                 orderBy: draft.selectedCOABalancesRange.orderBy,
             };
-            draft.isPeriodSelectionExplicit = false;
         },
         updateCOABalancesRange(draft, action) {
             draft.selectedCOABalancesRange = Object.assign({}, action.payload);
-            // A range without an anchor is a programmatic reset to the default, not a
-            // user pick (e.g. leaving the horizontal by-class layout).
-            draft.isPeriodSelectionExplicit = action.payload.thisPeriod != null;
         },
         updateFinanceStatementThisPeriod(draft, action) {
             const { firstMonthOfFY, coaBalances, maxNumOfPeriodsToHighlight } = action.payload;
@@ -68,7 +55,6 @@ const financeStatement = createSlice({
                     };
                     const newThisPeriod = getMatchingThisPeriod(action.payload.firstMonthOfFY, timeframe, action.payload.thisPeriod, safeCoaBalances);
                     if (newThisPeriod != null) {
-                        draft.isPeriodSelectionExplicit = true;
                         const selectionRanges = getSelectedAndHighlightedRangesForThisPeriod({
                             firstMonthOfFY,
                             thisPeriod: newThisPeriod,
@@ -109,14 +95,6 @@ const financeStatement = createSlice({
             draft.isAdditionalBalancesShown =
                 additionalBalances.length != 0 ? true : false;
             const { timeframe, selectedCOABalancesRange } = draft;
-            // The reports page fires this on load, on window resize and on every
-            // fetch-state transition (so also on report switch and on remount after a
-            // drill-down), carrying a viewport-derived period count. Re-deriving the
-            // range from that count silently discards a range the user chose, so only
-            // the column metadata above is applied once the selection is theirs.
-            if (draft.isPeriodSelectionExplicit) {
-                return;
-            }
             if (safeCoaBalances.length > 0) {
                 const thisPeriod = extractThisPeriod(action.payload.firstMonthOfFY, timeframe, selectedCOABalancesRange, safeCoaBalances);
                 if (thisPeriod != null) {

package/lib/view/financeStatement/financeStatementReducer.js

@@ -21,7 +21,6 @@ exports.initialFinanceStatementState = {
     displayOrder: 'ascending_date',
     reportFormat: 'more_detailed',
     lastFetchRequestTime: undefined,
-    isPeriodSelectionExplicit: false,
 };
 const financeStatement = (0, toolkit_1.createSlice)({
     name: 'financeStatement',
@@ -38,26 +37,14 @@ const financeStatement = (0, toolkit_1.createSlice)({
             },
         },
         updateFinanceStatementTimeframe(draft, action) {
-            // A month anchor is meaningless once the timeframe becomes quarter/year, so
-            // a real timeframe change has to drop it. Callers also re-dispatch the
-            // *current* timeframe as a side effect of unrelated UI (report format, view
-            // switches); dropping the anchor there silently resets the user's period to
-            // the latest one, so leave the selection alone when nothing changed.
-            if (draft.timeframe === action.payload) {
-                return;
-            }
             draft.timeframe = action.payload;
             draft.selectedCOABalancesRange = {
                 numberOfPeriods: draft.selectedCOABalancesRange.numberOfPeriods,
                 orderBy: draft.selectedCOABalancesRange.orderBy,
             };
-            draft.isPeriodSelectionExplicit = false;
         },
         updateCOABalancesRange(draft, action) {
             draft.selectedCOABalancesRange = Object.assign({}, action.payload);
-            // A range without an anchor is a programmatic reset to the default, not a
-            // user pick (e.g. leaving the horizontal by-class layout).
-            draft.isPeriodSelectionExplicit = action.payload.thisPeriod != null;
         },
         updateFinanceStatementThisPeriod(draft, action) {
             const { firstMonthOfFY, coaBalances, maxNumOfPeriodsToHighlight } = action.payload;
@@ -72,7 +59,6 @@ const financeStatement = (0, toolkit_1.createSlice)({
                     };
                     const newThisPeriod = (0, thisPeriodHelpers_1.getMatchingThisPeriod)(action.payload.firstMonthOfFY, timeframe, action.payload.thisPeriod, safeCoaBalances);
                     if (newThisPeriod != null) {
-                        draft.isPeriodSelectionExplicit = true;
                         const selectionRanges = (0, getSelectedAndHighlightedRanges_1.getSelectedAndHighlightedRangesForThisPeriod)({
                             firstMonthOfFY,
                             thisPeriod: newThisPeriod,
@@ -113,14 +99,6 @@ const financeStatement = (0, toolkit_1.createSlice)({
             draft.isAdditionalBalancesShown =
                 additionalBalances.length != 0 ? true : false;
             const { timeframe, selectedCOABalancesRange } = draft;
-            // The reports page fires this on load, on window resize and on every
-            // fetch-state transition (so also on report switch and on remount after a
-            // drill-down), carrying a viewport-derived period count. Re-deriving the
-            // range from that count silently discards a range the user chose, so only
-            // the column metadata above is applied once the selection is theirs.
-            if (draft.isPeriodSelectionExplicit) {
-                return;
-            }
             if (safeCoaBalances.length > 0) {
                 const thisPeriod = (0, thisPeriodHelpers_1.extractThisPeriod)(action.payload.firstMonthOfFY, timeframe, selectedCOABalancesRange, safeCoaBalances);
                 if (thisPeriod != null) {

package/lib/view/financeStatement/financeStatementState.d.ts

@@ -2,13 +2,6 @@ import { ReportUIState } from '../../commonStateTypes/viewAndReport/reportUIStat
 import { ReportID } from '../../commonStateTypes/viewAndReport/viewAndReport';
 import { ZeniDate } from '../../zeniDayJS';
 export interface FinanceStatementState extends ReportUIState {
-    /**
-     * True once the user has picked a period range themselves (date-range picker or
-     * a period column click), as opposed to the viewport-derived default the page
-     * applies on load. While true, automatic column refits must not resize the
-     * selection.
-     */
-    isPeriodSelectionExplicit: boolean;
     selectedReportId: ReportID;
     lastFetchRequestTime?: ZeniDate;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zeniai/client-epic-state",
-  "version": "5.2.39-betaRD0",
+  "version": "5.2.39",
   "description": "Shared module between Web & Mobile containing required abstractions for state management, async network communication. ",
   "main": "lib/index.js",
   "module": "lib/esm/index.js",