[MIG] users_ldap_populate: Migration to 16.0

users_ldap_populate/__init__.py

@@ -2,3 +2,4 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/gpl.html).
 
 from . import models
+from . import wizards

users_ldap_populate/__manifest__.py

@@ -1,9 +1,10 @@
 # Copyright 2012-2018 Therp BV <https://therp.nl>.
 # Copyright 2021 Tecnativa - João Marques
+# Copyright 2023 Tecnativa - Carolina Fernandez
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/gpl.html).
 {
     "name": "LDAP Populate",
-    "version": "14.0.1.0.0",
+    "version": "16.0.1.0.0",
     "author": "Therp BV, Odoo Community Association (OCA)",
     "maintainers": ["joao-p-marques"],
     "website": "https://github.com/OCA/server-auth",
@@ -13,8 +14,8 @@
     "external_dependencies": {"python": ["python-ldap"]},
     "data": [
         "security/ir.model.access.csv",
+        "wizards/populate_wizard.xml",
         "views/users_ldap.xml",
-        "views/populate_wizard.xml",
     ],
     "installable": True,
 }

users_ldap_populate/models/__init__.py

@@ -1,5 +1,5 @@
 # Copyright 2012 Therp BV (<http://therp.nl>)
+# Copyright 2023 Tecnativa - Carolina Fernandez
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/gpl.html).
 
 from . import users_ldap
-from . import populate_wizard

users_ldap_populate/models/users_ldap.py

@@ -71,7 +71,7 @@ def action_populate(self):
                     user_id = self.with_context(
                         no_reset_password=True
                     )._get_or_create_user(conf, login, result)
-                except AccessDenied:
+                except AccessDenied as e:
                     # this happens if the user exists but is active = False
                     # -> fetch the user again and reactivate it
                     self.env.cr.execute(
@@ -85,7 +85,7 @@ def action_populate(self):
                         )
                         user_id = res[0]
                     else:
-                        raise UserError(
+                        raise UserError from e(
                             _("Unable to process user with login %s") % login
                         )
                 finally:

users_ldap_populate/readme/CONTRIBUTORS.rst

@@ -4,3 +4,4 @@
   * Alexandre Díaz
   * João Marques
   * Carlos Roca
+  * Carolina Fernandez

users_ldap_populate/static/description/index.html

@@ -427,6 +427,7 @@ <h2><a class="toc-backref" href="#toc-entry-6">Contributors</a></h2>
 <li>Alexandre Díaz</li>
 <li>João Marques</li>
 <li>Carlos Roca</li>
+<li>Carolina Fernandez</li>
 </ul>
 </li>
 </ul>

users_ldap_populate/wizards/__init__.py

@@ -0,0 +1,3 @@
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/gpl.html).
+
+from . import populate_wizard

users_ldap_populate/models/populate_wizard.py → users_ldap_populate/wizards/populate_wizard.py

@@ -8,7 +8,7 @@ class CompanyLDAPPopulateWizard(models.TransientModel):
     _name = "res.company.ldap.populate_wizard"
     _description = "Populate users from LDAP"
 
-    name = fields.Char("Name", size=16)
+    name = fields.Char(size=16)
     ldap_id = fields.Many2one("res.company.ldap", "LDAP Configuration")
     users_created = fields.Integer("Number of users created", readonly=True)
     users_deactivated = fields.Integer("Number of users deactivated", readonly=True)