Merge lp:~daniel-watkins-credativ/poweremail/fix-server-actions into lp:~openerp-commiter/poweremail/poweremail-v6

[Daniel Watkins (credativ)]

Currently Poweremail Server Actions cannot be saved, as there is a required field that has not been overriden. This overrides that field.

[lp:~daniel-watkins-credativ/poweremail/fix-server-actions]

206. By Daniel Watkins (credativ) on 2010-09-07

Fix my fix of new required attribute.

[Niels Huylebroeck (red15)]

Just goes to show that openerp can sometimes really think backwards, why didn't they just say

attrs="{'required': [('state','=','copy_object')]}"

But I tested it and this fix works too :)

review: Approve

=== modified file 'poweremail_serveraction_view.xml'
--- poweremail_serveraction_view.xml	2009-08-04 10:33:25 +0000
+++ poweremail_serveraction_view.xml	2010-09-07 10:29:43 +0000
@@ -42,6 +42,20 @@
 				</xpath>
 			</field>
 		</record>
+		<record model="ir.ui.view"
+			id="view_server_action_form_poweremail_3">
+			<field name="name">ir.actions.server.form.poweremail_1</field>
+			<field name="model">ir.actions.server</field>
+			<field name="type">form</field>
+			<field name="inherit_id" ref="base.view_server_action_form" />
+			<field name="arch" type="xml">
+				<xpath
+					expr="/form/notebook/page/field[@name='copy_object']"
+					position="replace">
+                    <field name="copy_object" on_change="change_object(copy_object, state)" select="2" attrs="{'required':[('state','!=','dummy'), ('state','!=','sms'), ('state','!=','code'), ('state','!=','loop'), ('state','!=','trigger'), ('state','!=','object_write'), ('state','!=','object_create'), ('state','!=','client_action'), ('state','!=','email'), ('state','!=','sms'), ('state','!=','other'), ('state','!=','poweremail')]}"/>
+				</xpath>
+			</field>
+		</record>
 	</data>
 </openerp>