Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0027226 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] A. Platform | major | always | 2014-07-29 12:07 | 2014-08-11 09:44 | |||
| Reporter | caristu | View Status | public | |||||
| Assigned To | shankarb | |||||||
| Priority | high | Resolution | fixed | Fixed in Version | 3.0PR14Q4 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | be933e11432c | ||||
| Projection | none | ETA | none | Target Version | 3.0PR14Q4 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | pi | SCM revision | ||||||
| Merge Request Status | ||||||||
| Review Assigned To | alostale | |||||||
| OBNetwork customer | No | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Support ticket | 30085 | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0027226: Possible undefined value should be controlled in OBPlainToOBMasked method | |||||||
| Description | The OBPlainToOBMasked method in ob-utilities-number.js file receives several parameters. One of them is groupInterval and in some parts of the application this method is called without defining this parameter. For example in the blur function of the OBNumberItem class, this parameter is not defined. If this value is not defined a javascript error is raised when the execution arrives to the following point of the OBPlainToOBMasked method: if (groupCounter.toString() === groupInterval.toString() && k !== 1) { It should be checked if the groupInterval is null or undefined before this point. | |||||||
| Steps To Reproduce | 1) Create a new numeric base reference 2) Create a column and field in a window that uses the previous reference 3) Try to save a record that contains that field | |||||||
| Proposed Solution | Add a check in the OBPlainToOBMasked method like this: if (groupInterval === null || groupInterval === undefined) { groupInterval = OB.Format.defaultGroupingSize; } | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0068991) hgbot (developer) 2014-07-31 06:09 |
Repository: erp/devel/pi Changeset: be933e11432ce4785c2bea4ee38281f67d6f2c5e Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com> Date: Wed Jul 30 11:58:04 2014 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/be933e11432ce4785c2bea4ee38281f67d6f2c5e [^] Fixes issue 27226: Possible undefined value should be controlled in OBPlainToOBMasked method When grouping interval is undefined, use the default grouping interval. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/utilities/ob-utilities-number.js --- |
|
(0069248) hudsonbot (viewer) 2014-08-08 18:54 |
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/4450016dee64 [^] Maturity status: Test |
|
(0069286) alostale (viewer) 2014-08-11 09:44 |
code reviewed + tested |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2014-07-29 12:07 | caristu | New Issue | |
| 2014-07-29 12:07 | caristu | Assigned To | => AugustoMauch |
| 2014-07-29 12:07 | caristu | OBNetwork customer | => No |
| 2014-07-29 12:07 | caristu | Modules | => Core |
| 2014-07-29 12:07 | caristu | Support ticket | => 30085 |
| 2014-07-29 12:07 | caristu | Resolution time | => 1409263200 |
| 2014-07-29 12:07 | caristu | Triggers an Emergency Pack | => No |
| 2014-07-29 12:08 | caristu | Proposed Solution updated | |
| 2014-07-29 12:08 | caristu | Issue Monitored: networkb | |
| 2014-07-30 06:32 | shankarb | Assigned To | AugustoMauch => shankarb |
| 2014-07-30 08:32 | shankarb | Status | new => scheduled |
| 2014-07-30 08:32 | shankarb | fix_in_branch | => pi |
| 2014-07-31 06:05 | shankarb | Review Assigned To | => alostale |
| 2014-07-31 06:05 | shankarb | fix_in_branch | pi => |
| 2014-07-31 06:09 | hgbot | Checkin | |
| 2014-07-31 06:09 | hgbot | Note Added: 0068991 | |
| 2014-07-31 06:09 | hgbot | Status | scheduled => resolved |
| 2014-07-31 06:09 | hgbot | Resolution | open => fixed |
| 2014-07-31 06:09 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/be933e11432ce4785c2bea4ee38281f67d6f2c5e [^] |
| 2014-08-08 18:54 | hudsonbot | Checkin | |
| 2014-08-08 18:54 | hudsonbot | Note Added: 0069248 | |
| 2014-08-11 09:44 | alostale | Note Added: 0069286 | |
| 2014-08-11 09:44 | alostale | Status | resolved => closed |
| 2014-08-11 09:44 | alostale | Fixed in Version | => 3.0PR14Q4 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |