@ukhomeoffice/cop-react-form-renderer 4.73.0 → 4.75.0

package/dist/components/FormRenderer/onTaskAction.js

@@ -15,29 +15,32 @@ function _toPropertyKey(arg) { var key = _toPrimitive(arg, "string"); return _ty
 function _toPrimitive(input, hint) { if (_typeof(input) !== "object" || input === null) return input; var prim = input[Symbol.toPrimitive]; if (prim !== undefined) { var res = prim.call(input, hint || "default"); if (_typeof(res) !== "object") return res; throw new TypeError("@@toPrimitive must return a primitive value."); } return (hint === "string" ? String : Number)(input); }
 var onTaskAction = function onTaskAction(currentTask, pages, setCurrentTask, hubDetails, data, onPageChange, FormPages) {
   if (currentTask) {
-    currentTask.fullPages = [];
-    currentTask.pages.forEach(function (page) {
-      currentTask.fullPages.push(_helpers.default.getPage(page, pages));
+    var task = currentTask;
+    task.fullPages = [];
+    task.pages.forEach(function (page) {
+      task.fullPages.push(_helpers.default.getPage(page, pages));
     });
     setCurrentTask(currentTask);
-    var state = _objectSpread({}, currentTask);
-    if (currentTask.state === _models.TaskStates.TYPES.COMPLETE) {
+    var state = _objectSpread({}, task);
+    if (task.state === _models.TaskStates.TYPES.COMPLETE) {
       if (hubDetails !== null && hubDetails !== void 0 && hubDetails.noTaskCYAs) {
         // If the task is complete and there are no CYA's then show user first page
+        /* eslint-disable prefer-destructuring */
         var currentPage = data.formStatus.tasks[currentTask.name].currentPage;
-        onPageChange(currentTask.pages[0] || currentPage);
-      } else if (currentTask.customCYA) {
-        onPageChange(currentTask.customCYA);
+        onPageChange(task.pages[0] || currentPage);
+      } else if (task.customCYA) {
+        onPageChange(task.customCYA);
       } else {
         onPageChange(FormPages.CYA, state);
       }
-    } else if (currentTask.state === _models.TaskStates.TYPES.IN_PROGRESS) {
+    } else if (task.state === _models.TaskStates.TYPES.IN_PROGRESS) {
       var _currentPage = data.formStatus.tasks[currentTask.name].currentPage;
+      /* eslint-enable prefer-destructuring */
       onPageChange(_currentPage || currentTask.pages[0], state);
-    } else if (currentTask.firstPage) {
+    } else if (task.firstPage) {
       onPageChange(currentTask.firstPage, state);
     } else {
-      onPageChange(currentTask.pages[0], state);
+      onPageChange(task.pages[0], state);
     }
   }
 };

package/dist/components/PageActions/ActionButton.js

@@ -37,8 +37,5 @@ ActionButton.propTypes = {
   action: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.object]).isRequired,
   onAction: _propTypes.default.func.isRequired
 };
-ActionButton.defaultProps = {
-  action: ''
-};
 var _default = ActionButton;
 exports.default = _default;

package/dist/components/PageActions/PageActions.js

@@ -5,8 +5,8 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 var _copReactComponents = require("@ukhomeoffice/cop-react-components");
-var _propTypes = _interopRequireDefault(require("prop-types"));
 var _react = _interopRequireDefault(require("react"));
+var _propTypes = _interopRequireDefault(require("prop-types"));
 var _ActionButton = _interopRequireDefault(require("./ActionButton"));
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 // Global imports
@@ -28,7 +28,7 @@ var PageActions = function PageActions(_ref) {
   }));
 };
 PageActions.propTypes = {
-  actions: _propTypes.default.array,
+  actions: _propTypes.default.shape([]).isRequired,
   onAction: _propTypes.default.func.isRequired
 };
 var _default = PageActions;

package/dist/components/SummaryList/SummaryList.js

@@ -79,28 +79,33 @@ var SummaryList = function SummaryList(_ref) {
   }))));
 };
 SummaryList.propTypes = {
+  classBlock: _propTypes.default.string,
+  classModifiers: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.arrayOf(_propTypes.default.string)]),
+  className: _propTypes.default.string,
+  isGroup: _propTypes.default.bool,
+  noChangeAction: _propTypes.default.bool,
+  noGroupAction: _propTypes.default.bool,
   rows: _propTypes.default.arrayOf(_propTypes.default.shape({
     pageId: _propTypes.default.string.isRequired,
     fieldId: _propTypes.default.string.isRequired,
     full_path: _propTypes.default.string,
     key: _propTypes.default.string.isRequired,
-    value: _propTypes.default.any,
+    value: _propTypes.default.string,
     action: _propTypes.default.shape({
       page: _propTypes.default.string,
       label: _propTypes.default.string,
       aria_suffix: _propTypes.default.string,
       onAction: _propTypes.default.func
     })
-  })).isRequired,
-  noChangeAction: _propTypes.default.bool,
-  isGroup: _propTypes.default.bool,
-  classBlock: _propTypes.default.string,
-  classModifiers: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.arrayOf(_propTypes.default.string)]),
-  className: _propTypes.default.string
+  })).isRequired
 };
 SummaryList.defaultProps = {
   classBlock: DEFAULT_CLASS,
-  noChangeAction: false
+  classModifiers: [],
+  className: '',
+  isGroup: false,
+  noChangeAction: false,
+  noGroupAction: false
 };
 var _default = SummaryList;
 exports.default = _default;

package/dist/components/SummaryList/SummaryListHeadingRow.js

@@ -19,8 +19,12 @@ var SummaryListHeadingRow = function SummaryListHeadingRow(_ref) {
   }, size === 's' && /*#__PURE__*/_react.default.createElement(_copReactComponents.SmallHeading, null, title), size === 'm' && /*#__PURE__*/_react.default.createElement(_copReactComponents.MediumHeading, null, title));
 };
 SummaryListHeadingRow.propTypes = {
-  title: _propTypes.default.string.isRequired,
-  classes: _propTypes.default.func.isRequired
+  classes: _propTypes.default.func.isRequired,
+  size: _propTypes.default.string,
+  title: _propTypes.default.string.isRequired
+};
+SummaryListHeadingRow.defaultProps = {
+  size: 'm'
 };
 var _default = SummaryListHeadingRow;
 exports.default = _default;

package/dist/components/SummaryList/SummaryListRow.js

@@ -29,10 +29,11 @@ var SummaryListRow = function SummaryListRow(_ref) {
   })));
 };
 SummaryListRow.propTypes = {
+  classes: _propTypes.default.func.isRequired,
   row: _propTypes.default.shape({
     key: _propTypes.default.string.isRequired,
     required: _propTypes.default.bool,
-    value: _propTypes.default.any,
+    value: _propTypes.default.string,
     action: _propTypes.default.shape({
       page: _propTypes.default.string,
       label: _propTypes.default.string,
@@ -40,8 +41,10 @@ SummaryListRow.propTypes = {
       onAction: _propTypes.default.func
     })
   }).isRequired,
-  classes: _propTypes.default.func.isRequired,
   showAction: _propTypes.default.bool
 };
+SummaryListRow.defaultProps = {
+  showAction: false
+};
 var _default = SummaryListRow;
 exports.default = _default;

package/dist/components/SummaryList/helpers/getGroupActionAttributes.js

@@ -18,7 +18,8 @@ var getGroupActionAttributes = function getGroupActionAttributes(groupRow) {
         },
         tabIndex: '0'
       };
-    } else if (groupRow.action.page) {
+    }
+    if (groupRow.action.page) {
       return {
         href: "/".concat(groupRow.action.page)
       };

package/dist/components/SummaryList/helpers/getRowActionAttributes.js

@@ -18,7 +18,8 @@ var getRowActionAttributes = function getRowActionAttributes(row) {
         },
         tabIndex: '0'
       };
