diff --git a/portal_partner_select_all/__manifest__.py b/portal_partner_select_all/__manifest__.py index 2c4202475..911cec526 100644 --- a/portal_partner_select_all/__manifest__.py +++ b/portal_partner_select_all/__manifest__.py @@ -1,18 +1,13 @@ # Copyright 2018 Tecnativa - David Vidal # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html). { - 'name': 'Portal Partner Select All', - 'version': '12.0.1.0.2', - 'category': 'Custom', - 'author': 'Tecnativa,' - 'Odoo Community Association (OCA)', - 'website': 'https://www.github.com/OCA/partner-contact', + "name": "Portal Partner Select All", + "version": "12.0.1.0.2", + "category": "Custom", + "author": "Tecnativa," "Odoo Community Association (OCA)", + "website": "https://www.github.com/OCA/partner-contact", "license": "AGPL-3", - 'depends': [ - 'portal', - ], - 'data': [ - 'wizard/portal_wizard.xml', - ], - 'installable': True, + "depends": ["portal",], + "data": ["wizard/portal_wizard.xml",], + "installable": True, } diff --git a/portal_partner_select_all/tests/test_portal_partner_select_all.py b/portal_partner_select_all/tests/test_portal_partner_select_all.py index 5f7e7eb9d..51e65d18d 100644 --- a/portal_partner_select_all/tests/test_portal_partner_select_all.py +++ b/portal_partner_select_all/tests/test_portal_partner_select_all.py @@ -1,22 +1,27 @@ # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html). -from odoo.tests import common, Form +from odoo.tests import Form, common class TestPortalPartnerSelctAll(common.TransactionCase): - def setUp(self): super().setUp() Partner = self.env["res.partner"] - self.partner1 = Partner.create({'name': 'P1', 'email': 'p1@p1'}) - self.partner2 = Partner.create({'name': 'P2', 'email': 'p2@p2'}) - self.partner3 = Partner.create({'name': 'P3', 'email': 'p3@p3'}) - self.wizard_all = self.env["portal.wizard"].with_context({ - 'active_ids': [self.partner1.id, self.partner2.id] - }).create({}) - self.wizard_default = self.env["portal.wizard"].with_context({ - 'active_ids': [self.partner1.id, self.partner2.id, self.partner3.id] - }).create({}) + self.partner1 = Partner.create({"name": "P1", "email": "p1@p1"}) + self.partner2 = Partner.create({"name": "P2", "email": "p2@p2"}) + self.partner3 = Partner.create({"name": "P3", "email": "p3@p3"}) + self.wizard_all = ( + self.env["portal.wizard"] + .with_context({"active_ids": [self.partner1.id, self.partner2.id]}) + .create({}) + ) + self.wizard_default = ( + self.env["portal.wizard"] + .with_context( + {"active_ids": [self.partner1.id, self.partner2.id, self.partner3.id]} + ) + .create({}) + ) def test_portal_partner_select_all_wizard(self): # check selecting all @@ -27,8 +32,8 @@ class TestPortalPartnerSelctAll(common.TransactionCase): # partner should have user records with assigned portal group self.assertTrue(self.partner1.user_ids, "Partner should have user") self.assertTrue(self.partner2.user_ids, "Partner should have user") - self.assertTrue(self.partner1.user_ids[0].has_group('base.group_portal')) - self.assertTrue(self.partner2.user_ids[0].has_group('base.group_portal')) + self.assertTrue(self.partner1.user_ids[0].has_group("base.group_portal")) + self.assertTrue(self.partner2.user_ids[0].has_group("base.group_portal")) # checking toogle wizard_default_form = Form(self.wizard_default) diff --git a/portal_partner_select_all/wizard/portal_wizard.py b/portal_partner_select_all/wizard/portal_wizard.py index aa208db45..7065cb41f 100644 --- a/portal_partner_select_all/wizard/portal_wizard.py +++ b/portal_partner_select_all/wizard/portal_wizard.py @@ -6,20 +6,16 @@ from odoo import api, fields, models class PortalWizard(models.TransientModel): _inherit = "portal.wizard" - set_all_users = fields.Boolean( - string="Invite all the contacts", - default=False, - ) + set_all_users = fields.Boolean(string="Invite all the contacts", default=False,) - @api.onchange('set_all_users') + @api.onchange("set_all_users") def onchange_set_all_users(self): """Toggle between select all partners and the default""" if not self.set_all_users: for user in self.user_ids: - user.in_portal = ( - user.partner_id.user_ids and - user.partner_id.user_ids[0].has_group("base.group_portal") - ) + user.in_portal = user.partner_id.user_ids and user.partner_id.user_ids[ + 0 + ].has_group("base.group_portal") else: not_in_portal = self.user_ids.filtered(lambda x: not x.in_portal) - not_in_portal.update({'in_portal': True}) + not_in_portal.update({"in_portal": True}) diff --git a/portal_partner_select_all/wizard/portal_wizard.xml b/portal_partner_select_all/wizard/portal_wizard.xml index 3623de060..02d1dc168 100644 --- a/portal_partner_select_all/wizard/portal_wizard.xml +++ b/portal_partner_select_all/wizard/portal_wizard.xml @@ -1,21 +1,24 @@ - + - portal.wizard - + - + -
+
- You're about to invite to portal all the contacts list below. + You're about to invite to portal all the contacts list below.
-