Jul 29 [UPDT]: Updated 'odoo_health_report/'

odoo_health_report/README.rst

@@ -2,7 +2,7 @@
     :target: https://www.odoo.com/documentation/18.0/legal/licenses.html#odoo-apps
     :alt: License: OPL-1
 
-Odoo Health Report
+Odoo Module Health Report
 =================
 This module allows you to display Odoo app module details and print them as a PDF report.
 

odoo_health_report/__manifest__.py

@@ -20,8 +20,8 @@
 #
 ######################################################################################
 {
-    'name': 'Odoo Health Report',
-    'version': '18.0.2.1.1',
+    'name': 'Odoo Module Health Report',
+    'version': '18.0.2.2.1',
     'category': 'Productivity',
     'summary': "Odoo Module Health Monitoring Tool",
     'description': 'Displays odoo apps report in the menu and as a PDF report.',
@@ -38,6 +38,7 @@
             'bandit',
             'radon',
             'pylint',
+            'pylint-odoo',
             'flake8'
         ],
     },
@@ -50,7 +51,6 @@
     'assets': {
         'web.assets_backend': [
             'https://cdn.jsdelivr.net/npm/chart.js',
-            'https://cdnjs.cloudflare.com/ajax/libs/iconify/2.0.0/iconify.min.js',
             'https://fonts.googleapis.com/css2?family=Open+Sans:ital,wght@0,300..800;1,300..800&family=Public+Sans:ital,wght@0,100..900;1,100..900&display=swap.css',
             'odoo_health_report/static/src/css/main.css',
             'odoo_health_report/static/src/js/module_quality.js',

odoo_health_report/doc/RELEASE_NOTES.md

@@ -3,9 +3,20 @@
 #### 02.07.2025
 #### Version 18.0.1.0.1
 #### ADD
-- Initial Commit for Odoo Health Report
+- Initial Commit for Odoo Module Health Report
 
 #### 11.07.2025
 #### Version 18.0.2.1.1
 #### UPDATE
 - Added new features
+
+#### 22.07.2025
+#### Version 18.0.2.2.1
+#### FIX
+- Fixed checking violation bug
+#### UPDATE
+- Removed test folder for checking violations
+- Updated styles in dashboard and PDF
+- External dependency
+#### ADD
+- [requirements.txt](../requirements.txt) file

odoo_health_report/models/check_odoo_python_guidelines.py

@@ -433,8 +433,8 @@ def check_manifest_file(file_path):
             results.append({
                 "file": file_path_shorten,
                 "line": version.lineno if hasattr(version, "lineno") else 1,
-                "issue": f"Version '{version_val}' does not follow '16.0.1.0.0' format",
-                "suggestion": "Use 5-level semantic versioning like '16.0.1.0.0'"
+                "issue": f"Version '{version_val}' does not follow '18.0.1.0.0' format",
+                "suggestion": "Use 5-level semantic versioning like '18.0.1.0.0'"
             })
 
     # Depends check
@@ -684,6 +684,8 @@ def scan_directory(directory):
     """
     for root, _, files in os.walk(directory):
         for file in files:
+            if file.startswith("test_"):
+                continue
             if file == "__manifest__.py":
                 check_manifest_file(os.path.join(root, file))
             if file.endswith(".py") and file not in {"__init__.py", "__manifest__.py"}:

odoo_health_report/models/check_violations.py

@@ -88,7 +88,7 @@ def check_style_lint(module):
     Returns:
         list of dictionaries for the violations with the file name and line number.
     """
-    excluded_files = ['__init__.py', '__manifest__.py', '__pycache__', '*.pyc', '.git']
+    excluded_files = ['__init__.py', '__manifest__.py', '__pycache__', '*.pyc', '.git', 'tests']
     ignored_violations = ['E501', 'E301', 'E302']
 
     venv_python = sys.executable
@@ -131,6 +131,7 @@ def check_code_quality(module):
         '--load-plugins=pylint_odoo',
         '-d all',
         '-e odoolint',
+        '--ignore=tests',
         '--ignore-patterns=__init__.py,__manifest__.py',
         get_module_path(module)
     ],
@@ -182,6 +183,8 @@ def check_maintainability_index(module):
             rec = rec.split(' - ')
             if len(rec) >= 2:
                 file_path = f"{module}/{rec[0].split(f'/{module}/')[1]}"
+                if file_path.startswith(f'{module}/tests/'):
+                    continue
                 mi_list.append({'file': file_path, 'grade': rec[1]})
     return mi_list
 
@@ -200,6 +203,7 @@ def check_import_sort(module):
         '--check-only',
         '--skip', '__init__.py',
         '--skip', '__manifest__.py',
+        '--skip', 'tests',
         get_module_path(module),
     ],
         stdout=subprocess.PIPE,
