Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0031705Openbravo ERPA. Platformpublic2015-12-14 18:282016-10-24 10:18
egoitz 
caristu 
immediateminoralways
closedfixed 
5
 
3.0PR16Q1 
alostale
Core
Production - Confirmed Stable
2015-04-22
3.0PR15Q3
https://code.openbravo.com/erp/devel/pi/rev/b1614f4301585bb0f8affbf5cf53f451dd284851 [^]
No
0031705: a date used to filter on a grid dissapears after enter it
When you write a date on the filter of a grid, after loss the focus, the value entered is removed, but the filter works properly
-Go to sales order window with date 14-12-2015
-Remove all the filter on grid view
-Filter on the order date field by writing.
1412
-Then press tab key to change to next column
*The value entered is removed but the filter works properly
No tags attached.
related to defect 0034057 closed NaroaIriarte Wrong Tabulation Order in form fields in Organization window 
depends on backport 00317213.0PR15Q4.2 closed caristu a date used to filter on a grid dissapears after enter it 
depends on backport 00317223.0PR15Q3.5 closed caristu a date used to filter on a grid dissapears after enter it 
caused by defect 00295543.0PR15Q3 closed caristu Date filters are not cleared when switching between saved views 
has duplicate defect 00317133.0PR16Q1 closed platform Date value not maintained in grid when used to filter, while string values are maintained 
causes defect 0032474 closed caristu '[object Object]' instead of blank value after clearing value in selector 
Issue History
2015-12-14 18:28egoitzNew Issue
2015-12-14 18:28egoitzAssigned To => platform
2015-12-14 18:28egoitzModules => Core
2015-12-14 18:28egoitzResolution time => 1450479600
2015-12-14 18:28egoitzRegression level => Production - Confirmed Stable
2015-12-14 18:28egoitzTriggers an Emergency Pack => No
2015-12-15 11:02alostaleRegression introduced in release => 3.0PR15Q3
2015-12-15 11:03alostalePrioritynormal => immediate
2015-12-15 11:03alostaleStatusnew => acknowledged
2015-12-15 11:04alostaleRelationship addedhas duplicate 0031713
2015-12-15 13:32caristuRegression date => 2015-04-22
2015-12-15 13:32caristuRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/b1614f4301585bb0f8affbf5cf53f451dd284851 [^]
2015-12-15 13:32caristuNote Added: 0082764
2015-12-15 13:33caristuAssigned Toplatform => caristu
2015-12-15 13:33caristuStatusacknowledged => scheduled
2015-12-15 13:59alostaleStatusscheduled => acknowledged
2015-12-15 13:59alostaleStatusacknowledged => scheduled
2015-12-15 14:00alostaleRelationship addedcaused by 0029554
2015-12-15 18:42caristuNote Edited: 0082764bug_revision_view_page.php?bugnote_id=0082764#r10477
2015-12-15 19:00hgbotCheckin
2015-12-15 19:00hgbotNote Added: 0082784
2015-12-15 19:00hgbotStatusscheduled => resolved
2015-12-15 19:00hgbotResolutionopen => fixed
2015-12-15 19:00hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d85f65dd5246e3189c45b0b1a202340c930bade9 [^]
2015-12-15 19:01caristuReview Assigned To => alostale
2015-12-15 19:01caristuIssue Monitored: alostale
2015-12-16 09:07hgbotCheckin
2015-12-16 09:07hgbotNote Added: 0082793
2015-12-17 08:29alostaleNote Added: 0082809
2015-12-17 08:29alostaleStatusresolved => closed
2015-12-17 08:29alostaleFixed in Version => 3.0PR16Q1
2015-12-17 16:00caristuNote Added: 0082837
2015-12-17 16:00caristuNote Edited: 0082837bug_revision_view_page.php?bugnote_id=0082837#r10507
2015-12-17 17:49hudsonbotCheckin
2015-12-17 17:49hudsonbotNote Added: 0082844
2015-12-17 17:49hudsonbotCheckin
2015-12-17 17:49hudsonbotNote Added: 0082846
2016-02-02 09:43hgbotCheckin
2016-02-02 09:43hgbotNote Added: 0083830
2016-02-02 09:43hgbotCheckin
2016-02-02 09:43hgbotNote Added: 0083831
2016-03-15 10:27caristuRelationship addedcauses 0032474
2016-03-15 19:13hgbotCheckin
2016-03-15 19:13hgbotNote Added: 0084996
2016-03-16 11:55hgbotCheckin
2016-03-16 11:55hgbotNote Added: 0085001
2016-03-16 12:28hgbotCheckin
2016-03-16 12:28hgbotNote Added: 0085004
2016-03-17 10:56hudsonbotCheckin
2016-03-17 10:56hudsonbotNote Added: 0085199
2016-10-24 10:18NaroaIriarteRelationship addedrelated to 0034057

Notes
(0082764)
caristu   
2015-12-15 13:32   
(edited on: 2015-12-15 18:42)
The date is cleared because of since changeset[1] the date filters clear all its content on their clearValue() function.

The problem is caused because after clicking on the enter or tab button the date filter value is not recovered because the value is not being set accordingly.

[1] https://code.openbravo.com/erp/devel/pi/rev/b1614f4301585bb0f8affbf5cf53f451dd284851 [^]

(0082784)
hgbot   
2015-12-15 19:00   
Repository: erp/devel/pi
Changeset: d85f65dd5246e3189c45b0b1a202340c930bade9
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Tue Dec 15 18:59:05 2015 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/d85f65dd5246e3189c45b0b1a202340c930bade9 [^]

fixes issue 31705: a date used to filter on a grid dissapears after enter it

After solving issue 29554 the values of the date filters are completely removed when invoking the clearValue() method. For this reason, in order to prevent the clearing of the date filter it is necessary to call setValue().

This was already controlled when choosing the dates from the date pickers, but not after entering the date with the tab or enter keys or when using the 'Use as Filter' menu option.

Now this cases are controlled also, by calling setValue() on the setSingleDateValue() method.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-minidaterange.js
---
(0082793)
hgbot   
2015-12-16 09:07   
Repository: erp/devel/pi
Changeset: 17016914f029b502a391792438ec40750d2b95cd
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Wed Dec 16 09:05:20 2015 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/17016914f029b502a391792438ec40750d2b95cd [^]

related to issue 31705: do not set singleDateMode after clearing filter values

The singleDateMode should not be set to true when clearing the filter values because this is preventing the date range dialog to appear under some circumnstances.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-minidaterange.js
---
(0082809)
alostale   
2015-12-17 08:29   
code reviewed

Tested:

 1. Adding date filter by typing (as described in this issue)
 2. Adding date filter by right click in date field to "Use as filter" (as described in 0031713)
 3. Using date filters in saved views (see 0029554)
 4. Getting direct link for a grid filtered by date with set by all the 3 ways defined above
(0082837)
caristu   
2015-12-17 16:00   
An automated test has been created to cover the scenario of this issue. It has been added into TestLink also: https://testlink.openbravo.com/testlink/linkto.php?tprojectPrefix=Communit&item=testcase&id=Communit-9979 [^]

(0082844)
hudsonbot   
2015-12-17 17:49   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/7bb1cbcee3ce [^]
Maturity status: Test
(0082846)
hudsonbot   
2015-12-17 17:49   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/7bb1cbcee3ce [^]
Maturity status: Test
(0083830)
hgbot   
2016-02-02 09:43   
Repository: tools/automation/int
Changeset: e5f9606e5f846bddbb37676b9e0a118bb09b41c2
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Wed Dec 16 14:07:29 2015 +0100
URL: http://code.openbravo.com/tools/automation/int/rev/e5f9606e5f846bddbb37676b9e0a118bb09b41c2 [^]

added automated test for issue 31705

