[FIX] add_pricelist_price: fix styling issue

Fix the styling issue in the m 10000 odel files
odoo · shib-odoo · May 7, 2025 · May 8, 2025 · May 9, 2025 · May 12, 2025
commit 42fcb403926a9572212ad4bc3e35ce8637d1300f
diff --git a/add_pricelist_price/tests/test_book_price.py b/add_pricelist_price/tests/test_book_price.py
@@ -4,38 +4,38 @@
 
 class TestBookPrice(TransactionCase):
 
-    def setup(self):
+    def setUp(self):
         super().setup()
         Product = self.env['product.product']
         PriceList = self.env['product.pricelist']
 
         self.product = Product.create({
-            'name':'Test Book',
-            'list_price':100.0,
+            'name': 'Test Book',
+            'list_price': 100.0,
         })
 
         self.pricelist = PriceList.create({
-            'name':'Test Pricelist',
-            'item_ids':[(Command.create({
-               'applied_on':'0_product_variant',
-               'product_id':self.product.id,
-               'compute_price':'formula',
-               'base':'list_price',
-               'price_discount':20.0
+            'name': 'Test Pricelist',
+            'item_ids': [(Command.create({
+               'applied_on': '0_product_variant',
+               'product_id': self.product.id,
+               'compute_price': 'formula',
+               'base': 'list_price',
+               'price_discount': 20.0
             }))],
         })
         self.partner=self.env['res.partner'].create({
-            'name':'Test Partner',
+            'name': 'Test Partner',
         })
 
     def test_book_price_on_sale_order_line(self):
         sale_order = self.env['sale.order'].create({
-            'partner_id':self.partner.id,
-            'pricelist_id':self.pricelist.id,
-            'order_line':[Command.create({
-                'product_id':self.product.id,
-                'product_uom_qty':1,
-                'price_unit':100.0
+            'partner_id': self.partner.id,
+            'pricelist_id': self.pricelist.id,
+            'order_line': [Command.create({
+                'product_id': self.product.id,
+                'product_uom_qty': 1,
+                'price_unit': 100.0
             })]
         })
 
@@ -50,8 +50,8 @@ def test_book_price_on_sale_order_line(self):
 
     def test_book_price_fallback_to_list_price(self):
         sale_order = self.env['sale.order'].create({
-            'partner_id':self.partner.id,
-            'order_line':[Command.create({
+            'partner_id': self.partner.id,
+            'order_line': [Command.create({
                 'product_id':self.product.id,
                 'product_umo_qty':1,
                 'price_unit':100.0,
@@ -67,25 +67,25 @@ def test_book_price_fallback_to_list_price(self):
 
     def test_book_price_on_customer_invoice_line(self):
         sale_order = self.env['sale.order'].create({
-            'partner_id':self.partner.id,
-            'pricelist_id':self.pricelist.id,
-            'order_line':[Command.create({
+            'partner_id': self.partner.id,
+            'pricelist_id': self.pricelist.id,
+            'order_line': [Command.create({
                 'product_id':self.product.id,
                 'product_uom_qty':2,
                 'price_unit':200.0,
             })]
         })
         sale_order.action_confirm()
         invoice = self.env['account.move'].create({
-            'move_type':'out_invoice',
-            'partner_id':self.partner.id,
-            'invoice_line_ids':[Command.create({
+            'move_type': 'out_invoice',
+            'partner_id': self.partner.id,
+            'invoice_line_ids': [Command.create({
                 'product_id':self.product.id,
-                'invoice_line_ids':[Command.create({
-                    'product_id':self.product.id,
-                    'quantity':2,
-                    'price_unit':200.0,
-                    'sale_line_ids':[(6,0,[sale_order.order_line.id])],
+                'invoice_line_ids': [Command.create({
+                    'product_id': self.product.id,
+                    'quantity': 2,
+                    'price_unit': 200.0,
+                    'sale_line_ids': [(6, 0, [sale_order.order_line.id])],
                 })]
             })]
         })
@@ -100,7 +100,7 @@ def test_book_price_on_customer_invoice_line(self):
 
     def test_book_price_not_show_on_non_out_invoice(self):
         invoice = self.env['account.move'].create({
-            'move_type': 'out_refund',  # Customer credit note
+            'move_type': 'out_refund',
             'partner_id': self.partner.id,
             'invoice_line_ids': [(0, 0, {
                 'product_id': self.product.id,

diff --git a/add_pricelist_price/views/account_move_line.xml b/add_pricelist_price/views/account_move_line.xml
@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
-    <!-- Invoice Line View -->
     <record id="view_invoice_form_inherit_book_price" model="ir.ui.view">
         <field name="name">account.move.line.form.book.price</field>
         <field name="model">account.move</field>

diff --git a/add_pricelist_price/views/sale_order_line.xml b/add_pricelist_price/views/sale_order_line.xml
@@ -1,6 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
-    <!-- Sales Order Line View -->
     <record id="view_order_form_inherit_book_price" model="ir.ui.view">
         <field name="name">sale.order.line.form.book.price</field>
         <field name="model">sale.order</field>

diff --git a/awesome_dashboard/static/src/dashboard/components/number_card/number_card.js b/awesome_dashboard/static/src/dashboard/components/number_card/number_card.js
@@ -1,5 +1,3 @@
-/** @odoo-module **/
-
 import { Component } from "@odoo/owl";
 
 export class NumberCard extends Component {

diff --git a/awesome_dashboard/static/src/dashboard/components/pie_chart/pie_chart.js b/awesome_dashboard/static/src/dashboard/components/pie_chart/pie_chart.js
@@ -1,5 +1,3 @@
-/** @odoo-module **/
-
 import { Component, useRef, onWillStart, onMounted, onWillUpdateProps } from '@odoo/owl'
 import { loadJS } from "@web/core/assets";
 

diff --git a/awesome_dashboard/static/src/dashboard/dashboard.js b/awesome_dashboard/static/src/dashboard/dashboard.js
@@ -17,7 +17,7 @@ class AwesomeDashboard extends Component {
         this.statisticsService = useService("awesome_dashboard.statistics");
         this.items = registry.category("awesome_dashboard.items").getAll();
         this.removedItemIds = useState(this.getRemovedItems());
-        this.stats = useState(this.statisticsService.statistics);  // ✅ reactive subscription
+        this.stats = useState(this.statisticsService.statistics); 
     }
     getRemovedItems() {
         return JSON.parse(localStorage.getItem("awesome_dashboard.removed_items") || "[]");

diff --git a/awesome_dashboard/static/src/dashboard/dashboard_items.js b/awesome_dashboard/static/src/dashboard/dashboard_items.js
@@ -1,10 +1,8 @@
-/** @odoo-module **/
-
 import { registry } from "@web/core/registry";
 import { NumberCard } from "./components/number_card/number_card";
 import { PieChart } from "./components/pie_chart/pie_chart";
 import { _t } from "@web/core/l10n/translation";
-// List of items to register
+
 const dashboardItems = [
     {
         id: "nb_new_orders",
@@ -64,8 +62,6 @@ const dashboardItems = [
         props: (data) => ({ data: data.orders_by_size }),
     },
 ];
-
-// Register each item
 for (const item of dashboardItems) {
     registry.category("awesome_dashboard.items").add(item.id, item);
 }
diff --git a/awesome_owl/static/src/components/Todo/todo_item.xml b/awesome_owl/static/src/components/Todo/todo_item.xml
@@ -5,7 +5,6 @@
       class="d-flex align-items-center justify-content-between border rounded px-3 py-2 mb-2 shadow-sm"
       t-att-class="props.todo.isCompleted ? 'bg-light text-muted text-decoration-line-through' : ''"
     >
-      <!-- Left section: Checkbox, ID, Description -->
       <div class="d-flex align-items-center flex-wrap gap-2">
         <input
           type="checkbox"
@@ -16,8 +15,6 @@
         <strong><t t-esc="props.todo.id"/>.</strong>
         <span class="ms-1"><t t-esc="props.todo.description" /></span>
       </div>
-
-      <!-- Right section: Delete icon -->
       <span
         class="fa fa-times text-danger fs-5"
         role="button"

diff --git a/awesome_owl/static/src/components/Todo/todo_list.js b/awesome_owl/static/src/components/Todo/todo_list.js
@@ -36,7 +36,7 @@ export class TodoList extends Component {
   removeTodoItem = (todoId) => {
     const index = this.todos.findIndex((t) => t.id === todoId);
     if (index >= 0) {
-      this.todos.splice(index, 1); // ✅ mutates the reactive state
+      this.todos.splice(index, 1)
     }
   };
 }
diff --git a/awesome_owl/static/src/components/Todo/todo_list.xml b/awesome_owl/static/src/components/Todo/todo_list.xml
@@ -7,7 +7,6 @@
         <span class="badge bg-light text-dark"><t t-esc="todos.length"/> items</span>
       </div>
       <div class="card-body">
-        <!-- Input Field -->
         <div class="mb-3">
           <input
             type="text"
@@ -18,7 +17,6 @@
           />
         </div>
 
-        <!-- Todo Items -->
         <div>
           <t t-if="todos.length > 0">
             <t t-foreach="todos" t-as="todo" t-key="todo.id">

diff --git a/awesome_owl/static/src/components/playground/playground.xml b/awesome_owl/static/src/components/playground/playground.xml
@@ -2,11 +2,8 @@
 <templates xml:space="preserve">
   <t t-name="awesome_owl.playground">
     <div class="container p-4 rounded shadow-sm bg-white">
-      <!-- Playground Header -->
       <h3 class="mb-4 text-primary fw-bold">Counter Playground</h3>
       <hr class="mb-4"/>
-
-      <!-- Card Section -->
       <h4 class="mb-3 text-secondary">Card Examples</h4>
       <div class="row row-cols-1 row-cols-md-2 g-4">
         <div class="col d-flex flex-column gap-3">
@@ -29,8 +26,6 @@
           </Card>
         </div>
       </div>
-
-      <!-- Todo Section -->
       <h3 class="mt-5 mb-3 text-primary">Todo List</h3>
       <TodoList/>
     </div>

diff --git a/estate/views/estate_property_views.xml b/estate/views/estate_property_views.xml
@@ -6,13 +6,12 @@
         <field name="res_model">estate.property</field>
         <field name="view_mode">list,form,kanban</field>
         <field name="context">{"search_default_available": 1}</field>
-
     </record>
     <record id="action_open_properties_page" model="ir.actions.act_url">
         <field name="name">Property Listings (Website)</field>
         <field name="type">ir.actions.act_url</field>
         <field name="url">/properties</field>
-        <field name="target">new</field> <!-- new tab; use 'self' for same tab -->
+        <field name="target">new</field> 
     </record>
     <!-- List View -->
     <record id="estate_property_list" model="ir.ui.view">
@@ -67,8 +66,7 @@
                             <field name="date_availability" optional="show" />
                             <field name="property_type_id"
                                 options="{'no_create': True, 'no_open': True}" />
-                                <field name="tag_ids" string = "Tag" widget="many2many_tags" options="{'color_field': 'color'}"/>
-
+                            <field name="tag_ids" string = "Tag" widget="many2many_tags" options="{'color_field': 'color'}"/> 
                         </group>
                         <group>
                             <field name="expected_price" />