@@ -245,7 +249,7 @@ def check_code_format(module):
     result = subprocess.run([
         sys.executable, '-m', 'black',
         '--check',
-        '--exclude', '__init__.py|__manifest__.py',
+        '--exclude', '__init__.py|__manifest__.py|tests',
         get_module_path(module),
     ],
         stdout=subprocess.PIPE,
@@ -287,6 +291,11 @@ def check_code_complexity(module):
     if result.returncode == 0 and result.stdout:
         clean_stdout = re.sub(r'\x1b\[[0-9;]*m', '', result.stdout)
         cc_dict = json.loads(clean_stdout.strip())
+        cc_dict = {
+            k: v for k, v in cc_dict.items()
+            if '/tests/' not in k
+        }
+
         return cc_dict
     return {}
 
@@ -303,8 +312,8 @@ def scan_code_security(module):
     result = subprocess.run([
         sys.executable, '-m', 'bandit',
         '-r', get_module_path(module),
-        '-x', '__init__.py,__manifest__.py',
-        '-f', 'json'
+        '-x', '__init__.py,__manifest__.py,tests',
+        '-f', 'json', '--quiet'
     ],
         stdout=subprocess.PIPE,
         stderr=subprocess.PIPE,

odoo_health_report/models/module_quality_package.py

@@ -128,8 +128,7 @@ class ModuleQuality(models.AbstractModel):
         is_updated.update_module()
 
         for module in get_modules():
-            module_path = get_module_path(module)
-            if 'addons' not in module_path.split(os.sep) and module != 'odoo_health_report':
+            if module != 'odoo_health_report':
                 module_name = self.env.ref(f'base.module_{module}').display_name
                 module_icon = modules.module.get_module_icon(module)
                 module_and_icon[module] = [module_name, module_icon]

odoo_health_report/reports/ir_actions_report.xml

@@ -17,12 +17,12 @@
     </record>
 
     <record id="action_odoo_health_report" model="ir.actions.report">
-        <field name="name">Odoo Health Report</field>
+        <field name="name">Odoo Module Health Report</field>
         <field name="model">module.quality.package</field>
         <field name="report_type">qweb-pdf</field>
         <field name="report_name">odoo_health_report.odoo_health_report</field>
         <field name="report_file">odoo_health_report.odoo_health_report</field>
         <field name="paperformat_id" ref="odoo_health_report.paperformat_report"/>
-        <field name="print_report_name">Odoo Health Report</field>
+        <field name="print_report_name">Odoo Module Health Report</field>
     </record>
 </odoo>

odoo_health_report/reports/odoo_health_report.xml

@@ -1,15 +1,52 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
-  <template id="odoo_health_report" name="Odoo Health Report">
+  <template id="odoo_health_report" name="Odoo Module Health Report">
     <t t-call="web.html_container">
       <t t-call="web.external_layout">
-        <div class="footer">
-          <p>Cybrosys Technologies | www.cybrosys.com | info@cybrosys.com</p>
+        <style type="text/css">
+          .hr-module-title {
+            color: #000;
+            font-size: 18px;
+            font-weight: 700;
+            text-transform: capitalize;
+            margin-bottom: 10px;
+          }
+          table, th, td, tr, thead, tbody {
+            border: none !important;
+          }
+          .hr-table {
+            width: 100%;
+            overflow: hidden;
+          }
+          .hr-table thead {
+            background-color: #ddf66b;
+            color: #000;
+          }
+          .hr-table thead th {
+            font-size: 16px;
+            background-color: #DDF66B;
+            color: #000 !important;
+          }
+          .hr-table thead th:first-child {
+            border-top-left-radius: 10px;
+          }
+          .hr-table thead th:last-child {
+            border-top-right-radius: 10px;
+          }
+          .hr-table tbody tr td {
+            font-size: 14px !important;
+            color: #495057 !important;
+            border-bottom: 1px solid #ddd !important;
+          }
+        </style>
+
+        <div class="footer" style="display: flex; justify-content: space-between; align-items: center; padding: 10px 20px; font-size: 14px; border-top: 1px solid #ccc;">
+          <p style="text-align: left; margin: 0;">Cybrosys Technologies | www.cybrosys.com | info@cybrosys.com</p>
+          <p style="text-align: right; margin: 0; right: 40px; top: 10px; position:absolute;"><span class="page"/> | Page</p>
         </div>
-        <body style="background-color: beige; display: flex; justify-content: center; align-items: center; flex-flow: column; gap: 2rem; font-family: 'Public Sans', sans-serif; font-size: 1rem;">
-          <main style="width: 21cm; height: 29.7cm; background-color: white; padding: 25px; position: relative;">
-            <!-- Left Top Corner Decoration -->
-            <img src="/odoo_health_report/static/src/assets/images/icons_report/left.svg" alt="" style="position: absolute; top: 0; left: 0;"/>
+
+        <body style="display: flex; justify-content: center; align-items: center; flex-flow: column; gap: 2rem; font-family: 'Public Sans', sans-serif; font-size: 1rem;">
+          <main>
             <section>
               <t t-if="module_quality">
                 <t t-set="field_details" t-value="module_quality['field_details']"/>
@@ -26,51 +63,46 @@
                       Module Quality Report
                   </h3>
                 </div>
-                <table style="width: 100%; border-collapse: collapse; margin: 15px 0;">
+
+                <table style="width: 100%; margin: 15px 0;">
                   <tr>
                     <!-- Critical Overview -->
                     <td style="width: 40%; padding: 10px; vertical-align: top;">
                       <div style="background: url('/odoo_health_report/static/src/assets/images/icons_report/box.png') no-repeat; background-size: cover; border-radius: 15px; color: white; padding: 20px; position: relative; height: 135px;width: 100%;">
-                        <h5 style="margin: 0; font-size: 12pt; font-weight: normal;color: white;">Apps installed ratio</h5>
+                        <h5 style="margin: 0; font-size: 14pt; font-weight: normal;color: white;">Apps installed ratio</h5>
                         <img src="/odoo_health_report/static/src/assets/images/icons_report/graph.png" alt="" style="width: 155px; position: absolute; left: 16px; top: 52px;"/>
-                        <h6 style="position: absolute; bottom: 10px; left: 40px; font-size: 10pt; margin: 0;color: white;">
+                        <h6 style="position: absolute; bottom: 10px; left: 40px; font-size: 12pt; margin: 0;color: white;">
                           Overall <span style="font-weight: bold;"><t t-esc="field_details['critical_overview']['overall_percentage']['value']"/>%</span>
                         </h6>
                       </div>
                     </td>
                     <!-- Fields -->
                     <td style="width: 60%;">
-                    <table style="width: 100%; border-collapse: collapse;">
+                      <table style="width: 100%;">
                         <tr>
                           <td style="padding: 10px; width: 33.33%;">
                             <div style="background: #d9e8ff; border-radius: 15px; padding: 15px; position: relative; height: 135px;">
-                            <h3 style="margin-top: 30px; color: #7b7b7b; font-size: 10pt;">Total Fields</h3>
-                            <span style="position: absolute; right: 5px; top: 5px; background: #2b7fff; padding: 5px; border-radius: 40%; width: 40px; height: 40px; text-align: center;">
-                              <img src="/odoo_health_report/static/src/assets/images/icons_report/globe.svg" alt="" style="width: 25px; top: 8px;position: absolute; left: 8px;" />
-                            </span>
-                            <p style="margin: 8px 0; font-size: 10pt; color: black; font-weight: bold;">
+                              <h3 style="margin-top: 20px; color: #7b7b7b; font-size: 14pt;">Total Fields</h3>
+                              <img src="/odoo_health_report/static/src/assets/images/icons_report/icon-all-field.svg" style="position: absolute; right: 5px; top: 5px;"/>
+                              <p style="margin: 8px 0; font-size: 12pt; color: black; font-weight: bold;">
                                 <t t-esc="field_details['critical_overview']['total_fields']['value']"/>
                               </p>
                             </div>
                           </td>
                           <td style="padding: 10px; width: 33.33%;">
                             <div style="background: #d9e8ff; border-radius: 15px; padding: 15px; position: relative; height: 135px;">
-                            <h3 style="margin-top: 30px; color: #7b7b7b; font-size: 10pt;">Stored Fields</h3>
-                            <span style="position: absolute; right: 5px; top: 5px; background: #2b7fff; padding: 5px; border-radius: 40%; width: 40px; height: 40px; text-align: center;">
-                              <img src="/odoo_health_report/static/src/assets/images/icons_report/note.svg" alt="" style="width: 25px; top: 8px;position: absolute; left: 8px;"/>
-                            </span>
-                            <p style="margin: 8px 0; font-size: 10pt; color: black; font-weight: bold;">
+                              <h3 style="margin-top: 20px; color: #7b7b7b; font-size: 14pt;">Stored Fields</h3>
+                              <img src="/odoo_health_report/static/src/assets/images/icons_report/icon-stored.svg" style="position: absolute; right: 5px; top: 5px;"/>
+                              <p style="margin: 8px 0; font-size: 12pt; color: black; font-weight: bold;">
                                 <t t-esc="field_details['critical_overview']['stored']['value']"/>
                               </p>
                             </div>
                           </td>
                           <td style="padding: 10px; width: 33.33%;">
                             <div style="background: #d9e8ff; border-radius: 15px; padding: 15px; position: relative; height: 135px;">
-                            <h3 style="margin-top: 30px; color: #7b7b7b; font-size: 10pt;">Non-Stored Fields</h3>
-                            <span style="position: absolute; right: 5px; top: 5px; background: #2b7fff; padding: 5px; border-radius: 40%; width: 40px; height: 40px; text-align: center;">
-                              <img src="/odoo_health_report/static/src/assets/images/icons_report/hour-glass.svg" alt="" style="width: 25px; top: 8px;position: absolute; left: 8px;"/>
-                            </span>
-                            <p style="margin: 8px 0; font-size: 10pt; color: black; font-weight: bold;">
+                              <h3 style="margin-top: 20px; color: #7b7b7b; font-size: 14pt;">Non-Stored <br/>Fields</h3>
+                              <img src="/odoo_health_report/static/src/assets/images/icons_report/icon-non-stored.svg" style="position: absolute; right: 5px; top: 5px;"/>
+                              <p style="margin: 8px 0; font-size: 12pt; color: black; font-weight: bold;">
                                 <t t-esc="field_details['critical_overview']['non_stored']['value']"/>
                               </p>
                             </div>
@@ -83,11 +115,11 @@
                 <section>
                   <h3>Module Lines Of Code</h3>
                   <t t-foreach="count_lines.get('result')" t-as="author">
-                  <h3 style="font-size: 1rem; color: #333; margin: 0; margin-bottom: 10px">
+                    <h3 class="hr-module-title">
                       <t t-esc="author"/>
                     </h3>
-                  <table style="border-collapse: collapse; width: 100%; font-size: 0.8rem; border-radius: 10px; overflow: hidden; box-shadow: 0px 0px 20px rgb(0 0 0 / 5%);">
-                    <thead style="background-color: #ddf66b; color: #333">
+                    <table class="hr-table" style="margin-bottom: 15px;">
+                      <thead>
                         <th style="text-align: left; padding: 10px">Module</th>
                         <th style="text-align: center; padding: 10px">Technical Name</th>
                         <th style="text-align: center; padding: 10px">Java Script Lines</th>
@@ -127,12 +159,12 @@
                       <t t-foreach="violation_module['violations']" t-as="violations_categ">
                         <!--  Odoo Code Standards Check  -->
                         <t t-if="violations_categ == 'odoo_standards'">
-                        <h3 style="font-size: 1rem; color: #333; margin: 0; margin-bottom: 10px; font-weight: 700;">
+                          <h3 style="font-size: 1.1rem; color: #333; margin: 0; margin-bottom: 10px; font-weight: 700;">
                               Odoo Code Standards Check
                           </h3>
                           <t t-if="violation_module['violations']['odoo_standards']">
-                          <table style="border-collapse: collapse; width: 100%; font-size: 0.8rem; border-radius: 10px; overflow: hidden; box-shadow: 0px 0px 20px rgb(0 0 0 / 5%);">
-                            <thead style="background-color: #ddf66b; color: #333">
+                            <table class="hr-table">
+                              <thead>
                                 <th style="text-align: left; padding: 10px">File Name</th>
                                 <th style="text-align: center; padding: 10px">Line</th>
                                 <th style="text-align: left; padding: 10px">Violation Message</th>
@@ -157,8 +189,8 @@
                               Style &amp; Linting Check
                           </h3>
                           <t t-if="violation_module['violations']['style_lint']">
-                          <table style="border-collapse: collapse; width: 100%; font-size: 0.8rem; border-radius: 10px; overflow: hidden; box-shadow: 0px 0px 20px rgb(0 0 0 / 5%);">
-                            <thead style="background-color: #ddf66b; color: #333">
+                            <table class="hr-table">
+                              <thead>
                                 <th style="text-align: left; padding: 10px">File Name</th>
                                 <th style="text-align: center; padding: 10px">Line</th>
                                 <th style="text-align: left; padding: 10px">Violation Message</th>
@@ -181,8 +213,8 @@
                             General Code Quality
                           </h3>
                           <t t-if="violation_module['violations']['code_quality']">
-                          <table style="border-collapse: collapse; width: 100%; font-size: 0.8rem; border-radius: 10px; overflow: hidden; box-shadow: 0px 0px 20px rgb(0 0 0 / 5%);">
-                            <thead style="background-color: #ddf66b; color: #333">
+                            <table class="hr-table">
+                              <thead>
                                 <th style="text-align: left; padding: 10px">File Name</th>
                                 <th style="text-align: center; padding: 10px">Line</th>
                                 <th style="text-align: center; padding: 10px">Column</th>
@@ -209,8 +241,8 @@
                             Maintainability Index
                           </h3>
                           <t t-if="violation_module['violations']['maintainability_index']">
-                          <table style="border-collapse: collapse; width: 100%; font-size: 0.8rem; border-radius: 10px; overflow: hidden; box-shadow: 0px 0px 20px rgb(0 0 0 / 5%);">
-                            <thead style="background-color: #ddf66b; color: #333">
+                            <table class="hr-table">
+                              <thead>
                                 <th style="text-align: left; padding: 10px">File Name</th>
                                 <th style="text-align: center; padding: 10px">Grade</th>
                               </thead>
@@ -231,8 +263,8 @@
                             Import Sorting Check
                           </h3>
                           <t t-if="violation_module['violations']['import_sort']">
-                          <table style="border-collapse: collapse; width: 100%; font-size: 0.8rem; border-radius: 10px; overflow: hidden; box-shadow: 0px 0px 20px rgb(0 0 0 / 5%);">
-                            <thead style="background-color: #ddf66b; color: #333">
+                            <table class="hr-table">
+                              <thead>
                                 <th style="text-align: left; padding: 10px">File Name</th>
                                 <th style="text-align: left; padding: 10px">Message</th>
                               </thead>
@@ -253,8 +285,8 @@
                             Code Formatting Check
                           </h3>
                           <t t-if="violation_module['violations']['code_format']">
-                          <table style="border-collapse: collapse; width: 100%; font-size: 0.8rem; border-radius: 10px; overflow: hidden; box-shadow: 0px 0px 20px rgb(0 0 0 / 5%);">
-                            <thead style="background-color: #ddf66b; color: #333">
+                            <table class="hr-table">
+                              <thead>
                                 <th style="text-align: left; padding: 10px">File Name</th>
                                 <th style="text-align: left; padding: 10px">Action</th>
                               </thead>
@@ -278,8 +310,8 @@
                           <t t-foreach="violation_module['violations']['code_complexity'].items()" t-as="cc_files">
                             <h3 style="font-size: 1rem; color: #333; margin: 0; margin-bottom: 10px">File: <t t-esc="cc_files[0]"/></h3>
                             <t t-if="cc_files[1]">
-                            <table style="border-collapse: collapse; width: 100%; font-size: 0.8rem; border-radius: 10px; overflow: hidden; box-shadow: 0px 0px 20px rgb(0 0 0 / 5%);">
-                              <thead style="background-color: #ddf66b; color: #333">
+                              <table class="hr-table">
+                                <thead>
                                   <th style="text-align: left; padding: 10px">Type</th>
                                   <th style="text-align: left; padding: 10px">Name</th>
                                   <th style="text-align: left; padding: 10px">Class Name</th>
@@ -313,8 +345,8 @@
                             Security Scan
                           </h3>
                           <t t-if="violation_module['violations']['security_scan']">
-                          <table style="border-collapse: collapse; width: 100%; font-size: 0.8rem; border-radius: 10px; overflow: hidden; box-shadow: 0px 0px 20px rgb(0 0 0 / 5%);">
-                            <thead style="background-color: #ddf66b; color: #333">
+                            <table class="hr-table">
+                              <thead>
                                 <th style="text-align: left; padding: 10px">Test ID</th>
                                 <th style="text-align: center; padding: 10px">Test Name</th>
                                 <th style="text-align: center; padding: 10px">Severity</th>
@@ -348,5 +380,5 @@
         </body>
       </t>
     </t>
-</template>
+  </template>
 </odoo>

odoo_health_report/requirements.txt

@@ -0,0 +1,7 @@
+isort
+black
+bandit
+radon
+pylint
+pylint-odoo
+flake8

odoo_health_report/static/description/index.html

@@ -137,7 +137,7 @@
               font-size: 46px;
               font-weight: 700;
               line-height: normal;
-            ">Odoo Health Report</span>
+            ">Odoo Module Health Report</span>
                             </h1>
                         </div>
                         <div class="col-lg-12 d-flex justify-content-center align-items-center"
@@ -312,7 +312,7 @@
                     <div class="position-relative" style=" padding: 5rem 4rem 5rem 4rem; background-color: #0A1425; border-radius: 12px;">
                         <div class="d-flex flex-column gap-4">
                             <span class="wrapper-subtitle"
-                                  style="font-size: 40px; font-weight: 700; color: #fff;line-height: 60px; text-transform: capitalize; width: 450px;">Odoo Health Report</span>
+                                  style="font-size: 40px; font-weight: 700; color: #fff;line-height: 60px; text-transform: capitalize; width: 450px;">Odoo Module Health Report</span>
                             <h3 class="wrapper-details"
                                 style="font-size: 20px; font-weight: 400; color: #fff; line-height: 32px;  ">
                                 Are your modules up to standard?

odoo_health_report/static/src/assets/images/icons_report/icon-all-field.svg

@@ -0,0 +1,5 @@
+<svg width="60" height="60" viewBox="0 0 60 60" fill="none" xmlns="http://www.w3.org/2000/svg">
+<path d="M37 0C49.7025 0 60 10.2975 60 23V37C60 49.7025 49.7025 60 37 60H23C10.2975 60 0 49.7025 0 37V23C0 10.2975 10.2975 0 23 0H37Z" fill="#2B7FFF"/>
+<path d="M40 46H20C18.34 46 17 44.66 17 43V17C17 15.34 18.34 14 20 14H34.18C34.98 14 35.74 14.32 36.3 14.88L42.12 20.7C42.68 21.26 43 22.02 43 22.82V43C43 44.66 41.66 46 40 46ZM20 16C19.44 16 19 16.44 19 17V43C19 43.56 19.44 44 20 44H40C40.56 44 41 43.56 41 43V22.82C41.0016 22.6888 40.9757 22.5588 40.9241 22.4383C40.8724 22.3177 40.7961 22.2093 40.7 22.12L34.88 16.3C34.6954 16.1116 34.4438 16.0038 34.18 16H20Z" fill="white"/>
+<path d="M41.76 24H35.94C34.32 24 33 22.68 33 21.06V15.24C33 14.68 33.44 14.24 34 14.24C34.56 14.24 35 14.68 35 15.24V21.06C35 21.58 35.42 22 35.94 22H41.76C42.32 22 42.76 22.44 42.76 23C42.76 23.56 42.32 24 41.76 24ZM35 40H25C24.44 40 24 39.56 24 39C24 38.44 24.44 38 25 38H35C35.56 38 36 38.44 36 39C36 39.56 35.56 40 35 40ZM35 34H25C24.44 34 24 33.56 24 33C24 32.44 24.44 32 25 32H35C35.56 32 36 32.44 36 33C36 33.56 35.56 34 35 34ZM29 28H25C24.44 28 24 27.56 24 27C24 26.44 24.44 26 25 26H29C29.56 26 30 26.44 30 27C30 27.56 29.56 28 29 28Z" fill="white"/>
+</svg>

odoo_health_report/static/src/assets/images/icons_report/icon-non-stored.svg

@@ -0,0 +1,6 @@
+<svg width="60" height="60" viewBox="0 0 60 60" fill="none" xmlns="http://www.w3.org/2000/svg">
+<path d="M37 0C49.7025 0 60 10.2975 60 23V37C60 49.7025 49.7025 60 37 60H23C10.2975 60 0 49.7025 0 37V23C0 10.2975 10.2975 0 23 0H37Z" fill="#2B7FFF"/>
+<path d="M19.3335 44.3333V17.6667H34.6668L40.6668 24.6667V44.3333H19.3335Z" stroke="white" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
+<path d="M25.3335 27L34.6668 36.3333M36.6668 31.6667C36.6668 35.3487 33.6822 38.3333 30.0002 38.3333C26.3182 38.3333 23.3335 35.3487 23.3335 31.6667C23.3335 27.9847 26.3182 25 30.0002 25C31.7024 24.998 33.3405 25.6492 34.5768 26.8193C35.2378 27.4417 35.7642 28.1929 36.1237 29.0266C36.4831 29.8603 36.668 30.7588 36.6668 31.6667Z" stroke="white" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
+<path d="M36.6668 31.6667C36.6668 35.3487 33.6822 38.3333 30.0002 38.3333M23.3335 31.6667C23.3335 27.9847 26.3182 25 30.0002 25" stroke="white" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
+</svg>

odoo_health_report/static/src/assets/images/icons_report/icon-stored.svg

@@ -0,0 +1,4 @@
+<svg width="60" height="60" viewBox="0 0 60 60" fill="none" xmlns="http://www.w3.org/2000/svg">
+<path d="M37 0C49.7025 0 60 10.2975 60 23V37C60 49.7025 49.7025 60 37 60H23C10.2975 60 0 49.7025 0 37V23C0 10.2975 10.2975 0 23 0H37Z" fill="#2B7FFF"/>
+<path d="M32.6668 15.6667H22.0002C20.5335 15.6667 19.3468 16.8667 19.3468 18.3333L19.3335 39.6667C19.3335 41.1333 20.5202 42.3333 21.9868 42.3333H38.0002C39.4668 42.3333 40.6668 41.1333 40.6668 39.6667V23.6667L32.6668 15.6667ZM38.0002 39.6667H22.0002V18.3333H31.5602L38.0002 24.7733V39.6667ZM25.2668 27.0933L23.7068 25.5333V30.3333H28.5068L26.5868 28.4133C26.9386 27.729 27.4715 27.1545 28.1275 26.7524C28.7836 26.3503 29.5374 26.1362 30.3068 26.1333C31.4207 26.1333 32.489 26.5758 33.2767 27.3635C34.0643 28.1511 34.5068 29.2194 34.5068 30.3333C34.5068 31.4472 34.0643 32.5155 33.2767 33.3032C32.489 34.0908 31.4207 34.5333 30.3068 34.5333C29.6307 34.5334 28.9646 34.3697 28.3656 34.0562C27.7665 33.7427 27.2523 33.2888 26.8668 32.7333H24.8002C25.2682 33.8014 26.0366 34.7104 27.0119 35.3496C27.9873 35.9887 29.1274 36.3305 30.2935 36.3333C33.6002 36.3333 36.2935 33.64 36.2935 30.3333C36.2935 27.0267 33.6002 24.3333 30.2935 24.3333C28.1735 24.3333 26.3335 25.44 25.2668 27.0933Z" fill="white"/>
+</svg>

odoo_health_report/static/src/css/main.css

@@ -988,11 +988,14 @@
   margin-top: 10px;
   border-radius: 12px;
   border: 1px solid #e5e5e5;
+  overflow-y: scroll;
+  max-height: 500px;
 }
 .hr-module-table-container-title {
   color: #000;
   font-size: 18px;
   font-weight: 700;
+  text-transform: capitalize;
 }
 
 .hr-accordion-item {
@@ -1002,16 +1005,6 @@
   border-radius: 10px;
 }
 
-.hr-accordion-item div.mt-4 {
-  max-height: 500px;
-  overflow-y: scroll;
-}
-
-.hr-accordion-item div.mt-4 h3 {
-  position: sticky;
-  top: 0;
-}
-
 .hr-accordion-item .hr-accordion-header {
   width: 100%;
   border: 1px solid #e5e5e5;
@@ -1038,6 +1031,9 @@
 .hr-accordion-item .hr-accordion-header:hover {
   background-color: #f8f8f8;
 }
+.hr-accordion-header.selected {
+  border: 1px solid #2B7FFF;
+}
 .hr-accordion-item .icon {
   font-size: 20px;
 }
@@ -1047,6 +1043,9 @@
 .hr-accordion-item h4 {
   margin-bottom: unset;
 }
+.hr-accordion-item h3.mt-4 {
+  font-weight: 700;
+}
 .hr-accordion-item .hr-accordion-content {
   display: none;
   background-color: #fff;
@@ -1418,7 +1417,12 @@ button.hrl-secondary-button:active {
   font-style: normal;
   font-weight: 700;
 }
-
+.hr-modal_dialog {
+	width: 450px;
+}
+.hr-modal_dialog .modal-content {
+    border-radius: 12px;
+}
 .hr-modal .modal-header {
   border-bottom: none;
   padding: 24px !important;
@@ -1444,6 +1448,9 @@ button.hrl-secondary-button:active {
   width: 24px;
   height: 24px;
 }
+.hr-modal-body .hr-form_input:checked {
+	background-color: #007bff;
+}
 .hr-modal-body .hr-form_input:focus {
   -webkit-box-shadow: none !important;
           box-shadow: none !important;
@@ -2342,6 +2349,7 @@ button.hrl-secondary-button:active {
 
 .scrollbar {
   /* Scrollbar Styling */
+  scrollbar-color: #2B7FFF;
 }
 .scrollbar ::-webkit-scrollbar {
   width: 7px;
@@ -2354,6 +2362,17 @@ button.hrl-secondary-button:active {
   border-radius: 10px;
   background: #bababa;
 }
+.scrollbar.hr-module-table-container::-webkit-scrollbar {
+  width: 5px;
+}
+.scrollbar.hr-module-table-container::-webkit-scrollbar-track {
+  background: #E4E4E4;
+  border-radius: 10px;
+}
+.scrollbar.hr-module-table-container::-webkit-scrollbar-thumb {
+  background: #2B7FFF;
+  border-radius: 10px;
+}
 
 .table-container {
   width: 100%;
@@ -2363,8 +2382,7 @@ button.hrl-secondary-button:active {
   -webkit-box-pack: center;
       -ms-flex-pack: center;
           justify-content: center;
-  overflow-y: scroll;
-  max-height: 500px;
+  margin-top: 16px;
 }
 
 table {
@@ -2373,14 +2391,13 @@ table {
   text-align: left;
 }
 table thead {
-  background-color: #d0e97b;
-  position: sticky;
-  top: 0px;
+  color: black;
 }
 table thead th {
   padding: 12px;
-  border-bottom: 2px solid #ddd;
   font-size: 16px;
+    background-color: #DDF66B;
+
 }
 table thead th:first-child {
   border-top-left-radius: 10px;
@@ -2392,12 +2409,9 @@ table .error {
   color: #ff0000;
 }
 table tbody tr {
-  border-left: 1px solid #ddd;
-  border-right: 1px solid #ddd;
   font-size: 14px;
-}
-table tbody tr {
   border-bottom: 1px solid #ddd;
+  box-shadow: inset 1px 0 0 #ddd, inset -1px 0 0 #ddd;
 }
 table tbody tr:hover {
   background-color: #f9f9f9;
@@ -2523,3 +2537,16 @@ table tbody tr td {
 .text-orange { color: #fd7e14; }
 
 .text-dark-red { color: #8B0000; }
+
+.animated-checkbox {
+  transition: background-color 0.3s ease, border-color 0.3s ease;
+}
+
+.animated-checkbox::after {
+  transform: scale(0) rotate(45deg);
+  transition: transform 0.2s ease;
+}
+
+.animated-checkbox:checked::after {
+  transform: scale(1) rotate(45deg);
+}

odoo_health_report/static/src/js/module_quality.js

@@ -25,6 +25,7 @@ export class ModuleQuality extends Component {
             all_modules : {},
             loading_accordian: false,
             expanded_accordian: false,
+            selectedModule: false,
         });
 
         this.orm = useService("orm");
@@ -94,7 +95,6 @@ export class ModuleQuality extends Component {
     }
 
     displayViolations(name) {
-        // If clicking the same module again, toggle the table visibility
         if (this.state.selectedModule === name) {
             this.state.selectedModule = null;
             this.state.module_selected = null;

odoo_health_report/static/src/xml/health_dashboard.xml

@@ -68,7 +68,7 @@
                                     <div class="modal-body hr-modal-body">
                                         <!--  Module -->
                                         <div class="form-check mb-4">
-                                            <input class="form-check-input hr-form_input" type="checkbox" value="" id="qualityCheck" style="width: 20px; height: 20px; border: 2px solid #007bff; border-radius: 4px; cursor: pointer;" t-model="state.module"/>
+                                            <input class="form-check-input hr-form_input animated-checkbox" type="checkbox" value="" id="qualityCheck" style="width: 20px; height: 20px; border: 2px solid #007bff; border-radius: 4px; cursor: pointer;" t-model="state.module"/>
                                             <label class="form-check-label hr-form_label" for="qualityCheck" style="margin-left: 8px; font-weight: 500;">Module Quality</label>
                                         </div>
                                         <t t-if="state.module">
@@ -76,16 +76,21 @@
                                                 <label class="form-label" style="font-weight: 500; margin-bottom: 8px;">Installed Modules</label>
                                                 <t t-if="state.select_module">
                                                     <div style="position: relative;">
-                                                        <div t-on-click="() => state.showOptions = !state.showOptions" style="width: 98%; display: flex; flex-wrap: wrap; align-items: center; padding: 6px; border: 1px solid #d1d5db; border-radius: 6px; cursor: pointer; background-color: white; min-height: 46px; gap: 4px;" class="hr-custom-select">
+                                                        <div t-on-click="() => state.showOptions = !state.showOptions" style="width: 98%; display: flex; flex-wrap: wrap; align-items: center; padding: 6px; border: 1px solid #d1d5db; border-radius: 12px; cursor: pointer; background-color: white; min-height: 46px; gap: 4px; box-shadow: none;" class="hr-custom-select">
                                                             <t t-set="selectedModules" t-value="Object.keys(state.module_selected_desc)" />
                                                             <t t-if="selectedModules.length">
                                                                 <t t-foreach="selectedModules" t-as="module" t-key="module_index">
-                                                                    <div style="display: flex; align-items: center; background-color: #e6f0ff; color: #000000; padding: 4px 8px; border-radius: 16px; font-size: 14px; margin: 2px;">
+                                                                    <div style="display: flex; align-items: center; background-color: #e6f0ff; color: #000000; padding: 4px 8px; border-radius: 12px; font-size: 14px; margin: 2px;">
                                                                         <span style="margin-right: 6px;" t-log="module">
                                                                             <t t-esc="state.module_selected_desc[module]"/>
                                                                         </span>
                                                                         <div t-on-click="onModuleSelectDesc" t-att-data-mod-name="module"
-                                                                             style="background-color: #007bff; color: white; width: 18px; height: 18px; display: flex; align-items: center; justify-content: center; cursor: pointer; font-size: 12px; font-weight: bold;">×</div>
+                                                                             style="background-color: #007bff; color: white; width: 18px; height: 18px; display: flex; align-items: center; justify-content: center; cursor: pointer; font-size: 12px; font-weight: bold; border-radius: 3px;">
+                                                                            <svg width="8" height="7" viewBox="0 0 8 7" fill="none" xmlns="http://www.w3.org/2000/svg">
+                                                                                <path fill-rule="evenodd" clip-rule="evenodd" d="M1 0.5L7 6.5L1 0.5ZM7 0.5L1 6.5L7 0.5Z" fill="white"/>
+                                                                                <path d="M1 0.5L7 6.5M7 0.5L1 6.5" stroke="white" stroke-linecap="round" stroke-linejoin="round"/>
+                                                                            </svg>
+                                                                        </div>
                                                                     </div>
                                                                 </t>
                                                             </t>
@@ -98,13 +103,13 @@
                                                         </div>
                                                         <!-- Dropdown menu -->
                                                         <t t-if="state.showOptions">
-                                                            <div style="position: absolute; top: 100%; left: 0; width: 95%; background-color: white; border: 1px solid #e0e0e0; border-radius: 4px; box-shadow: 0 2px 8px rgba(0,0,0,0.1); z-index: 100; margin-top: 4px; display: flex;position: relative;flex-direction: column;">
+                                                            <div style="top: 100%; left: 0; width: 98%; background-color: white; border: none; border-radius: 0 0 12px 12px; box-shadow: 0 2px 8px rgba(0,0,0,0.1); z-index: 999;  display: flex;position: relative;flex-direction: column;">
                                                                 <div style="max-height: 200px; overflow-y: auto; padding: 8px 12px;">
                                                                     <t t-foreach="state.select_module" t-as="mod" t-key="mod_index">
                                                                         <div style="padding: 6px 0;">
                                                                             <label style="display: flex; align-items: center; cursor: pointer; width: 100%;">
                                                                                 <input type="checkbox" t-att-value="mod.name" t-model="state.module_selected[mod.name]" style="display: none;" />
-                                                                                <input class="form-check-input hr-form_input" type="checkbox" t-att-checked="state.module_selected[mod.name]" id="serverCheck" t-on-click="onModuleSelect"
+                                                                                <input class="form-check-input hr-form_input animated-checkbox" type="checkbox" t-att-checked="state.module_selected[mod.name]" id="serverCheck" t-on-click="onModuleSelect"
                                                                                        t-att-data-mod-name="mod.name" t-att-data-mod-shortdesc="mod.shortdesc" style="width: 24px; height: 24px; border: 2px solid #007bff; border-radius: 4px; cursor: pointer;" />
                                                                                 <label for="serverCheck" style="margin-left: 10px;" t-att-data-mod-name="mod.name">
                                                                                     <t t-esc="mod.shortdesc"/>
@@ -122,8 +127,8 @@
 
                                         <!--  Footer  -->
                                         <div class="d-flex justify-content-between hr-modal-footer" style="gap: 16px; padding-top: 16px; border-top: 1px solid #DEDEDE;">
-                                            <button class="w-50 me-2 hr-primary_btn" style="padding: 12px; border-radius: 6px; border: none; background-color: #007bff; color: white; font-weight: 500; cursor: pointer; transition: background-color 0.2s ease;" t-on-click="onGenerateReport">Print report</button>
-                                            <button class="hr-secondary_btn w-50" data-bs-dismiss="modal" aria-label="Close" style="padding: 12px; border-radius: 6px; border: 1px solid #6c757d; background-color: transparent; color: #6c757d; font-weight: 500; cursor: pointer; transition: all 0.2s ease;">Cancel</button>
+                                            <button class="w-50 me-2 hr-primary_btn" style="padding: 12px; border-radius: 12px; border: none; background-color: #007bff; color: white; font-weight: 500; cursor: pointer; transition: background-color 0.2s ease;" t-on-click="onGenerateReport">Print report</button>
+                                            <button class="hr-secondary_btn w-50" data-bs-dismiss="modal" aria-label="Close" style="padding: 12px; border-radius: 12px; border: 1px solid #6c757d; background-color: transparent; color: #6c757d; font-weight: 500; cursor: pointer; transition: all 0.2s ease;">Cancel</button>
                                         </div>
                                     </div>
                                 </div>

odoo_health_report/static/src/xml/module_quality.xml

@@ -50,10 +50,11 @@
             <t t-esc="state.field['critical_overview']['total_fields']['value']"/>
           </p>
         </div>
-        <div class="icon">
-          <svg xmlns="http://www.w3.org/2000/svg" width="32" height="32" viewBox="0 0 32 32" fill="none">
-            <rect width="32" height="32" fill="#2B7FFF"/>
-            <path d="M30 20L32 30H0L2 20H5.625C5.09375 19.0833 4.69271 18.1198 4.42188 17.1094C4.15104 16.099 4.01042 15.0625 4 14C4 12.8958 4.14062 11.8333 4.42188 10.8125C4.70312 9.79167 5.10938 8.83854 5.64062 7.95312C6.17188 7.06771 6.79688 6.25521 7.51562 5.51562C8.23438 4.77604 9.04167 4.15104 9.9375 3.64062C10.8333 3.13021 11.7917 2.72917 12.8125 2.4375C13.8333 2.14583 14.8958 2 16 2C17.1042 2 18.1667 2.14062 19.1875 2.42188C20.2083 2.70312 21.1615 3.10938 22.0469 3.64062C22.9323 4.17188 23.7448 4.79688 24.4844 5.51562C25.224 6.23438 25.849 7.04167 26.3594 7.9375C26.8698 8.83333 27.2708 9.79167 27.5625 10.8125C27.8542 11.8333 28 12.8958 28 14C28 15.0625 27.8646 16.099 27.5938 17.1094C27.3229 18.1198 26.9167 19.0833 26.375 20H30ZM26 14C26 13.6667 25.9844 13.3333 25.9531 13C25.9219 12.6667 25.8698 12.3333 25.7969 12H21.9062C21.9688 12.6771 22 13.3438 22 14C22 14.6667 21.9688 15.3333 21.9062 16H25.7969C25.8594 15.6667 25.9062 15.3333 25.9375 15C25.9688 14.6667 25.9896 14.3333 26 14ZM12.0938 16H19.9062C19.9688 15.3333 20 14.6667 20 14C20 13.3438 19.9688 12.6771 19.9062 12H12.0938C12.0312 12.6771 12 13.3438 12 14C12 14.6667 12.0312 15.3333 12.0938 16ZM19.625 18H12.375C12.4167 18.2396 12.4896 18.5521 12.5938 18.9375C12.6979 19.3229 12.8281 19.7448 12.9844 20.2031C13.1406 20.6615 13.3229 21.1094 13.5312 21.5469C13.7396 21.9844 13.9688 22.3906 14.2188 22.7656C14.4688 23.1406 14.7448 23.4375 15.0469 23.6562C15.349 23.875 15.6667 23.9896 16 24C16.3333 24 16.6458 23.8854 16.9375 23.6562C17.2292 23.4271 17.5052 23.1302 17.7656 22.7656C18.026 22.401 18.2552 21.9948 18.4531 21.5469C18.651 21.099 18.8333 20.651 19 20.2031C19.1667 19.7552 19.2969 19.3385 19.3906 18.9531C19.4844 18.5677 19.5625 18.25 19.625 18ZM12.1719 23.2344C11.6927 22.4427 11.3073 21.599 11.0156 20.7031C10.724 19.8073 10.5 18.9062 10.3438 18H6.84375C7.09375 18.5521 7.40625 19.0885 7.78125 19.6094C8.15625 20.1302 8.57292 20.625 9.03125 21.0938C9.48958 21.5625 9.98958 21.974 10.5312 22.3281C11.0729 22.6823 11.6198 22.9844 12.1719 23.2344ZM21.6562 18C21.4896 18.8958 21.2656 19.7917 20.9844 20.6875C20.7031 21.5833 20.3229 22.4323 19.8438 23.2344C20.4062 22.9844 20.9479 22.6771 21.4688 22.3125C21.9896 21.9479 22.4844 21.5417 22.9531 21.0938C23.4219 20.6458 23.8438 20.1562 24.2188 19.625C24.5938 19.0938 24.9062 18.5521 25.1562 18H21.6562ZM25.1562 10C24.625 8.82292 23.9062 7.78125 23 6.875C22.0938 5.96875 21.0365 5.26562 19.8281 4.76562C20.3073 5.55729 20.6875 6.40104 20.9688 7.29688C21.25 8.19271 21.4792 9.09375 21.6562 10H25.1562ZM16 4C15.6667 4 15.3542 4.11458 15.0625 4.34375C14.7708 4.57292 14.4948 4.86979 14.2344 5.23438C13.974 5.59896 13.7448 6.00521 13.5469 6.45312C13.349 6.90104 13.1667 7.34896 13 7.79688C12.8333 8.24479 12.6979 8.66146 12.5938 9.04688C12.4896 9.43229 12.4167 9.75 12.375 10H19.625C19.5729 9.76042 19.5 9.44792 19.4062 9.0625C19.3125 8.67708 19.1823 8.25521 19.0156 7.79688C18.849 7.33854 18.6667 6.89062 18.4688 6.45312C18.2708 6.01562 18.0417 5.60938 17.7812 5.23438C17.5208 4.85938 17.2448 4.5625 16.9531 4.34375C16.6615 4.125 16.3438 4.01042 16 4ZM12.1719 4.76562C10.974 5.26562 9.92188 5.96875 9.01562 6.875C8.10938 7.78125 7.38542 8.82292 6.84375 10H10.3438C10.5104 9.09375 10.7344 8.19271 11.0156 7.29688C11.2969 6.40104 11.6823 5.55729 12.1719 4.76562ZM6.20312 12C6.14062 12.3333 6.09375 12.6667 6.0625 13C6.03125 13.3333 6.01042 13.6667 6 14C6 14.3333 6.01562 14.6667 6.04688 15C6.07812 15.3333 6.13021 15.6667 6.20312 16H10.0938C10.0312 15.3333 10 14.6667 10 14C10 13.3438 10.0312 12.6771 10.0938 12H6.20312ZM2.4375 28H29.5625L28.3594 22H24.9219C24.3594 22.6354 23.7396 23.1979 23.0625 23.6875C22.3854 24.1771 21.6615 24.599 20.8906 24.9531C20.1198 25.3073 19.3281 25.5677 18.5156 25.7344C17.7031 25.901 16.8646 25.9896 16 26C15.1458 26 14.3073 25.9115 13.4844 25.7344C12.6615 25.5573 11.8698 25.2969 11.1094 24.9531C10.349 24.6094 9.63021 24.1927 8.95312 23.7031C8.27604 23.2135 7.65104 22.6458 7.07812 22H3.64062L2.4375 28Z" fill="#ECF1FF"/>
+        <div class="">
+          <svg width="60" height="60" viewBox="0 0 60 60" fill="none" xmlns="http://www.w3.org/2000/svg">
+            <path d="M37 0C49.7025 0 60 10.2975 60 23V37C60 49.7025 49.7025 60 37 60H23C10.2975 60 0 49.7025 0 37V23C0 10.2975 10.2975 0 23 0H37Z" fill="#2B7FFF"/>
+            <path d="M40 46H20C18.34 46 17 44.66 17 43V17C17 15.34 18.34 14 20 14H34.18C34.98 14 35.74 14.32 36.3 14.88L42.12 20.7C42.68 21.26 43 22.02 43 22.82V43C43 44.66 41.66 46 40 46ZM20 16C19.44 16 19 16.44 19 17V43C19 43.56 19.44 44 20 44H40C40.56 44 41 43.56 41 43V22.82C41.0016 22.6888 40.9757 22.5588 40.9241 22.4383C40.8724 22.3177 40.7961 22.2093 40.7 22.12L34.88 16.3C34.6954 16.1116 34.4438 16.0038 34.18 16H20Z" fill="white"/>
+            <path d="M41.76 24H35.94C34.32 24 33 22.68 33 21.06V15.24C33 14.68 33.44 14.24 34 14.24C34.56 14.24 35 14.68 35 15.24V21.06C35 21.58 35.42 22 35.94 22H41.76C42.32 22 42.76 22.44 42.76 23C42.76 23.56 42.32 24 41.76 24ZM35 40H25C24.44 40 24 39.56 24 39C24 38.44 24.44 38 25 38H35C35.56 38 36 38.44 36 39C36 39.56 35.56 40 35 40ZM35 34H25C24.44 34 24 33.56 24 33C24 32.44 24.44 32 25 32H35C35.56 32 36 32.44 36 33C36 33.56 35.56 34 35 34ZM29 28H25C24.44 28 24 27.56 24 27C24 26.44 24.44 26 25 26H29C29.56 26 30 26.44 30 27C30 27.56 29.56 28 29 28Z" fill="white"/>
           </svg>
         </div>
       </div>
@@ -67,10 +68,10 @@
             <t t-esc="state.field['critical_overview']['stored']['value']"/>
           </p>
         </div>
-        <div class="icon">
-          <svg xmlns="http://www.w3.org/2000/svg" width="32" height="32" viewBox="0 0 32 32" fill="none">
-            <path d="M10 13C10 12.7348 10.1054 12.4804 10.2929 12.2929C10.4804 12.1054 10.7348 12 11 12H21C21.2652 12 21.5196 12.1054 21.7071 12.2929C21.8946 12.4804 22 12.7348 22 13C22 13.2652 21.8946 13.5196 21.7071 13.7071C21.5196 13.8946 21.2652 14 21 14H11C10.7348 14 10.4804 13.8946 10.2929 13.7071C10.1054 13.5196 10 13.2652 10 13ZM11 18C10.7348 18 10.4804 18.1054 10.2929 18.2929C10.1054 18.4804 10 18.7348 10 19C10 19.2652 10.1054 19.5196 10.2929 19.7071C10.4804 19.8946 10.7348 20 11 20H21C21.2652 20 21.5196 19.8946 21.7071 19.7071C21.8946 19.5196 22 19.2652 22 19C22 18.7348 21.8946 18.4804 21.7071 18.2929C21.5196 18.1054 21.2652 18 21 18H11ZM10 25C10 24.7348 10.1054 24.4804 10.2929 24.2929C10.4804 24.1054 10.7348 24 11 24H21C21.2652 24 21.5196 24.1054 21.7071 24.2929C21.8946 24.4804 22 24.7348 22 25C22 25.2652 21.8946 25.5196 21.7071 25.7071C21.5196 25.8946 21.2652 26 21 26H11C10.7348 26 10.4804 25.8946 10.2929 25.7071C10.1054 25.5196 10 25.2652 10 25ZM11 6C10.7348 6 10.4804 6.10536 10.2929 6.29289C10.1054 6.48043 10 6.73478 10 7C10 7.26522 10.1054 7.51957 10.2929 7.70711C10.4804 7.89464 10.7348 8 11 8H16V6H11Z" fill="white"/>
-            <path fill-rule="evenodd" clip-rule="evenodd" d="M27.9996 9.14114C28.0214 8.98289 28.0049 8.82173 27.9516 8.67114L27.9256 8.54514C27.852 8.23869 27.7305 7.94577 27.5656 7.67714C27.3816 7.37714 27.1216 7.11714 26.6016 6.59714L21.4216 1.41714C20.9036 0.897142 20.6436 0.637142 20.3416 0.451142C20.0366 0.264372 19.7007 0.133648 19.3496 0.0651418C19.199 0.0118321 19.0379 -0.00462724 18.8796 0.0171418C18.6576 0.00914179 18.3876 0.00914179 18.0296 0.00914179H13.6096C10.2496 0.00914179 8.56962 0.0091418 7.28963 0.663142C6.16337 1.24042 5.2469 2.15689 4.66963 3.28314C4.01563 4.56714 4.01562 6.24314 4.01562 9.60314V22.4031C4.01562 25.7631 4.01563 27.4431 4.66963 28.7231C5.2469 29.8494 6.16337 30.7659 7.28963 31.3431C8.57363 31.9971 10.2496 31.9971 13.6096 31.9971H18.4096C21.7696 31.9971 23.4496 31.9971 24.7296 31.3431C25.8575 30.7681 26.7746 29.851 27.3496 28.7231C28.0036 27.4391 28.0036 25.7631 28.0036 22.4031V9.98114C28.0036 9.62514 28.0036 9.35514 27.9936 9.13114L27.9996 9.14114ZM22.1796 29.9411C21.2796 30.0151 20.1196 30.0171 18.3996 30.0171H13.5996C11.8856 30.0171 10.7196 30.0151 9.81962 29.9411C8.94362 29.8691 8.49363 29.7391 8.18163 29.5811C7.42901 29.1977 6.81711 28.5858 6.43363 27.8331C6.27363 27.5211 6.14363 27.0711 6.07363 26.1951C5.99963 25.2951 5.99763 24.1351 5.99763 22.4151V9.61514C5.99763 7.90114 5.99963 6.73514 6.07363 5.83514C6.14563 4.95914 6.27563 4.50914 6.43363 4.19714C6.81763 3.44514 7.42963 2.83314 8.18163 2.44914C8.49363 2.28914 8.94362 2.15914 9.81962 2.08914C10.7196 2.01514 11.8796 2.01314 13.5996 2.01314H17.9996V9.01314C17.9996 9.27836 18.105 9.53271 18.2925 9.72025C18.4801 9.90778 18.7344 10.0131 18.9996 10.0131H25.9996V22.4131C25.9996 24.1271 25.9996 25.2931 25.9236 26.1931C25.8536 27.0691 25.7236 27.5191 25.5636 27.8331C25.1798 28.585 24.5679 29.1962 23.8156 29.5791C23.5036 29.7391 23.0556 29.8691 22.1776 29.9391L22.1796 29.9411ZM19.9996 2.94114L25.1796 8.12114H19.9996V2.94114Z" fill="white"/>
+        <div class="">
+          <svg width="60" height="60" viewBox="0 0 60 60" fill="none" xmlns="http://www.w3.org/2000/svg">
+            <path d="M37 0C49.7025 0 60 10.2975 60 23V37C60 49.7025 49.7025 60 37 60H23C10.2975 60 0 49.7025 0 37V23C0 10.2975 10.2975 0 23 0H37Z" fill="#2B7FFF"/>
+            <path d="M32.6668 15.6667H22.0002C20.5335 15.6667 19.3468 16.8667 19.3468 18.3333L19.3335 39.6667C19.3335 41.1333 20.5202 42.3333 21.9868 42.3333H38.0002C39.4668 42.3333 40.6668 41.1333 40.6668 39.6667V23.6667L32.6668 15.6667ZM38.0002 39.6667H22.0002V18.3333H31.5602L38.0002 24.7733V39.6667ZM25.2668 27.0933L23.7068 25.5333V30.3333H28.5068L26.5868 28.4133C26.9386 27.729 27.4715 27.1545 28.1275 26.7524C28.7836 26.3503 29.5374 26.1362 30.3068 26.1333C31.4207 26.1333 32.489 26.5758 33.2767 27.3635C34.0643 28.1511 34.5068 29.2194 34.5068 30.3333C34.5068 31.4472 34.0643 32.5155 33.2767 33.3032C32.489 34.0908 31.4207 34.5333 30.3068 34.5333C29.6307 34.5334 28.9646 34.3697 28.3656 34.0562C27.7665 33.7427 27.2523 33.2888 26.8668 32.7333H24.8002C25.2682 33.8014 26.0366 34.7104 27.0119 35.3496C27.9873 35.9887 29.1274 36.3305 30.2935 36.3333C33.6002 36.3333 36.2935 33.64 36.2935 30.3333C36.2935 27.0267 33.6002 24.3333 30.2935 24.3333C28.1735 24.3333 26.3335 25.44 25.2668 27.0933Z" fill="white"/>
           </svg>
         </div>
       </div>
@@ -84,9 +85,12 @@
             <t t-esc="state.field['critical_overview']['non_stored']['value']"/>
           </p>
         </div>
-        <div class="icon">
-          <svg xmlns="http://www.w3.org/2000/svg" width="32" height="32" viewBox="0 0 32 32" fill="none">
-            <path d="M7.19159 1.375V1.95894C7.19159 8.331 9.57647 13.8119 13.213 15.8398V16.2206C9.57641 18.2484 7.19153 23.7294 7.19153 30.1015V30.6855H25.1133V30.1015C25.1133 23.692 22.688 18.2094 19.0177 16.2148V15.8456C22.6878 13.8508 25.1133 8.36812 25.1133 1.95888V1.375H7.19141H7.19159ZM8.37516 2.54281H23.9298C23.775 8.57062 21.2585 13.513 18.1837 14.9744L17.8498 15.1327V16.9296L18.1838 17.0877C21.2583 18.5492 23.7748 23.4901 23.9298 29.5175H8.37516C8.52866 23.5239 11.008 18.5751 14.0548 17.0839L14.383 16.9237V15.1366L14.0548 14.9765C11.0081 13.4852 8.52859 8.53644 8.37516 2.54281ZM21.7287 7.92562C18.1282 9.66375 13.9458 9.66562 10.631 8.18156C11.5469 11.3631 13.3695 13.6276 15.5139 14.0019C15.4139 14.134 15.3597 14.2951 15.3595 14.4609C15.3595 14.8834 15.7025 15.2265 16.1252 15.2265C16.5478 15.2265 16.8908 14.8834 16.8908 14.4609C16.8908 14.2909 16.8341 14.1347 16.7404 14.0077C18.9545 13.6402 20.8334 11.2533 21.7287 7.92562ZM16.1252 16.166C15.7025 16.166 15.3595 16.509 15.3595 16.9316C15.3595 17.3542 15.7025 17.6973 16.1252 17.6973C16.5478 17.6973 16.8908 17.3542 16.8908 16.9316C16.8908 16.509 16.5478 16.166 16.1252 16.166ZM16.1252 18.621C15.7025 18.621 15.3595 18.9641 15.3595 19.3866C15.3595 19.8093 15.7025 20.1523 16.1252 20.1523C16.5478 20.1523 16.8908 19.8093 16.8908 19.3866C16.8908 18.9641 16.5478 18.621 16.1252 18.621ZM16.1252 21.0918C15.7025 21.0918 15.3595 21.4348 15.3595 21.8574C15.3595 22.2799 15.7025 22.6231 16.1252 22.6231C16.5478 22.6231 16.8908 22.2799 16.8908 21.8574C16.8908 21.4348 16.5478 21.0918 16.1252 21.0918ZM16.133 23.5859C14.6823 23.6034 13.2461 25.1966 11.3947 28.3144H20.9572C19.0495 25.1268 17.5836 23.5684 16.133 23.5859Z" fill="white"/>
+        <div class="">
+          <svg width="60" height="60" viewBox="0 0 60 60" fill="none" xmlns="http://www.w3.org/2000/svg">
+            <path d="M37 0C49.7025 0 60 10.2975 60 23V37C60 49.7025 49.7025 60 37 60H23C10.2975 60 0 49.7025 0 37V23C0 10.2975 10.2975 0 23 0H37Z" fill="#2B7FFF"/>
+            <path d="M19.3335 44.3333V17.6667H34.6668L40.6668 24.6667V44.3333H19.3335Z" stroke="white" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
+            <path d="M25.3335 27L34.6668 36.3333M36.6668 31.6667C36.6668 35.3487 33.6822 38.3333 30.0002 38.3333C26.3182 38.3333 23.3335 35.3487 23.3335 31.6667C23.3335 27.9847 26.3182 25 30.0002 25C31.7024 24.998 33.3405 25.6492 34.5768 26.8193C35.2378 27.4417 35.7642 28.1929 36.1237 29.0266C36.4831 29.8603 36.668 30.7588 36.6668 31.6667Z" stroke="white" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
+            <path d="M36.6668 31.6667C36.6668 35.3487 33.6822 38.3333 30.0002 38.3333M23.3335 31.6667C23.3335 27.9847 26.3182 25 30.0002 25" stroke="white" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
           </svg>
         </div>
       </div>
@@ -101,20 +105,9 @@
           <t t-foreach="Object.keys(state.quality_monitoring)" t-as="author" t-key="author_index">
             <t t-set="isFirst" t-value="authorIndex === 0"/>
             <t t-set="authorIndex" t-value="authorIndex + 1"/>
-
-            <div class="hr-accordion-item">
-              <button class="hr-accordion-header" t-on-click="() => this.displayAuthorModules(author)">
-                <div class="d-flex align-items-center gap-2">
-                  <h4 class="cursor-pointer">
-                    <t t-esc="author"/>
-                  </h4>
-                </div>
-                <span class="icon">+</span>
-              </button>
-              <t t-if="state.selectedAuthor === author or (state.selectedAuthor === null and isFirst)">
-                <div class="mt-4 active table-container">
-                  <div class="table-container">
-                    <table class="table table-bordered">
+            <span class="hr-module-table-container-title" t-on-click="() => this.displayAuthorModules(author)"><t t-esc="author"/></span>
+            <div class="table-container" style="margin-bottom: 15px;">
+              <table>
                 <thead>
                   <tr>
                     <th>Module</th>
@@ -127,29 +120,16 @@
                 <tbody>
                   <t t-foreach="state.quality_monitoring[author]" t-as="author_module" t-key="author_module_index">
                     <tr>
-                            <td>
-                              <t t-esc="author_module.module_name"/>
-                            </td>
-                            <td>
-                              <t t-esc="author_module.technical_name"/>
-                            </td>
-                            <td>
-                              <t t-esc="author_module.js_lines"/>
-                            </td>
-                            <td>
-                              <t t-esc="author_module.py_lines"/>
-                            </td>
-                            <td>
-                              <t t-esc="author_module.xml_lines"/>
-                            </td>
+                      <td><t t-esc="author_module.module_name"/></td>
+                      <td><t t-esc="author_module.technical_name"/></td>
+                      <td><t t-esc="author_module.js_lines"/></td>
+                      <td><t t-esc="author_module.py_lines"/></td>
+                      <td><t t-esc="author_module.xml_lines"/></td>
                     </tr>
                   </t>
                 </tbody>
               </table>
             </div>
-                </div>
-              </t>
-            </div>
           </t>
         </div>
       </div>
@@ -163,27 +143,49 @@
           <t t-if="state.all_modules">
             <t t-foreach="Object.entries(state.all_modules)" t-as="module" t-key="module_index">
               <div class="hr-accordion-item">
-                <button class="hr-accordion-header" t-on-click="() => this.displayViolations(module[0])">
+                <button class="hr-accordion-header" t-att-class="module[0] === state.selectedModule ? 'selected' : ''"
+                        t-att-data-module="module[0]" t-on-click="() => this.displayViolations(module[0])">
                   <div class="d-flex align-items-center gap-2">
                     <span class="icon-box">
                       <img t-att-src="module[1][1]" width="30" height="30" viewBox="0 0 48 48"/>
                     </span>
-                    <h4 class="cursor-pointer" t-esc="module[1][0]"/>
+                    <span t-esc="module[1][0]"/>
+                  </div>
+                  <t t-if="module[0] === state.selectedModule &amp;&amp; ! state.loading_accordian">
+                    <span class="icon">
+                      <svg width="16" height="2" viewBox="0 0 16 2" fill="none" xmlns="http://www.w3.org/2000/svg">
+                        <path fill-rule="evenodd" clip-rule="evenodd" d="M0.850098 0.25H15.1521C15.351 0.25 15.5418 0.329018 15.6824 0.46967C15.8231 0.610323 15.9021 0.801088 15.9021 1C15.9021 1.19891 15.8231 1.38968 15.6824 1.53033C15.5418 1.67098 15.351 1.75 15.1521 1.75H0.850098C0.651185 1.75 0.46042 1.67098 0.319767 1.53033C0.179115 1.38968 0.100098 1.19891 0.100098 1C0.100098 0.801088 0.179115 0.610323 0.319767 0.46967C0.46042 0.329018 0.651185 0.25 0.850098 0.25Z" fill="black"/>
+                      </svg>
+                    </span>
+                  </t>
+                  <t t-elif="module[0] === state.selectedModule &amp;&amp; state.loading_accordian">
+                    <span class="icon">
+                      <div class="text-center">
+                        <i class="fa fa-spinner fa-spin me-2"/>
                       </div>
-                  <span class="icon">+</span>
+                    </span>
+                  </t>
+                  <t t-else="">
+                    <span class="icon">
+                      <svg width="18" height="18" viewBox="0 0 18 18" fill="none" xmlns="http://www.w3.org/2000/svg">
+                        <path d="M9 1.5V16.5M16.5 9H1.5" stroke="black" stroke-width="1.5" stroke-linecap="round" stroke-linejoin="round"/>
+                      </svg>
+                    </span>
+                  </t>
                 </button>
                 <t t-if="state.selectedModule === module[0] and state.expanded_accordian">
                   <t t-if="state.loading_accordian">
-                    <div class="text-center">
-                      <i class="fa fa-spinner fa-spin me-2"/> Loading...
+                    <div class="text-center mt-3">
+                      <i class="fa fa-spinner fa-spin me-2"/> Analysing the module...
                     </div>
                   </t>
                   <t t-else="">
-                    <t t-if="state.module_selected">
+                    <div t-if="state.module_selected" class="hr-accordion-content show">
+
                       <!--  Odoo Code Standards  -->
-                      <div class="mt-4">
-                        <h3>Odoo Code Standards Check for: <t t-esc="module[1][0]"/></h3>
-                        <table t-if="state.module_selected.odoo_standards_check" class="table table-bordered">
+                      <h3 class="mt-4">Odoo Code Standards Check for: <t t-esc="module[1][0]"/></h3>
+                      <div class="table-container">
+                        <table t-if="state.module_selected.odoo_standards_check">
                           <thead>
                             <tr>
                               <th>File</th>
@@ -209,9 +211,9 @@
                       </div>
 
                       <!--  Style &amp; Linting Check  -->
-                      <div class="mt-4">
-                        <h3>Style &amp; Linting Check for: <t t-esc="module[1][0]"/></h3>
-                        <table t-if="state.module_selected.style_lint_check" class="table table-bordered">
+                      <h3 class="mt-4">Style &amp; Linting Check for: <t t-esc="module[1][0]"/></h3>
+                      <div class="table-container">
+                        <table t-if="state.module_selected.style_lint_check">
                           <thead>
                             <tr>
                               <th>File Name</th>
@@ -237,9 +239,9 @@
                       </div>
 
                       <!--  General Code Quality  -->
-                      <div class="mt-4">
-                        <h3>General Code Quality for: <t t-esc="module[1][0]"/></h3>
-                        <table t-if="state.module_selected.code_quality_check" class="table table-bordered">
+                      <h3 class="mt-4">General Code Quality for: <t t-esc="module[1][0]"/></h3>
+                      <div class="table-container">
+                        <table t-if="state.module_selected.code_quality_check">
                           <thead>
                             <tr>
                               <th>File Name</th>
@@ -268,10 +270,44 @@
                         </t>
                       </div>
 
+                      <!--  Security scan  -->
+                      <h3 class="mt-4">Security Scan for: <t t-esc="module[1][0]"/></h3>
+                      <div class="table-container">
+                        <table t-if="state.module_selected.security_scan.length >= 1">
+                          <thead>
+                            <tr>
+                              <th>Test ID</th>
+                              <th>Test Name</th>
+                              <th>Severity</th>
+                              <th>Confidence</th>
+                              <th>Issue Text</th>
+                              <th>File</th>
+                              <th>Line</th>
+                            </tr>
+                          </thead>
+                          <tbody>
+                            <t t-foreach="state.module_selected.security_scan" t-as="selected" t-key="selected_index">
+                              <tr>
+                                <td t-esc="selected.test_id"/>
+                                <td t-esc="selected.test_name"/>
+                                <td t-esc="selected.issue_severity"/>
+                                <td t-esc="selected.issue_confidence"/>
+                                <td t-esc="selected.issue_text"/>
+                                <td t-esc="selected.filename"/>
+                                <td t-esc="selected.line_number"/>
+                              </tr>
+                            </t>
+                          </tbody>
+                        </table>
+                        <t t-else="">
+                          No issues found
+                        </t>
+                      </div>
+
                       <!--  Maintainability Index  -->
-                      <div class="mt-4">
-                        <h3>Maintainability Index for: <t t-esc="module[1][0]"/></h3>
-                        <table t-if="state.module_selected.mi_check" class="table table-bordered">
+                      <h3 class="mt-4">Maintainability Index for: <t t-esc="module[1][0]"/></h3>
+                      <div class="table-container">
+                        <table t-if="state.module_selected.mi_check">
                           <thead>
                             <tr>
                               <th>File Name</th>
@@ -293,9 +329,9 @@
                       </div>
 
                       <!--  Import Sorting  -->
-                      <div class="mt-4">
-                        <h3>Import Sorting Check for: <t t-esc="module[1][0]"/></h3>
-                        <table t-if="state.module_selected.import_sort_check" class="table table-bordered">
+                      <h3 class="mt-4">Import Sorting Check for: <t t-esc="module[1][0]"/></h3>
+                      <div class="table-container">
+                        <table t-if="state.module_selected.import_sort_check">
                           <thead>
                             <tr>
                               <th>File Name</th>
@@ -317,9 +353,9 @@
                       </div>
 
                       <!--  Coding Format  -->
-                      <div class="mt-4">
-                        <h3>Code Formatting Check for: <t t-esc="module[1][0]"/></h3>
-                        <table t-if="state.module_selected.code_format_check" class="table table-bordered">
+                      <h3 class="mt-4">Code Formatting Check for: <t t-esc="module[1][0]"/></h3>
+                      <div class="table-container">
+                        <table t-if="state.module_selected.code_format_check">
                           <thead>
                             <tr>
                               <th>File Name</th>
@@ -341,12 +377,12 @@
                       </div>
 
                       <!--  Code Complexity  -->
-                      <div class="mt-4">
                       <t t-set="cc_colors" t-value="{'A': 'text-success', 'B': 'text-primary', 'C': 'text-warning', 'D': 'text-orange', 'E': 'text-danger', 'F': 'text-dark-red'}"/>
-                        <h3>Code Complexity for: <t t-esc="module[1][0]"/></h3>
+                      <h3 class="mt-4">Code Complexity for: <t t-esc="module[1][0]"/></h3>
                       <t t-foreach="Object.entries(state.module_selected.cc_check)" t-as="cc_files" t-key="cc_files_index">
-                          <h5>File: <t t-esc="cc_files[0]"/></h5>
-                          <table t-if="cc_files[1]" class="table table-bordered">
+                        <h5 style="color: grey;">File: <t t-esc="cc_files[0]"/></h5>
+                        <div class="table-container mb-4">
+                          <table t-if="cc_files[1]">
                             <thead>
                               <tr>
                                 <th>Type</th>
@@ -375,45 +411,11 @@
                           <t t-else="">
                             No issues found
                           </t>
-                        </t>
                         </div>
-
-                      <!--  Security scan  -->
-                      <div class="mt-4">
-                        <h3>Security Scan for: <t t-esc="module[1][0]"/></h3>
-                        <table t-if="state.module_selected.security_scan.length >= 1" class="table table-bordered">
-                          <thead>
-                            <tr>
-                              <th>Test ID</th>
-                              <th>Test Name</th>
-                              <th>Severity</th>
-                              <th>Confidence</th>
-                              <th>Issue Text</th>
-                              <th>File</th>
-                              <th>Line</th>
-                            </tr>
-                          </thead>
-                          <tbody>
-                            <t t-foreach="state.module_selected.security_scan" t-as="selected" t-key="selected_index">
-                              <tr>
-                                <td t-esc="selected.test_id"/>
-                                <td t-esc="selected.test_name"/>
-                                <td t-esc="selected.issue_severity"/>
-                                <td t-esc="selected.issue_confidence"/>
-                                <td t-esc="selected.issue_text"/>
-                                <td t-esc="selected.filename"/>
-                                <td t-esc="selected.line_number"/>
-                              </tr>
-                            </t>
-                          </tbody>
-                        </table>
-                        <t t-else="">
-                          No issues found
                       </t>
                     </div>
                   </t>
                 </t>
-                </t>
               </div>
             </t>
           </t>