---
M src-test/com/openbravo/test/integration/erp/modules/client/application/gui/GeneratedTab.java
M src-test/com/openbravo/test/integration/erp/modules/client/application/gui/OBGrid.java
M src-test/com/openbravo/test/integration/erp/modules/client/application/gui/OBStandardView.java
A src-test/com/openbravo/test/integration/erp/modules/client/application/testscripts/standardview/grid/DateFilterPersistsAfterPressingTabKey.java
---
(0083831)
hgbot   
2016-02-02 09:43   
Repository: tools/automation/int
Changeset: 9e4f70bc7b145a07138a84764448bd31fdec958c
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Thu Dec 17 13:08:57 2015 +0100
URL: http://code.openbravo.com/tools/automation/int/rev/9e4f70bc7b145a07138a84764448bd31fdec958c [^]

add automated test for issue 31705 into a test suite

---
M src-test/com/openbravo/test/integration/erp/modules/client/application/testsuites/standardview/grid/GridSuite.java
M src-test/com/openbravo/test/integration/erp/modules/client/application/testsuites/standardview/grid/build.xml
A src-test/com/openbravo/test/integration/erp/modules/client/application/testsuites/standardview/grid/GFI001_DateFilterPersistsAfterPressingKey.java
R src-test/com/openbravo/test/integration/erp/modules/client/application/testscripts/standardview/grid/DateFilterPersistsAfterPressingTabKey.java
---
(0084996)
hgbot   
2016-03-15 19:13   
Repository: erp/devel/pi
Changeset: 330cd48c5a70635408ead8b3fbfc72659ed18f52
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Tue Mar 15 19:12:36 2016 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/330cd48c5a70635408ead8b3fbfc72659ed18f52 [^]

fixes issue 32474: [object Object] appears after clearing value in selector

Having singleDateMode as false after clearing the filter content of the date causes the return of an empty object in the getValuesAsCriteria method of the grid. The this.singleDateMode was being set to true in clearFilterValues function, but this statement was deleted to solve issue 31705, because it was preventing to show the range dialog after clearing the filters, when pressing the Enter key.

To solve both problems the change regarding setting singleDateMode in clearFilterValues has been reverted, and in order to ensure that the range dialog is displayed when it should be, a new check has been included in the keyPress method.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-minidaterange.js
---
(0085001)
hgbot   
2016-03-16 11:55   
Repository: erp/backports/3.0PR16Q1.1
Changeset: 5ae13dc119e9fd18489af9f8bed42a4aea9b1991
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Wed Mar 16 11:55:00 2016 +0100
URL: http://code.openbravo.com/erp/backports/3.0PR16Q1.1/rev/5ae13dc119e9fd18489af9f8bed42a4aea9b1991 [^]

fixes issue 32477: [object Object] appears after clearing value in selector

Having singleDateMode as false after clearing the filter content of the date causes the return of an empty object in the getValuesAsCriteria method of the grid. The this.singleDateMode was being set to true in clearFilterValues function, but this statement was deleted to solve issue 31705, because it was preventing to show the range dialog after clearing the filters, when pressing the Enter key.

To solve both problems the change regarding setting singleDateMode in clearFilterValues has been reverted, and in order to ensure that the range dialog is displayed when it should be, a new check has been included in the keyPress method.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-minidaterange.js
---
(0085004)
hgbot   
2016-03-16 12:28   
Repository: erp/backports/3.0PR15Q4.5
Changeset: d6a7ec729aa86b7fe2c8e29d663e4e8086177f77
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Tue Mar 15 19:22:47 2016 +0100
URL: http://code.openbravo.com/erp/backports/3.0PR15Q4.5/rev/d6a7ec729aa86b7fe2c8e29d663e4e8086177f77 [^]

fixes issue 32478: [object Object] appears after clearing value in selector

Having singleDateMode as false after clearing the filter content of the date causes the return of an empty object in the getValuesAsCriteria method of the grid. The this.singleDateMode was being set to true in clearFilterValues function, but this statement was deleted to solve issue 31705, because it was preventing to show the range dialog after clearing the filters, when pressing the Enter key.

To solve both problems the change regarding setting singleDateMode in clearFilterValues has been reverted, and in order to ensure that the range dialog is displayed when it should be, a new check has been included in the keyPress method.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-minidaterange.js
---
(0085199)
hudsonbot   
2016-03-17 10:56   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/b22fb0500156 [^]
Maturity status: Test