[FIX] estate: Error fixed

Author: niyp-odoo

estate/__manifest__.py

@@ -1,18 +1,18 @@
 {
     'name': 'Estate',
     "version": "1.0",
+    'application': True,
+    'installable': True,
+    'auto_install': False,
     "category": "Real Estate/Brokerage",
     'data': [
-        'security/ir.model.access.csv',
         'security/security.xml',
+        'security/ir.model.access.csv',
         'views/estate_property_views.xml',
         'views/estate_property_type_views.xml',
         'views/estate_property_tag_views.xml',
-        'views/res_users_views.xml',
+        'views/inherited_model_view.xml',
         'views/estate_menus.xml',
     ],
-    'installable': True,
-    'application': True,
-    'auto_install': False,
     'license': 'LGPL-3',
 }

estate/models/__init__.py

@@ -2,4 +2,4 @@
 from . import estate_property_type
 from . import estate_property_tag
 from . import estate_property_offer
-from . import res_user
+from . import inherited_model

estate/models/estate_property.py

@@ -1,7 +1,7 @@
 from datetime import date
 from dateutil.relativedelta import relativedelta
-from odoo import fields, models, api # type: ignore
-from odoo.exceptions import UserError # type: ignore
+from odoo import fields, models, api
+from odoo.exceptions import UserError
 
 
 class EstateProperty(models.Model):

estate/models/estate_property_offer.py

@@ -1,7 +1,7 @@
 from datetime import timedelta
-from odoo import api, fields, models # type: ignore
-from odoo.exceptions import UserError, ValidationError # type: ignore
-from odoo.tools import float_compare # type: ignore
+from odoo import api, fields, models
+from odoo.exceptions import UserError, ValidationError
+from odoo.tools import float_compare
 
 
 class EstatePropertyOffer(models.Model):

estate/models/estate_property_tag.py

@@ -1,4 +1,4 @@
-from odoo import fields, models # type: ignore
+from odoo import fields, models
 
 
 class EstatePropertyTag(models.Model):

estate/models/estate_property_type.py

@@ -1,4 +1,4 @@
-from odoo import fields, models, api # type: ignore
+from odoo import fields, models, api
 
 
 class EstatePropertyType(models.Model):

estate/models/res_user.py renamed to estate/models/inherited_model.py

@@ -1,4 +1,4 @@
-from odoo import models, fields # type: ignore
+from odoo import models, fields
 
 
 class InheritedModel(models.Model):

estate/security/security.xml

@@ -1,16 +1,21 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
     <record id="estate_group_user" model="res.groups">
-            <field name="name">Agent</field>
-            <field name="category_id" ref="base.module_category_real_estate_brokerage"/>
+        <field name="name">Agent</field>
+        <field name="comment">Real estate agents can manage the properties under their care, or properties which are not specifically under the care of any agent.</field>
+        <field name="category_id" ref="base.module_category_real_estate_brokerage"/>
     </record>
+
     <record id="estate_group_manager" model="res.groups">
         <field name="name">Manager</field>
+        <field name="comment">Real estate managers can configure the system (manage available types and tags) as well as oversee every property in the pipeline.</field>
         <field name="category_id" ref="base.module_category_real_estate_brokerage"/>
-        <field name="implied_ids" eval="[(4,ref('estate_group_user'))]" />
+        <field name="implied_ids" eval="[(4, ref('estate_group_user'))]"/>
     </record>
+
     <record id="rule_id" model="ir.rule">
-        <field name="name">Agents: see or modify properties with no salesperson or themselves as salesperson</field>
+        <field name="name">Agents: see or modify properties with no salesman or themselves as
+            salesman</field>
         <field name="model_id" ref="model_estate_property" />
         <field name="perm_read" eval="True" />
         <field name="perm_write" eval="True" />

estate/views/res_users_views.xml renamed to estate/views/inherited_model_view.xml

@@ -1,11 +1,11 @@
 {
     'name': 'Estate Account',
-    'depends': ['base','estate', 'account'],
-    'data': [
-        'security/ir.model.access.csv',
-    ],
     'installable': True,
     'application': True,
     'auto_install': False,
+    'depends': ['estate', 'account'],
+    'data': [
+        'security/ir.model.access.csv',
+    ],
     'license': 'LGPL-3',
 }