Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0006874Openbravo ERPA. Platformpublic2009-01-13 18:042009-04-21 12:39
Reportershuehner 
Assigned Todbaz 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0006874: Validation of fields in the search popup does not work for numeric fields
DescriptionWhen you enable an extra filter field (i.e. Grand Total for the table C_Order) and open the search popup for i.e. the Sales Order window. Then the validation (if the content is a number) does not work for this new field.

Two Problems:
- the import for the javascript file containing the validation function is missing
- the html-code is generated with the wrong fieldname when calling this validation function.
Steps To ReproduceSwitch to System Administrator role
Go to "Tables and column", and search C_Order.
Check GrandTotal column as Filter column
Switch to i.e. openbravo admin role
Go to Sales Order window, and open seach pop-up.
enter "hello world" in i.e. the "Grand Total from" field
See that no error message is shown next to the field saying this text is not a number
Proposed Solution- Add the missing js-import if a numeric field is shown
- fix the generated html to output the correct fieldname when calling the validateNumberBo function
Additional Information
TagsNo tags attached.
Relationships
Attached Filesdiff fix_for_6874_part1.diff (924) 2009-01-13 19:40
https://issues.openbravo.com/file_download.php?file_id=636&type=bug
Issue History
Date ModifiedUsernameFieldChange
2009-01-13 18:04shuehnerNew Issue
2009-01-13 18:04shuehnerAssigned To => rafaroda
2009-01-13 18:04shuehnersf_bug_id0 => 2504957
2009-01-13 18:04shuehnerOBNetwork customer => No
2009-01-13 18:04shuehnerRegression testing => No
2009-01-13 19:08shuehnerNote Added: 0012153
2009-01-13 19:40shuehnerFile Added: fix_for_6874_part1.diff
2009-01-13 19:41shuehnerNote Added: 0012160
2009-01-19 10:51psarobeStatusnew => scheduled
2009-01-19 10:51psarobeAssigned Torafaroda => dbaz
2009-01-19 10:51psarobefix_in_branch => trunk
2009-02-02 21:41svnbotCheckin
2009-02-02 21:41svnbotNote Added: 0013042
2009-02-02 21:41svnbotStatusscheduled => resolved
2009-02-02 21:41svnbotResolutionopen => fixed
2009-02-02 21:41svnbotsvn_revision => 12690
2009-04-21 12:39psarobeStatusresolved => closed

Notes
(0012153)
shuehner   
2009-01-13 19:08   
This issue exists in trunk,2.40,2.35
(0012160)
shuehner   
2009-01-13 19:41   
Attached patch for first part of the problem (missing js-script import for validateNumberBox). The wrong parameter/field-name problem on calling the validation function is still open.
(0013042)
svnbot   
2009-02-02 21:41   
Repository: openbravo
Revision: 12690
Author: dbaz
Date: 2009-02-02 21:41:17 +0100 (Mon, 02 Feb 2009)

Fixed bug 0006874: added ids to numeric inputs of Buscador.java and fixed js error of auto_complete_number

---
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CopyFromOrder.html
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CreateFromMultiple_Shipment.html
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CreateFrom_Settlement.html
U trunk/src/org/openbravo/erpCommon/ad_forms/MaterialReceiptPending.html
U trunk/src/org/openbravo/erpCommon/ad_forms/RequisitionToOrder.html
U trunk/src/org/openbravo/erpCommon/ad_forms/ShowSessionPreferences.html
U trunk/src/org/openbravo/erpCommon/businessUtility/Buscador.java
U trunk/src/org/openbravo/erpCommon/info/AttributeSetInstance.java
U trunk/src-wad/lib/openbravo-wad.jar
U trunk/src-wad/src/org/openbravo/wad/WadActionButton.java
U trunk/src-wad/src/org/openbravo/wad/WadUtility.java
U trunk/web/js/utils.js
---

https://dev.openbravo.com/websvn/openbravo/?rev=12690&sc=1 [^]