Merge pull request #538 from rodriguezc/event-normalization

Event normalization for React Form libraries
primefaces · Sep 20, 2018 · 762d108 · 762d108
2 parents 41fa890 + f542224
commit 762d108
Show file tree

Hide file tree

Showing 17 changed files with 212 additions and 27 deletions.
diff --git a/src/components/autocomplete/AutoComplete.js b/src/components/autocomplete/AutoComplete.js
@@ -198,10 +198,18 @@ export class AutoComplete extends Component {
     }
 
     updateModel(event, value) {
+        const resolvedFieldData = this.props.field ? ObjectUtils.resolveFieldData(value, this.props.field) : undefined;
         if(this.props.onChange) {
             this.props.onChange({
                 originalEvent: event,
-                value: value
+                value: value,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value: resolvedFieldData ? resolvedFieldData: value
+                }
             });
         }
     }

diff --git a/src/components/calendar/Calendar.js b/src/components/calendar/Calendar.js
@@ -667,7 +667,14 @@ export class Calendar extends Component {
         if (this.props.onChange) {
             this.props.onChange({
                 originalEvent: event,
-                value: value
+                value: value,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value
+                }
             });
         }
     }

diff --git a/src/components/checkbox/Checkbox.js b/src/components/checkbox/Checkbox.js
@@ -53,7 +53,16 @@ export class Checkbox extends Component {
             this.props.onChange({
                 originalEvent: e,
                 value: this.props.value,
-                checked: !this.props.checked
+                checked: !this.props.checked,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    type: 'checkbox',
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value: this.props.value,
+                    checked: !this.props.checked,
+                }
             });
 
             this.input.checked = !this.props.checked;

diff --git a/src/components/chips/Chips.js b/src/components/chips/Chips.js
@@ -85,7 +85,14 @@ export class Chips extends Component {
         if (this.props.onChange) {
             this.props.onChange({
                 originalEvent: event,
-                value: values
+                value: values,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value : values
+                }
             });
         }
     }
@@ -122,7 +129,14 @@ export class Chips extends Component {
                     if (this.props.onChange) {
                         this.props.onChange({
                             originalEvent: event,
-                            value: values
+                            value: values,
+                            stopPropagation : () =>{},
+                            preventDefault : () =>{},
+                            target: {
+                                name: this.props.name,
+                                id :  this.props.id,
+                                value : values
+                            }
                         });
                     }
                 }

diff --git a/src/components/colorpicker/ColorPicker.js b/src/components/colorpicker/ColorPicker.js
@@ -161,7 +161,14 @@ export class ColorPicker extends Component {
     onChange(value) {
         if(this.props.onChange) {
             this.props.onChange({
-                value: value
+                value: value,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value
+                }
             })
         }
     }

diff --git a/src/components/dropdown/Dropdown.js b/src/components/dropdown/Dropdown.js
@@ -258,7 +258,14 @@ export class Dropdown extends Component {
     onEditableInputChange(event) {              
         this.props.onChange({
             originalEvent: event.originalEvent,
-            value: event.target.value
+            value: event.target.value,
+            stopPropagation : () =>{},
+            preventDefault : () =>{},
+            target: {
+                name: this.props.name,
+                id :  this.props.id,
+                value : event.target.value,
+            }
         });
     }
 
@@ -304,7 +311,14 @@ export class Dropdown extends Component {
     clear(event) {
         this.props.onChange({
             originalEvent: event,
-            value: null
+            value: null,
+            stopPropagation : () =>{},
+            preventDefault : () =>{},
+            target: {
+                name: this.props.name,
+                id :  this.props.id,
+                value : null
+            }
         });
         this.updateEditableLabel();
     }
@@ -316,7 +330,14 @@ export class Dropdown extends Component {
             this.updateEditableLabel(event.option);
             this.props.onChange({
                 originalEvent: event.originalEvent,
-                value: this.props.optionLabel ? event.option : event.option.value
+                value: this.props.optionLabel ? event.option : event.option.value,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value : this.props.optionLabel ? event.option : event.option.value,
+                }
             });
         } 
     }

diff --git a/src/components/inputmask/InputMask.js b/src/components/inputmask/InputMask.js
@@ -442,7 +442,14 @@ export class InputMask extends Component {
             var val = this.props.unmask ? this.getUnmaskedValue() : e.target.value;
             this.props.onChange({
                 originalEvent: e,
-                value: (this.defaultBuffer !== val) ? val : ''
+                value: (this.defaultBuffer !== val) ? val : '',
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value : (this.defaultBuffer !== val) ? val : '',
+                }
             })
         }
     }

diff --git a/src/components/inputswitch/InputSwitch.js b/src/components/inputswitch/InputSwitch.js
@@ -57,7 +57,14 @@ export class InputSwitch extends Component {
         if (this.props.onChange) {
             this.props.onChange({
                 originalEvent: event,
-                value: !this.props.checked
+                value: !this.props.checked,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value: !this.props.checked,
+                }
             });
         }
     }

