Merge pull request #205 from hbrunn/8.0-fetchmail_attach_from_folder

[ADD] honor sequence, add active flag

fetchmail_attach_from_folder/model/fetchmail_server.py

@@ -33,7 +33,8 @@ class fetchmail_server(models.Model):
     _inherit = 'fetchmail.server'
 
     folder_ids = fields.One2many(
-        'fetchmail.server.folder', 'server_id', 'Folders')
+        'fetchmail.server.folder', 'server_id', 'Folders',
+        context={'active_test': False})
     object_id = fields.Many2one(required=False)
 
     _defaults = {
@@ -67,7 +68,7 @@ class fetchmail_server(models.Model):
                 })
 
             connection = this.connect()
-            for folder in this.folder_ids:
+            for folder in this.folder_ids.filtered('active'):
                 this.handle_folder(connection, folder)
             connection.close()
 
@@ -216,7 +217,7 @@ class fetchmail_server(models.Model):
             this.write({'state': 'draft'})
             connection = this.connect()
             connection.select()
-            for folder in this.folder_ids:
+            for folder in this.folder_ids.filtered('active'):
                 if connection.select(folder.path)[0] != 'OK':
                     raise exceptions.ValidationError(
                         _('Mailbox %s not found!') % folder.path)

fetchmail_attach_from_folder/model/fetchmail_server_folder.py

@@ -26,6 +26,7 @@ from .. import match_algorithm
 class fetchmail_server_folder(models.Model):
     _name = 'fetchmail.server.folder'
     _rec_name = 'path'
+    _order = 'sequence'
 
     def _get_match_algorithms(self):
         def get_all_subclasses(cls):
@@ -93,10 +94,12 @@ class fetchmail_server_folder(models.Model):
         'Message state',
         help='The state messages fetched from this folder should be '
         'assigned in Odoo')
+    active = fields.Boolean('Active')
 
     _defaults = {
         'flag_nonmatching': True,
         'msg_state': 'received',
+        'active': True,
     }
 
     @api.multi

fetchmail_attach_from_folder/view/fetchmail_server.xml

@@ -16,8 +16,9 @@
                                 name="folder_ids"
                                 nolabel="1"
                                 on_change="onchange_server_type(type, is_ssl, object_id)">
-                                <tree>
-                                    <field name="sequence" invisible="1" />
+                                <tree colors="gray:active==False">
+                                    <field name="active" invisible="True" />
+                                    <field name="sequence" widget="handle" />
                                     <field name="path" />
                                     <field name="model_id" />
                                     <field name="model_field" />
@@ -37,6 +38,7 @@
                                             <field name="mail_field" placeholder="to,from" />
                                         </group>
                                         <group>
+                                            <field name="active" />
                                             <field name="delete_matching" />
                                             <field name="flag_nonmatching" />
                                             <field name="match_first" />