Openbravo Issue Tracking System - Openbravo ERP |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0023675 | Openbravo ERP | A. Platform | public | 2013-04-29 18:25 | 2013-04-30 21:05 |
|
| Reporter | marvintm | |
| Assigned To | AugustoMauch | |
| Priority | normal | Severity | major | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | | Fixed in Version | 3.0MP23 | |
| Merge Request Status | |
| Review Assigned To | alostale |
| 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 | 0023675: OBContext seems to initially contain a wrong value for the Warehouse |
| Description | It seems initially OBContext has a wrong value for the Warehouse. The warehouse contained in the OBContext should be the one displayed in the "User Info Widget". However, it seems it's a different one. |
| Steps To Reproduce | - In the "Product" selector, set the following default expression in the Warehouse field:
if (OB.getContext().getWarehouse() != null) { OB.getContext().getWarehouse().id; }
- As System Admin, open the User window, open in edit mode the Openbravo record and delete the default client, organization and warehouse.
- Log in the application in the F&B client, with a Spanish warehouse.
- Go to the Sales Invoice window. Create a new sales invoice, and a new line for it.
- In the line, verify that no products are shown in the selector. Open the selector, and realize that a US warehouse is picked, even though we defined the default expression to get the warehouse from the OBContext.
- If now you reload the page via the User Info Widget (by clicking on "Apply", without changing anything), and repeat the previous steps, then the result will be correct. |
| Proposed Solution | |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | |
| Attached Files | |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2013-04-29 18:25 | marvintm | New Issue | |
| 2013-04-29 18:25 | marvintm | Assigned To | => AugustoMauch |
| 2013-04-29 18:25 | marvintm | Modules | => Core |
| 2013-04-29 18:25 | marvintm | Triggers an Emergency Pack | => No |
| 2013-04-30 09:19 | shankarb | Assigned To | AugustoMauch => shankarb |
| 2013-04-30 09:38 | shankarb | Assigned To | shankarb => AugustoMauch |
| 2013-04-30 10:57 | AugustoMauch | Steps to Reproduce Updated | bug_revision_view_page.php?rev_id=4636#r4636 |
| 2013-04-30 11:22 | AugustoMauch | Issue Monitored: alostale | |
| 2013-04-30 11:22 | AugustoMauch | Review Assigned To | => alostale |
| 2013-04-30 11:27 | hgbot | Checkin | |
| 2013-04-30 11:27 | hgbot | Note Added: 0058334 | |
| 2013-04-30 11:27 | hgbot | Status | new => resolved |
| 2013-04-30 11:27 | hgbot | Resolution | open => fixed |
| 2013-04-30 11:27 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/a7efc1231554de4989245a9f006cecad40bd8ecb [^] |
| 2013-04-30 11:45 | alostale | Note Added: 0058335 | |
| 2013-04-30 11:45 | alostale | Status | resolved => closed |
| 2013-04-30 11:45 | alostale | Fixed in Version | => 3.0MP23 |
| 2013-04-30 21:05 | hudsonbot | Checkin | |
| 2013-04-30 21:05 | hudsonbot | Note Added: 0058391 | |
|
Notes |
|
|
(0058334)
|
|
hgbot
|
|
2013-04-30 11:27
|
|
Repository: erp/devel/pi
Changeset: a7efc1231554de4989245a9f006cecad40bd8ecb
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Tue Apr 30 11:27:05 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/a7efc1231554de4989245a9f006cecad40bd8ecb [^]
Fixes issue 23675: OBContext initializes properly the user default warehouse
If the user did not have a default warehouse defined, OBContext was not initializing the warehouse properly. Instead of using the previosly obtained default organization to fetch the warehouse, it was using any organization in the user accessible tree. This way, the default warehouse could belong to a organization differente than the default organization.
To fix this, if a default organization is available, it will use it. If there is no default organization available or if there a no warehouses that belong to the default organization, the user accessible organization tree will be used.
---
M src/org/openbravo/base/secureApp/LoginUtils.java
---
|
|
|
|
|
|
Code reviewed + verified on pi@a7efc1231554 |
|
|
|
|
|