[FIX] web_tree_dynamic_colored_field: Fix pre-commit

Author: SMaciasOSI

web_tree_dynamic_colored_field/README.rst

@@ -163,7 +163,8 @@ Contributors
 -  Artem Kostyuk <a.kostyuk@mobilunity.com>
 -  Guewen Baconnier <guewen.baconnier@camptocamp.com>
 -  Phuc Tran Thanh <phuc@trobz.com>
--  Sylvain LE GAL <https://twitter.com/legalsylvain>
+-  Sylvain LE GAL
+   <`https://twitter.com/legalsylvain <https://twitter.com/legalsylvain>`__>
 -  Jurgis Pralgauskis <jurgis@versada.eu>
 
 Other credits

web_tree_dynamic_colored_field/static/description/index.html

@@ -8,10 +8,11 @@
 
 /*
 :Author: David Goodger (goodger@python.org)
-:Id: $Id: html4css1.css 8954 2022-01-20 10:10:25Z milde $
+:Id: $Id: html4css1.css 9511 2024-01-13 09:50:07Z milde $
 :Copyright: This stylesheet has been placed in the public domain.
 
 Default cascading style sheet for the HTML output of Docutils.
+Despite the name, some widely supported CSS2 features are used.
 
 See https://docutils.sourceforge.io/docs/howto/html-stylesheets.html for how to
 customize this style sheet.
@@ -274,7 +275,7 @@
   margin-left: 2em ;
   margin-right: 2em }
 
-pre.code .ln { color: grey; } /* line numbers */
+pre.code .ln { color: gray; } /* line numbers */
 pre.code, code { background-color: #eeeeee }
 pre.code .comment, code .comment { color: #5C6576 }
 pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
@@ -300,7 +301,7 @@
 span.pre {
   white-space: pre }
 
-span.problematic {
+span.problematic, pre.problematic {
   color: red }
 
 span.section-subtitle {
@@ -505,7 +506,8 @@ <h1>Contributors</h1>
 <li>Artem Kostyuk &lt;<a class="reference external" href="mailto:a.kostyuk&#64;mobilunity.com">a.kostyuk&#64;mobilunity.com</a>&gt;</li>
 <li>Guewen Baconnier &lt;<a class="reference external" href="mailto:guewen.baconnier&#64;camptocamp.com">guewen.baconnier&#64;camptocamp.com</a>&gt;</li>
 <li>Phuc Tran Thanh &lt;<a class="reference external" href="mailto:phuc&#64;trobz.com">phuc&#64;trobz.com</a>&gt;</li>
-<li>Sylvain LE GAL &lt;<a class="reference external" href="https://twitter.com/legalsylvain">https://twitter.com/legalsylvain</a>&gt;</li>
+<li>Sylvain LE GAL
+&lt;<a class="reference external" href="https://twitter.com/legalsylvain">https://twitter.com/legalsylvain</a>&gt;</li>
 <li>Jurgis Pralgauskis &lt;<a class="reference external" href="mailto:jurgis&#64;versada.eu">jurgis&#64;versada.eu</a>&gt;</li>
 </ul>
 </div>
@@ -520,7 +522,9 @@ <h1>Other credits</h1>
 <div class="section" id="maintainers">
 <h1>Maintainers</h1>
 <p>This module is maintained by the OCA.</p>
-<a class="reference external image-reference" href="https://odoo-community.org"><img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" /></a>
+<a class="reference external image-reference" href="https://odoo-community.org">
+<img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" />
+</a>
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>

web_tree_dynamic_colored_field/static/src/js/list_renderer.esm.js

@@ -1,8 +1,8 @@
 /** @odoo-module **/
 
-import {patch} from "@web/core/utils/patch";
 import {ListRenderer} from "@web/views/list/list_renderer";
 import {evaluateExpr} from "@web/core/py_js/py";
+import {patch} from "@web/core/utils/patch";
 
 patch(ListRenderer.prototype, "web_tree_dynamic_colored_field_list_renderer", {
     /**
@@ -11,36 +11,35 @@ patch(ListRenderer.prototype, "web_tree_dynamic_colored_field_list_renderer", {
      * @returns {String} style code for the html element
      */
     getDynamicColoredStyle(column, record) {
-        let definition
-            expression
-            color
-        let style = ''
-        if (column.options){
-            if (column?.options?.bg_color){
-                definition = column.options.bg_color
+        let definition = false;
+        var expression = false;
+        var color = false;
+        let style = "";
+        var pairList = false;
+        if (column.options) {
+            if (column && column.options && column.options.bg_color) {
+                definition = column.options.bg_color;
                 for (const color_def of definition.split(";")) {
-                    var pairList = color_def.split(":"),
-                    color = pairList[0],
+                    pairList = color_def.split(":");
+                    color = pairList[0];
                     expression = pairList[1] ? pairList[1] : "True";
                     if (evaluateExpr(expression, record.evalContext)) {
                         style += `background-color: ${color} !important;`;
                     }
                 }
             }
-            if (column?.options?.fg_color){
-                definition = column.options.fg_color
+            if (column && column.options && column.options.fg_color) {
+                definition = column.options.fg_color;
                 for (const color_def of definition.split(";")) {
-                    var pairList = color_def.split(":"),
-                    color = pairList[0],
+                    pairList = color_def.split(":");
+                    color = pairList[0];
                     expression = pairList[1] ? pairList[1] : "True";
-                    console.log("expression", expression)
                     if (evaluateExpr(expression, record.evalContext)) {
                         style += `color: ${color} !important`;
                     }
                 }
             }
         }
-        return style
+        return style;
     },
-
 });