@abgov/jsonforms-components 2.73.1 → 2.74.0

package/index.esm.js

@@ -12791,6 +12791,33 @@ class ListWithDetailControl extends React.Component {
   }
 }
 
+// clean-code-ignore: RULE-19 — covered by ./useReviewChange.spec.tsx, colocated. The rule looks for a
+// .test.ts sibling; the hook has to be exercised through a rendered component, so its spec is .tsx.
+/**
+ * The single way a review summary's Change button reports a click.
+ *
+ * Two things have to happen and neither is optional. The stepper moves to the page, which is all a
+ * host needs when the summary and the form are the same JsonForms instance. The review context is
+ * also told, and that is the only signal a host whose summary lives on a *different* page can see —
+ * it is what turns into a deep link back into the form.
+ *
+ * Renderers used to wire these up one at a time, and every renderer except the plain-input one
+ * called goToPage alone, so a composite control's Change button was silently inert for a cross-page
+ * host. Going through here makes forgetting the second half impossible.
+ */
+const useReviewChange = () => {
+  const stepperContext = useContext(JsonFormsStepperContext);
+  const reviewContext = useReviewContext();
+  return useCallback((stepId, scope) => {
+    // Outside a stepper there is no page to move to, but the host may still resolve the field's
+    // own page from the scope, so the report below happens either way.
+    if (stepId !== undefined) {
+      stepperContext == null || stepperContext.goToPage(stepId, scope);
+    }
+    reviewContext == null || reviewContext.onChangeDispatch(stepId, scope != null ? scope : '');
+  }, [stepperContext, reviewContext]);
+};
+
 const GoAInputBaseTableReview = props => {
   var _uischema$options, _uischema$scope2, _jsonForms$core, _uischema$options4, _uischema$options5, _uischema$options6;
   const {
@@ -12806,8 +12833,7 @@ const GoAInputBaseTableReview = props => {
     required,
     visible
   } = props;
-  const context = useContext(JsonFormsStepperContext);
-  const reviewCtx = useReviewContext();
+  const reportChange = useReviewChange();
   const showChangeButtons = useShowChangeButtons();
   const jsonForms = useJsonForms();
   const reviewLabel = typeof ((_uischema$options = uischema.options) == null ? void 0 : _uischema$options.reviewLabel) === 'string' ? uischema.options.reviewLabel : '';
@@ -12979,8 +13005,7 @@ const GoAInputBaseTableReview = props => {
   // eslint-disable-next-line
   const stepId = (_uischema$options5 = uischema.options) == null ? void 0 : _uischema$options5.stepId;
   function handleChangeClick() {
-    context == null || context.goToPage(stepId, uischema.scope);
-    reviewCtx == null || reviewCtx.onChangeDispatch(stepId, uischema.scope);
+    reportChange(stepId, uischema.scope);
   }
   const showInlineValue = reviewText === null || reviewText === undefined || typeof reviewText === 'string' || typeof reviewText === 'number' || /*#__PURE__*/React.isValidElement(reviewText) || Array.isArray(reviewText) && reviewText.length === 0;
   const isValueEmpty = !isBooleanRequiredError && (reviewText === null || reviewText === undefined || reviewText === '' || Array.isArray(reviewText) && reviewText.length === 0);
@@ -13521,7 +13546,7 @@ const FullNameTester = rankWith(4, isFullName);
 
 const FullNameControlReview = props => {
   var _props$uischema, _props$schema$require, _props$schema;
-  const context = useContext(JsonFormsStepperContext);
+  const reportChange = useReviewChange();
   const showChangeButtons = useShowChangeButtons();
   const stepId = (_props$uischema = props.uischema) == null || (_props$uischema = _props$uischema.options) == null ? void 0 : _props$uischema.stepId;
   const {
@@ -13543,7 +13568,7 @@ const FullNameControlReview = props => {
             })]
           }), showChangeButtons && stepId !== undefined && !((_uischema$options = uischema.options) != null && (_uischema$options = _uischema$options.componentProps) != null && _uischema$options.readOnly) && jsx(GoabButton, {
             type: "text",
-            onClick: () => context == null ? void 0 : context.goToPage(stepId, uischema.scope),
+            onClick: () => reportChange(stepId, uischema.scope),
             testId: `${fieldName}-change-btn`,
             children: "Change"
           })]
@@ -13575,7 +13600,7 @@ const FullNameDobReviewControl = props => {
     id,
     uischema
   } = props;
-  const context = useContext(JsonFormsStepperContext);
+  const reportChange = useReviewChange();
   const showChangeButtons = useShowChangeButtons();
   const stepId = uischema == null || (_uischema$options = uischema.options) == null ? void 0 : _uischema$options.stepId;
   const requiredFields = (_props$schema$require = (_props$schema = props.schema) == null ? void 0 : _props$schema.required) != null ? _props$schema$require : [];
@@ -13593,7 +13618,7 @@ const FullNameDobReviewControl = props => {
           }), showChangeButtons && stepId !== undefined && !((_uischema$options2 = uischema.options) != null && (_uischema$options2 = _uischema$options2.componentProps) != null && _uischema$options2.readOnly) && jsx(GoabButton, {
             type: "text",
             size: "compact",
-            onClick: () => context == null ? void 0 : context.goToPage(stepId, uischema.scope),
+            onClick: () => reportChange(stepId, uischema.scope),
             testId: `${fieldName}-change-btn`,
             children: "Change"
           })]
