Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0021405 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 04. Warehouse management | minor | have not tried | 2012-08-21 10:46 | 2012-09-26 17:19 | |||
| Reporter | gorkaion | View Status | public | |||||
| Assigned To | naiaramartinez | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | ||||
| Status | closed | Fix in branch | pi | Fixed in SCM revision | 3292566b5a1b | |||
| Projection | none | ETA | none | Target Version | ||||
| OS | Linux 32 bit | Database | Oracle | Java version | 1.5.0_15 | |||
| OS Version | Gentoo | Database version | 11g | Ant version | 1.7.0 | |||
| Product Version | SCM revision | |||||||
| Review Assigned To | dmiguelez | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0021405: Costing Background process cannot be executed by a role with limited access | |||||||
| Description | The costing background process cannot be executed by a role that for example does not have access to the Product window. If a role is granted access to the process it should be able to run it. The process do not show any error in the Process Monitor, but the logs show a ''Entity MaterialMgmtMaterialTransaction is not writable by this user'' error. | |||||||
| Steps To Reproduce | 1. Create a manual role and give access to the Process Request and Process Monitor windows and the Costing migration process. 2. Ensure that there are transactions pending to calculate, if there isn't any create a shipment/inventory/movement with any other role. 3. Schedule the Costing migration process to be run immediately. using the new role. 4. Check in the logs that it shows an error like | |||||||
| Proposed Solution | 1. Fix the Costing Migration Process with a call to setAdminMode. 2. Fix the Costing Migration Process with a global catch to properly set the error message in the result bundle. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0051987) naiaramartinez (developer) 2012-09-11 12:04 edited on: 2012-09-13 12:45 |
Test Plan 1: 1. Create a manual role and give access to the Process Request and Process Monitor windows and the Costing background process. 2. Ensure that there are transactions pending to calculate, if there isn't any create a shipment/inventory/movement with any other role. 3. Schedule the Costing background process to be run immediately. using the new role. 4. Check in the logs(Process Monitor window) that process is completed succesfully. Test Plan 2: 1.If in the code you try to comment the following line: OBContext.setAdminMode(false); 2. Create a manual role and give access to the Process Request and Process Monitor windows and the Costing background process. 3. Ensure that there are transactions pending to calculate, if there isn't any create a shipment/inventory/movement with any other role. 4. Schedule the Costing background process to be run immediately. using the new role. 5. Check in the logs(Process Monitor window) that error is shown. |
|
(0052056) hgbot (developer) 2012-09-13 12:46 |
Repository: erp/devel/pi Changeset: 3292566b5a1bf439516d26f3cb35638e5effbb29 Author: Naiara Martinez <naiara.martinez <at> openbravo.com> Date: Tue Sep 11 12:02:07 2012 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/3292566b5a1bf439516d26f3cb35638e5effbb29 [^] fixed bug 21405 Costing Background process can't be executed by a role with limited access --- M src/org/openbravo/costing/CostingBackground.java --- |
|
(0052057) dmiguelez (developer) 2012-09-13 12:47 |
Code Review + Testing OK |
|
(0052741) hudsonbot (developer) 2012-09-26 17:19 |
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/b780e90c6452 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2012-08-21 10:46 | gorkaion | New Issue | |
| 2012-08-21 10:46 | gorkaion | Assigned To | => mirurita |
| 2012-08-21 10:46 | gorkaion | Modules | => Core |
| 2012-08-21 10:50 | gorkaion | Description Updated | View Revisions |
| 2012-08-21 10:50 | gorkaion | Steps to Reproduce Updated | View Revisions |
| 2012-08-21 10:50 | gorkaion | Proposed Solution updated | |
| 2012-08-22 13:21 | naiaramartinez | Assigned To | mirurita => naiaramartinez |
| 2012-08-22 13:30 | naiaramartinez | Status | new => scheduled |
| 2012-08-22 13:30 | naiaramartinez | fix_in_branch | => pi |
| 2012-09-11 12:04 | naiaramartinez | Note Added: 0051987 | |
| 2012-09-11 17:39 | naiaramartinez | Note Edited: 0051987 | View Revisions |
| 2012-09-13 12:45 | naiaramartinez | Note Edited: 0051987 | View Revisions |
| 2012-09-13 12:46 | hgbot | Checkin | |
| 2012-09-13 12:46 | hgbot | Note Added: 0052056 | |
| 2012-09-13 12:46 | hgbot | Status | scheduled => resolved |
| 2012-09-13 12:46 | hgbot | Resolution | open => fixed |
| 2012-09-13 12:46 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/3292566b5a1bf439516d26f3cb35638e5effbb29 [^] |
| 2012-09-13 12:47 | dmiguelez | Closed by | => dmiguelez |
| 2012-09-13 12:47 | dmiguelez | Note Added: 0052057 | |
| 2012-09-13 12:47 | dmiguelez | Status | resolved => closed |
| 2012-09-26 17:19 | hudsonbot | Checkin | |
| 2012-09-26 17:19 | hudsonbot | Note Added: 0052741 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |