openerp-india team mailing list archive

["Yannick Vaucher @ Camptocamp" <1309105@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

In multi company context, when using the group by 'state' in Delivery
Orders I encountered the following error:

(Document type: Picking List, Operation: read)
Traceback (most recent call last):
  File ".../server/openerp/netsvc.py", line 296, in dispatch_rpc
    result = ExportService.getService(service_name).dispatch(method, params)
  File ".../server/openerp/service/web_services.py", line 626, in dispatch
    res = fn(db, uid, *params)
  File ".../server/openerp/osv/osv.py", line 190, in execute_kw
    return self.execute(db, uid, obj, method, *args, **kw or {})
  File ".../server/openerp/osv/osv.py", line 132, in wrapper
    return f(self, dbname, *args, **kwargs)
  File ".../server/openerp/osv/osv.py", line 199, in execute
    res = self.execute_cr(cr, uid, obj, method, *args, **kw)
  File ".../server/openerp/osv/osv.py", line 187, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File ".../server/openerp/osv/orm.py", line 2759, in read_group
    data = self.read(cr, uid, data_ids, [groupby], context=context)
  File ".../addons/stock/stock.py", line 3128, in read
    return self.pool.get('stock.picking').read(cr, uid, ids, fields=fields, context=context, load=load)
  File ".../server/openerp/osv/orm.py", line 3680, in read
    result = self._read_flat(cr, user, select, fields, context, load)
  File ".../server/openerp/osv/orm.py", line 3735, in _read_flat
    self._check_record_rules_result_count(cr, user, sub_ids, result_ids, 'read', context=context)
  File ".../server/openerp/osv/orm.py", line 3939, in _check_record_rules_result_count
    (self._description, operation))
except_osv: (u'Access Denied', u'The requested operation cannot be completed due to security restrictions. Please contact your system administrator.\n\n(Document type: Picking List, Operation: read)')

After digging a bit, I found that orm method read_group calls

orm.py: 2640
        self._apply_ir_rules(cr, uid, query, 'read', context=context)

In there, we get the get domain on stock.picking.out which will be empty
by default. So query will not have any ir_rule applied.

Then, in read_group again, there is the big query that will get the
counts. However it will also takes min(id).

The min id will be passed in read() to which will again be applied the
ir_rules. But this time it won't be None, it will read stock.picking
rule and not the unexistant stock.picking.out rule.

Thus it will fail if min id is a stock picking out from the other
company than the current user's company.

As a temporary patch I duplicated stock.stock_picking_rule for
stock.picking.in and stock.picking.out

I'll provide a patch to solve it on the read_group method in stock
module

** Affects: openobject-addons
     Importance: Undecided
         Status: New

** Description changed:

  In multi company context, when using the group by 'state' in Delivery
  Orders I encountered the following error:
  
- 
  (Document type: Picking List, Operation: read)
  Traceback (most recent call last):
-   File ".../server/openerp/netsvc.py", line 296, in dispatch_rpc
-     result = ExportService.getService(service_name).dispatch(method, params)
-   File ".../server/openerp/service/web_services.py", line 626, in dispatch
-     res = fn(db, uid, *params)
-   File ".../server/openerp/osv/osv.py", line 190, in execute_kw
-     return self.execute(db, uid, obj, method, *args, **kw or {})
-   File ".../server/openerp/osv/osv.py", line 132, in wrapper
-     return f(self, dbname, *args, **kwargs)
-   File ".../server/openerp/osv/osv.py", line 199, in execute
-     res = self.execute_cr(cr, uid, obj, method, *args, **kw)
-   File ".../server/openerp/osv/osv.py", line 187, in execute_cr
-     return getattr(object, method)(cr, uid, *args, **kw)
-   File ".../server/openerp/osv/orm.py", line 2759, in read_group
-     data = self.read(cr, uid, data_ids, [groupby], context=context)
-   File ".../addons/stock/stock.py", line 3128, in read
-     return self.pool.get('stock.picking').read(cr, uid, ids, fields=fields, context=context, load=load)
-   File ".../server/openerp/osv/orm.py", line 3680, in read
-     result = self._read_flat(cr, user, select, fields, context, load)
-   File ".../server/openerp/osv/orm.py", line 3735, in _read_flat
-     self._check_record_rules_result_count(cr, user, sub_ids, result_ids, 'read', context=context)
-   File ".../server/openerp/osv/orm.py", line 3939, in _check_record_rules_result_count
-     (self._description, operation))
+   File ".../server/openerp/netsvc.py", line 296, in dispatch_rpc
+     result = ExportService.getService(service_name).dispatch(method, params)
+   File ".../server/openerp/service/web_services.py", line 626, in dispatch
+     res = fn(db, uid, *params)
+   File ".../server/openerp/osv/osv.py", line 190, in execute_kw
+     return self.execute(db, uid, obj, method, *args, **kw or {})
+   File ".../server/openerp/osv/osv.py", line 132, in wrapper
+     return f(self, dbname, *args, **kwargs)
+   File ".../server/openerp/osv/osv.py", line 199, in execute
+     res = self.execute_cr(cr, uid, obj, method, *args, **kw)
+   File ".../server/openerp/osv/osv.py", line 187, in execute_cr
+     return getattr(object, method)(cr, uid, *args, **kw)
+   File ".../server/openerp/osv/orm.py", line 2759, in read_group
+     data = self.read(cr, uid, data_ids, [groupby], context=context)
+   File ".../addons/stock/stock.py", line 3128, in read
+     return self.pool.get('stock.picking').read(cr, uid, ids, fields=fields, context=context, load=load)
+   File ".../server/openerp/osv/orm.py", line 3680, in read
+     result = self._read_flat(cr, user, select, fields, context, load)
+   File ".../server/openerp/osv/orm.py", line 3735, in _read_flat
+     self._check_record_rules_result_count(cr, user, sub_ids, result_ids, 'read', context=context)
+   File ".../server/openerp/osv/orm.py", line 3939, in _check_record_rules_result_count
+     (self._description, operation))
  except_osv: (u'Access Denied', u'The requested operation cannot be completed due to security restrictions. Please contact your system administrator.\n\n(Document type: Picking List, Operation: read)')
  
