Add helper function

dmethvin-gov · Sep 14, 2018 · 31fd754 · 31fd754
1 parent 56c8743
commit 31fd754
Show file tree

Hide file tree

Showing 12 changed files with 51 additions and 33 deletions.
diff --git a/lib/js/helpers.js b/lib/js/helpers.js
diff --git a/lib/js/helpers.js.map b/lib/js/helpers.js.map
diff --git a/lib/js/review/ReviewChapters.js b/lib/js/review/ReviewChapters.js
diff --git a/lib/js/review/ReviewChapters.js.map b/lib/js/review/ReviewChapters.js.map
diff --git a/lib/js/review/SubmitController.js b/lib/js/review/SubmitController.js
diff --git a/lib/js/review/SubmitController.js.map b/lib/js/review/SubmitController.js.map
diff --git a/lib/js/routing.js b/lib/js/routing.js
diff --git a/lib/js/routing.js.map b/lib/js/routing.js.map
diff --git a/src/js/helpers.js b/src/js/helpers.js
@@ -523,6 +523,22 @@ export function expandArrayPages(pageList, data) {
   return result.currentList;
 }
 
+/**
+ * Gets active and expanded pages, in the correct order
+ *
+ * Any `showPagePerItem` pages are expanded to create items for each array item.
+ * We update the `path` for each of those pages to replace `:index` with the current item index.
+ *
+ * @param pages {Array<Object>} List of page configs
+ * @param data {Object} Current form data
+ * @returns {Array<Object>} A list of pages, includeing individual array
+ *   pages that are active
+ */
+export function getActiveExpandedPages(pages, data) {
+  const expandedPages = expandArrayPages(pages, data);
+  return getActivePages(expandedPages, data);
+}
+
 /**
  * getPageKeys returns a list of keys for the currently active pages
  *
@@ -532,8 +548,7 @@ export function expandArrayPages(pageList, data) {
  *   and the index if it’s a pagePerItem page
  */
 export function getPageKeys(pages, formData) {
-  const eligiblePageList = getActivePages(pages, formData);
-  const expandedPageList = expandArrayPages(eligiblePageList, formData);
+  const expandedPageList = getActiveExpandedPages(pages, formData);
 
   return expandedPageList.map(page => {
     let pageKey = page.pageKey;
@@ -554,8 +569,7 @@ export function getPageKeys(pages, formData) {
 export function getActiveChapters(formConfig, formData) {
   const formPages = createFormPageList(formConfig);
   const pageList = createPageList(formConfig, formPages);
-  const eligiblePageList = getActivePages(pageList, formData);
-  const expandedPageList = expandArrayPages(eligiblePageList, formData);
+  const expandedPageList = getActiveExpandedPages(pageList, formData);
 
   return _.uniq(expandedPageList.map(p => p.chapterKey).filter(key => !!key && key !== 'review'));
 }
@@ -578,3 +592,4 @@ export function omitRequired(schema) {
 
   return newSchema;
 }
+
diff --git a/src/js/review/ReviewChapters.jsx b/src/js/review/ReviewChapters.jsx
@@ -7,9 +7,8 @@ import Scroll from 'react-scroll';
 import ReviewCollapsibleChapter from './ReviewCollapsibleChapter';
 import {
   createPageListByChapter,
-  expandArrayPages,
+  getActiveExpandedPages,
   getActiveChapters,
-  getActivePages,
   getPageKeys
 } from '../helpers';
 import {
@@ -124,8 +123,7 @@ export function mapStateToProps(state, ownProps) {
   const chapters = chapterNames.map(chapterName => {
     const pages = pagesByChapter[chapterName];
 
-    let expandedPages = expandArrayPages(pages, formData);
-    expandedPages = getActivePages(expandedPages, formData);
+    const expandedPages = getActiveExpandedPages(pages, formData);
     const chapterFormConfig = formConfig.chapters[chapterName];
     const open = openChapters.includes(chapterName);
     const pageKeys = getPageKeys(pages, formData);

diff --git a/src/js/review/SubmitController.jsx b/src/js/review/SubmitController.jsx
@@ -8,8 +8,7 @@ import { PreSubmitSection } from '../components/PreSubmitSection';
 import { isValidForm } from '../validation';
 import {
   createPageListByChapter,
-  expandArrayPages,
-  getActivePages
+  getActiveExpandedPages
 } from '../helpers';
 import {
   setPreSubmit,
@@ -45,8 +44,7 @@ class SubmitController extends React.Component {
       router
     } = this.props;
 
-    const eligiblePageList = getActivePages(pageList, form.data);
-    const expandedPageList = expandArrayPages(eligiblePageList, this.props.form.data);
+    const expandedPageList = getActiveExpandedPages(pageList, form.data);
 
     // TODO: Fix this bug that assumes there is a confirmation page.
     // Actually, it assumes the app also doesn't add routes at the end!

diff --git a/src/js/routing.js b/src/js/routing.js
@@ -1,15 +1,12 @@
 import _ from 'lodash/fp';
-import { expandArrayPages, getActivePages } from './helpers';
+import { getActiveExpandedPages } from './helpers';
 
 /*
  * Returns the page list without conditional pages that have not satisfied
  * their dependencies and therefore should be skipped.
  */
 function getEligiblePages(pageList, data, pathname) {
-  // Any `showPagePerItem` pages are expanded to create items for each array item.
-  // We update the `path` for each of those pages to replace `:index` with the current item index.
-  const expandedPageList = expandArrayPages(pageList, data);
-  const eligiblePageList = getActivePages(expandedPageList, data);
+  const eligiblePageList = getActiveExpandedPages(pageList, data);
   const pageIndex = _.findIndex(item => item.path === pathname, eligiblePageList);
   return { pages: eligiblePageList, pageIndex };
 }