Openbravo Issue Tracking System - Openbravo ERP | |||||
View Issue Details | |||||
ID | Project | Category | View Status | Date Submitted | Last Update |
0019411 | Openbravo ERP | A. Platform | public | 2012-01-04 13:15 | 2012-01-12 10:26 |
Reporter | dbaz | ||||
Assigned To | dbaz | ||||
Priority | urgent | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Platform | OS | 5 | OS Version | ||
Product Version | |||||
Target Version | Fixed in Version | 3.0MP8 | |||
Merge Request Status | |||||
Review Assigned To | |||||
OBNetwork customer | No | ||||
Web browser | |||||
Modules | Core | ||||
Support ticket | |||||
Regression level | |||||
Regression date | |||||
Regression introduced in release | |||||
Regression introduced by commit | |||||
Triggers an Emergency Pack | No | ||||
Summary | 0019411: Wrong built OB.OnChangeRegistry function | ||||
Description | Right now there is a "registry = []" array. Then there is an assignation like registry[tabId] = {}; This is wrong because registry is defined as an array and not like an object. This could cause things like tabId = '1000500000'; registry[tabId] = {}; Generate a registry.length of 1000500001 | ||||
Steps To Reproduce | You can do it all in the console. Just paste this: OB.OnChangeRegistry.register('1000500000', 'test', function() { return true; }); console.log(OB.OnChangeRegistry.registry.length); Notice that the returned length is 1000500001 | ||||
Proposed Solution | In ob-change-registry.js change line 25. registry: [], by registry: {}, | ||||
Additional Information | |||||
Tags | No tags attached. | ||||
Relationships | |||||
Attached Files | |||||
Issue History | |||||
Date Modified | Username | Field | Change | ||
2012-01-04 13:15 | dbaz | New Issue | |||
2012-01-04 13:15 | dbaz | Assigned To | => dbaz | ||
2012-01-04 13:15 | dbaz | Modules | => Core | ||
2012-01-04 13:15 | dbaz | OBNetwork customer | => No | ||
2012-01-04 13:22 | hgbot | Checkin | |||
2012-01-04 13:22 | hgbot | Note Added: 0044116 | |||
2012-01-04 13:22 | hgbot | Status | new => resolved | ||
2012-01-04 13:22 | hgbot | Resolution | open => fixed | ||
2012-01-04 13:22 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/edf472b58e99d5c8ba2835799c1c514aa105d0ef [^] | ||
2012-01-04 21:31 | mtaal | Note Added: 0044125 | |||
2012-01-04 21:31 | mtaal | Status | resolved => closed | ||
2012-01-04 21:31 | mtaal | Fixed in Version | => 3.0MP8 | ||
2012-01-12 10:26 | hudsonbot | Checkin | |||
2012-01-12 10:26 | hudsonbot | Note Added: 0044287 |
Notes | |||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|