-    } else if (row.action.page) {
+    }
+    if (row.action.page) {
       return {
         href: "/".concat(row.action.page)
       };

package/dist/components/TaskList/Task.js

@@ -59,7 +59,7 @@ var Task = function Task(_ref) {
 Task.propTypes = {
   task: _propTypes.default.shape({
     name: _propTypes.default.string.isRequired,
-    pages: _propTypes.default.array.isRequired,
+    pages: _propTypes.default.shape([]).isRequired,
     state: _propTypes.default.string,
     displayName: _propTypes.default.string
   }).isRequired,

package/dist/components/TaskList/TaskList.js

@@ -46,7 +46,7 @@ var TaskList = function TaskList(_ref) {
     attrs = _objectWithoutProperties(_ref, _excluded);
   var classes = _copReactComponents.Utils.classBuilder(classBlock, classModifiers, className);
 
-  //TODO state will be retrieved from a document in S3 rather than given in the component definition, covered under COP-9885
+  // TODO state will be retrieved from a document in S3 rather than given in the component definition, covered under COP-9885
   var _sections$filter$redu = sections.filter(function (section) {
       return !section.skipped;
     }).reduce(function (acc, current) {
@@ -61,7 +61,7 @@ var TaskList = function TaskList(_ref) {
     completeSections = _sections$filter$redu2[0],
     totalSections = _sections$filter$redu2[1];
   var notesId = "".concat(id, "Notes");
-  var _ref2 = notes ? notes : {},
+  var _ref2 = notes && notes || {},
     notesTitle = _ref2.title,
     notesText = _ref2.text;
   var onClick = function onClick(task) {
@@ -115,32 +115,41 @@ var TaskList = function TaskList(_ref) {
   }));
 };
 TaskList.propTypes = {
+  classBlock: _propTypes.default.string,
+  classModifiers: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.arrayOf(_propTypes.default.string)]),
+  className: _propTypes.default.string,
+  fieldId: _propTypes.default.string,
+  formData: _propTypes.default.shape({}),
   id: _propTypes.default.string,
-  refTitle: _propTypes.default.string,
-  refNumber: _propTypes.default.string,
+  incompleteTitle: _propTypes.default.string,
   notes: _propTypes.default.shape({
     title: _propTypes.default.string,
     text: _propTypes.default.string
   }),
-  incompleteTitle: _propTypes.default.string,
+  onTaskAction: _propTypes.default.func,
+  refTitle: _propTypes.default.string.isRequired,
+  refNumber: _propTypes.default.string.isRequired,
   sections: _propTypes.default.arrayOf(_propTypes.default.shape({
     name: _propTypes.default.string.isRequired,
     label: _propTypes.default.string,
     tasks: _propTypes.default.arrayOf(_propTypes.default.shape({
       name: _propTypes.default.string.isRequired,
-      pages: _propTypes.default.array.isRequired,
+      pages: _propTypes.default.shape([]).isRequired,
       state: _propTypes.default.string,
       displayName: _propTypes.default.string
     })).isRequired
-  })).isRequired,
-  classBlock: _propTypes.default.string,
-  classModifiers: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.arrayOf(_propTypes.default.string)]),
-  className: _propTypes.default.string,
-  formData: _propTypes.default.shape({})
+  })).isRequired
 };
 TaskList.defaultProps = {
   classBlock: DEFAULT_CLASS,
-  incompleteTitle: DEFAULT_INCOMPLETE_TITLE
+  classModifiers: [],
+  className: '',
+  fieldId: '',
+  formData: {},
+  id: '',
+  incompleteTitle: DEFAULT_INCOMPLETE_TITLE,
+  onTaskAction: undefined,
+  notes: ''
 };
 var _default = TaskList;
 exports.default = _default;

package/dist/components/TaskList/TaskState.js

@@ -9,9 +9,9 @@ var _react = _interopRequireDefault(require("react"));
 var _copReactComponents = require("@ukhomeoffice/cop-react-components");
 var _models = require("../../models");
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-//Global Imports
+// Global Imports
 
-//Local imports
+// Local imports
 
 var DEFAULT_CLASS = 'hods-task-list';
 exports.DEFAULT_CLASS = DEFAULT_CLASS;

package/dist/components/index.js

