Openbravo Issue Tracking System - Openbravo ERP | |||||
| View Issue Details | |||||
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0021620 | Openbravo ERP | A. Platform | public | 2012-09-12 12:14 | 2012-10-01 13:25 |
| Reporter | marvintm | ||||
| Assigned To | marvintm | ||||
| Priority | immediate | Severity | major | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Platform | OS | 5 | OS Version | ||
| Product Version | |||||
| Target Version | Fixed in Version | 3.0MP16 | |||
| 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 | 0021620: There has been an API change in the Ad_Ref_Data_Loaded table | ||||
| Description | A change in the Ad_Ref_Data_Loaded has changed the API. This change was intended. The Ad_Module_Id column incorrectly had tabledir reference, and has been changed to String. This table stores the correspondence between generic ids and specific ids when importing reference data using DAL. The previous reference for ad_module_id was incorrect because there could be cases in which the ad_ref_data_loaded records pointed to a non-existant module (for example, in case somebody installed a module with reference data, applied the reference data, and then uninstalled it). Maintaining referencial integrity in this case doesn't make sense. However, due to this reference, DAL checks when importing a client if the referenced modules exist, and fails if they don't. This is very very unlikely to happen in a customer environment, but it's a very inconvenient problem when managing our demo data, both the standard included in Core and the retail demo data. As this is a very internal table, the chances that somebody is using the DAL-generated class is very low, and moreover, it's important to remark that the api was broken anyway (the previously generated getModule() method would fail if the referenced module didn't exist), so we think the risk of this change affecting any real customer is extremely low. | ||||
| Steps To Reproduce | ... | ||||
| Proposed Solution | |||||
| Additional Information | |||||
| Tags | No tags attached. | ||||
| Relationships | |||||
| Attached Files | |||||
| Issue History | |||||
| Date Modified | Username | Field | Change | ||
| 2012-09-12 12:14 | marvintm | New Issue | |||
| 2012-09-12 12:14 | marvintm | Assigned To | => iciordia | ||
| 2012-09-12 12:14 | marvintm | Modules | => Core | ||
| 2012-09-12 12:14 | marvintm | OBNetwork customer | => No | ||
| 2012-09-12 15:43 | iciordia | Note Added: 0052022 | |||
| 2012-09-12 15:43 | iciordia | Assigned To | iciordia => marvintm | ||
| 2012-09-12 15:45 | marvintm | Status | new => scheduled | ||
| 2012-09-12 15:45 | marvintm | fix_in_branch | => pi | ||
| 2012-09-12 15:46 | marvintm | Note Added: 0052023 | |||
| 2012-09-12 15:46 | marvintm | Status | scheduled => resolved | ||
| 2012-09-12 15:46 | marvintm | Fixed in SCM revision | => ... | ||
| 2012-09-12 15:46 | marvintm | Resolution | open => fixed | ||
| 2012-09-12 16:16 | hgbot | Checkin | |||
| 2012-09-12 16:16 | hgbot | Note Added: 0052025 | |||
| 2012-09-12 16:16 | hgbot | Fixed in SCM revision | ... => http://code.openbravo.com/erp/devel/api-checks/rev/8fdf08cdb63e14dae6605a07139e3e36e1404c3c [^] | ||
| 2012-10-01 13:25 | alostale | Note Added: 0052956 | |||
| 2012-10-01 13:25 | alostale | Status | resolved => closed | ||
| 2012-10-01 13:25 | alostale | Fixed in Version | => 3.0MP16 | ||
| Notes | |||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||