Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

My First Odoo Application #70

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

My First Odoo Application #70

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b1ef1d8
My First Commit
ncarpio Sep 30, 2024
e86ea8b
My First Odoo Application
ncarpio Oct 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion estate/__manifest__.py
View file
Original file line number Diff line number Diff line change
@@ -4,7 +4,14 @@
"application": True, # This line says the module is an App, and not a module
"depends": ["base"], # dependencies
"data": [

"security/ir.model.access.csv",

"views/estate_property_views.xml",
"views/property_type_views.xml",
"views/property_tag_views.xml",
"views/offers_view.xml",

"actions/estate_property_menus.xml",
],
"installable": True,
'license': 'LGPL-3',
10 changes: 10 additions & 0 deletions estate/actions/estate_property_menus.xml
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
<?xml version="1.0"?>
<odoo>
<menuitem id="estate_menu_root" name="Real Estate">
<menuitem id="estate_property_menu_action" name="Properties" action="estate_property_action"/>
<menuitem id="estate_property_settings" name="Settings">
<menuitem id="estate_property_type_menu_action" name="Property Types" action="property_type_action"/>
<menuitem id="estate_property_tag_menu_action" name="Property Tags" action="property_tag_action"/>
</menuitem>
</menuitem>
</odoo>
Empty file removed estate/models.py
View file
Empty file.
4 changes: 4 additions & 0 deletions estate/models/__init__.py
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
from . import estate_property
from . import property_type
from . import property_tag
from . import offer
98 changes: 98 additions & 0 deletions estate/models/estate_property.py
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,98 @@
from odoo import _, api, fields, models
from odoo.exceptions import UserError, ValidationError
from odoo.tools.float_utils import float_is_zero, float_compare

class EstateProperty(models.Model):
_name = "estate.property"
_description = "Estate Property"

name = fields.Char(required=True)
property_tag_ids = fields.Many2many("estate.property.tag", string="Tags")

description = fields.Text()
address = fields.Text()
postcode = fields.Char()
date_availability = fields.Date(copy=False, default=fields.Date.add(fields.Date.today(), months=3))
expected_price = fields.Float()
selling_price = fields.Float(readonly=False, copy=False)
bedrooms = fields.Integer(default=2)
living_area = fields.Float(string="Living Area (sqm)")
garden = fields.Boolean()
garden_area = fields.Float(string="Garden Area (sqm)")
garden_orientation = fields.Selection([("north", "North"), ("south", "South"), ("east", "East"), ("west", "West")])

total_area = fields.Float(string="Total Area (sqm)", readonly=True, compute="_compute_total_area", store=True)
best_price = fields.Float(readonly=True,compute="_compute_best_price")

active = fields.Boolean(default=True)
state = fields.Selection(
selection=[
("new", "New"),
("offer_received", "Offer received"),
("offer_accepted", "Offer accepted"),
("sold", "Sold"),
("canceled", "Canceled"),
],
default="new",
copy=False,
required=True,
)

property_type_id = fields.Many2one("estate.property.type", string="Property Type")

salesman_id = fields.Many2one("res.users", string="Salesman")
buyer_id = fields.Many2one("res.partner", string="Partner")

offer_ids = fields.One2many("estate.property.offer", "property_id", string="Offers")

_sql_constraints = [
('check_expected_price', 'CHECK(expected_price >= 0)', 'The expected price must me be at least 100.')
]

@api.onchange("garden")
def _onchange_garden(self):
if self.garden:
self.garden_area = 10
self.garden_orientation = "north"
else:
self.garden_area = self.garden_orientation = False

@api.depends("living_area", "garden_area")
def _compute_total_area(self):
self.total_area = self.living_area + self.garden_area

@api.depends("offer_ids.price")
def _compute_best_price(self):
for property in self:
if property.offer_ids:
property.best_price = max(property.offer_ids.mapped("price"))
else:
property.best_price = 0

def action_sell_property(self):
for property in self:
if property.state == "cancelled":
raise UserError(_("Cancelled properties cannot be sold!"))
property.state = "sold"

