fix(datepicker): corrige disparo do p-change

O evento p-change só emitia valor quando saísse do campo (blur). Foi redefinido para que dispare ao selecionar nova data.

Fixes DTHFUI-904

Author: Samir Hassan Ayoub

projects/ui/src/lib/components/po-field/po-datepicker/po-datepicker.component.html

@@ -15,7 +15,6 @@
       [readonly]="readonly"
       [required]="required"
       (blur)="eventOnBlur($event)"
-      (change)="eventOnChange($event)"
       (click)="eventOnClick($event)">
 
     <div class="po-field-icon-container-right">

projects/ui/src/lib/components/po-field/po-datepicker/po-datepicker.component.spec.ts

@@ -412,17 +412,6 @@ describe('PoDatepickerComponent:', () => {
     expect(component.callOnChange).toHaveBeenCalled();
   });
 
-  it('should emit onchange when it is not mobile', () => {
-    const input = fixture.debugElement.nativeElement.querySelector('input');
-    input.value = '01/01/2000';
-
-    spyOn(component.onchange, 'emit');
-
-    component.eventOnChange({});
-
-    expect(component.onchange.emit).toHaveBeenCalled();
-  });
-
   it('simulate click out behavior the datepicker field read-only', () => {
     component.dialogPicker = undefined;
     fixture.detectChanges();
@@ -436,18 +425,6 @@ describe('PoDatepickerComponent:', () => {
 
   // Testes de mobile
 
-  it('should emit onchange when it is mobile', () => {
-    spyOn(component, 'verifyMobile').and.returnValue(['Android']);
-
-    spyOn(component, 'callOnChange');
-    spyOn(component, 'controlModel');
-
-    component.eventOnChange({});
-
-    expect(component.callOnChange).toHaveBeenCalled();
-    expect(component.controlModel).toHaveBeenCalled();
-  });
-
   it('should call eventOnClick when have mobile', () => {
     spyOn(component, 'verifyMobile').and.returnValue(['Android']);
 
@@ -745,13 +722,15 @@ describe('PoDatepickerComponent:', () => {
       expect(component['closeCalendar']).toHaveBeenCalled();
     });
 
-    it('dateSelected: should call `controlModel`', () => {
+    it('dateSelected: should call `controlModel` and `controlChangeEmitter`', () => {
 
       spyOn(component, 'controlModel');
+      spyOn(component, <any>'controlChangeEmitter');
 
       component.dateSelected();
 
       expect(component.controlModel).toHaveBeenCalled();
+      expect(component['controlChangeEmitter']).toHaveBeenCalled();
     });
 
     it('formatToDate: should call `formatYear` with date year', () => {
@@ -970,24 +949,6 @@ describe('PoDatepickerComponent:', () => {
       expect(fakeThis.onchange.emit).not.toHaveBeenCalled();
     });
 
-    it('eventOnChange: should emit value', () => {
-      const value = '30/08/2018' ;
-      const fakeThis = {
-        verifyMobile: () => false,
-        inputEl: {
-          nativeElement: {
-            value: value
-          }
-        },
-        onchange: { emit: () => {} }
-      };
-
-      spyOn(fakeThis.onchange, 'emit');
-      component['eventOnChange'].call(fakeThis);
-
-      expect(fakeThis.onchange.emit).toHaveBeenCalledWith(value);
-    });
-
     it('isValidDateIso: should return `true` if value format is `yyyy-mm-dd`.', () => {
       const expectedValue = true;
       const isoDate = '2018-09-29';

projects/ui/src/lib/components/po-field/po-datepicker/po-datepicker.component.ts

@@ -124,6 +124,7 @@ export class PoDatepickerComponent extends PoDatepickerBaseComponent implements
     this.inputEl.nativeElement.focus();
     this.inputEl.nativeElement.value = this.formatToDate(this.date);
     this.controlModel(this.date);
+    this.controlChangeEmitter();
     this.closeCalendar();
   }
 
@@ -212,15 +213,6 @@ export class PoDatepickerComponent extends PoDatepickerBaseComponent implements
 
   }
 
-  eventOnChange($event) {
-    const elementValue = this.inputEl.nativeElement.value;
-    if (this.verifyMobile()) {
-      this.controlModel(elementValue);
-      this.callOnChange(elementValue);
-    }
-    this.onchange.emit(elementValue);
-  }
-
   eventOnClick($event) {
     if (this.verifyMobile()) {
       $event.target.blur();