Merge lp:~openerp-dev/openobject-addons/6.1-opw-574714-skh into lp:openobject-addons/6.1

[Somesh Khare]

Hello Sir,

[FIX] Idea : Comments posted along with a vote is not stored/displayed in idea.vote object.

This branch fixes this issue.

Kindly review the branch and please share your views on it.

Thanks
Somesh Khare

[Naresh(OpenERP) (nch-openerp)]

Hello,

This bug was qualified as Not Relevant on Trunk (means that architecture on trunk has changed and so this bug has no meaning anymore). If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh Soni

Unmerged revisions

6795. By Somesh Khare on 2012-05-11

[FIX] Idea : Comments posted along with a vote is not stored/displayed in idea.vote object (Case: Ref 574714)

=== modified file 'idea/idea.py'
--- idea/idea.py	2011-12-31 07:57:20 +0000
+++ idea/idea.py	2012-05-11 07:26:18 +0000
@@ -160,17 +160,22 @@
         @param ids: List of vote save’s IDs """
 
         vote_obj = self.pool.get('idea.vote')
+        comment_obj = self.pool.get('idea.comment')
         vote = vote_obj.search(cr, uid, [('idea_id', '=', id), ('user_id', '=', uid)])
         textual_value = str(field_value)
-
+        comment = comment_obj.read(cr, uid,id,['content'], context=context)
+        if comment:
+            vote_comment = comment['content']
+        else:
+            vote_comment =''
         if vote:
             if int(field_value) >= 0:
-                vote_obj.write(cr, uid, vote, {'score': textual_value })
+                vote_obj.write(cr, uid, vote, {'score': textual_value ,'comment':vote_comment})
             else:
                 vote_obj.unlink(cr, uid, vote)
         else:
             if int(field_value) >= 0:
-                vote_obj.create(cr, uid, {'idea_id': id, 'user_id': uid, 'score': textual_value })
+                vote_obj.create(cr, uid, {'idea_id': id, 'user_id': uid, 'score': textual_value ,'comment':vote_comment})
 
     _columns = {
         'user_id': fields.many2one('res.users', 'Creator', required=True, readonly=True),