Merged PR 2239: refactor(newappeal): replace BuildCheckRow formatter chain with map (Slice 6)

refactor(newappeal): replace BuildCheckRow formatter chain with map (Slice 6)

Related work items: #22587
This commit is contained in:
Robert Bond
2026-04-13 11:53:45 +00:00
parent 20eea5b128
commit 783786b9eb
2 changed files with 141 additions and 180 deletions
+101 -140
View File
@@ -138,72 +138,59 @@ let BuildCheckRow = (props) => {
// datafieldname[0].value // datafieldname[0].value
// ] != "undefined" // ] != "undefined"
// ) { // ) {
return ( const fieldTypeValue = fieldtype[0].value;
<div className="govuk-summary-list__row" key={key}> const fieldValue =
<dt className="govuk-summary-list__key"> props.props.form["appealForm"].values[
{FieldsTranslations(rows[0].value)} datafieldname[0].value
</dt> ];
<dd className="govuk-summary-list__value"> const renderFileUploadValue = () => (
{fieldtype[0].value ==
"{E0DECE4B-6666-4a8f-A065-082708572369}" ? (
<div
dangerouslySetInnerHTML={{
__html: props.props.form[
"appealForm"
].values[datafieldname[0].value]
}}
/>
) : fieldtype[0].value ==
"{5B773807-9FB2-42db-97C3-7A91EFF8ADFF}" ? (
formatDate(
props.props.form["appealForm"].values[
datafieldname[0].value
]
)
) : fieldtype[0].value ==
"{3EF39988-22BB-4f0b-BBBE-64B5A3748AEE}" ? (
getPickListLabel(
props.props.formData.pickListData,
datafieldname[0].value,
props.props.form["appealForm"].values[
datafieldname[0].value
]
)
) : fieldtype[0].value ==
"{07FAC785-CD58-4f9f-ABB3-4B7DDC6ED5ED}" ? (
getRadioLabel(
props.props.formData.pickListData,
datafieldname[0].value,
props.props.form["appealForm"].values[
datafieldname[0].value
]
)
) : fieldtype[0].value ==
"{67FAC785-CD58-4f9f-ABB3-4B7DDC6ED5ED}" ? (
props.props.form["appealForm"].values[
datafieldname[0].value
] == "false" ? (
router.locale == "cy" ? (
"Nac ydw"
) : (
"No"
)
) : router.locale == "cy" ? (
"Oes"
) : (
"Yes"
)
) : fieldtype[0].value ==
"{16D63FD6-119B-4353-BDCA-18358721C3FE}" ? (
<> <>
{props.filesArray.map((blob, i) => ( {props.filesArray.map((blob, i) => (
<div key={i}>
{getDocumentTypeFromFilename(blob.name) ===
documentTypeCode[0].value && (
<div
style={{
display: "flex",
alignItems: "flex-start"
}}
className="govuk-!-margin-bottom-2"
>
<img
style={{
minWidth: "50",
width: "50"
}}
src={getThumbnailIconByExtension(
blob.name
)}
alt={blob.name}
width="50"
/>
<span
style={{
maxWidth: "85%"
}}
className="govuk-body govuk-!-font-size-14 govuk-!-margin-left-2"
>
{blob.name} -{" "}
{bytesToSize(blob.size)}{" "}
</span>
</div>
)}
</div>
))}
<br />
{typeof fieldValue != "undefined" &&
fieldValue.map(
(blob, i) =>
isObjectAndNotEmpty(blob) && (
<div key={i}> <div key={i}>
{getDocumentTypeFromFilename( {getDocumentTypeFromFilename(
blob.name blob.name
) === ) ===
documentTypeCode[0] documentTypeCode[0].value && (
.value && (
<div <div
style={{ style={{
display: "flex", display: "flex",
@@ -237,81 +224,14 @@ let BuildCheckRow = (props) => {
</div> </div>
)} )}
</div> </div>
))}
<br />
{typeof props.props.form["appealForm"]
.values[datafieldname[0].value] !=
"undefined" &&
props.props.form[
"appealForm"
].values[
datafieldname[0].value
].map(
(blob, i) =>
isObjectAndNotEmpty(
blob
) && (
<div key={i}>
{getDocumentTypeFromFilename(
blob.name
) ===
documentTypeCode[0]
.value && (
<div
style={{
display:
"flex",
alignItems:
"flex-start"
}}
className="govuk-!-margin-bottom-2"
>
<img
style={{
minWidth:
"50",
width: "50"
}}
src={getThumbnailIconByExtension(
blob.name
)}
alt={
blob.name
}
width="50"
/>
<span
style={{
maxWidth:
"85%"
}}
className="govuk-body govuk-!-font-size-14 govuk-!-margin-left-2"
>
{
blob.name
}{" "}
-{" "}
{bytesToSize(
blob.size
)}{" "}
</span>
</div>
)}
</div>
) )
)} )}
</> </>
) : fieldtype[0].value == );
"{0273EDBD-AC1D-40d3-9FB2-095C621B552D}" ? (
typeof props.props.form["appealForm"] const renderTenantValue = () =>
.values[datafieldname[0].value] != typeof fieldValue != "undefined" && fieldValue != null ? (
"undefined" && fieldValue.map((tenant, index) => {
props.props.form["appealForm"].values[
datafieldname[0].value
] != null ? (
props.props.form["appealForm"].values[
datafieldname[0].value
].map((tenant, index) => {
console.log( console.log(
typeof tenant.pinswg_owners_firstname typeof tenant.pinswg_owners_firstname
); );
@@ -338,9 +258,7 @@ let BuildCheckRow = (props) => {
</div> </div>
<div className="govuk-!-width-one-half"> <div className="govuk-!-width-one-half">
Served:{" "} Served:{" "}
{formatDate( {formatDate(tenant.dateserved)}
tenant.dateserved
)}
</div> </div>
</div> </div>
) : ( ) : (
@@ -349,12 +267,55 @@ let BuildCheckRow = (props) => {
}) })
) : ( ) : (
"N/A" "N/A"
) );
) : (
props.props.form["appealForm"].values[ const formatterMap = {
datafieldname[0].value "{E0DECE4B-6666-4a8f-A065-082708572369}": () => (
] <div
)} dangerouslySetInnerHTML={{
__html: fieldValue
}}
/>
),
"{5B773807-9FB2-42db-97C3-7A91EFF8ADFF}": () =>
formatDate(fieldValue),
"{3EF39988-22BB-4f0b-BBBE-64B5A3748AEE}": () =>
getPickListLabel(
props.props.formData.pickListData,
datafieldname[0].value,
fieldValue
),
"{07FAC785-CD58-4f9f-ABB3-4B7DDC6ED5ED}": () =>
getRadioLabel(
props.props.formData.pickListData,
datafieldname[0].value,
fieldValue
),
"{67FAC785-CD58-4f9f-ABB3-4B7DDC6ED5ED}": () =>
fieldValue == "false"
? router.locale == "cy"
? "Nac ydw"
: "No"
: router.locale == "cy"
? "Oes"
: "Yes",
"{16D63FD6-119B-4353-BDCA-18358721C3FE}":
renderFileUploadValue,
"{0273EDBD-AC1D-40d3-9FB2-095C621B552D}":
renderTenantValue
};
const defaultFormatter = () => fieldValue;
const renderFieldValue =
formatterMap[fieldTypeValue] || defaultFormatter;
return (
<div className="govuk-summary-list__row" key={key}>
<dt className="govuk-summary-list__key">
{FieldsTranslations(rows[0].value)}
</dt>
<dd className="govuk-summary-list__value">
{renderFieldValue()}
</dd> </dd>
<dd className="govuk-summary-list__actions"> <dd className="govuk-summary-list__actions">
+2 -2
View File
@@ -4,7 +4,7 @@ Base branch for this tracker: `refactor`
## Status ## Status
Current slice: Slice 6BuildCheckRow Formatter Map Current slice: Slice 7Props Boundary Cleanup
Status: READY TO START Status: READY TO START
--- ---
@@ -62,7 +62,7 @@ Status: COMPLETE
### Slice 6 — BuildCheckRow Formatter Map ### Slice 6 — BuildCheckRow Formatter Map
Status: NOT STARTED Status: COMPLETE
--- ---