openerp-india team mailing list archive

[xrg <xrg@xxxxxxxxx>]

On Thursday 22 March 2012, you wrote:
> Public bug reported:
> 
> In ir_attachement.py line 57:
> In the override method _search, if the option count is True, the method
> _search return an number of ids instead of ids. In this case the method
> should return this number directly without trying to read the
> attachements.

I think you are wrong.

"count" mode in search() should return a reliable count of the items. So, 
going all through the expensive calculation of ids makes sense.

Your patch returns the wrong kind of result. Count is supposed to always 
return a number, not a list or so.

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

Title:
  [6.1][ir_attachement] _search method should work with count option

Status in OpenERP Server:
  New

Bug description:
  In ir_attachement.py line 57: 
  In the override method _search, if the option count is True, the method _search return an number of ids instead of ids.
  In this case the method should return this number directly without trying to read the attachements.

  So I propose to move up : 
  if count:
      return ids
  As I submit in the attachement of the bug

  Best regards,

  Benoît

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