Useless loop in Budget code
Bug #1275464 reported by
Lionel Sausin - Initiatives/Numérigraphe
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Odoo Addons (MOVED TO GITHUB) |
New
|
Undecided
|
Unassigned |
Bug Description
I noticed something weird in the code of Account Budget Lines.
It seems like no-op proxy functions have been introduced in this revision ages ago : revid:<email address hidden>
Here is the first (the other read mostly the same):
def _prac(self, cr, uid, ids, name, args, context=None):
res={}
for line in self.browse(cr, uid, ids, context=context):
return res
Am I missing something, or is this just returning the exact same value as _prac_amt in a totally inefficient way?
Using browse() to get the ids is strange. And it's just rebuilding a dictionary it already has isn't it?
description: | updated |
To post a comment you must log in.