@ukhomeoffice/cop-react-form-renderer 5.13.0-alpha → 5.13.1

package/dist/components/CollectionPage/CollectionPage.js

@@ -123,6 +123,7 @@ CollectionPage.propTypes = {
     actions: _propTypes.default.arrayOf(_propTypes.default.oneOfType([_propTypes.default.shape({}), _propTypes.default.string])),
     formData: _propTypes.default.shape({}).isRequired
   }).isRequired,
+  onCollectionChange: _propTypes.default.func,
   onAction: _propTypes.default.func.isRequired,
   hashLink: _propTypes.default.bool,
   classBlock: _propTypes.default.string,
@@ -130,6 +131,7 @@ CollectionPage.propTypes = {
   className: _propTypes.default.string
 };
 CollectionPage.defaultProps = {
+  onCollectionChange: undefined,
   hashLink: false,
   classBlock: DEFAULT_CLASS,
   classModifiers: [],

package/dist/components/FormComponent/Container.js

@@ -5,11 +5,11 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = exports.DEFAULT_CLASS = void 0;
+var _Utils = require("@ukhomeoffice/cop-react-components/dist/utils/Utils");
 var _react = _interopRequireDefault(require("react"));
 var _propTypes = _interopRequireDefault(require("prop-types"));
 var _models = require("../../models");
 var _utils = _interopRequireDefault(require("../../utils"));
-var _cleanAttributes = _interopRequireDefault(require("../../utils/Component/cleanAttributes"));
 var _FormComponent = _interopRequireDefault(require("./FormComponent"));
 var _excluded = ["container", "value", "formData", "onChange", "wrap", "onTopLevelChange"];
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
@@ -61,8 +61,8 @@ var Container = function Container(_ref) {
     }), formData);
   };
   var classes = _utils.default.classBuilder(DEFAULT_CLASS, [], container.className);
