Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0021405Openbravo ERP04. Warehouse managementpublic2012-08-21 10:462012-09-26 17:19
gorkaion 
naiaramartinez 
normalminorhave not tried
closedfixed 
20Gentoo
 
 
dmiguelez
Core
No
0021405: Costing Background process cannot be executed by a role with limited access
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.
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
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.
No tags attached.
Issue History
2012-08-21 10:46gorkaionNew Issue
2012-08-21 10:46gorkaionAssigned To => mirurita
2012-08-21 10:46gorkaionModules => Core
2012-08-21 10:50gorkaionDescription Updatedbug_revision_view_page.php?rev_id=3734#r3734
2012-08-21 10:50gorkaionSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=3736#r3736
2012-08-21 10:50gorkaionProposed Solution updated
2012-08-22 13:21naiaramartinezAssigned Tomirurita => naiaramartinez
2012-08-22 13:30naiaramartinezStatusnew => scheduled
2012-08-22 13:30naiaramartinezfix_in_branch => pi
2012-09-11 12:04naiaramartinezNote Added: 0051987
2012-09-11 17:39naiaramartinezNote Edited: 0051987bug_revision_view_page.php?bugnote_id=0051987#r3821
2012-09-13 12:45naiaramartinezNote Edited: 0051987bug_revision_view_page.php?bugnote_id=0051987#r3844
2012-09-13 12:46hgbotCheckin
2012-09-13 12:46hgbotNote Added: 0052056
2012-09-13 12:46hgbotStatusscheduled => resolved
2012-09-13 12:46hgbotResolutionopen => fixed
2012-09-13 12:46hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/3292566b5a1bf439516d26f3cb35638e5effbb29 [^]
2012-09-13 12:47dmiguelezClosed by => dmiguelez
2012-09-13 12:47dmiguelezNote Added: 0052057
2012-09-13 12:47dmiguelezStatusresolved => closed
2012-09-26 17:19hudsonbotCheckin
2012-09-26 17:19hudsonbotNote Added: 0052741

Notes
(0051987)
naiaramartinez   
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   
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   
2012-09-13 12:47   
Code Review + Testing OK
(0052741)
hudsonbot   
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