From 4b9ea58f1b368bb813dae601a087d1557117de87 Mon Sep 17 00:00:00 2001
From: Juho Kettunen <juho.kettunen@nordcloud.com>
Date: Wed, 13 Nov 2024 13:15:36 +0200
Subject: [PATCH] Fix hiding the field from MaKe

---
 .../components/leaseSections/rent/ContractRentEdit.tsx   | 8 ++++----
 .../components/leaseSections/rent/ContractRents.tsx      | 4 +++-
 .../components/leaseSections/rent/ContractRentsEdit.tsx  | 9 +++------
 src/leases/components/leaseSections/rent/RentItem.tsx    | 4 +++-
 .../components/leaseSections/rent/RentItemEdit.tsx       | 6 ++++--
 src/leases/components/leaseSections/rent/Rents.tsx       | 2 +-
 src/leases/components/leaseSections/rent/RentsEdit.tsx   | 9 ++++++---
 7 files changed, 24 insertions(+), 18 deletions(-)

diff --git a/src/leases/components/leaseSections/rent/ContractRentEdit.tsx b/src/leases/components/leaseSections/rent/ContractRentEdit.tsx
index 3a10f111e..b3e8b4168 100644
--- a/src/leases/components/leaseSections/rent/ContractRentEdit.tsx
+++ b/src/leases/components/leaseSections/rent/ContractRentEdit.tsx
@@ -35,6 +35,7 @@ type Props = {
   receivableTypes: Array<Record<string, any>>;
   rentField: string;
   rentType: string;
+  serviceUnitId: number;
   showRemove: boolean;
   usersPermissions: UsersPermissionsType;
   change: (...args: Array<any>) => any;
@@ -73,6 +74,7 @@ class ContractRentEdit extends PureComponent<Props> {
       onRemove,
       receivableTypes,
       rentType,
+      serviceUnitId: leaseServiceUnitId,
       showRemove,
       usersPermissions
     } = this.props;
@@ -223,11 +225,9 @@ class ContractRentEdit extends PureComponent<Props> {
       </Authorization>;
 
     const endDateField = <Authorization allow={isFieldAllowedToRead(leaseAttributes, LeaseRentContractRentsFieldPaths.END_DATE)}>
-        {leaseAttributes.service_unit.id !== ServiceUnitIds.MAKE &&
           <FormField disableTouched={isSaveClicked} fieldAttributes={getFieldAttributes(leaseAttributes, LeaseRentContractRentsFieldPaths.END_DATE)} name={`${field}.end_date`} overrideValues={{
             label: LeaseRentContractRentsFieldTitles.END_DATE
           }} enableUiDataEdit uiDataKey={getUiDataLeaseKey(LeaseRentContractRentsFieldPaths.END_DATE)} />
-        }
       </Authorization>;
 
     const indexField = <Authorization allow={isFieldAllowedToRead(leaseAttributes, LeaseRentContractRentsFieldPaths.INDEX)}>
@@ -245,9 +245,9 @@ class ContractRentEdit extends PureComponent<Props> {
         <Column small={6} medium={4} large={2}>
           {intendedUseField}
         </Column>
-        <Column small={6} medium={4} large={2}>
+        { leaseServiceUnitId !== ServiceUnitIds.MAKE && <Column small={6} medium={4} large={2}>
           {overrideReceivableTypeField}
-        </Column>
+        </Column>}
         {rentType === RentTypes.INDEX2022 && <Column small={6} medium={4} large={2}>
             {indexField}
           </Column>}
diff --git a/src/leases/components/leaseSections/rent/ContractRents.tsx b/src/leases/components/leaseSections/rent/ContractRents.tsx
index bd565387d..5d43553a2 100644
--- a/src/leases/components/leaseSections/rent/ContractRents.tsx
+++ b/src/leases/components/leaseSections/rent/ContractRents.tsx
@@ -20,6 +20,7 @@ type Props = {
   largeScreen: boolean;
   leaseAttributes: Attributes;
   rentType: string;
+  serviceUnitId: number;
 };
 type State = {
   amountPeriodOptions: Array<Record<string, any>>;
@@ -61,6 +62,7 @@ class ContractRents extends PureComponent<Props, State> {
       largeScreen,
       leaseAttributes,
       rentType,
+      serviceUnitId,
     } = this.props;
     const {
       amountPeriodOptions,
@@ -125,7 +127,7 @@ class ContractRents extends PureComponent<Props, State> {
                     </>
                   </Authorization>
                 </Column>
-                {(leaseAttributes.service_unit.id !== ServiceUnitIds.MAKE) && <Column small={6} medium={4} large={3}>
+                {(serviceUnitId !== ServiceUnitIds.MAKE) && <Column small={6} medium={4} large={3}>
                   <Authorization allow={isFieldAllowedToRead(leaseAttributes, LeaseRentContractRentsFieldPaths.OVERRIDE_RECEIVABLE_TYPE)}>
                     <>
                     <FormTextTitle uiDataKey={getUiDataLeaseKey(LeaseRentContractRentsFieldPaths.OVERRIDE_RECEIVABLE_TYPE)}>
diff --git a/src/leases/components/leaseSections/rent/ContractRentsEdit.tsx b/src/leases/components/leaseSections/rent/ContractRentsEdit.tsx
index bd38dc92c..5f569771c 100644
--- a/src/leases/components/leaseSections/rent/ContractRentsEdit.tsx
+++ b/src/leases/components/leaseSections/rent/ContractRentsEdit.tsx
@@ -11,23 +11,21 @@ import { ConfirmationModalTexts } from "@/enums";
 import { ButtonColors } from "@/components/enums";
 import { UsersPermissions } from "@/usersPermissions/enums";
 import { hasPermissions } from "@/util/helpers";
-import { getAttributes as getLeaseAttributes } from "@/leases/selectors";
 import { getUsersPermissions } from "@/usersPermissions/selectors";
-import type { Attributes } from "types";
 import type { UsersPermissions as UsersPermissionsType } from "@/usersPermissions/types";
 type Props = {
   fields: any;
-  leaseAttributes: Attributes;
   rentField: string;
   rentType: string;
+  serviceUnitId: number;
   usersPermissions: UsersPermissionsType;
 };
 
 const ContractRentsEdit = ({
   fields,
-  leaseAttributes,
   rentField,
   rentType,
+  serviceUnitId,
   usersPermissions
 }: Props) => {
   const handleAdd = () => {
@@ -60,7 +58,7 @@ const ContractRentsEdit = ({
               });
             };
 
-            return <ContractRentEdit key={index} field={rent} onRemove={handleRemove} rentField={rentField} rentType={rentType} showRemove={!!fields && fields.length > 1} />;
+            return <ContractRentEdit key={index} field={rent} onRemove={handleRemove} rentField={rentField} rentType={rentType} showRemove={!!fields && fields.length > 1} serviceUnitId={serviceUnitId} />;
           })}
               </BoxItemContainer>}
 
@@ -78,7 +76,6 @@ const ContractRentsEdit = ({
 
 export default connect(state => {
   return {
-    leaseAttributes: getLeaseAttributes(state),
     usersPermissions: getUsersPermissions(state)
   };
 })(ContractRentsEdit);
diff --git a/src/leases/components/leaseSections/rent/RentItem.tsx b/src/leases/components/leaseSections/rent/RentItem.tsx
index 915a29e10..97b71ab93 100644
--- a/src/leases/components/leaseSections/rent/RentItem.tsx
+++ b/src/leases/components/leaseSections/rent/RentItem.tsx
@@ -32,6 +32,7 @@ type Props = {
   rents: Array<Record<string, any>>;
   rentAdjustmentsCollapseState: boolean;
   rentCollapseState: boolean;
+  serviceUnitId: number;
 };
 
 const RentItem = ({
@@ -46,6 +47,7 @@ const RentItem = ({
   rents,
   rentAdjustmentsCollapseState,
   rentCollapseState,
+  serviceUnitId,
 }: Props) => {
   const handleCollapseToggle = (key: string, val: boolean) => {
     receiveCollapseStates({
@@ -118,7 +120,7 @@ const RentItem = ({
 
       <Authorization allow={isFieldAllowedToRead(leaseAttributes, LeaseRentContractRentsFieldPaths.CONTRACT_RENTS)}>
         {(rentTypeIsIndex || rentTypeIsIndex2022 || rentTypeIsFixed || rentTypeIsManual) && <Collapse className='collapse__secondary' defaultOpen={contractRentsCollapseState !== undefined ? contractRentsCollapseState : true} headerTitle={`${LeaseRentContractRentsFieldTitles.CONTRACT_RENTS} (${contractRents.length})`} onToggle={handleContractRentsCollapseToggle} uiDataKey={getUiDataLeaseKey(LeaseRentContractRentsFieldPaths.CONTRACT_RENTS)}>
-            <ContractRents contractRents={contractRents} rentType={rentType} />
+            <ContractRents contractRents={contractRents} rentType={rentType} serviceUnitId={serviceUnitId}/>
           </Collapse>}
       </Authorization>
 
diff --git a/src/leases/components/leaseSections/rent/RentItemEdit.tsx b/src/leases/components/leaseSections/rent/RentItemEdit.tsx
index e9f7edd71..4b491abf1 100644
--- a/src/leases/components/leaseSections/rent/RentItemEdit.tsx
+++ b/src/leases/components/leaseSections/rent/RentItemEdit.tsx
@@ -49,6 +49,7 @@ type Props = {
   rentId: number;
   rents: Array<Record<string, any>>;
   rentType: string | null | undefined;
+  serviceUnitId: number;
   usersPermissions: UsersPermissionsType;
 };
 type State = {
@@ -237,6 +238,7 @@ class RentItemEdit extends PureComponent<Props, State> {
       rentCollapseState,
       rentType,
       rents,
+      serviceUnitId,
       usersPermissions
     } = this.props;
     const {
@@ -277,7 +279,7 @@ class RentItemEdit extends PureComponent<Props, State> {
 
         <Authorization allow={isFieldAllowedToRead(leaseAttributes, LeaseRentContractRentsFieldPaths.CONTRACT_RENTS)}>
           {(rentTypeIsIndex || rentTypeIsIndex2022 || rentTypeIsFixed || rentTypeIsManual) && <Collapse className='collapse__secondary' defaultOpen={contractRentsCollapseState !== undefined ? contractRentsCollapseState : true} hasErrors={isSaveClicked && !isEmpty(contractRentErrors)} headerTitle={`${LeaseRentContractRentsFieldTitles.CONTRACT_RENTS} (${contractRents ? contractRents.length : 0})`} onToggle={this.handleContractRentsCollapseToggle} enableUiDataEdit uiDataKey={getUiDataLeaseKey(LeaseRentContractRentsFieldPaths.CONTRACT_RENTS)}>
-              <FieldArray component={ContractRentsEdit} isSaveClicked={isSaveClicked} name={`${field}.contract_rents`} rentField={field} rentType={rentType} />
+              <FieldArray component={ContractRentsEdit} isSaveClicked={isSaveClicked} name={`${field}.contract_rents`} rentField={field} rentType={rentType} serviceUnitId={serviceUnitId}/>
             </Collapse>}
         </Authorization>
 
@@ -340,4 +342,4 @@ export default connect((state, props) => {
 }, {
   change,
   receiveCollapseStates
-})(RentItemEdit);
\ No newline at end of file
+})(RentItemEdit);
diff --git a/src/leases/components/leaseSections/rent/Rents.tsx b/src/leases/components/leaseSections/rent/Rents.tsx
index 2a3932ecc..4c281098c 100644
--- a/src/leases/components/leaseSections/rent/Rents.tsx
+++ b/src/leases/components/leaseSections/rent/Rents.tsx
@@ -67,7 +67,7 @@ const Rents = ({
         <>
         {!rents || !rents.length && <FormText className='no-margin'>Ei vuokria</FormText>}
         {rents && !!rents.length && rents.map(rent => {
-        return <RentItem key={rent.id} rent={rent} rents={rents} />;
+        return <RentItem key={rent.id} rent={rent} rents={rents} serviceUnitId={currentLease.service_unit.id} />;
       })}
 
         {!!rentsArchived.length && <h3 style={{
diff --git a/src/leases/components/leaseSections/rent/RentsEdit.tsx b/src/leases/components/leaseSections/rent/RentsEdit.tsx
index ef741b82c..7bdebdfa5 100644
--- a/src/leases/components/leaseSections/rent/RentsEdit.tsx
+++ b/src/leases/components/leaseSections/rent/RentsEdit.tsx
@@ -58,6 +58,7 @@ type RentsProps = {
   archived: boolean;
   fields: any;
   rents: Array<Record<string, any>>;
+  serviceUnitId: number;
   usersPermissions: UsersPermissionsType;
 };
 
@@ -65,6 +66,7 @@ const renderRents = ({
   archived,
   fields,
   rents,
+  serviceUnitId,
   usersPermissions
 }: RentsProps): ReactElement => {
   const handleAdd = () => {
@@ -99,7 +101,7 @@ const renderRents = ({
             });
           };
 
-          return <RentItemEdit key={index} field={item} index={index} onRemove={handleRemove} rents={rents} />;
+          return <RentItemEdit key={index} field={item} index={index} onRemove={handleRemove} rents={rents} serviceUnitId={serviceUnitId}/>;
         })}
             {!archived && <Authorization allow={hasPermissions(usersPermissions, UsersPermissions.ADD_RENT)}>
                 <Row>
@@ -219,6 +221,7 @@ class RentsEdit extends PureComponent<Props, State> {
 
   render() {
     const {
+      currentLease,
       editedActiveBasisOfRents,
       editedArchivedBasisOfRents,
       isRentInfoComplete,
@@ -282,7 +285,7 @@ class RentsEdit extends PureComponent<Props, State> {
                 <Field name='rentWarnings' component={RentWarnings} leaseAttributes={leaseAttributes} showWarning={true} />
                 <Divider />
 
-                <FieldArray component={renderRents} archive={false} name='rents' rents={rents} usersPermissions={usersPermissions} />
+                <FieldArray component={renderRents} archived={false} name='rents' rents={rents} usersPermissions={usersPermissions} serviceUnitId={currentLease.service_unit.id} />
 
                 {
               /* Archived rents */
@@ -344,4 +347,4 @@ export default flowRight(withRouter, connect(state => {
   destroyOnUnmount: false,
   validate: validateRentForm,
   warn: warnRentForm
-}))(RentsEdit) as React.ComponentType<any>;
\ No newline at end of file
+}))(RentsEdit) as React.ComponentType<any>;