@@ -9,6 +9,12 @@ Object.defineProperty(exports, "CheckYourAnswers", {
     return _CheckYourAnswers.default;
   }
 });
+Object.defineProperty(exports, "Container", {
+  enumerable: true,
+  get: function get() {
+    return _Container.default;
+  }
+});
 Object.defineProperty(exports, "FormComponent", {
   enumerable: true,
   get: function get() {
@@ -39,6 +45,7 @@ Object.defineProperty(exports, "SummaryList", {
     return _SummaryList.default;
   }
 });
+var _Container = _interopRequireDefault(require("./FormComponent/Container"));
 var _CheckYourAnswers = _interopRequireDefault(require("./CheckYourAnswers"));
 var _FormComponent = _interopRequireDefault(require("./FormComponent"));
 var _FormPage = _interopRequireDefault(require("./FormPage"));

package/dist/context/HooksContext/HooksContext.js

@@ -6,6 +6,8 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = exports.HooksContext = exports.ALLOWED_HOOKS = void 0;
 var _react = _interopRequireWildcard(require("react"));
+var _propTypes = _interopRequireDefault(require("prop-types"));
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || _typeof(obj) !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); enumerableOnly && (symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
@@ -87,4 +89,34 @@ var HooksContextProvider = function HooksContextProvider(_ref) {
   }, children);
 };
 var _default = HooksContextProvider;
-exports.default = _default;
+exports.default = _default;
+HooksContextProvider.propTypes = {
+  children: _propTypes.default.node,
+  overrides: _propTypes.default.shape({
+    onCancel: _propTypes.default.func,
+    onChange: _propTypes.default.func,
+    onFormComplete: _propTypes.default.func,
+    onFormLoad: _propTypes.default.func,
+    onGetComponent: _propTypes.default.func,
+    onGetCYA: _propTypes.default.func,
+    onGoingBack: _propTypes.default.func,
+    onPageChange: _propTypes.default.func,
+    onRequest: _propTypes.default.func,
+    onSubmit: _propTypes.default.func
+  })
+};
+HooksContextProvider.defaultProps = {
+  children: null,
+  overrides: {
+    onCancel: undefined,
+    onChange: undefined,
+    onFormComplete: undefined,
+    onFormLoad: undefined,
+    onGetComponent: undefined,
+    onGetCYA: undefined,
+    onGoingBack: undefined,
+    onPageChange: undefined,
+    onRequest: undefined,
+    onSubmit: undefined
+  }
+};

package/dist/context/ValidationContext/ValidationContext.js

@@ -6,6 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = exports.ValidationContext = void 0;
 var _react = _interopRequireWildcard(require("react"));
+var _propTypes = _interopRequireDefault(require("prop-types"));
 var _utils = _interopRequireDefault(require("../../utils"));
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
@@ -87,4 +88,10 @@ var ValidationContextProvider = function ValidationContextProvider(_ref) {
   }, children);
 };
 var _default = ValidationContextProvider;
-exports.default = _default;
+exports.default = _default;
+ValidationContextProvider.propTypes = {
+  children: _propTypes.default.node
+};
+ValidationContextProvider.defaultProps = {
+  children: null
+};

package/dist/context/ValidationContext/ValidationContext.test.js

@@ -4,6 +4,7 @@ function _typeof(obj) { "@babel/helpers - typeof"; return _typeof = "function" =
 var _react = require("@testing-library/react");
 var _copReactComponents = require("@ukhomeoffice/cop-react-components");
 var _react2 = _interopRequireWildcard(require("react"));
+var _propTypes = _interopRequireDefault(require("prop-types"));
 var _hooks = require("../../hooks");
 var _ValidationContext = _interopRequireDefault(require("./ValidationContext"));
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
@@ -23,7 +24,6 @@ var TestComponent = function TestComponent(_ref) {
     if (customErrors) {
       addErrors(customErrors);
     }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [customErrors]);
   return /*#__PURE__*/_react2.default.createElement(_react2.default.Fragment, null, typeof addErrors === 'function' && /*#__PURE__*/_react2.default.createElement("span", null, "addErrors is a function"), typeof clearErrors === 'function' && /*#__PURE__*/_react2.default.createElement("span", null, "clearErrors is a function"), typeof validate.page === 'function' && /*#__PURE__*/_react2.default.createElement("span", null, "validate.page is a function"), Array.isArray(errors) && /*#__PURE__*/_react2.default.createElement("span", null, "errors is an array of length ", errors.length), (errors === null || errors === void 0 ? void 0 : errors.length) > 0 && /*#__PURE__*/_react2.default.createElement(_copReactComponents.ErrorSummary, {
     errors: errors
@@ -74,4 +74,10 @@ describe('context.ValidationContext', function () {
       }
     }, _callee2);
   })));
