hr_timesheet : on_change_user_id does not pass proper uid!
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
Odoo Addons (MOVED TO GITHUB) | Status tracked in Trunk | |||||
5.0 |
Fix Released
|
Undecided
|
Unassigned | |||
Trunk |
Fix Released
|
Medium
|
JMA(Open ERP) |
Bug Description
Hello,
on v6 trunk at least and very likely on 5.0 too (can see that on older revision in like v5.0.12 at least), in the hr_timesheet module:
if you enter some hr.analytic.
Indeed, the on_change_user_id method was implementation is totally wrong and doesn't even try to use the user_id (guys you never tried that code, even once, right?)
Here is an attached patch that fixes it (it doesn't claim that the overall resulting code will be clean and consistent but at least makes it work, BTW, names of function do not enforce PEP naming conventions in that file (Java camel case style instead)).
Hope this help. I'm sad to see things that have obviously never been tested presented as ""quality"" addons.
Changed in openobject-addons: | |
assignee: | nobody → JMA(Open ERP) (jma-openerp) |
Changed in openobject-addons: | |
status: | New → Confirmed |
Guys, I re-attached a new patch as the previous had a type (= symbol after return). This one works.