From afcb7a912f5980777544e0e37887a124522a8242 Mon Sep 17 00:00:00 2001
From: Jorge Cloquell <jorge.cloquell@optibus.com>
Date: Wed, 8 Mar 2023 01:33:50 +0100
Subject: [PATCH] Fix the allowEditing prop which was being accidentally
 ignored

---
 example/src/App.tsx                                         | 4 ++++
 src/Date/DatePickerModalContent.tsx                         | 2 +-
 src/Date/DatePickerModalContentHeader.tsx                   | 6 ++----
 .../Date/__snapshots__/AnimatedCrossView.test.tsx.snap      | 1 +
 src/__tests__/Date/__snapshots__/Calendar.test.tsx.snap     | 1 +
 src/__tests__/Date/__snapshots__/YearPicker.test.tsx.snap   | 1 +
 6 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/example/src/App.tsx b/example/src/App.tsx
index 570ec782..86e8a0fd 100644
--- a/example/src/App.tsx
+++ b/example/src/App.tsx
@@ -306,6 +306,8 @@ function App() {
         // animationType="slide" // optional, default is slide on ios/android and none on web
         // startYear={2000} // optional, default is 1800
         // endYear={2100} // optional, default is 2200
+        // allowEditing={false} // optional, default is true
+        // inputEnabled={false} // optional, default is true
       />
 
       <DatePickerModal
@@ -327,6 +329,8 @@ function App() {
         // animationType="slide" // optional, default is 'slide' on ios/android and 'none' on web
         // startYear={2000} // optional, default is 1800
         // endYear={2100} // optional, default is 2200
+        // allowEditing={false} // optional, default is true
+        // inputEnabled={false} // optional, default is true
       />
 
       <DatePickerModal
diff --git a/src/Date/DatePickerModalContent.tsx b/src/Date/DatePickerModalContent.tsx
index 74bad774..eefca0f5 100644
--- a/src/Date/DatePickerModalContent.tsx
+++ b/src/Date/DatePickerModalContent.tsx
@@ -165,7 +165,7 @@ export function DatePickerModalContent(
           locale={locale}
           editIcon={props?.editIcon}
           calendarIcon={props.calendarIcon}
-          allowEditing={props.allowEditing || true}
+          allowEditing={props.allowEditing ?? true}
         />
       </DatePickerModalHeaderBackground>
       <AnimatedCrossView
diff --git a/src/Date/DatePickerModalContentHeader.tsx b/src/Date/DatePickerModalContentHeader.tsx
index c3c4feea..20a238b2 100644
--- a/src/Date/DatePickerModalContentHeader.tsx
+++ b/src/Date/DatePickerModalContentHeader.tsx
@@ -28,7 +28,6 @@ export interface HeaderContentProps extends HeaderPickProps {
   collapsed: boolean
   onToggle: () => any
   locale: string | undefined
-  inputDate?: boolean
 }
 
 function getLabel(
@@ -62,7 +61,6 @@ export default function DatePickerModalContentHeader(
     uppercase,
     editIcon,
     calendarIcon,
-    inputDate,
     allowEditing,
   } = props
   const theme = useTheme()
@@ -70,7 +68,7 @@ export default function DatePickerModalContentHeader(
 
   const color = useHeaderTextColor()
 
-  const isAllowEditing = allowEditing && mode !== 'multiple' && inputDate !== undefined
+  const isEditingEnabled = allowEditing && mode !== 'multiple'
 
   const supportingTextColor = theme.isV3 ? theme.colors.onSurfaceVariant : color
 
@@ -103,7 +101,7 @@ export default function DatePickerModalContentHeader(
         </View>
       </View>
       <View style={styles.fill} />
-      {isAllowEditing ? (
+      {isEditingEnabled ? (
         <IconButton
           icon={
             collapsed
diff --git a/src/__tests__/Date/__snapshots__/AnimatedCrossView.test.tsx.snap b/src/__tests__/Date/__snapshots__/AnimatedCrossView.test.tsx.snap
index 8cfde1f1..18a1056d 100644
--- a/src/__tests__/Date/__snapshots__/AnimatedCrossView.test.tsx.snap
+++ b/src/__tests__/Date/__snapshots__/AnimatedCrossView.test.tsx.snap
@@ -488,6 +488,7 @@ exports[`renders collapsed AnimatedCrossView 1`] = `
           onScrollEndDrag={[Function]}
           removeClippedSubviews={false}
           renderItem={[Function]}
+          renderScrollComponent={[Function]}
           scrollEventThrottle={50}
           stickyHeaderIndices={[]}
           style={
diff --git a/src/__tests__/Date/__snapshots__/Calendar.test.tsx.snap b/src/__tests__/Date/__snapshots__/Calendar.test.tsx.snap
index a2eead9e..f2bfb2cf 100644
--- a/src/__tests__/Date/__snapshots__/Calendar.test.tsx.snap
+++ b/src/__tests__/Date/__snapshots__/Calendar.test.tsx.snap
@@ -463,6 +463,7 @@ exports[`renders Calendar 1`] = `
       onScrollEndDrag={[Function]}
       removeClippedSubviews={false}
       renderItem={[Function]}
+      renderScrollComponent={[Function]}
       scrollEventThrottle={50}
       stickyHeaderIndices={[]}
       style={
diff --git a/src/__tests__/Date/__snapshots__/YearPicker.test.tsx.snap b/src/__tests__/Date/__snapshots__/YearPicker.test.tsx.snap
index 5541a032..e855554a 100644
--- a/src/__tests__/Date/__snapshots__/YearPicker.test.tsx.snap
+++ b/src/__tests__/Date/__snapshots__/YearPicker.test.tsx.snap
@@ -444,6 +444,7 @@ exports[`renders YearPicker 1`] = `
     onScrollEndDrag={[Function]}
     removeClippedSubviews={false}
     renderItem={[Function]}
+    renderScrollComponent={[Function]}
     scrollEventThrottle={50}
     stickyHeaderIndices={[]}
     style={