[FIX] warnings and style issues

fixed issue in the property type action button in the form view, the manifist data order was incorrect

fixed the warning of batch create in th property offer,

fixed some code style issues in the code

Author: A-Yehia19

awesome_owl/static/src/card/card.xml

@@ -11,5 +11,5 @@
         </div>
 
     </t>
-    
+
 </templates>

awesome_owl/static/src/counter/counter.xml

@@ -2,10 +2,12 @@
 <templates xml:space="preserve">
 
     <t t-name="awesome_owl.counter">
-        <div style="border: 1px solid #ccc; padding: 15px; margin: 10px; width: fit-content; height:fit-content; display: inline">
+        <div style="border: 1px solid #ccc; padding: 15px; margin: 10px; width: fit-content; display: inline">
             <label style="margin:10px">Counter: <t t-esc="count.value"/></label>
-            <button class="btn" t-on-click="increment" style="background-color: purple; color: white">Increment</button>
+            <button class="btn" t-on-click="increment" style="background-color: purple; color: white">
+                Increment
+            </button>
         </div>
     </t>
-    
+
 </templates>

awesome_owl/static/src/playground.js

@@ -1,8 +1,8 @@
 /** @odoo-module **/
 
 import { Component, useState } from "@odoo/owl";
-import { Counter } from "./counter/counter";
 import { Card } from "./card/card";
+import { Counter } from "./counter/counter";
 import { TodoList } from "./todo_list/todo_list";
 
 export class Playground extends Component {
@@ -14,7 +14,7 @@ export class Playground extends Component {
     setup() {
         this.content1 = "<u>Card Content 1</u>";
         this.content2 = "<u>Card Content 2</u>";
-        this.state = useState({ 
+        this.state = useState({
             sum: 0,
         });
     }

awesome_owl/static/src/playground.xml

@@ -26,5 +26,5 @@
         </div>
 
     </t>
-    
+
 </templates>

awesome_owl/static/src/todo_list/todo_item/todo_item.xml

@@ -2,9 +2,11 @@
 <templates xml:space="preserve">
 
     <t t-name="awesome_owl.todo_item">
-        <p class="card-text" t-att-class="props.todo.isCompleted ? 'text-decoration-line-through text-muted' : '' ">
-            <t t-esc="props.todo.id"/>. <t t-esc="props.todo.description"/>
+        <p 
+            class="card-text" 
+            t-att-class="props.todo.isCompleted ? 'text-decoration-line-through text-muted' : '' ">
+                <t t-esc="props.todo.id"/>. <t t-esc="props.todo.description"/>
         </p>
     </t>
-    
+
 </templates>

awesome_owl/static/src/todo_list/todo_list.js

@@ -22,7 +22,7 @@ export class TodoList extends Component {
         },
     };
 
-    addTodo(ev){
+    addTodo(ev) {
         let input_val = ev.target.value;
         if (ev.keyCode === 13 && input_val != "") {
             this.todos.push({

estate/__manifest__.py

@@ -10,9 +10,9 @@
     'data': [
         'security/ir.model.access.csv',
         'views/estate_property_views.xml',
-        'views/estate_property_type_views.xml',
         'views/estate_property_tag_views.xml',
         'views/estate_property_type_offer.xml',
+        'views/estate_property_type_views.xml',
         'views/estate_user_views.xml',
         'views/estate_menus.xml',
     ],

estate/models/estate_property_offer.py

@@ -56,11 +56,12 @@ def action_refuse_offer(self):
             record.status = 'refused'
         return True
 
-    @api.model
+    @api.model_create_multi
     def create(self, vals):
-        if vals['price'] <= self.env['estate.property'].browse(vals['property_id']).best_price:
-            raise UserError(self.env._('The offer price must be greater than the best offer.'))
+        for val in vals:
+            if val['price'] <= self.env['estate.property'].browse(val['property_id']).best_price:
+                raise UserError(self.env._('The offer price must be greater than the best offer.'))
 
-        if self.env['estate.property'].browse(vals['property_id']).state == 'new':
-            self.env['estate.property'].browse(vals['property_id']).state = 'offer_received'
-        return super(TestModel, self).create(vals)
+            if self.env['estate.property'].browse(val['property_id']).state == 'new':
+                self.env['estate.property'].browse(val['property_id']).state = 'offer_received'
+        return super().create(vals)