[FIX] API compatibility of monkeypatched method

[IMP] Make _register_hook idempotent as we have to call it in tests

base_technical_features/models/basemodel_monkeypatch.py

@@ -1,26 +1,31 @@
 # -*- coding: utf-8 -*-
-from openerp import models
+from openerp import models, api
 
 
 class BaseModelMonkeyPatch(models.AbstractModel):
     _name = 'basemodel.monkeypatch'
 
     def _register_hook(self, cr):
-        models.BaseModel.base_technical_features_user_has_groups = (
-            models.BaseModel.user_has_groups)
+        if not hasattr(
+                models.BaseModel, 'base_technical_features_user_has_groups'):
 
-        def user_has_groups(self, cr, uid, groups, context=None):
-            """ Return True for users in the technical features group when
-            membership of the original group is checked, even if debug mode is
-            not enabled.
-            """
-            if ('base.group_no_one' in groups.split(',') and
-                self.pool['res.users'].has_group(
-                    cr, uid,
-                    'base_technical_features.group_technical_features')):
-                return True
-            return self.base_technical_features_user_has_groups(
-                cr, uid, groups, context=context)
+            models.BaseModel.base_technical_features_user_has_groups = (
+                models.BaseModel.user_has_groups)
+
+            @api.cr_uid_context
+            def user_has_groups(self, cr, uid, groups, context=None):
+                """ Return True for users in the technical features group when
+                membership of the original group is checked, even if debug mode
+                is not enabled.
+                """
+                if ('base.group_no_one' in groups.split(',') and
+                    self.pool['res.users'].has_group(
+                        cr, uid,
+                        'base_technical_features.group_technical_features')):
+                    return True
+                return self.base_technical_features_user_has_groups(
+                    cr, uid, groups, context=context)
+
+            models.BaseModel.user_has_groups = user_has_groups
 
-        models.BaseModel.user_has_groups = user_has_groups
         return super(BaseModelMonkeyPatch, self)._register_hook(cr)