Openbravo Issue Tracking System - Openbravo ERP | |||||
| View Issue Details | |||||
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0020659 | Openbravo ERP | A. Platform | public | 2012-06-01 10:05 | 2012-06-07 12:37 |
| Reporter | mtaal | ||||
| Assigned To | AugustoMauch | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Platform | OS | 5 | OS Version | ||
| Product Version | |||||
| Target Version | 3.0MP13 | Fixed in Version | 3.0MP13 | ||
| Merge Request Status | |||||
| Review Assigned To | |||||
| OBNetwork customer | |||||
| Web browser | |||||
| Modules | Core | ||||
| Support ticket | |||||
| Regression level | |||||
| Regression date | |||||
| Regression introduced in release | |||||
| Regression introduced by commit | |||||
| Triggers an Emergency Pack | No | ||||
| Summary | 0020659: OrganizationStructureProvider throws NPE if passed org id does not belong to the client | ||||
| Description | The OrganizationStructureProvider class throws an NPE in this line (271): return new HashSet<String>(childByOrganizationID.get(orgId)); this because childByOrganizationID.get(orgId) returns null and the HashSet constructor can't handle that. It is nicer to either throw a more meaningfull exception (organization not valid for client or something) or return an empty result. | ||||
| Steps To Reproduce | Get the organization structure provider from the context: Set<String> orgChildTree = OBContext.getOBContext().getOrganizationStructureProvider() .getChildTree(strOrg, true); the strOrg should be a non-existing id a NPE will be thrown it is nice to check this issue through a junit testcase | ||||
| Proposed Solution | |||||
| Additional Information | |||||
| Tags | No tags attached. | ||||
| Relationships | |||||
| Attached Files | |||||
| Issue History | |||||
| Date Modified | Username | Field | Change | ||
| 2012-06-01 10:05 | mtaal | New Issue | |||
| 2012-06-01 10:05 | mtaal | Assigned To | => marvintm | ||
| 2012-06-01 10:05 | mtaal | Modules | => Core | ||
| 2012-06-04 12:13 | marvintm | Assigned To | marvintm => AugustoMauch | ||
| 2012-06-05 15:09 | hgbot | Checkin | |||
| 2012-06-05 15:09 | hgbot | Note Added: 0049619 | |||
| 2012-06-05 15:09 | hgbot | Status | new => resolved | ||
| 2012-06-05 15:09 | hgbot | Resolution | open => fixed | ||
| 2012-06-05 15:09 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/b9a5fed6dd96ef644fcf8c6e31260a4f93ccf340 [^] | ||
| 2012-06-07 12:37 | guilleaer | Note Added: 0049667 | |||
| 2012-06-07 12:37 | guilleaer | Status | resolved => closed | ||
| 2012-06-07 12:37 | guilleaer | Fixed in Version | => 3.0MP13 | ||
| Notes | |||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||