From 6021b0fe8fd8fd4173c25245950f0eb09d144a32 Mon Sep 17 00:00:00 2001
From: Mitesh Savani <msa@openerp.com>
Date: Fri, 1 Aug 2014 17:09:09 +0530
Subject: [PATCH] [FIX] base_gengo, website_gengo: groups on the gengo key.

Basically the admin group can read the gengo key. When adding website_gengo that adds the
website groups, the groups allowed to access the key are updated.

This is done because in base_gengo the website groups are not necesssarily defined. Website
is not a dependency of base_gengo.
---
 addons/base_gengo/wizard/base_gengo_translations.py   |  3 ++-
 addons/website_gengo/__init__.py                      |  3 ++-
 addons/website_gengo/models/__init__.py               |  1 +
 .../website_gengo/models/base_gengo_translations.py   | 11 +++++++++++
 4 files changed, 16 insertions(+), 2 deletions(-)
 create mode 100644 addons/website_gengo/models/__init__.py
 create mode 100644 addons/website_gengo/models/base_gengo_translations.py

diff --git a/addons/base_gengo/wizard/base_gengo_translations.py b/addons/base_gengo/wizard/base_gengo_translations.py
index 680be04d408..1521ac3dce5 100644
--- a/addons/base_gengo/wizard/base_gengo_translations.py
+++ b/addons/base_gengo/wizard/base_gengo_translations.py
@@ -40,6 +40,7 @@ GENGO_DEFAULT_LIMIT = 20
 
 class base_gengo_translations(osv.osv_memory):
     GENGO_KEY = "Gengo.UUID"
+    GROUPS = ['base.group_system']
 
     _name = 'base.gengo.translations'
     _columns = {
@@ -57,7 +58,7 @@ class base_gengo_translations(osv.osv_memory):
     def init(self, cr):
         icp = self.pool['ir.config_parameter']
         if not icp.get_param(cr, SUPERUSER_ID, self.GENGO_KEY, default=None):
-            icp.set_param(cr, SUPERUSER_ID, self.GENGO_KEY, str(uuid.uuid4()), groups=['base.group_website_designer', 'base.group_website_publisher'])
+            icp.set_param(cr, SUPERUSER_ID, self.GENGO_KEY, str(uuid.uuid4()), groups=self.GROUPS)
 
     def get_gengo_key(self, cr):
         icp = self.pool['ir.config_parameter']
diff --git a/addons/website_gengo/__init__.py b/addons/website_gengo/__init__.py
index 6da11461444..afa6323de2c 100644
--- a/addons/website_gengo/__init__.py
+++ b/addons/website_gengo/__init__.py
@@ -19,4 +19,5 @@
 #
 ##############################################################################
 
-import controllers
\ No newline at end of file
+import controllers
+import models
diff --git a/addons/website_gengo/models/__init__.py b/addons/website_gengo/models/__init__.py
new file mode 100644
index 00000000000..435cb9b9ee9
--- /dev/null
+++ b/addons/website_gengo/models/__init__.py
@@ -0,0 +1 @@
+import base_gengo_translations
diff --git a/addons/website_gengo/models/base_gengo_translations.py b/addons/website_gengo/models/base_gengo_translations.py
new file mode 100644
index 00000000000..da8e0422c7f
--- /dev/null
+++ b/addons/website_gengo/models/base_gengo_translations.py
@@ -0,0 +1,11 @@
+# -*- coding: utf-8 -*-
+
+from openerp.osv import osv
+
+
+class base_gengo_translations(osv.TransientModel):
+    _inherit = 'base.gengo.translations'
+    # update GROUPS, that are the groups allowing to access the gengo key.
+    # this is done here because in the base_gengo module, the various website
+    # groups do not exist, limiting the access to the admin group.
+    GROUPS = ['base.group_website_designer', 'base.group_website_publisher']