Comment on lines +73 to +77
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was the solution of the slides. I suggest that you check the filtered function from Odoo to see how you could change this code with it :D

def action_cancel_property(self):
self.state = "canceled"

@api.constrains("selling_price", "expected_price")
def _check_selling_price(self):
for property in self:
if (not float_is_zero(property.selling_price, precision_rounding=0.01) and
float_compare(property.selling_price, 0.9 * property.expected_price, precision_rounding=0.01) < 0
):
raise ValidationError(_("The selling price should not be lower than 90% of the expected price!"))

@api.ondelete(at_uninstall=False)
def _unlink_if_new_or_canceled(self):
for property in self:
if property.state not in ("new", "canceled"):
raise UserError(_("Only new or canceled property can be deleted!"))
Comment on lines +90 to +93
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The same here, you can check the filtered function :)


@api.model_create_multi
def create(self, vals_list):
for vals in vals_list:
property = self.env["estate.property"]
Comment on lines +96 to +98
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not finished function ?

41 changes: 41 additions & 0 deletions estate/models/offer.py
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
from odoo import api, fields, models

class Offer(models.Model):
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Usually we uses the same name as the name of the model so here EstatePropertyOffer but it is more of a preference

_name = "estate.property.offer"
_description = "Offers"

price = fields.Float(required=True)
buyer_id = fields.Many2one("res.partner", string="Partner")
validity = fields.Integer(default=7, required=True, string="Validity (days)")
deadline = fields.Date(compute="_compute_deadline", inverse="_inverse_deadline", string="Deadline Date")
status = fields.Selection(
selection=[
("new", "New"),
("accepted", "Accepted"),
("refused", "Refused"),
],
default="new",
copy=False,
required=True,
)

property_id = fields.Many2one("estate.property", required=True)

@api.depends("validity", "create_date")
def _compute_deadline(self):
for estate in self:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would more appropriate to call your record with the model name

Suggested change
for estate in self:
for offer in self:

create_date = estate.create_date or fields.Date.today()
estate.deadline = fields.Date.add(create_date, days=estate.validity)

def _inverse_deadline(self):
for estate in self:
estate.validity = (estate.deadline - fields.Date.to_date(estate.create_date)).days

def action_accept_offer(self):
self.status = "accepted"
for offer in self:
offer.property_id.selling_price = offer.price

def action_refuse_offer(self):
self.status = "refused"

9 changes: 9 additions & 0 deletions estate/models/property_tag.py
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
from odoo import fields, models

class PropertyTag(models.Model):
_name = "estate.property.tag"
_description = "Property Tag"

name = fields.Char(required=True)
description = fields.Text()
active = fields.Boolean(default=True)
9 changes: 9 additions & 0 deletions estate/models/property_type.py
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
from odoo import fields, models

class PropertyType(models.Model):
_name = "estate.property.type"
_description = "Property Type"

name = fields.Char(required=True)
description = fields.Text()
active = fields.Boolean(default=True)
5 changes: 5 additions & 0 deletions estate/security/ir.model.access.csv
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
access_estate_property,access_estate_property,model_estate_property,base.group_user,1,1,1,1
access_property_type,access_property_type,model_estate_property_type,base.group_user,1,1,1,1
access_property_tag,access_property_tag,model_estate_property_tag,base.group_user,1,1,1,1
access_offers,access_offers,model_estate_property_offer,base.group_user,1,1,1,1
123 changes: 123 additions & 0 deletions estate/views/estate_property_views.xml
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,123 @@
<?xml version="1.0"?>
<odoo>
<record id="estate_property_action" model="ir.actions.act_window">
<field name="name">Real Estate</field>
<field name="res_model">estate.property</field>
<field name="view_mode">list,form</field>
</record>

<record id="view_estate_property_list" model="ir.ui.view">
<field name="name">estate.property.list</field>
<field name="model">estate.property</field>
<field name="arch" type="xml">
<list string="Properties">
<field name="name"/>
<field name="property_type_id"/>
<field name="address"/>
<field name="postcode"/>
<field name="date_availability"/>
<field name="bedrooms"/>
<field name="expected_price"/>
<field name="state"/>
<field name="active"/>
</list>
</field>
</record>