- 
- After digging a bit, I found that orm method read_group calls 
+ After digging a bit, I found that orm method read_group calls
  
  orm.py: 2640
-         self._apply_ir_rules(cr, uid, query, 'read', context=context)
+         self._apply_ir_rules(cr, uid, query, 'read', context=context)
  
  In there, we get the get domain on stock.picking.out which will be empty
  by default. So query will not have any ir_rule applied.
  
  Then, in read_group again, there is the big query that will get the
  counts. However it will also takes min(id).
  
  The min id will be passed in read() to which will again be applied the
  ir_rules. But this time it won't be None, it will read stock.picking
  rule and not the unexistant stock.picking.out rule.
  
- Thus it will fail
+ Thus it will fail if min id is a stock picking out from the other
+ company than the current user's company.
  
+ As a temporary patch I duplicated stock.stock_picking_rule for
+ stock.picking.in and stock.picking.out
  
- As a temporary patch I duplicated stock.stock_picking_rule for stock.picking.in and stock.picking.out
- 
- 
- I'll provide a patch to solve it on the read_group method in stock module
+ I'll provide a patch to solve it on the read_group method in stock
+ module

-- 
You received this bug notification because you are a member of OpenERP
Indian Team, which is subscribed to OpenERP Addons.
https://bugs.launchpad.net/bugs/1309105

Title:
  stock.picking.out group by in multi companies

Status in OpenERP Addons (modules):
  New

Bug description:
  In multi company context, when using the group by 'state' in Delivery
  Orders I encountered the following error:

  (Document type: Picking List, Operation: read)
  Traceback (most recent call last):
    File ".../server/openerp/netsvc.py", line 296, in dispatch_rpc
      result = ExportService.getService(service_name).dispatch(method, params)
    File ".../server/openerp/service/web_services.py", line 626, in dispatch
      res = fn(db, uid, *params)
    File ".../server/openerp/osv/osv.py", line 190, in execute_kw
      return self.execute(db, uid, obj, method, *args, **kw or {})
    File ".../server/openerp/osv/osv.py", line 132, in wrapper
      return f(self, dbname, *args, **kwargs)
    File ".../server/openerp/osv/osv.py", line 199, in execute
      res = self.execute_cr(cr, uid, obj, method, *args, **kw)
    File ".../server/openerp/osv/osv.py", line 187, in execute_cr
      return getattr(object, method)(cr, uid, *args, **kw)
    File ".../server/openerp/osv/orm.py", line 2759, in read_group
      data = self.read(cr, uid, data_ids, [groupby], context=context)
    File ".../addons/stock/stock.py", line 3128, in read
      return self.pool.get('stock.picking').read(cr, uid, ids, fields=fields, context=context, load=load)
    File ".../server/openerp/osv/orm.py", line 3680, in read
      result = self._read_flat(cr, user, select, fields, context, load)
    File ".../server/openerp/osv/orm.py", line 3735, in _read_flat
      self._check_record_rules_result_count(cr, user, sub_ids, result_ids, 'read', context=context)
    File ".../server/openerp/osv/orm.py", line 3939, in _check_record_rules_result_count
      (self._description, operation))
  except_osv: (u'Access Denied', u'The requested operation cannot be completed due to security restrictions. Please contact your system administrator.\n\n(Document type: Picking List, Operation: read)')

  After digging a bit, I found that orm method read_group calls

  orm.py: 2640
          self._apply_ir_rules(cr, uid, query, 'read', context=context)

  In there, we get the get domain on stock.picking.out which will be
  empty by default. So query will not have any ir_rule applied.

  Then, in read_group again, there is the big query that will get the
  counts. However it will also takes min(id).

  The min id will be passed in read() to which will again be applied the
  ir_rules. But this time it won't be None, it will read stock.picking
  rule and not the unexistant stock.picking.out rule.

  Thus it will fail if min id is a stock picking out from the other
  company than the current user's company.

  As a temporary patch I duplicated stock.stock_picking_rule for
  stock.picking.in and stock.picking.out

  I'll provide a patch to solve it on the read_group method in stock
  module

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-addons/+bug/1309105/+subscriptions