openerp-dev-web team mailing list archive

["ssi \(Open ERP\)" <ssi@xxxxxxxxxxx>]

ssi (Open ERP) has proposed merging lp:~openerp-dev/openobject-addons/ssi-dev-addons1 into lp:~openerp-dev/openobject-addons/trunk-dev-addons1.

Requested reviews:
  OpenERP Core Team (openerp)


Hello
FIX]Idea: solved problem of visibility(If True creator of the idea will be visible to others) not working. added this functionality on write too.
Kindly check it

Thanks
ssi
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/ssi-dev-addons1/+merge/40088
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/ssi-dev-addons1.

=== modified file 'idea/idea.py'
--- idea/idea.py	2010-09-16 12:51:36 +0000
+++ idea/idea.py	2010-11-04 13:26:05 +0000
@@ -200,12 +200,10 @@
         @return: Returns an id of the new record
         """
         visibility = False
-
         if vals.get('category_id', False):
             category_pool = self.pool.get('idea.category')
             category = category_pool.browse(cr, user, vals.get('category_id'), context)
             visibility = category.visibility
-
         vals.update({
             'visibility':visibility
         })
@@ -252,7 +250,15 @@
         if vals.get('my_vote', False):
             if vals.get('state', state) != 'open':
                 raise osv.except_osv(_("Warning !"), _("Draft/Accepted/Cancelled ideas Could not be voted"))
-
+        
+        visibility = False
+        if vals.get('category_id', False):
+            category_pool = self.pool.get('idea.category')
+            category = category_pool.browse(cr, user, vals.get('category_id'), context)
+            visibility = category.visibility
+        vals.update({
+            'visibility':visibility
+        })
         res = super(idea_idea, self).write(cr, user, ids, vals, context)
         return res
 

=== modified file 'idea/idea_view.xml'
--- idea/idea_view.xml	2010-11-04 06:18:12 +0000
+++ idea/idea_view.xml	2010-11-04 13:26:05 +0000
@@ -198,6 +198,7 @@
                 <field name="name" select="1" />
                 <field name="category_id" select="1"/>
                 <field name="open_date" select="1"/>
+                <field name="visibility" invisible="1"/>	
                 <field name="user_id" attrs="{'invisible':[('visibility','=',False)]}"/>
                 <field name="vote_avg" widget="progressbar"/>
                 <group col="4" colspan="2">