Skip to content

Commit

Permalink
Merge pull request #64 from acsone/fix-uppercase-addon-names
Browse files Browse the repository at this point in the history
Fix for addons with upper case names
  • Loading branch information
sbidoul authored Dec 22, 2023
2 parents 298e4c4 + d2f27b9 commit e2338ac
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/manifestoo_core/metadata.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
re.IGNORECASE,
)
ODOO_ADDON_METADATA_NAME_RE = re.compile(
r"^odoo(\d{1,2})?[-_]addon[-_](?P<addon_name>[a-z0-9_-]+)$",
r"^odoo(\d{1,2})?[-_]addon[-_](?P<addon_name>[a-zA-Z0-9_-]+)$",
)

__all__ = [
Expand Down
1 change: 1 addition & 0 deletions tests/test_metadata.py
Original file line number Diff line number Diff line change
Expand Up @@ -718,6 +718,7 @@ def test_distribution_name_to_addon_name() -> None:
assert distribution_name_to_addon_name("odoo-addon-addon1") == "addon1"
assert distribution_name_to_addon_name("odoo-addon-addon-1") == "addon_1"
assert distribution_name_to_addon_name("odoo-addon-addon_1") == "addon_1"
assert distribution_name_to_addon_name("odoo-addon-aDDon_1") == "aDDon_1"
with pytest.raises(InvalidDistributionName):
distribution_name_to_addon_name("odoo14-addon-")
with pytest.raises(InvalidDistributionName):
Expand Down

0 comments on commit e2338ac

Please sign in to comment.