[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

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)