From 17b7a2045f401d7ee8fb99361864dc992fff6bcc Mon Sep 17 00:00:00 2001
From: Keyur Gajjar <kga@odoo.com>
Date: Thu, 13 Aug 2015 12:26:04 +0530
Subject: [PATCH] [MOV] fetchmail: splitted models and views through spearated
 files

- splitted the models/mail_mail.py from models/fetchmail.py.
- splitted the views/mail_mail_views.xml from views/fetchmail_views.xml
---
 addons/fetchmail/__openerp__.py            |  1 +
 addons/fetchmail/models/__init__.py        |  1 +
 addons/fetchmail/models/fetchmail.py       | 27 --------------------
 addons/fetchmail/models/mail_mail.py       | 29 ++++++++++++++++++++++
 addons/fetchmail/views/fetchmail_views.xml | 16 ------------
 addons/fetchmail/views/mail_mail_views.xml | 23 +++++++++++++++++
 6 files changed, 54 insertions(+), 43 deletions(-)
 create mode 100644 addons/fetchmail/models/mail_mail.py
 create mode 100644 addons/fetchmail/views/mail_mail_views.xml

diff --git a/addons/fetchmail/__openerp__.py b/addons/fetchmail/__openerp__.py
index 68c880ea415..4213734ccc4 100644
--- a/addons/fetchmail/__openerp__.py
+++ b/addons/fetchmail/__openerp__.py
@@ -39,6 +39,7 @@ For more specific needs, you may also assign custom-defined actions
         'data/fetchmail_data.xml',
         'security/ir.model.access.csv',
         'views/fetchmail_views.xml',
+        'views/mail_mail_views.xml',
     ],
     'demo': [],
     'installable': True,
diff --git a/addons/fetchmail/models/__init__.py b/addons/fetchmail/models/__init__.py
index d25bd214b77..b09bd3023dd 100644
--- a/addons/fetchmail/models/__init__.py
+++ b/addons/fetchmail/models/__init__.py
@@ -1,3 +1,4 @@
 # -*- coding: utf-8 -*-
 import fetchmail
 import res_config
+import mail_mail
diff --git a/addons/fetchmail/models/fetchmail.py b/addons/fetchmail/models/fetchmail.py
index 203c3ee1fbc..b21ee4a9a7e 100644
--- a/addons/fetchmail/models/fetchmail.py
+++ b/addons/fetchmail/models/fetchmail.py
@@ -270,30 +270,3 @@ openerp_mailgate: "|/path/to/openerp-mailgate.py --host=localhost -u %(uid)d -p
         res = super(fetchmail_server, self).unlink(cr, uid, ids, context=context)
         self._update_cron(cr, uid, context=context)
         return res
-
-class mail_mail(osv.osv):
-    _inherit = "mail.mail"
-    _columns = {
-        'fetchmail_server_id': fields.many2one('fetchmail.server', "Inbound Mail Server",
-                                               readonly=True,
-                                               select=True,
-                                               oldname='server_id'),
-    }
-
-    def create(self, cr, uid, values, context=None):
-        if context is None:
-            context = {}
-        fetchmail_server_id = context.get('fetchmail_server_id')
-        if fetchmail_server_id:
-            values['fetchmail_server_id'] = fetchmail_server_id
-        res = super(mail_mail, self).create(cr, uid, values, context=context)
-        return res
-
-    def write(self, cr, uid, ids, values, context=None):
-        if context is None:
-            context = {}
-        fetchmail_server_id = context.get('fetchmail_server_id')
-        if fetchmail_server_id:
-            values['fetchmail_server_id'] = fetchmail_server_id
-        res = super(mail_mail, self).write(cr, uid, ids, values, context=context)
-        return res
diff --git a/addons/fetchmail/models/mail_mail.py b/addons/fetchmail/models/mail_mail.py
new file mode 100644
index 00000000000..57c5fca4561
--- /dev/null
+++ b/addons/fetchmail/models/mail_mail.py
@@ -0,0 +1,29 @@
+# -*- coding: utf-8 -*-
+from openerp.osv import fields, osv
+
+class mail_mail(osv.osv):
+    _inherit = "mail.mail"
+    _columns = {
+        'fetchmail_server_id': fields.many2one('fetchmail.server', "Inbound Mail Server",
+                                               readonly=True,
+                                               select=True,
+                                               oldname='server_id'),
+    }
+
+    def create(self, cr, uid, values, context=None):
+        if context is None:
+            context = {}
+        fetchmail_server_id = context.get('fetchmail_server_id')
+        if fetchmail_server_id:
+            values['fetchmail_server_id'] = fetchmail_server_id
+        res = super(mail_mail, self).create(cr, uid, values, context=context)
+        return res
+
+    def write(self, cr, uid, ids, values, context=None):
+        if context is None:
+            context = {}
+        fetchmail_server_id = context.get('fetchmail_server_id')
+        if fetchmail_server_id:
+            values['fetchmail_server_id'] = fetchmail_server_id
+        res = super(mail_mail, self).write(cr, uid, ids, values, context=context)
+        return res
diff --git a/addons/fetchmail/views/fetchmail_views.xml b/addons/fetchmail/views/fetchmail_views.xml
index 9c746573414..63708c3fe1a 100644
--- a/addons/fetchmail/views/fetchmail_views.xml
+++ b/addons/fetchmail/views/fetchmail_views.xml
@@ -119,21 +119,5 @@
             groups="base.group_no_one"
         />
 
-      <record model="ir.ui.view" id="email_message_tree_view">
-          <field name="name">mail.mail.form.fetchmail</field>
-          <field name="model">mail.mail</field>
-          <field name="inherit_id" ref="mail.view_mail_form"/>
-          <field name="arch" type="xml">
-              <field name="references" position="after">
-                  <field name="fetchmail_server_id"/>
-              </field>
-          </field>
-      </record>
-
-      <act_window
-            context="{'search_default_server_id': active_id, 'default_fetchmail_server_id': active_id}"
-            id="act_server_history" name="Messages" domain="[('email_from', '!=', False), ('fetchmail_server_id', '=', active_id)]"
-            res_model="mail.mail" src_model="fetchmail.server"/>
-
     </data>
 </openerp>
diff --git a/addons/fetchmail/views/mail_mail_views.xml b/addons/fetchmail/views/mail_mail_views.xml
new file mode 100644
index 00000000000..8a808b368b0
--- /dev/null
+++ b/addons/fetchmail/views/mail_mail_views.xml
@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+
+    <record  id="email_message_tree_view" model="ir.ui.view">
+        <field name="name">mail.mail.form.fetchmail</field>
+        <field name="model">mail.mail</field>
+        <field name="inherit_id" ref="mail.view_mail_form"/>
+        <field name="arch" type="xml">
+            <field name="references" position="after">
+                <field name="fetchmail_server_id"/>
+            </field>
+        </field>
+    </record>
+
+    <act_window
+        id="act_server_history"
+        name="Messages"
+        res_model="mail.mail"
+        src_model="fetchmail.server"
+        domain="[('email_from', '!=', False), ('fetchmail_server_id', '=', active_id)]"
+        context="{'search_default_server_id': active_id, 'default_fetchmail_server_id': active_id}"/>
+
+</odoo>