-  var htmlAttrs = (0, _cleanAttributes.default)(attrs);
-  return /*#__PURE__*/_react.default.createElement("div", _extends({}, htmlAttrs, {
+  var cleanedAttrs = (0, _Utils.cleanHtmlAttributes)(attrs);
+  return /*#__PURE__*/_react.default.createElement("div", _extends({}, cleanedAttrs, {
     className: classes(),
     id: container.id
   }), container.components && container.components.filter(shouldShow).map(function (component, index) {

package/dist/components/FormRenderer/onCYAAction.js

@@ -101,7 +101,11 @@ var onCYAAction = function onCYAAction(setPagePoint, action, pages, validate, co
     onPageChange(_helpers.default.getNextPageId(type, pages, pageId, action, (_formState$page = formState.page) === null || _formState$page === void 0 ? void 0 : _formState$page.formData));
   }
   if (action.type === _models.PageAction.TYPES.CANCEL) {
-    hooks.onCancel();
+    var formData = _utils.default.Format.form({
+      pages: pages,
+      components: components
+    }, _objectSpread({}, data), _models.EventTypes.CANCEL);
+    hooks.onCancel(formData);
   }
 };
 var _default = onCYAAction;

package/dist/components/FormRenderer/onCYAAction.test.js

@@ -102,15 +102,15 @@ describe('components.FormRenderer.onCYAAction', function () {
     onFormComplete: function onFormComplete() {
       this.onFormCompleteCalls += 1;
     },
-    onCancelCalls: 0,
-    onCancel: function onCancel() {
-      this.onCancelCalls += 1;
+    onCancelArgs: [],
+    onCancel: function onCancel(data) {
+      this.onCancelArgs.push(data);
     },
     reset: function reset() {
       this.onSubmitCalls = 0;
       this.onSubmitArgs = [];
       this.onFormCompleteCalls = 0;
-      this.onCancelCalls = 0;
+      this.onCancelArgs = [];
     }
   };
   var setPagePointCalls = 0;
@@ -578,6 +578,9 @@ describe('components.FormRenderer.onCYAAction', function () {
       }
     });
     _onCYAAction.default.apply(void 0, Object.values(CUSTOM_ARGS));
-    expect(MOCK_HOOKS.onCancelCalls).toEqual(1);
+    expect(MOCK_HOOKS.onCancelArgs.length).toEqual(1);
+    expect(MOCK_HOOKS.onCancelArgs[0]).toMatchObject({
+      id: 'formId'
+    });
   });
 });

package/dist/components/FormRenderer/onPageAction.js

@@ -20,11 +20,6 @@ function _toPrimitive(input, hint) { if (_typeof(input) !== "object" || input ==
 // patchLabel captures non-ID values 
 // for display purposes after submission.)
 var onPageAction = function onPageAction(action, patch, patchLabel, hooks, data, formState, validate, onPageChange, type, pages, components, pageId, setPagePoint, currentTask, setData, hubDetails, setSubmitted, addErrors) {
-  if (action.type === _models.PageAction.TYPES.CANCEL) {
-    hooks.onCancel();
-    return;
-  }
-  ;
   // Save a copy of data in case submit errors and we need to revert
   var preSubmitData = _objectSpread({}, data);
   var form = formState;
@@ -36,6 +31,14 @@ var onPageAction = function onPageAction(action, patch, patchLabel, hooks, data,
   // This should normally have no effect but will prevent issues 
   // with validation if formData happens to have been wiped.
   form.page.formData = _objectSpread(_objectSpread({}, form.page.formData), _patch);
+
+  // Exposes form data to consuming application.
+  if (action.type === _models.PageAction.TYPES.CANCEL) {
+    hooks.onCancel(form.page.formData);
+    return;
+  }
+  ;
+
   // Check to see whether the action is able to proceed, which in
   // in the case of a submission will validate the fields in the page.
   if (_helpers.default.canActionProceed(action, form.page, validate.page)) {

package/dist/components/FormRenderer/onPageAction.test.js

@@ -121,15 +121,15 @@ describe('components.FormRenderer.onPageAction', function () {
     onFormComplete: function onFormComplete() {
       this.onFormCompleteCalls += 1;
     },
-    onCancelCalls: 0,
-    onCancel: function onCancel() {
-      this.onCancelCalls += 1;
+    onCancelArgs: [],
+    onCancel: function onCancel(data) {
+      this.onCancelArgs.push(data);
     },
     reset: function reset() {
       this.onSubmitCalls = 0;
       this.onSubmitArgs = [];
       this.onFormCompleteCalls = 0;
-      this.onCancelCalls = 0;
+      this.onCancelArgs = [];
       this.onActionCalls = 0;
     },
     onActionCalls: 0,
@@ -290,7 +290,8 @@ describe('components.FormRenderer.onPageAction', function () {
       action: ACTION
     });
     _onPageAction.default.apply(void 0, Object.values(CUSTOM_ARGS));
-    expect(MOCK_HOOKS.onCancelCalls).toEqual(1);
+    expect(MOCK_HOOKS.onCancelArgs.length).toEqual(1);
+    expect(MOCK_HOOKS.onCancelArgs[0]).toMatchObject({});
     // Shouldn't get to the point of validation if
     // action is a cancel action.
     expect(_helpers.default.canActionProceedCalls).toEqual(0);

package/dist/components/SummaryList/SummaryList.js

@@ -29,6 +29,7 @@ var SummaryList = function SummaryList(_ref) {
     className = _ref.className,
     attrs = _objectWithoutProperties(_ref, _excluded);
   var classes = _copReactComponents.Utils.classBuilder(classBlock, classModifiers, className);
+  var cleanedHtmlAttrs = _copReactComponents.Utils.cleanHtmlAttributes(attrs);
   var groupActionRow = null;
   if (isGroup) {
     var rowActions = rows.filter(function (r) {
@@ -38,7 +39,7 @@ var SummaryList = function SummaryList(_ref) {
   }
   return /*#__PURE__*/_react.default.createElement("div", {
     className: "group-of-rows"
-  }, /*#__PURE__*/_react.default.createElement("dl", _extends({}, attrs, {
+  }, /*#__PURE__*/_react.default.createElement("dl", _extends({}, cleanedHtmlAttrs, {
     className: classes()
   }), rows.map(function (row) {
     var key = "".concat(row.pageId, "_").concat(row.full_path || row.fieldId);

package/dist/components/TaskList/TaskList.js

@@ -45,7 +45,7 @@ var TaskList = function TaskList(_ref) {
     formData = _ref.formData,
     attrs = _objectWithoutProperties(_ref, _excluded);
   var classes = _copReactComponents.Utils.classBuilder(classBlock, classModifiers, className);
-
+  var cleanedHtmlAttrs = _copReactComponents.Utils.cleanHtmlAttributes(attrs);
   // 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;
@@ -71,7 +71,7 @@ var TaskList = function TaskList(_ref) {
   };
   return /*#__PURE__*/_react.default.createElement("div", _extends({
     id: id
-  }, attrs, {
+  }, cleanedHtmlAttrs, {
     className: classes()
   }), refNumber && refTitle && /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, /*#__PURE__*/_react.default.createElement("p", {
     className: "govuk-body govuk-!-margin-bottom-0"
@@ -103,7 +103,7 @@ var TaskList = function TaskList(_ref) {
       className: classes('section')
     }, sections.length > 1 ? "".concat(index + 1, ". ") : '', section.name), /*#__PURE__*/_react.default.createElement("ol", {
       className: classes('items')
-    }, /*#__PURE__*/_react.default.createElement(_dist.Hint, null, section.label ? section.label : ''), section.tasks.filter(function (task) {
+    }, /*#__PURE__*/_react.default.createElement("li", null, /*#__PURE__*/_react.default.createElement(_dist.Hint, null, section.label ? section.label : '')), section.tasks.filter(function (task) {
       return task.state !== _models.TaskStates.TYPES.SKIPPED;
     }).map(function (task) {
       return /*#__PURE__*/_react.default.createElement(_Task.default, {

package/dist/context/HooksContext/HooksContext.js

@@ -56,7 +56,7 @@ var DEFAULT_HOOKS = {
     return errors;
   },
   // Returns the list of errors already found in FR by default.
-  onCancel: function onCancel() {} // Consuming application must provide function to action, else this does nothing.
+  onCancel: function onCancel(data) {} // Consuming application must provide function to action, else this does nothing.
 };
 
 var ALLOWED_HOOKS = Object.keys(DEFAULT_HOOKS);

package/dist/models/EventTypes.js

@@ -6,9 +6,11 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = void 0;
 var TYPE_BLUR = 'blur';
 var TYPE_SUBMIT = 'submit';
+var TYPE_CANCEL = 'cancel';
 var EventTypes = {
   BLUR: TYPE_BLUR,
-  SUBMIT: TYPE_SUBMIT
+  SUBMIT: TYPE_SUBMIT,
+  CANCEL: TYPE_CANCEL
 };
 var _default = EventTypes;
 exports.default = _default;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ukhomeoffice/cop-react-form-renderer",
-  "version": "5.13.0-alpha",
+  "version": "5.13.1",
   "private": false,
   "scripts": {
     "clean": "rimraf dist",
@@ -16,7 +16,7 @@
     "post-compile": "rimraf dist/*.test.* dist/**/*.test.* dist/**/*.stories.* dist/docs dist/assets"
   },
   "dependencies": {
-    "@ukhomeoffice/cop-react-components": "^3.4.0",
+    "@ukhomeoffice/cop-react-components": "^3.4.1",
     "axios": "^0.23.0",
     "dayjs": "^1.11.0",
     "govuk-frontend": "^4.3.1",