<record id="view_estate_property_form" model="ir.ui.view">
<field name="name">estate.property.form</field>
<field name="model">estate.property</field>
<field name="arch" type="xml">
<form string="New Property">
<header>
<button name="action_sell_property" string="Sold" type="object" icon="fa-check"/>
<button name="action_cancel_property" string="Cancel" type="object" icon="fa-ban"/>
</header>
<sheet>
<group>
<h1 class="mb32">
<field name="name" class="mb16"/>
</h1>
</group>
<group>
<group>
<field name="active"/>
<field name="property_type_id"/>
<field name="property_tag_ids" widget="many2many_tags"/>
</group>

<group>
<field name="date_availability"/>
<field name="expected_price"/>
<field name="selling_price"/>
<field name="best_price"/>
</group>

<group>
<field name="state"/>
</group>
</group>
<notebook>
<page string="Descriptions">
<group>
<group>
<field name="description"/>
</group>

<group>
<field name="address"/>
<field name="postcode"/>
</group>

<group>
<field name="bedrooms"/>
<field name="living_area"/>

<field name="garden"/>
<field name="garden_orientation"/>
<field name="garden_area"/>

<field name="total_area"/>
</group>
</group>
</page>

<page string="Other Information">
<group>
<group>
<field name="salesman_id"/>
<field name="buyer_id"/>
</group>
</group>
</page>

<page string="Offers">
<field name="offer_ids"/>
</page>

</notebook>
</sheet>
</form>
</field>
</record>

<record id="view_estate_property_search" model="ir.ui.view">
<field name="name">estate.property.search</field>
<field name="model">estate.property</field>
<field name="arch" type="xml">
<search string="Estate Property">
<field name="name" string="Estate" />
<field name="bedrooms"/>
<field name="property_type_id"/>
<separator/>
<filter string="Active" name="active" domain="[('active', '=', True)]"/>
<filter string="Inactive" name="inactive" domain="[('active', '=', False)]"/>
<separator/>
<filter string="Available" name="available" domain="[('state', 'in', ['new', 'offer_received'])]"/>
<group expand="1" string="Group By">
<filter string="Postcode" name="postcode" context="{'group_by':'postcode'}"/>
</group>
</search>
</field>
</record>
</odoo>
57 changes: 57 additions & 0 deletions estate/views/offers_view.xml
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
<?xml version="1.0"?>
<odoo>
<record id="estate_property_offer_action" model="ir.actions.act_window">
<field name="name">Offers</field>
<field name="res_model">estate.property.offer</field>
<field name="view_mode">list,form</field>
</record>

<record id="view_estate_property_offer_list" model="ir.ui.view">
<field name="name">estate.property.offer.list</field>
<field name="model">estate.property.offer</field>
<field name="arch" type="xml">
<list string="Properties" >
<field name="price"/>
<field name="buyer_id"/>
<field name="validity"/>
<field name="deadline"/>
<field name="status"/>
<button name="action_accept_offer" string="Accept" type="object" icon="fa-check"/>
<button name="action_refuse_offer" string="Refuse" type="object" icon="fa-ban"/>
</list>
</field>
</record>

<record id="view_estate_property_offer_form" model="ir.ui.view">
<field name="name">estate.property.offer.form</field>
<field name="model">estate.property.offer</field>
<field name="arch" type="xml">
<form string="New Offer">
<sheet>
<group>
<group>
<field name="price"/>
<field name="buyer_id"/>
<field name="validity"/>
<field name="deadline"/>
<field name="status"/>
</group>
</group>
</sheet>
</form>
</field>
</record>

<record id="view_estate_property_offer_search" model="ir.ui.view">
<field name="name">estate.property.offer.search</field>
<field name="model">estate.property.offer</field>
<field name="arch" type="xml">
<search string="Estate Property Offer">
<field name="price" string="Offers" />
<field name="buyer_id"/>
<separator/>
<filter string="Status" name="status" domain="[('status', 'in', ['new', 'received'])]"/>
</search>
</field>
</record>
</odoo>
Loading