@zeniai/client-epic-state 5.2.12 → 5.2.13-betaAK0

package/lib/view/invoicing/invoicingQboView/invoicingQboViewSelector.js

@@ -1,11 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getInvoicingQboView = exports.invoicingAccountHierarchyForTypes = exports.invoicingAccountsForQboAccountTypes = exports.invoicingQboClassByQboId = exports.invoicingQboAccountByQboId = exports.INVOICING_INCOME_ACCOUNT_TYPES = exports.getInvoicingQboConnectionFetchState = exports.isInvoicingAccountingPickerDataLoading = exports.isInvoicingQboConnected = exports.INVOICING_ACCOUNT_LIST_KEY = void 0;
+exports.getInvoicingQboView = exports.invoicingAccountsForQboAccountTypes = exports.getInvoicingCatalogAccountingLookup = exports.invoicingQboClassByQboId = exports.invoicingQboAccountByQboId = exports.getInvoicingQboConnectionFetchState = exports.isInvoicingAccountingPickerDataLoading = exports.isInvoicingQboConnected = exports.INVOICING_ACCOUNT_LIST_KEY = void 0;
+const toolkit_1 = require("@reduxjs/toolkit");
 const accountListSelector_1 = require("../../accountList/accountListSelector");
 const classListSelector_1 = require("../../classList/classListSelector");
 // The shared bucket, fetched with the same arguments as the other consumers so
-// whichever screen loads first satisfies the rest. Invoicing narrows to income
-// accounts when it builds its options, so it does not need its own copy.
+// whichever screen loads first satisfies the rest. Invoicing renders the same
+// accounts they do, so it does not need its own copy.
 exports.INVOICING_ACCOUNT_LIST_KEY = 'accountList';
 /** Whether QBO is connected, without building the option lists to find out. */
 const isInvoicingQboConnected = (state) => state.invoicingQboViewState.connection?.connected === true;
@@ -35,14 +36,6 @@ exports.isInvoicingAccountingPickerDataLoading = isInvoicingAccountingPickerData
 /** The QBO connection fetch on its own, without building the bundled view. */
 const getInvoicingQboConnectionFetchState = (state) => state.invoicingQboViewState.connectionFetchState;
 exports.getInvoicingQboConnectionFetchState = getInvoicingQboConnectionFetchState;
-/**
- * Account types that can back invoicing revenue. Named here so screens ask for
- * "income accounts" rather than repeating the accounting service's vocabulary.
- */
-exports.INVOICING_INCOME_ACCOUNT_TYPES = [
-    'income',
-    'other_income',
-];
 /**
  * The account behind a picker value. The pickers submit the QuickBooks id, so
  * that is what callers hold; the payload builders need the whole entity to send
@@ -85,6 +78,17 @@ const invoicingQboClassByQboId = (state, qboId) => {
     }
 };
 exports.invoicingQboClassByQboId = invoicingQboClassByQboId;
+/**
+ * The resolver every catalog and settings payload builder needs: a picked
+ * QuickBooks id turned back into the account or class record the request body
+ * carries. Memoized so callers get one stable object rather than a fresh pair of
+ * closures per store change, which in `mapStateToProps` would re-render on every
+ * unrelated action.
+ */
+exports.getInvoicingCatalogAccountingLookup = (0, toolkit_1.createSelector)((state) => state, (state) => ({
+    accountByQboId: (qboId) => (0, exports.invoicingQboAccountByQboId)(state, qboId),
+    classByQboId: (qboId) => (0, exports.invoicingQboClassByQboId)(state, qboId),
+}));
 /**
  * Accounts and classes as the invoicing pickers need them. Both come from the
  * shared accounting-service lists, so whatever another screen already loaded
@@ -92,17 +96,12 @@ exports.invoicingQboClassByQboId = invoicingQboClassByQboId;
  * API stores; an account without one cannot be posted, so it is dropped rather
  * than offered.
  */
-const invoicingAccountingOptions = (state, accountTypes) => {
+const invoicingAccountingOptions = (state) => {
     const { accounts } = (0, accountListSelector_1.getAccountList)(state.accountState, state.accountListState, exports.INVOICING_ACCOUNT_LIST_KEY);
     return accounts.flatMap((account) => {
         if (account.qboId == null || account.isDeleted === true) {
             return [];
         }
-        if (accountTypes != null &&
-            (account.accountType == null ||
-                !accountTypes.includes(account.accountType))) {
-            return [];
-        }
         return [
             {
                 id: String(account.qboId),
@@ -160,27 +159,6 @@ const invoicingAccountingClassOptions = (state) => {
             },
         ]);
 };
-/**
- * The account tree narrowed to the given account types. The tree itself carries
- * no account type, so the flat account list supplies that and the matching
- * `qboId`s are kept — which is why callers that only need "income accounts, as a
- * tree" do not have to hold a second flat list to filter with.
- */
-const invoicingAccountHierarchyForTypes = (state, accountTypes) => {
-    const allowedQboIds = new Set(invoicingAccountingOptions(state, accountTypes).map((account) => account.id));
-    const keepMatching = (nodes) => nodes.flatMap((node) => {
-        const children = keepMatching(node.children);
-        const isMatch = node.qboId != null && allowedQboIds.has(String(node.qboId));
-        // A parent that is not itself a match is still kept when it holds one, so
-        // the tree's shape survives the filter.
-        if (!isMatch && children.length === 0) {
-            return [];
-        }
-        return [{ ...node, children }];
-    });
-    return keepMatching((0, accountListSelector_1.getNestedAccountListHierarchy)(state.accountListState, exports.INVOICING_ACCOUNT_LIST_KEY));
-};
-exports.invoicingAccountHierarchyForTypes = invoicingAccountHierarchyForTypes;
 /**
  * Single bundled view for the QBO integration settings screen — the
  * connection, the account and class options, sync health, and the fetch/action

package/package.json

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