From 4ef3f9218bb97b66b301f84ed49661d3be79a909 Mon Sep 17 00:00:00 2001
From: "Purnendu Singh (OpenERP)" <psi@tinyerp.com>
Date: Wed, 3 Oct 2012 15:12:42 +0530
Subject: [PATCH] [IMP] document: add on_change on fname so it will update the
 download link with the name of attached document

bzr revid: psi@tinyerp.com-20121003094242-5wu6pki5cx442jte
---
 addons/document/document.py       | 10 ++++++++--
 addons/document/document_view.xml | 13 +++----------
 2 files changed, 11 insertions(+), 12 deletions(-)

diff --git a/addons/document/document.py b/addons/document/document.py
index 3ae783f49e7..cbc20edc044 100644
--- a/addons/document/document.py
+++ b/addons/document/document.py
@@ -36,7 +36,6 @@ DMS_ROOT_PATH = tools.config.get('document_path', os.path.join(tools.config['roo
 
 class document_file(osv.osv):
     _inherit = 'ir.attachment'
-    _rec_name = 'datas_fname'
 
 
     def _attach_parent_id(self, cr, uid, ids=None, context=None):
@@ -149,7 +148,14 @@ class document_file(osv.osv):
     _sql_constraints = [
         # filename_uniq is not possible in pure SQL
     ]
-    def _check_duplication(self, cr, uid, vals, ids=None, op='create'):
+
+    def onchange_file(self, cr, uid, ids, datas_fname=False, context=None):
+        res = {'value':{}}
+        if datas_fname:
+            res['value'].update({'name': datas_fname})
+        return res
+
+    def _check_duplication(self, cr, uid, vals, ids=[], op='create'):
         name = vals.get('name', False)
         parent_id = vals.get('parent_id', False)
         res_model = vals.get('res_model', False)
diff --git a/addons/document/document_view.xml b/addons/document/document_view.xml
index 8d1ff0850b3..54776c4d36e 100644
--- a/addons/document/document_view.xml
+++ b/addons/document/document_view.xml
@@ -228,9 +228,11 @@
         <field name="priority" eval="1"/>
         <field name="inherit_id" ref="base.view_attachment_form"/>
         <field name="arch" type="xml">
+            <xpath expr="//field[@name='datas_fname']" position="replace">
+                <field name="datas_fname" invisible="1" on_change="onchange_file(datas_fname)"/>
+            </xpath>
             <field name="url" position="after">
                 <field name="user_id"/>
-                <field name="datas_fname" select="1"/>
             </field>
             <field name="company_id" position="before">
                 <field name="parent_id"/>
@@ -339,15 +341,6 @@
         <field name="value" eval="'ir.actions.act_window,%d'%action_document_file_directory_form"/>
     </record>
 
-    <record model="ir.ui.view" id="view_attachment_form_inherit">
-        <field name="name">ir.attachment.view.inherit</field>
-        <field name="model">ir.attachment</field>
-        <field name="inherit_id" ref="base.view_attachment_form"/>
-        <field name="arch" type="xml">
-            <field name="datas_fname" position = "replace" >
-            </field>
-        </field>
-    </record>
 
     <record model="ir.ui.view" id="view_process_node_form_inherit1">
         <field name="name">process.node.form</field>