Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0016300Openbravo ERPB. User interfacepublic2011-03-15 12:512012-10-11 18:37
Reporterplujan 
Assigned Todbaz 
PriorityhighSeveritytrivialReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version3.0MP17 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0016300: I get no feedback when the values are not correct
DescriptionIf I misspelled a date filter, I get no feedback at all. I was expecting the wrong values to turn red as a warning. See attach.
Steps To Reproduce
Proposed Solution(Rob): in case of invalid values, paint the text red, identical to the behavior in a form, see image.
Additional Information
TagsNo tags attached.
Relationships
related to design defect 0016258 acknowledged dbaz [KS] Date column filter does not have a keyboard shortcut to launch range dialog 
Attached Filespng screenshot-12.png (7,306) 2011-03-15 12:51
https://issues.openbravo.com/file_download.php?file_id=3684&type=bug
png

png red.PNG (6,397) 2011-05-18 12:33
https://issues.openbravo.com/file_download.php?file_id=4040&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2011-03-15 12:51plujanNew Issue
2011-03-15 12:51plujanAssigned To => mtaal
2011-03-15 12:51plujanFile Added: screenshot-12.png
2011-03-15 12:51plujanModules => Core
2011-03-16 08:23alostaleTarget Version => 3.0RC6
2011-03-16 08:30alostaleStatusnew => scheduled
2011-04-04 10:30mtaalTarget Version3.0RC6 => 3.0RC7
2011-05-06 18:39dmitry_mezentsevNote Added: 0036531
2011-05-06 18:39dmitry_mezentsevTarget Version3.0RC7 => 3.0MP0
2011-05-18 12:33rgorisRelationship addedrelated to 0016258
2011-05-18 12:33rgorisFile Added: red.PNG
2011-05-18 12:34rgorisProposed Solution updated
2011-05-23 11:33plujanTarget Version3.0MP0 => 3.0MP1
2011-06-22 19:41dmitry_mezentsevTarget Version3.0MP1 => 3.0MP3
2011-08-31 09:48mtaalTarget Version3.0MP3 => 3.0MP4
2011-09-07 13:21mtaalAssigned Tomtaal => dbaz
2011-09-07 13:28mtaalPrioritynormal => low
2011-09-26 12:42dbazTarget Version3.0MP4 =>
2011-11-24 18:09dbazNote Added: 0043193
2011-11-24 18:09dbazTypedefect => design defect
2012-09-24 23:22AugustoMauchNote Added: 0052457
2012-09-24 23:22AugustoMauchPrioritylow => high
2012-10-08 13:39dbazNote Added: 0053108
2012-10-08 13:39dbazTypedesign defect => defect
2012-10-08 13:42dbazReview Assigned To => AugustoMauch
2012-10-08 13:43dbazNote Deleted: 0043193
2012-10-08 13:43dbazIssue Monitored: AugustoMauch
2012-10-08 13:44hgbotCheckin
2012-10-08 13:44hgbotNote Added: 0053109
2012-10-08 13:44hgbotStatusscheduled => resolved
2012-10-08 13:44hgbotResolutionopen => fixed
2012-10-08 13:44hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/f5556ad1abb1e8c86a3fd41fdb02b7ac3525a5a4 [^]
2012-10-08 13:50hgbotCheckin
2012-10-08 13:50hgbotNote Added: 0053112
2012-10-11 07:50hudsonbotCheckin
2012-10-11 07:50hudsonbotNote Added: 0053258
2012-10-11 07:50hudsonbotCheckin
2012-10-11 07:50hudsonbotNote Added: 0053259
2012-10-11 18:37AugustoMauchNote Added: 0053324
2012-10-11 18:37AugustoMauchStatusresolved => closed
2012-10-11 18:37AugustoMauchFixed in Version => 3.0MP17

Notes
(0036531)
dmitry_mezentsev   
2011-05-06 18:39   
Moving to MP0 because significant redesign of date filter is planned for it.
(0052457)
AugustoMauch   
2012-09-24 23:22   
Impact: mid
Plan: short
(0053108)
dbaz   
2012-10-08 13:39   
Moving it to defect since there are also more cases where the bahavior is not the same:

a) Write "03-10" and blur. In regular date field the date is autocompleted, in the filter case it isn't
b) Write "03-10-99" and blur. In regular date field the date is autocompleted to "03-10-1999", in filter case it isn't
c) Write "03/09/2012" and blur. In regular date field the date is changed to "03-09-2012", in the filter case it isn't


To test the issues using both fields (regular date fields and filter date fields).

Also do some changes in Openbravo.properties, to ensure that it works also in other kind of configurations. Example:

dateFormat.js=%Y/%m/%d
(0053109)
hgbot   
2012-10-08 13:44   
Repository: erp/devel/pi
Changeset: f5556ad1abb1e8c86a3fd41fdb02b7ac3525a5a4
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Mon Oct 08 13:43:19 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/f5556ad1abb1e8c86a3fd41fdb02b7ac3525a5a4 [^]

Fixed issue 16300: Now filter date fields have the same behavior
than regular date fields

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-minidaterange.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/utilities/ob-utilities-date.js
M modules/org.openbravo.client.application/web/org.openbravo.userinterface.smartclient/openbravo/skins/Default/org.openbravo.client.application/ob-form-styles.css
M modules/org.openbravo.client.application/web/org.openbravo.userinterface.smartclient/openbravo/skins/Default/org.openbravo.client.application/ob-form-styles.js
---
(0053112)
hgbot   
2012-10-08 13:50   
Repository: erp/devel/pi
Changeset: 4e56b87ebd326bab01ecf0471cc8c8440cd6317c
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Mon Oct 08 13:48:48 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/4e56b87ebd326bab01ecf0471cc8c8440cd6317c [^]

Related to issue 16300: Small code reorganization

---
M modules/org.openbravo.client.application/web/org.openbravo.userinterface.smartclient/openbravo/skins/Default/org.openbravo.client.application/ob-form-styles.css
---
(0053258)
hudsonbot   
2012-10-11 07:50   
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/63dfc135f2a1 [^]

Maturity status: Test
(0053259)
hudsonbot   
2012-10-11 07:50   
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/63dfc135f2a1 [^]

Maturity status: Test
(0053324)
AugustoMauch   
2012-10-11 18:37   
Code reviewed and verified in pi@ae79cd15bab6