@@ -13962,7 +13987,7 @@ const ContractInfoControl = props => {
 
 const ContractInfoControlReview = props => {
   var _props$uischema, _schema$required, _options$enableEmail, _options$enablePhone, _options$emailFirst;
-  const context = useContext(JsonFormsStepperContext);
+  const reportChange = useReviewChange();
   const showChangeButtons = useShowChangeButtons();
   const stepId = (_props$uischema = props.uischema) == null || (_props$uischema = _props$uischema.options) == null ? void 0 : _props$uischema.stepId;
   const {
@@ -13984,7 +14009,7 @@ const ContractInfoControlReview = props => {
           }), showChangeButtons && stepId !== undefined && !((_uischema$options = uischema.options) != null && (_uischema$options = _uischema$options.componentProps) != null && _uischema$options.readOnly) && jsx(GoabButton, {
             type: "text",
             size: "compact",
-            onClick: () => context == null ? void 0 : context.goToPage(stepId, uischema.scope),
+            onClick: () => reportChange(stepId, uischema.scope),
             testId: `${fieldName}-change-btn`,
             children: "Change"
           })]
@@ -14611,7 +14636,7 @@ const ObjectArrayControl = props => {
   const [open, setOpen] = useState(false);
   const [rowData, setRowData] = useState(0);
   const [maxItemsError, setMaxItemsError] = useState('');
-  const context = useContext(JsonFormsStepperContext);
+  const reportChange = useReviewChange();
   const showChangeButtons = useShowChangeButtons();
   const {
       label,
@@ -14798,7 +14823,7 @@ const ObjectArrayControl = props => {
             size: "compact",
             onClick: () => {
               var _uischema$options8;
-              return context == null ? void 0 : context.goToPage((_uischema$options8 = uischema.options) == null ? void 0 : _uischema$options8.stepId);
+              return reportChange((_uischema$options8 = uischema.options) == null ? void 0 : _uischema$options8.stepId, uischema.scope);
             },
             children: "Change"
           })]
@@ -15316,7 +15341,7 @@ const AddressLoopUpControlTableReview = props => {
   } = props;
   const jsonForms = useJsonForms();
   const stepId = (_uischema$options = uischema.options) == null ? void 0 : _uischema$options.stepId;
-  const formStepperCtx = useContext(JsonFormsStepperContext);
+  const reportChange = useReviewChange();
   const showChangeButtons = useShowChangeButtons();
   const isAlbertaAddress = (schema == null || (_schema$properties2 = schema.properties) == null || (_schema$properties2 = _schema$properties2.subdivisionCode) == null ? void 0 : _schema$properties2.const) === 'AB';
   const provinces = [{
@@ -15429,7 +15454,7 @@ const AddressLoopUpControlTableReview = props => {
           }), showButton && showChangeButtons && stepId !== undefined && !((_uischema$options2 = uischema.options) != null && (_uischema$options2 = _uischema$options2.componentProps) != null && _uischema$options2.readOnly) && jsx(GoabButton, {
             type: "text",
             size: "compact",
-            onClick: () => formStepperCtx == null ? void 0 : formStepperCtx.goToPage(stepId, targetScope),
+            onClick: () => reportChange(stepId, targetScope),
             testId: "address-change-btn",
             children: "Change"
           })]
@@ -15455,7 +15480,7 @@ const AddressLoopUpControlTableReview = props => {
           }), showChangeButtons && stepId !== undefined && !((_uischema$options3 = uischema.options) != null && (_uischema$options3 = _uischema$options3.componentProps) != null && _uischema$options3.readOnly) && jsx(GoabButton, {
             type: "text",
             size: "compact",
-            onClick: () => formStepperCtx == null ? void 0 : formStepperCtx.goToPage(stepId, targetScope),
+            onClick: () => reportChange(stepId, targetScope),
             testId: "address-change-btn",
             children: "Change"
           })]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abgov/jsonforms-components",
-  "version": "2.73.1",
+  "version": "2.74.0",
   "license": "Apache-2.0",
   "description": "Government of Alberta - React renderers for JSON Forms based on the design system.",
   "repository": "https://github.com/GovAlta/adsp-monorepo",

package/renderer-catalog.json

@@ -1,7 +1,7 @@
 {
   "schemaVersion": "1.0.0",
-  "generatedAt": "2026-08-27T22:10:25.781Z",
-  "sourceCommit": "8963e4fedf4c8f323ddc21c852273f0d75b2f99f",
+  "generatedAt": "2026-09-03T20:26:40.184Z",
+  "sourceCommit": "a9f11f1d8a8d4394f1d1bdfcd5591d245e024d51",
   "sourcePath": "libs/jsonforms-components/src/index.ts",
   "rendererCount": 33,
   "renderers": [

package/src/lib/Controls/FormStepper/context/useReviewChange.d.ts

@@ -0,0 +1,13 @@
+/**
+ * The single way a review summary's Change button reports a click.
+ *
+ * Two things have to happen and neither is optional. The stepper moves to the page, which is all a
+ * host needs when the summary and the form are the same JsonForms instance. The review context is
+ * also told, and that is the only signal a host whose summary lives on a *different* page can see —
+ * it is what turns into a deep link back into the form.
+ *
+ * Renderers used to wire these up one at a time, and every renderer except the plain-input one
+ * called goToPage alone, so a composite control's Change button was silently inert for a cross-page
+ * host. Going through here makes forgetting the second half impossible.
+ */
+export declare const useReviewChange: () => ((stepId: number | undefined, scope?: string) => void);