diff --git a/src/components/listbox/ListBox.js b/src/components/listbox/ListBox.js
@@ -158,7 +158,14 @@ export class ListBox extends Component {
         if(valueChanged) {
             this.props.onChange({
                 originalEvent: event,
-                value: value
+                value: value,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value
+                }
             });
         }
     }
@@ -171,7 +178,14 @@ export class ListBox extends Component {
         if(this.props.onChange) {
             this.props.onChange({
                 originalEvent: event,
-                value: value
+                value: value,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value
+                }
             });
         }
     }

diff --git a/src/components/multiselect/MultiSelect.js b/src/components/multiselect/MultiSelect.js
@@ -120,7 +120,14 @@ export class MultiSelect extends Component {
         if(this.props.onChange) {
             this.props.onChange({
                 originalEvent: event,
-                value: value
+                value: value,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value
+                }
             });
         }
     }

diff --git a/src/components/radiobutton/RadioButton.js b/src/components/radiobutton/RadioButton.js
@@ -51,7 +51,14 @@ export class RadioButton extends Component {
             this.props.onChange({
                 originalEvent: e,
                 value: this.props.value,
-                checked: !this.props.checked
+                checked: !this.props.checked,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value:  this.props.value,
+                }
             });
 
             this.input.checked = !this.props.checked;

diff --git a/src/components/rating/Rating.js b/src/components/rating/Rating.js
@@ -42,7 +42,14 @@ export class Rating extends Component {
         if (!this.props.readonly && !this.props.disabled && this.props.onChange) {
             this.props.onChange({
                 originalEvent: event,
-                value: i
+                value: i,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value: i
+                }
             });
         }
 
@@ -53,7 +60,14 @@ export class Rating extends Component {
         if (!this.props.readonly && !this.props.disabled && this.props.onChange) {
             this.props.onChange({
                 originalEvent: event,
-                value: null
+                value: null,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value: null
+                }
             });
         }
 

diff --git a/src/components/selectbutton/SelectButton.js b/src/components/selectbutton/SelectButton.js
@@ -89,7 +89,14 @@ export class SelectButton extends Component {
         if(this.props.onChange) {
             this.props.onChange({
                 originalEvent: event.originalEvent,
-                value: newValue
+                value: newValue,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value: newValue,
+                }
             });
         }
     }

diff --git a/src/components/slider/Slider.js b/src/components/slider/Slider.js
@@ -195,7 +195,14 @@ export class Slider extends Component {
             if (this.props.onChange) {
                 this.props.onChange({
                     originalEvent: event,
-                    value: this.values
+                    value: this.values,
+                    stopPropagation : () =>{},
+                    preventDefault : () =>{},
+                    target: {
+                        name: this.props.name,
+                        id :  this.props.id,
+                        value:  this.values,
+                    }
                 })
             }
         }
@@ -219,7 +226,14 @@ export class Slider extends Component {
             if (this.props.onChange) {
                 this.props.onChange({
                     originalEvent: event,
-                    value: this.value
+                    value: this.value,
+                    stopPropagation : () =>{},
+                    preventDefault : () =>{},
+                    target: {
+                        name: this.props.name,
+                        id :  this.props.id,
+                        value:  this.value,
+                    }
                 })
             }
         }

diff --git a/src/components/spinner/Spinner.js b/src/components/spinner/Spinner.js
@@ -116,7 +116,14 @@ export class Spinner extends Component {
         if (this.props.onChange) {
             this.props.onChange({
                 originalEvent: event,
-                value: newValue
+                value: newValue,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value: newValue
+                }
             });
         }
     }
@@ -239,7 +246,14 @@ export class Spinner extends Component {
         if (this.props.onChange) {
             this.props.onChange({
                 originalEvent: event,
-                value: event.target.value
+                value: event.target.value,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value: event.target.value
+                }   
             });
         }
     } 
@@ -248,15 +262,29 @@ export class Spinner extends Component {
         DomHandler.removeClass(this.element, 'p-inputwrapper-focus');
 
         if (this.props.onChange) {
+            const parsedValue =  this.parseValue(event.target.value);
             this.props.onChange({
                 originalEvent: event,
-                value: this.parseValue(event.target.value)
+                value: parsedValue,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                    value: parsedValue,
+                }
             });
         }
 
         if (this.props.onBlur) {
             this.props.onBlur({
-                event: event
+                event: event,
+                stopPropagation : () =>{},
+                preventDefault : () =>{},
+                target: {
+                    name: this.props.name,
+                    id :  this.props.id,
+                }
             });
         }
     }
@@ -334,4 +362,4 @@ export class Spinner extends Component {
             </span>
         );
     }
-}
+}