Merge lp:~openerp-dev/openobject-addons/7.0-opw-592256-rgo into lp:openobject-addons/7.0

Proposed by Ravi Gohil (OpenERP)
Status: Merged
Approved by: Naresh(OpenERP)
Approved revision: no longer in the source branch.
Merged at revision: 9708
Proposed branch: lp:~openerp-dev/openobject-addons/7.0-opw-592256-rgo
Merge into: lp:openobject-addons/7.0
Diff against target: 12 lines (+1/-1)
1 file modified
document/report/document_report.py (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/7.0-opw-592256-rgo
Reviewer Review Type Date Requested Status
Naresh(OpenERP) (community) Approve
Review via email: mp+162760@code.launchpad.net

Description of the change

"user_id" field is defined as integer for postgresql report "report_document_user", which is wrong, it should be of type many2one as it is referred in fields.related defined in same class.

Relevant code of 'report.document.user':
        'user_id':fields.integer('Owner', readonly=True),
        'user': fields.related('user_id', 'name', type='char', size=64, readonly=True),

There isn't any specific menu defined to access this in official module, but there may be some community module using it.

This fix will fix this issue, kindly review it.

Note: After applying the patch, user need to upgrade the document module for patch to take the effect.

Thanks.

To post a comment you must log in.
Revision history for this message
Naresh(OpenERP) (nch-openerp) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'document/report/document_report.py'
2--- document/report/document_report.py 2012-12-06 14:56:32 +0000
3+++ document/report/document_report.py 2013-05-07 12:21:26 +0000
4@@ -31,7 +31,7 @@
5 'name': fields.char('Year', size=64,readonly=True),
6 'month':fields.selection([('01','January'), ('02','February'), ('03','March'), ('04','April'), ('05','May'), ('06','June'),
7 ('07','July'), ('08','August'), ('09','September'), ('10','October'), ('11','November'), ('12','December')],'Month',readonly=True),
8- 'user_id':fields.integer('Owner', readonly=True),
9+ 'user_id': fields.many2one('res.users', 'Owner', readonly=True),
10 'user': fields.related('user_id', 'name', type='char', size=64, readonly=True),
11 'directory': fields.char('Directory',size=64,readonly=True),
12 'datas_fname': fields.char('File Name',size=64,readonly=True),