[IMP] *: use svg icons instead of png

Prior to this commit, some icons used the png version.
After this commit, the svg version is used instead.

task-2818586

Part-of: odoo/odoo#116641

addons/account/views/account_menuitem.xml

@@ -4,7 +4,7 @@
     <menuitem name="Invoicing"
         id="menu_finance"
         groups="account.group_account_readonly,account.group_account_invoice"
-        web_icon="account,static/description/icon.png"
+        web_icon="account,static/description/icon.svg"
         sequence="55">
         <menuitem id="menu_board_journal_1" name="Dashboard" action="open_account_journal_dashboard_kanban" groups="account.group_account_readonly" sequence="1"/>
         <menuitem id="menu_finance_receivables" name="Customers" sequence="2">

addons/im_livechat/tests/test_message.py

@@ -65,7 +65,7 @@ class TestImLivechatMessage(TransactionCase):
             'message_type': 'notification',
             'messageReactionGroups': [],
             'model': 'discuss.channel',
-            'module_icon': '/mail/static/description/icon.png',
+            'module_icon': '/mail/static/description/icon.svg',
             'needaction_partner_ids': [],
             'notifications': [],
             'pinned_at': False,

odoo/addons/base/tests/test_module.py

@@ -48,7 +48,7 @@ class TestModuleManifest(BaseCase):
             'depends': [],
             'description': '',
             'external_dependencies': {},
-            'icon': '/base/static/description/icon.png',
+            'icon': '/base/static/description/icon.svg',
             'init_xml': [],
             'installable': True,
             'images': [],

odoo/addons/base/views/base_menus.xml

@@ -4,10 +4,10 @@
       <!-- Top menu item -->
       <menuitem name="Settings"
           id="menu_administration"
-          web_icon="base,static/description/settings.png"
+          web_icon="base,static/description/settings.svg"
           sequence="550"
           groups="base.group_erp_manager"/>
-          <menuitem id="menu_management" name="Apps" sequence="500" web_icon="base,static/description/modules.png" groups="base.group_system"/>
+          <menuitem id="menu_management" name="Apps" sequence="500" web_icon="base,static/description/modules.svg" groups="base.group_system"/>
           <menuitem id="menu_administration_shortcut" parent="menu_administration" name="Custom Shortcuts" sequence="50"/>
           <!-- FYI The group no_one on 'User & Companies' and 'Translations' is a FP/APR request -->
           <menuitem id="menu_users" name="Users &amp; Companies" parent="menu_administration" sequence="1"/>

odoo/addons/test_lint/tests/test_manifests.py

@@ -93,7 +93,7 @@ class ManifestLinter(BaseCase):
         )
         self.assertNotEqual(
             value,
-            f"/{module}/static/description/icon.png",
+            f"/{module}/static/description/icon.svg",
             f"Setting manifest key icon to the default manifest value for module {module!r}. "
             "You can remove this key from the dict to reduce noise/inconsistencies between manifests specifications"
             " and ease understanding of manifest content."

odoo/modules/module.py

@@ -37,7 +37,7 @@ _DEFAULT_MANIFEST = {
     'depends': [],
     'description': '',
     'external_dependencies': {},
-    #icon: f'/{module}/static/description/icon.png',  # automatic
+    #icon: f'/{module}/static/description/icon.svg',  # automatic
     'init_xml': [],
     'installable': True,
     'images': [],  # website
@@ -240,7 +240,7 @@ def get_resource_from_path(path):
     return None
 
 def get_module_icon(module):
-    iconpath = ['static', 'description', 'icon.png']
+    iconpath = ['static', 'description', 'icon.svg']
     if get_module_resource(module, *iconpath):
         return ('/' + module + '/') + '/'.join(iconpath)
     return '/base/'  + '/'.join(iconpath)