[IMP] ir.filters: unassigned filters are now global

bzr revid: odo@openerp.com-20120511094654-0qn0yr9yz4hbbzab

diff --git a/openerp/addons/base/__openerp__.py b/openerp/addons/base/__openerp__.py
index 393e765..e523659 100644 (file)
--- a/openerp/addons/base/__openerp__.py
+++ b/openerp/addons/base/__openerp__.py
@@ -42,6 +42,7 @@
         'base_update.xml',
         'ir/wizard/wizard_menu_view.xml',
         'ir/ir.xml',
+        'ir/ir_filters.xml',
         'ir/ir_config_parameter_view.xml',
         'ir/workflow/workflow_view.xml',
         'ir/report/ir_report.xml',

diff --git a/openerp/addons/base/ir/ir.xml b/openerp/addons/base/ir/ir.xml
index 518ea1e..2e938c8 100644 (file)
--- a/openerp/addons/base/ir/ir.xml
+++ b/openerp/addons/base/ir/ir.xml
@@ -777,7 +777,7 @@
             <field name="view_id" ref="view_view_tree"/>
             <field name="help">Views allows you to personalize each view of OpenERP. You can add new fields, move fields, rename them or delete the ones that you do not need.</field>
         </record>
-        <menuitem action="action_ui_view" id="menu_action_ui_view" parent="base.next_id_2"/>
+        <menuitem action="action_ui_view" id="menu_action_ui_view" parent="base.next_id_2" sequence="2"/>
 
 
         <!-- View customizations -->
@@ -822,7 +822,7 @@
             <field name="res_model">ir.ui.view.custom</field>
             <field name="help">Customized views are used when users reorganize the content of their dashboard views (via web client)</field>
         </record>
-        <menuitem id="menu_action_ui_view_custom" action="action_ui_view_custom" parent="base.next_id_2"/>
+        <menuitem id="menu_action_ui_view_custom" action="action_ui_view_custom" parent="base.next_id_2" sequence="3"/>
 
 
         <!-- Attachment -->

diff --git a/openerp/addons/base/ir/ir_filters.py b/openerp/addons/base/ir/ir_filters.py
index 19b2676..6ee6766 100644 (file)
--- a/openerp/addons/base/ir/ir_filters.py
+++ b/openerp/addons/base/ir/ir_filters.py
@@ -37,35 +37,53 @@ class ir_filters(osv.osv):
         name = self.read(cr, uid, [id], ['name'])[0]['name']
         default.update({'name':_('%s (copy)') % name})
         return super(ir_filters, self).copy(cr, uid, id, default, context)
-   
+
     def get_filters(self, cr, uid, model):
-        act_ids = self.search(cr,uid,[('model_id','=',model),('user_id','=',uid)])
-        my_acts = self.read(cr, uid, act_ids, ['name', 'domain','context'])
+        """Obtain the list of filters available for the user on the given model.
+
+        :return: list of :meth:`~osv.read`-like dicts containing the ``name``,
+            ``domain``, ``user_id`` (m2o tuple) and ``context`` of the matching ``ir.filters``.
+        """
+        # available filters: private filters (user_id=uid) and public filters (uid=NULL) 
+        act_ids = self.search(cr, uid, [('model_id','=',model),('user_id','in',[uid, False])])
+        my_acts = self.read(cr, uid, act_ids, ['name', 'domain', 'context', 'user_id'])
         return my_acts
 
     def create_or_replace(self, cr, uid, vals, context=None):
-        filter_id = None
         lower_name = vals['name'].lower()
-        matching_filters = [x for x in self.get_filters(cr, uid, vals['model_id'])
-                                if x['name'].lower() == lower_name]
+        matching_filters = [f for f in self.get_filters(cr, uid, vals['model_id'])
+                                if f['name'].lower() == lower_name
+                                if (f['user_id'] and f['user_id'][0]) == vals.get('user_id', False)]
+        # When a filter exists for the same (name, model, user) triple, we simply
+        # replace its definition.
         if matching_filters:
             self.write(cr, uid, matching_filters[0]['id'], vals, context)
-            return False
+            return matching_filters[0]['id']
         return self.create(cr, uid, vals, context)
 
+    _sql_constraints = [
+        # Partial constraint, complemented by unique index (see below)
+        # Still useful to keep because it provides a proper error message when a violation
+        # occurs, as it shares the same prefix as the unique index. 
+        ('name_model_uid_unique', 'unique (name, model_id, user_id)', 'Filter names must be unique'),
+    ]
+
     def _auto_init(self, cr, context=None):
         super(ir_filters, self)._auto_init(cr, context)
         # Use unique index to implement unique constraint on the lowercase name (not possible using a constraint)
         cr.execute("SELECT indexname FROM pg_indexes WHERE indexname = 'ir_filters_name_model_uid_unique_index'")
         if not cr.fetchone():
-            cr.execute('CREATE UNIQUE INDEX "ir_filters_name_model_uid_unique_index" ON ir_filters (lower(name), model_id, user_id)')
+            cr.execute("""CREATE UNIQUE INDEX "ir_filters_name_model_uid_unique_index" ON ir_filters
+                            (lower(name), model_id, COALESCE(user_id,-1))""")
 
     _columns = {
         'name': fields.char('Filter Name', size=64, translate=True, required=True),
-        'user_id':fields.many2one('res.users', 'User', help="The user this filter is available to. When left empty the filter is usable by the system only."),
-        'domain': fields.text('Domain Value', required=True),
-        'context': fields.text('Context Value', required=True),
-        'model_id': fields.selection(_list_all_models, 'Object', size=64, required=True),
+        'user_id': fields.many2one('res.users', 'User', ondelete='cascade',
+                                   help="The user this filter is private to. When left empty the filter is public "
+                                        "and available to all users."),
+        'domain': fields.text('Domain', required=True),
+        'context': fields.text('Context', required=True),
+        'model_id': fields.selection(_list_all_models, 'Model', required=True),
     }
     _defaults = {
         'domain': '[]',

diff --git a/openerp/addons/base/ir/ir_filters.xml b/openerp/addons/base/ir/ir_filters.xml
index 93f725e..a0e37b4 100644 (file)
--- a/openerp/addons/base/ir/ir_filters.xml
+++ b/openerp/addons/base/ir/ir_filters.xml
@@ -1,5 +1,15 @@
 <?xml version="1.0" encoding="utf-8"?>
 <openerp>
+
+    <data noupdate="1">
+        <!-- Restrict modifications on ir.filters to owner only -->
+        <record id="ir_filters_rule" model="ir.rule">
+            <field name="name">ir.filters.owner</field>
+            <field model="ir.model" name="model_id" ref="model_ir_filters"/>
+            <field name="domain_force">[('user_id','in',[False,user.id])]</field>
+        </record>
+    </data>
+
     <data>
         <record id="ir_filters_view_form" model="ir.ui.view">
             <field name="name">ir.filters.form</field>
@@ -43,13 +53,15 @@
             <field name="type">search</field>
             <field name="arch" type="xml">
                 <search string="Filters">
-                    <filter name="Private" domain="[('user_id','!=',False))]"/>
-                    <filter name="Public" domain="[('user_id','=',False))]"/>
+                    <filter string="Personal" domain="[('user_id','!=',False)]" help="Filters visible only for one user"/>
+                    <filter string="Shared" domain="[('user_id','=',False)]" help="Filters shared with all users"/>
                     <separator orientation="vertical"/>
                     <field name="name"/>
                     <field name="model_id"/>
                     <field name="user_id">
-                        <filter icon="terp-personal" domain="[('user_id','in',(uid, False))]" name="My Filters" />
+                        <filter icon="terp-personal" domain="[('user_id','in',(uid, False))]"
+                                name="my_filters"
+                                string="My Filters" />
                     </field>
                 </search>
             </field>
@@ -61,7 +73,7 @@
             <field name="res_model">ir.filters</field>
         </record>
 
-        <menuitem parent="base.next_id_6" name="User-defined Filters"
-            id="menu_ir_filters" action="actions_ir_filters_view"/>
+        <menuitem parent="base.next_id_2" name="User-defined Filters"
+            id="menu_ir_filters" action="actions_ir_filters_view" sequence="5"/>
     </data>
 </openerp>
\ No newline at end of file