-});
+});
+TestComponent.propTypes = {
+  customErrors: _propTypes.default.shape([])
+};
+TestComponent.defaultProps = {
+  customErrors: null
+};

package/dist/hooks/useGetRequest.js

@@ -22,9 +22,10 @@ function _arrayWithHoles(arr) { if (Array.isArray(arr)) return arr; }
 var cache = {};
 var errorCache = {};
 var clearCache = function clearCache(cacheObject) {
-  var keys = Object.keys(cacheObject);
+  var cachedObject = cacheObject;
+  var keys = Object.keys(cachedObject);
   keys.forEach(function (key) {
-    delete cacheObject[key];
+    delete cachedObject[key];
   });
 };
 var clear = function clear() {

package/dist/hooks/useRefData.js

@@ -28,6 +28,7 @@ exports.STATUS_LOADING = STATUS_LOADING;
 var STATUS_COMPLETE = 'complete';
 exports.STATUS_COMPLETE = STATUS_COMPLETE;
 var getRefDataUrl = function getRefDataUrl(component) {
+  // eslint-disable-next-line prefer-destructuring
   var data = component.data;
   if (data && !data.options) {
     return _copReactComponents.Utils.interpolateString(data.url, component.formData);

package/dist/index.js

@@ -4,6 +4,12 @@ function _typeof(obj) { "@babel/helpers - typeof"; return _typeof = "function" =
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
+Object.defineProperty(exports, "Container", {
+  enumerable: true,
+  get: function get() {
+    return _components.Container;
+  }
+});
 Object.defineProperty(exports, "FormComponent", {
   enumerable: true,
   get: function get() {

package/dist/utils/CheckYourAnswers/getCYAAction.js

@@ -29,12 +29,12 @@ var getPageFromCYALink = function getPageFromCYALink(link) {
  * - `href: '/bravo'` => `page: 'bravo'`
  */
 var getCYAAction = function getCYAAction(readonly, page, onAction) {
-  var cya_link = page === null || page === void 0 ? void 0 : page.cya_link;
-  if (readonly !== true && cya_link) {
+  var cyaLink = page === null || page === void 0 ? void 0 : page.cya_link;
+  if (readonly !== true && cyaLink) {
     return {
-      page: getPageFromCYALink(cya_link) || page.id || '#',
-      label: cya_link.label || 'Change',
-      aria_suffix: cya_link.aria_suffix,
+      page: getPageFromCYALink(cyaLink) || page.id || '#',
+      label: cyaLink.label || 'Change',
+      aria_suffix: cyaLink.aria_suffix,
       onAction: onAction
     };
   }

package/dist/utils/CheckYourAnswers/getCYAAction.test.js

@@ -7,9 +7,9 @@ function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { de
 describe('utils', function () {
   describe('CheckYourAnswers', function () {
     describe('getCYAAction', function () {
-      var getPage = function getPage(cya_link, id) {
+      var getPage = function getPage(cyaLink, id) {
         return {
-          cya_link: cya_link,
+          cya_link: cyaLink,
           id: id
         };
       };
@@ -20,100 +20,100 @@ describe('utils', function () {
         expect((0, _getCYAAction.default)(false, null, function () {})).toBeNull();
       });
       it('should return a default action if the cya_link is empty and the page has no id', function () {
-        var CYA_LINK = {};
-        var ON_ACTION = function ON_ACTION() {};
-        expect((0, _getCYAAction.default)(false, getPage(CYA_LINK), ON_ACTION)).toEqual({
+        var cyaLink = {};
+        var onAction = function onAction() {};
+        expect((0, _getCYAAction.default)(false, getPage(cyaLink), onAction)).toEqual({
           page: '#',
           label: 'Change',
-          onAction: ON_ACTION
+          onAction: onAction
         });
       });
       it('should return a default action if the cya_link is empty but the page has an id', function () {
         var PAGE_ID = 'page-id';
-        var CYA_LINK = {};
-        var ON_ACTION = function ON_ACTION() {};
-        expect((0, _getCYAAction.default)(false, getPage(CYA_LINK, PAGE_ID), ON_ACTION)).toEqual({
+        var cyaLink = {};
+        var onAction = function onAction() {};
+        expect((0, _getCYAAction.default)(false, getPage(cyaLink, PAGE_ID), onAction)).toEqual({
           page: PAGE_ID,
           label: 'Change',
-          onAction: ON_ACTION
+          onAction: onAction
         });
       });
       it('should use page specified in cya_link', function () {
         var PAGE = 'alpha';
-        var CYA_LINK = {
+        var cyaLink = {
           page: PAGE
         };
-        var ON_ACTION = function ON_ACTION() {};
-        expect((0, _getCYAAction.default)(false, getPage(CYA_LINK), ON_ACTION)).toEqual({
+        var onAction = function onAction() {};
+        expect((0, _getCYAAction.default)(false, getPage(cyaLink), onAction)).toEqual({
           page: PAGE,
           label: 'Change',
-          onAction: ON_ACTION
+          onAction: onAction
         });
       });
       it('should convert an href specified in cya_link', function () {
         var HREF = '/alpha/bravo';
-        var CYA_LINK = {
+        var cyaLink = {
           href: HREF
         };
-        var ON_ACTION = function ON_ACTION() {};
-        expect((0, _getCYAAction.default)(false, getPage(CYA_LINK), ON_ACTION)).toEqual({
+        var onAction = function onAction() {};
+        expect((0, _getCYAAction.default)(false, getPage(cyaLink), onAction)).toEqual({
           page: 'bravo',
           label: 'Change',
-          onAction: ON_ACTION
+          onAction: onAction
         });
       });
       it('should convert a url specified in cya_link', function () {
         var URL = '/alpha/bravo';
-        var CYA_LINK = {
+        var cyaLink = {
           url: URL
         };
-        var ON_ACTION = function ON_ACTION() {};
-        expect((0, _getCYAAction.default)(false, getPage(CYA_LINK), ON_ACTION)).toEqual({
+        var onAction = function onAction() {};
+        expect((0, _getCYAAction.default)(false, getPage(cyaLink), onAction)).toEqual({
           page: 'bravo',
           label: 'Change',
-          onAction: ON_ACTION
+          onAction: onAction
         });
       });
       it('should use label specified in cya_link', function () {
         var LABEL = 'Alpha Bravo Charlie';
-        var CYA_LINK = {
+        var cyaLink = {
           label: LABEL
         };
-        var ON_ACTION = function ON_ACTION() {};
-        expect((0, _getCYAAction.default)(false, getPage(CYA_LINK), ON_ACTION)).toEqual({
+        var onAction = function onAction() {};
+        expect((0, _getCYAAction.default)(false, getPage(cyaLink), onAction)).toEqual({
           page: '#',
           label: LABEL,
-          onAction: ON_ACTION
+          onAction: onAction
         });
       });
       it('should use aria_suffix specified in cya_link', function () {
         var ARIA_SUFFIX = 'This is hidden text';
-        var CYA_LINK = {
+        var cyaLink = {
           aria_suffix: ARIA_SUFFIX
         };
-        var ON_ACTION = function ON_ACTION() {};
-        expect((0, _getCYAAction.default)(false, getPage(CYA_LINK), ON_ACTION)).toEqual({
+        var onAction = function onAction() {};
+        expect((0, _getCYAAction.default)(false, getPage(cyaLink), onAction)).toEqual({
           page: '#',
           label: 'Change',
           aria_suffix: ARIA_SUFFIX,
-          onAction: ON_ACTION
+          onAction: onAction
         });
       });
       it('should use all properties specified in cya_link', function () {
         var PAGE = 'alpha';
         var LABEL = 'Alpha Bravo Charlie';
         var ARIA_SUFFIX = 'This is hidden text';
-        var CYA_LINK = {
+        var cyaLink = {
           label: LABEL,
           page: PAGE,
           aria_suffix: ARIA_SUFFIX
         };
-        var ON_ACTION = function ON_ACTION() {};
-        expect((0, _getCYAAction.default)(false, getPage(CYA_LINK), ON_ACTION)).toEqual({
+        var onAction = function onAction() {};
+        expect((0, _getCYAAction.default)(false, getPage(cyaLink), onAction)).toEqual({
           page: PAGE,
           label: LABEL,
           aria_suffix: ARIA_SUFFIX,
-          onAction: ON_ACTION
+          onAction: onAction
         });
       });
     });

package/dist/utils/CheckYourAnswers/getCYACollectionChangeAction.js

@@ -21,7 +21,8 @@ var getCYACollectionChangeAction = function getCYACollectionChangeAction(page, i
     type: 'action',
     action: {
       onAction: function onAction(row) {
-        var changeCallback = function changeCallback(data) {
+        var changeCallback = function changeCallback(cyaData) {
+          var data = cyaData;
           data["".concat(page.collection.name, "ActiveId")] = item.id;
           return data;
         };

package/dist/utils/CheckYourAnswers/getCYACollectionChangeAction.test.js

@@ -32,9 +32,9 @@ describe('utils', function () {
         collectionNameActiveId: 3
       };
       it('should change activeId to the selected item and call onAction with the correct page', function () {
-        var ON_ACTION = function ON_ACTION(row, call_back) {
+        var ON_ACTION = function ON_ACTION(row, callback) {
           expect(row.id).toEqual(2);
-          TEST_DATA = call_back(TEST_DATA);
+          TEST_DATA = callback(TEST_DATA);
         };
         var action = (0, _getCYACollectionChangeAction.default)(PAGE, ITEM, ON_ACTION, 0, {});
         action.action.onAction(PAGE);
@@ -58,9 +58,9 @@ describe('utils', function () {
             }]
           }
         };
-        var ON_ACTION = function ON_ACTION(page, call_back) {
+        var ON_ACTION = function ON_ACTION(page, callback) {
           expect(page.pageId).toEqual('confirmDelete');
-          TEST_DATA = call_back(TEST_DATA);
+          TEST_DATA = callback(TEST_DATA);
         };
         var action = (0, _getCYACollectionChangeAction.default)(PAGE_WITH_CUSTOM_DESTINATION, ITEM, ON_ACTION, 0, PAGE_WITH_CUSTOM_DESTINATION.collection.actions[0]);
         action.action.onAction({});

package/dist/utils/CheckYourAnswers/getCYACollectionDeleteAction.js

@@ -21,7 +21,8 @@ var getCYACollectionDeleteAction = function getCYACollectionDeleteAction(page, i
     type: 'action',
     action: {
       onAction: function onAction() {
-        var deleteCallback = function deleteCallback(data) {
+        var deleteCallback = function deleteCallback(cyaData) {
+          var data = cyaData;
           data["".concat(page.collection.name)] = data["".concat(page.collection.name)].filter(function (i) {
             return i.id !== item.id;
           });

package/dist/utils/CheckYourAnswers/getCYACollectionDeleteAction.test.js

@@ -35,8 +35,8 @@ describe('utils', function () {
       };
       it('should remove selected item from the form data', function () {
         var data = null;
-        var ON_ACTION = function ON_ACTION(_, call_back) {
-          data = call_back(TEST_DATA);
+        var ON_ACTION = function ON_ACTION(_, callback) {
+          data = callback(TEST_DATA);
         };
         var action = (0, _getCYACollectionDeleteAction.default)(PAGE, ITEM, ON_ACTION, 0, {});
         action.action.onAction();