Merge lp:~openerp-dev/openobject-addons/trunk-bug-1103072-ishwar into lp:openobject-addons

[Ishwar Malvi(OpenERP)]

Hello,

      I have fixed a problem of survey : survey invited user can't able to answer survey.

Thanks,
Ishwar Malvi

[Stefan Rijnhart (Opener) (stefan-opener)]

We feel that this change is granting too broad permissions to the invited respondents. My colleague Holger proposes a more refined change here:

https://code.launchpad.net/~therp-nl/openobject-addons/7.0_make_survey_module_usable_for_external_users/+merge/180328

review: Disapprove

Unmerged revisions

8639. By Ishwar Malvi(OpenERP) on 2013-03-21

[FIX] survey : In Survey Invitations invited user can't view survey

=== modified file 'survey/security/survey_security.xml'
--- survey/security/survey_security.xml	2012-04-04 09:08:30 +0000
+++ survey/security/survey_security.xml	2013-03-21 10:57:43 +0000
@@ -16,4 +16,14 @@
             <field name="users" eval="[(4, ref('base.user_root'))]"/>
         </record>
     </data>
+    <data noupdate="1">
+		<record id="survey_rule" model="ir.rule">
+			<field name="name">Survey</field>
+			<field name="model_id" ref="model_survey" />
+			<field name="global" eval="True" />
+			<field name="domain_force">
+				[('invited_user_ids', '=', user.id)]
+			</field>
+		</record>
+	</data>
 </openerp>

=== modified file 'survey/wizard/survey_send_invitation.py'
--- survey/wizard/survey_send_invitation.py	2012-12-06 14:56:32 +0000
+++ survey/wizard/survey_send_invitation.py	2013-03-21 10:57:43 +0000
@@ -108,8 +108,10 @@
         mail_message = self.pool.get('mail.message')
 
         model_data_obj = self.pool.get('ir.model.data')
-        group_id = model_data_obj._get_id(cr, uid, 'base', 'group_survey_user')
-        group_id = model_data_obj.browse(cr, uid, group_id, context=context).res_id
+        group_id1= model_data_obj._get_id(cr, uid, 'base', 'group_survey_user')
+        group_id1 = model_data_obj.browse(cr, uid, group_id1, context=context).res_id
+        group_id2= model_data_obj._get_id(cr, uid, 'base', 'group_tool_manager')
+        group_id2 = model_data_obj.browse(cr, uid, group_id2, context=context).res_id
 
         act_id = self.pool.get('ir.actions.act_window')
         act_id = act_id.search(cr, uid, [('res_model', '=' , 'survey.name.wiz'), \
@@ -186,7 +188,7 @@
                                 'login': partner.email,
                                 'password': passwd,
                                 'address_id': partner.id,
-                                'groups_id': [[6, 0, [group_id]]],
+                                'groups_id': [[6, 0, [group_id1,group_id2]]],
                                 'action_id': act_id[0],
                                 'survey_id': [[6, 0, survey_ids]]
                                }