fix(DatePicker): avoid aggressive range input clearing (#8893)

emyarod · kodiakhq[bot] · web-flow · commit 0107fb1ac0fe · 2021-06-16T16:09:20.000Z
* refactor(DatePicker): optionally chain `this.cal` reference check

* fix(DatePicker): avoid aggressive range input clearing

Co-authored-by: kodiakhq[bot] &lt;49736102+kodiakhq[bot]@users.noreply.github.com&gt;
diff --git a/packages/react/src/components/DatePicker/DatePicker.js b/packages/react/src/components/DatePicker/DatePicker.js
@@ -445,12 +445,8 @@ export default class DatePicker extends Component {
     }
   }
 
-  onChange = (e) => {
-    if (
-      e.target.value === '' &&
-      this.cal &&
-      this.cal.selectedDates.length > 0
-    ) {
+  onChange = () => {
+    if (this.inputField.value === '' && this.cal?.selectedDates.length) {
       this.cal.clear();
     }
   };

Original file line number	Diff line number	Diff line change
`@@ -445,12 +445,8 @@ export default class DatePicker extends Component {`
`445`	`445`	`}`
`446`	`446`	`}`
`447`	`447`
`448`		`- onChange = (e) => {`
`449`		`- if (`
`450`		`- e.target.value === '' &&`
`451`		`- this.cal &&`
`452`		`- this.cal.selectedDates.length > 0`
`453`		`- ) {`
	`448`	`+ onChange = () => {`
	`449`	`+ if (this.inputField.value === '' && this.cal?.selectedDates.length) {`
`454`	`450`	`this.cal.clear();`
`455`	`451`	`}`
`456`	`452`	`};`