Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0035899
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementmajoralways2017-05-02 18:392017-05-29 17:43
ReporterngarciaView Statuspublic 
Assigned Tocollazoandy4 
PriorityurgentResolutionfixedFixed in Version3.0PR17Q3
StatusclosedFix in branchFixed in SCM revision7ea7f1503427
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Toaferraz
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0035899: In Stock Report, you can see stock for warehouse's that belong to an organization you don't have access to

DescriptionIn Stock Report, you can see stock for warehouse's that belong to an organization you don't have access to

The problem is that the records in m_transaction are filtered by the natural tree of the organization the user has access to and warehouses are not filtered at all
Steps To ReproduceAs group admin role:
   Create a new product for F&B España, S.A.
   Create a Goods Receipt for F&B España, S.A. and receive the previously created product in a storage bin of España Region Norte and a storage bin of España Region Sur
   Create a goods shipment for F&B España Region Norte and add the previously created product, reduce the shipped quantity not to leave its stock to 0
   Create a new advanced role
   Assign it to Openbravo user
   Remove the access to all the organizations except F&B España Region Sur
   Log out and log in
   Select the previously created role
   Launch the Stock Report and filter by the previously created role
   Check the stock in España Region Norte and Region Sur warehouses is shown which is not correct. Moreover, you see more stock than the available one in España Region Norte
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0096430)
collazoandy4 (reporter)
2017-05-05 23:12

As group admin role:
   Go to Product window and create a new record:
     Organization: F&B España, S.A.
     Search Key/Name: Product
   Go to Goods Receipt and create a new record:
     Organization: F&B España, S.A.
     Business Partner: Bebidas Alegres, S.L.
     Warehouse: España Región Norte
     Create a new record in Lines tab:
       Product: Product
       Movement Quantity: 10
     Complete the receipt
   Create another Goods Receipt record:
     Organization: F&B España, S.A.
     Business Partner: Bebidas Alegres, S.L.
     Warehouse: España Región Sur
     Create a new record in Lines tab:
       Product: Product
       Movement Quantity: 15
     Complete the receipt
   Go to Goods Shipment window and create a new record:
     Organization: F&B España - Región Norte
     Business Partner: Alimentos y Supermercados, S.A
     Warehouse: España Región Norte
     Create a new record in Lines tab:
       Product: Product
       Movement Quantity: 5
     Complete the shipment
   Go to Role window and create a new record:
     Name: Role
     User Level: Client+Organization
     Remove all organizations acces lines except F&B España Region Sur
     Create a new record in User Assigment tab:
       User/Contact: Openbravo
   Log out and log in
   Select the previously created role
   Launch the Stock Report and filter by the previously created product
   Check the stock in España Region Norte warehouses is not shown
(0096538)
hgbot (developer)
2017-05-11 19:46

Repository: erp/devel/pi
Changeset: 7ea7f150342767534c16da2ccf25016adbb7cb5b
Author: Armaignac <collazoandy4 <at> gmail.com>
Date: Fri May 05 17:05:02 2017 -0400
URL: http://code.openbravo.com/erp/devel/pi/rev/7ea7f150342767534c16da2ccf25016adbb7cb5b [^]

Fixes issue 35899: Stock Report shows stocks from org that don't have access

The Stock Report shown stocks from warehouse's that belong to an organization that the user
don't have access because the query doesn't filter by the warehouse organization

Now the records of M_LOCATOR are filtered by the natural tree of the organization the
user has access to.

---
M src/org/openbravo/erpCommon/ad_reports/ReportWarehousePartner_data.xsql
---
(0096541)
aferraz (manager)
2017-05-11 19:47
edited on: 2017-05-12 13:14

Test plan II:

As group admin role:
   Create a new product for F&B España, S.A.
   Create a new warehouse and storage bin for F&B España, S.A.
   Create a Goods Receipt for España Norte and receive the previously created product in the previously created storage bin.
   Create a new advanced role
   Assign it to Openbravo user
   Remove the access to all the organizations except F&B España Region Sur
   Log out and log in
   Select the previously created role
   Launch the Stock Report and filter by the previously created product
   Check the stock in new warehouse is shown as it belongs to F&B España, S.A. organization, which is in the natural tree of F&B España Region Sur organization.

(0096563)
hgbot (developer)
2017-05-12 13:11

Repository: erp/devel/pi
Changeset: 7db0e715083badf6c02bcdab8439ed0e81c66723
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Fri May 12 13:11:06 2017 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/7db0e715083badf6c02bcdab8439ed0e81c66723 [^]

Related to issue 35899: Code review improvements

Do not filter m_transaction records by the organization natural tree.
Only m_locator records will be filtered by the organization natural tree.

---
M src/org/openbravo/erpCommon/ad_reports/ReportWarehousePartner_data.xsql
---
(0096564)
aferraz (manager)
2017-05-12 13:19

Code review + Testing OK
(0096959)
hudsonbot (developer)
2017-05-29 17:43

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/1ee70113bdc4 [^]
Maturity status: Test
(0096963)
hudsonbot (developer)
2017-05-29 17:43

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/1ee70113bdc4 [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2017-05-02 18:39 ngarcia New Issue
2017-05-02 18:39 ngarcia Assigned To => Triage Finance
2017-05-02 18:39 ngarcia Modules => Core
2017-05-02 18:39 ngarcia Resolution time => 1495490400
2017-05-02 18:39 ngarcia Triggers an Emergency Pack => No
2017-05-02 18:39 ngarcia Issue Monitored: networkb
2017-05-02 23:24 eugeni Issue Monitored: eugeni
2017-05-04 16:28 aferraz Assigned To Triage Finance => collazoandy4
2017-05-05 23:12 collazoandy4 Status new => scheduled
2017-05-05 23:12 collazoandy4 Note Added: 0096430
2017-05-11 19:46 hgbot Checkin
2017-05-11 19:46 hgbot Note Added: 0096538
2017-05-11 19:46 hgbot Status scheduled => resolved
2017-05-11 19:46 hgbot Resolution open => fixed
2017-05-11 19:46 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/7ea7f150342767534c16da2ccf25016adbb7cb5b [^]
2017-05-11 19:47 aferraz Review Assigned To => aferraz
2017-05-11 19:47 aferraz Note Added: 0096541
2017-05-11 19:47 aferraz Status resolved => closed
2017-05-11 19:47 aferraz Fixed in Version => 3.0PR17Q3
2017-05-12 13:11 hgbot Checkin
2017-05-12 13:11 hgbot Note Added: 0096563
2017-05-12 13:14 aferraz Note Edited: 0096541 View Revisions
2017-05-12 13:19 aferraz Note Added: 0096564
2017-05-29 17:43 hudsonbot Checkin
2017-05-29 17:43 hudsonbot Note Added: 0096959
2017-05-29 17:43 hudsonbot Checkin
2017-05-29 17:43 hudsonbot Note Added: 0096963


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker