[IMP] account_banking*: Show identifiers at payment mode level

Now the visibility is controlled through a security group.

account_banking_pain_base/README.rst

@@ -25,7 +25,18 @@ This module is part of the OCA/bank-payment suite.
 Configuration
 =============
 
-No configuration required.
+#. Go to Accounting > Configuration > Settings.
+#. On the fields "Initiating Party Issuer" and "Initiating Party Identifier",
+   in the section *SEPA/PAIN*, you can fill the corresponding identifiers.
+
+If your country requires several identifiers (like Spain), you must:
+
+#. Go to *Accounting > Configuration > Settings*.
+#. On the section *SEPA/PAIN*, check the mark "Multiple identifiers".
+#. Now go to *Accounting > Configuration > Management > Payment Modes*.
+#. Create a payment mode for your specific bank.
+#. Fill the specific identifiers on the fields "Initiating Party Identifier"
+   and "Initiating Party Issuer".
 
 Usage
 =====
@@ -34,7 +45,7 @@ See 'readme' files of the OCA/bank-payment suite.
 
 .. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
    :alt: Try me on Runbot
-   :target: https://runbot.odoo-community.org/runbot/173/9.0
+   :target: https://runbot.odoo-community.org/runbot/173/10.0
 
 Known issues / Roadmap
 ======================
@@ -70,10 +81,12 @@ Maintainer
 
 .. image:: http://odoo-community.org/logo.png
    :alt: Odoo Community Association
-   :target: http://odoo-community.org
+   :target: https://odoo-community.org
 
 This module is maintained by the OCA.
 
-OCA, or the Odoo Community Association, is a nonprofit organization whose mission is to support the collaborative development of Odoo features and promote its widespread use.
+OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.
 
-To contribute to this module, please visit http://odoo-community.org.
+To contribute to this module, please visit https://odoo-community.org.

account_banking_pain_base/__manifest__.py

@@ -1,26 +1,26 @@
 # -*- coding: utf-8 -*-
-# © 2013-2016 Akretion - Alexis de Lattre <alexis.delattre@akretion.com>
-# © 2014 Tecnativa - Pedro M. Baeza
-# © 2016 Tecnativa - Antonio Espinosa
+# Copyright 2013-2016 Akretion - Alexis de Lattre
+# Copyright 2014-2017 Tecnativa - Pedro M. Baeza
+# Copyright 2016 Tecnativa - Antonio Espinosa
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
 {
     'name': 'Account Banking PAIN Base Module',
     'summary': 'Base module for PAIN file generation',
-    'version': '10.0.1.0.0',
+    'version': '10.0.1.1.0',
     'license': 'AGPL-3',
     'author': "Akretion, "
               "Noviat, "
               "Tecnativa, "
               "Odoo Community Association (OCA)",
     'website': 'https://github.com/OCA/bank-payment',
-    'contributors': ['Pedro M. Baeza <pedro.baeza@serviciosbaeza.com>'],
     'category': 'Hidden',
     'depends': ['account_payment_order'],
     'external_dependencies': {
         'python': ['unidecode', 'lxml'],
     },
     'data': [
+        'security/security.xml',
         'views/account_payment_line.xml',
         'views/account_payment_order.xml',
         'views/bank_payment_line_view.xml',

account_banking_pain_base/models/account_config_settings.py

@@ -1,5 +1,6 @@
 # -*- coding: utf-8 -*-
-# © 2016 Akretion - Alexis de Lattre <alexis.delattre@akretion.com>
+# Copyright 2016 Akretion - Alexis de Lattre <alexis.delattre@akretion.com>
+# Copyright 2017 Tecnativa - Pedro M. Baeza
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
 from odoo import models, fields
@@ -12,3 +13,10 @@ class AccountConfigSettings(models.TransientModel):
         related='company_id.initiating_party_issuer')
     initiating_party_identifier = fields.Char(
         related='company_id.initiating_party_identifier')
+    group_pain_multiple_identifier = fields.Boolean(
+        string='Multiple identifiers',
+        implied_group='account_banking_pain_base.'
+                      'group_pain_multiple_identifier',
+        help="Enable this option if your country requires several SEPA/PAIN "
+             "identifiers like in Spain.",
+    )

account_banking_pain_base/security/security.xml

@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+
+    <record id="group_pain_multiple_identifier" model="res.groups">
+        <field name="name">SEPA/PAIN Identifiers on Payment Modes</field>
+        <field name="category_id" ref="base.module_category_hidden"/>
+    </record>
+
+</odoo>
+

account_banking_pain_base/views/account_config_settings.xml

@@ -12,9 +12,10 @@
     <field name="inherit_id" ref="account.view_account_config_settings"/>
     <field name="arch" type="xml">
         <xpath expr="//separator[@name='analytic_account']" position="before">
-            <group name="pain">
+            <group name="pain" string="SEPA/PAIN">
                 <field name="initiating_party_identifier"/>
                 <field name="initiating_party_issuer"/>
+                <field name="group_pain_multiple_identifier"/>
             </group>
         </xpath>
     </field>

account_banking_pain_base/views/account_payment_mode.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
   Copyright (C) 2013-2016 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
-  © 2015 Antiun Ingenieria S.L. - Antonio Espinosa
+  Copyright 2015-2017 Tecnativa
   License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 -->
 <odoo>
@@ -13,9 +13,8 @@
     <field name="inherit_id" ref="account_payment_order.account_payment_mode_form"/>
     <field name="arch" type="xml">
         <group name="main" position="inside">
-            <!-- To be set visible in the localisation modules that need it -->
-            <field name="initiating_party_identifier" invisible="1"/>
-            <field name="initiating_party_issuer" invisible="1"/>
+            <field name="initiating_party_identifier" groups="account_banking_pain_base.group_pain_multiple_identifier"/>
+            <field name="initiating_party_issuer" groups="account_banking_pain_base.group_pain_multiple_identifier"/>
         </group>
     </field>
 </record>