diff --git a/client/src/components/bill-detail-edit/bill-detail-edit-component.jsx b/client/src/components/bill-detail-edit/bill-detail-edit-component.jsx index 9fdb915e5..d04f86cd2 100644 --- a/client/src/components/bill-detail-edit/bill-detail-edit-component.jsx +++ b/client/src/components/bill-detail-edit/bill-detail-edit-component.jsx @@ -56,7 +56,7 @@ export function BillDetailEditcontainer({ insertAuditTrail, bodyshop }) { const handleSave = () => { //It's got a previously deducted bill line! if ( - data.bills_by_pk.billlines.filter((b) => b.deductedfromlbr).length > 0 || + data?.bills_by_pk?.billlines.filter((b) => b.deductedfromlbr).length > 0 || form.getFieldValue("billlines").filter((b) => b.deductedfromlbr).length > 0 ) setOpen(true); @@ -84,7 +84,7 @@ export function BillDetailEditcontainer({ insertAuditTrail, bodyshop }) { //Find bill lines that were deleted. const deletedJobLines = []; - data.bills_by_pk.billlines.forEach((a) => { + data?.bills_by_pk?.billlines.forEach((a) => { const matchingRecord = billlines.find((b) => b.id === a.id); if (!matchingRecord) { deletedJobLines.push(a); @@ -151,8 +151,8 @@ export function BillDetailEditcontainer({ insertAuditTrail, bodyshop }) { if (error) return ; if (!search.billid) return <>; //
{t("bills.labels.noneselected")}
; - const exported = data?.bills_by_pk && data.bills_by_pk.exported; - const isinhouse = data?.bills_by_pk && data.bills_by_pk.isinhouse; + const exported = data?.bills_by_pk && data?.bills_by_pk?.exported; + const isinhouse = data?.bills_by_pk && data?.bills_by_pk?.isinhouse; return ( <> @@ -160,7 +160,7 @@ export function BillDetailEditcontainer({ insertAuditTrail, bodyshop }) { {data && ( <> @@ -192,15 +192,15 @@ export function BillDetailEditcontainer({ insertAuditTrail, bodyshop }) { {t("general.labels.media")} {bodyshop.uselocalmediaserver ? ( ) : ( )} @@ -212,7 +212,7 @@ export function BillDetailEditcontainer({ insertAuditTrail, bodyshop }) { } const transformData = (data) => { - return data + return data?.bills_by_pk ? { ...data.bills_by_pk, diff --git a/client/src/components/bill-form/bill-form.lines.component.jsx b/client/src/components/bill-form/bill-form.lines.component.jsx index fea62cbe0..396516903 100644 --- a/client/src/components/bill-form/bill-form.lines.component.jsx +++ b/client/src/components/bill-form/bill-form.lines.component.jsx @@ -5,6 +5,7 @@ import { useTranslation } from "react-i18next"; import { connect } from "react-redux"; import { createStructuredSelector } from "reselect"; import { selectBodyshop } from "../../redux/user/user.selectors"; +import { selectDarkMode } from "../../redux/application/application.selectors"; import CiecaSelect from "../../utils/Ciecaselect"; import InstanceRenderManager from "../../utils/instanceRenderMgr"; import BillLineSearchSelect from "../bill-line-search-select/bill-line-search-select.component"; @@ -14,7 +15,8 @@ import { bodyshopHasDmsKey } from "../../utils/dmsUtils.js"; const mapStateToProps = createStructuredSelector({ //currentUser: selectCurrentUser - bodyshop: selectBodyshop + bodyshop: selectBodyshop, + isDarkMode: selectDarkMode }); const mapDispatchToProps = () => ({ //setUserLanguage: language => dispatch(setUserLanguage(language)) @@ -22,6 +24,7 @@ const mapDispatchToProps = () => ({ export function BillEnterModalLinesComponent({ bodyshop, + isDarkMode, disabled, lineData, discount, @@ -249,19 +252,26 @@ export function BillEnterModalLinesComponent({ }; }, formInput: (record, index) => ( - - {() => { - const line = getFieldsValue(["billlines"]).billlines[index]; - if (!line) return null; - let lineDiscount = 1 - line.actual_cost / line.actual_price; - if (isNaN(lineDiscount)) lineDiscount = 0; - return ( - + + + + {() => { + const line = getFieldsValue(["billlines"]).billlines[index]; + if (!line) return null; + let lineDiscount = 1 - line.actual_cost / line.actual_price; + if (isNaN(lineDiscount)) lineDiscount = 0; + return ( + +
- - ); - }} - - } - /> +
+
+ ); + }} +
+
) // additional: (record, index) => ( // @@ -627,7 +637,8 @@ const EditableCell = ({ // DO NOT mutate rawProps; omit `key` immutably const propsFinal = rawProps ? (() => { - const { ...rest } = rawProps; + // eslint-disable-next-line no-unused-vars + const { key, ...rest } = rawProps; return rest; })() : undefined;