Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0012714Openbravo ERPB. User interfacepublic2010-03-17 22:462010-04-19 21:10
dbaz 
dbaz 
urgentmajoralways
closedfixed 
5
 
 
Core
No
0012714: maxlength is not taked into account when introducing a comma as decimal separator
With Format.xml set as the comma "," is the decimal separator, maxlength property of the input is not taked into account while introducing "," with the numeric keyboard
Open Format.xml and set
decimal="," grouping="."
all the fields.

Make an install.source

Go to Master Data Management || Business Partner || Business Partner

Go to "Consumption Days" field

You can see that you are able to write up to 10 characters... but if the 10 characters are written and you press the numerical keyboard dot "." a new comma is added (which is wrong, since the 10 characters limit is reached)
No tags attached.
Issue History
2010-03-17 22:46dbazNew Issue
2010-03-17 22:46dbazAssigned To => dbaz
2010-03-17 22:49hgbotCheckin
2010-03-17 22:49hgbotNote Added: 0025583
2010-03-17 22:49hgbotStatusnew => resolved
2010-03-17 22:49hgbotResolutionopen => fixed
2010-03-17 22:49hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/a94300f1b39422948e4d4924e9cf8cfc5a6d8e8b [^]
2010-03-17 22:50hgbotCheckin
2010-03-17 22:50hgbotNote Added: 0025584
2010-03-17 22:50hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/a94300f1b39422948e4d4924e9cf8cfc5a6d8e8b [^] => http://code.openbravo.com/erp/mods/org.openbravo.userinterface.smartclient/rev/45f8efa1e94d6828979c72e3ac1ec9b5fe467501 [^]
2010-03-17 22:50dbazStatusresolved => new
2010-03-17 22:50dbazResolutionfixed => open
2010-03-17 22:50dbazFixed in SCM revisionhttp://code.openbravo.com/erp/mods/org.openbravo.userinterface.smartclient/rev/45f8efa1e94d6828979c72e3ac1ec9b5fe467501 [^] => http://code.openbravo.com/erp/devel/pi/rev/a94300f1b39422948e4d4924e9cf8cfc5a6d8e8b [^]
2010-03-17 22:51dbazNote Edited: 0025584bug_revision_view_page.php?bugnote_id=0025584#r167
2010-03-17 22:51dbazStatusnew => acknowledged
2010-03-17 22:51dbazStatusacknowledged => scheduled
2010-03-17 22:51dbazfix_in_branch => pi
2010-03-17 22:51dbazStatusscheduled => resolved
2010-03-17 22:51dbazResolutionopen => fixed
2010-03-18 00:41hgbotCheckin
2010-03-18 00:41hgbotNote Added: 0025585
2010-04-15 10:33iperdomoNote Added: 0026232
2010-04-15 10:33iperdomoStatusresolved => closed
2010-04-16 00:00anonymoussf_bug_id0 => 2987969
2010-04-19 21:10hudsonbotCheckin
2010-04-19 21:10hudsonbotNote Added: 0026353

Notes
(0025583)
hgbot   
2010-03-17 22:49   
Repository: erp/devel/pi
Changeset: a94300f1b39422948e4d4924e9cf8cfc5a6d8e8b
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Wed Mar 17 22:56:26 2010 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/a94300f1b39422948e4d4924e9cf8cfc5a6d8e8b [^]

Fixed issue 12714: Now maxlength is taked into account when typing comma

---
M src/org/openbravo/erpCommon/security/Login_F1.html
M web/js/utils.js
---
(0025584)
hgbot   
2010-03-17 22:50   
(edited on: 2010-03-17 22:51)
Related fix to the Smartclient number widget

Repository: erp/mods/org.openbravo.userinterface.smartclient
Changeset: 45f8efa1e94d6828979c72e3ac1ec9b5fe467501
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Wed Mar 17 22:57:20 2010 +0100
URL: http://code.openbravo.com/erp/mods/org.openbravo.userinterface.smartclient/rev/45f8efa1e94d6828979c72e3ac1ec9b5fe467501 [^]

Fixed issue 12714: Now maxlength is taked into account when typing comma

---
M web/org.openbravo.userinterface.smartclient/openbravo/system/OB_Widgets.js
---

(0025585)
hgbot   
2010-03-18 00:41   
Repository: erp/mods/org.openbravo.userinterface.smartclient
Changeset: 098d2a19624ac883a5633dae333218bef8d78df1
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Thu Mar 18 00:48:55 2010 +0100
URL: http://code.openbravo.com/erp/mods/org.openbravo.userinterface.smartclient/rev/098d2a19624ac883a5633dae333218bef8d78df1 [^]

Related to issue 12714: Now maxlength is taked into account when typing comma

---
M web/org.openbravo.userinterface.smartclient/openbravo/system/ob_formitem_widgets.js
---
(0026232)
iperdomo   
2010-04-15 10:33   
Tested on PI @ rev 7076c219cdd9
(0026353)
hudsonbot   
2010-04-19 21:10   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/a94300f1b394 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/91d98bda46c1 [^]
Tests: http://builds.openbravo.com/view/devel-int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.17088.obx [^]