Skip to content

Commit a7da780

Browse files
juancarlosonate-tecnativajuancarlosonate-tecnativa
committedFeb 27, 2025·
[MIG] partner_delivery_zone: Migration to 17.0
TT52339
1 parent 8d8992b commit a7da780

11 files changed

+22
-51
lines changed
 

‎partner_delivery_zone/README.rst

+1
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ Contributors
9696
- Sergio Teruel
9797
- César A. Sánchez
9898
- Luis Daniel Lafaurie
99+
- Juan Carlos Oñate
99100

100101
- `Pesol <https://www.pesol.es>`__
101102

‎partner_delivery_zone/__manifest__.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
{
44
"name": "Partner Delivery Zone",
55
"summary": "Enables partner delivery zones for physical products",
6-
"version": "16.0.1.3.0",
6+
"version": "17.0.1.0.0",
77
"development_status": "Beta",
88
"category": "Delivery",
99
"website": "https://github.com/OCA/delivery-carrier",

‎partner_delivery_zone/models/res_partner.py

-3
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,6 @@ class ResPartner(models.Model):
1919
def get_view(self, view_id=None, view_type="form", **options):
2020
"""The purpose of this is to write a context on "child_ids" field
2121
respecting other contexts on this field.
22-
There is a PR (https://github.com/odoo/odoo/pull/26607) to odoo for
23-
avoiding this. If merged, remove this method and add the attribute
24-
in the field.
2522
"""
2623
res = super().get_view(view_id, view_type, **options)
2724
if view_type == "form":

‎partner_delivery_zone/readme/CONTRIBUTORS.md

+1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
- Sergio Teruel
33
- César A. Sánchez
44
- Luis Daniel Lafaurie
5+
- Juan Carlos Oñate
56
- [Pesol](https://www.pesol.es)
67
- Angel Moya Pardo \<<angel.moya@pesol.es>\>
78
- Antonio J Rubio Lorente \<<antonio.rubio@pesol.es>\>

‎partner_delivery_zone/static/description/index.html

+1
Original file line numberDiff line numberDiff line change
@@ -445,6 +445,7 @@ <h2><a class="toc-backref" href="#toc-entry-7">Contributors</a></h2>
445445
<li>Sergio Teruel</li>
446446
<li>César A. Sánchez</li>
447447
<li>Luis Daniel Lafaurie</li>
448+
<li>Juan Carlos Oñate</li>
448449
</ul>
449450
</li>
450451
<li><a class="reference external" href="https://www.pesol.es">Pesol</a><ul>

‎partner_delivery_zone/tests/test_partner_delivery_zone.py

+9-10
Original file line numberDiff line numberDiff line change
@@ -2,19 +2,20 @@
22
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
33
from lxml import etree
44

5-
from odoo.tests import TransactionCase
5+
from odoo import Command
6+
from odoo.tests import tagged
67

7-
from odoo.addons.base.tests.common import DISABLED_MAIL_CONTEXT
8+
from odoo.addons.base.tests.common import BaseCommon
89

910

10-
class TestPartnerDeliveryZone(TransactionCase):
11+
@tagged("post_install", "-at_install")
12+
class TestPartnerDeliveryZone(BaseCommon):
1113
at_install = False
1214
post_install = True
1315

1416
@classmethod
1517
def setUpClass(cls):
1618
super().setUpClass()
17-
cls.env = cls.env(context=dict(cls.env.context, **DISABLED_MAIL_CONTEXT))
1819
cls.warehouse = cls.env.ref("stock.warehouse0")
1920
cls.delivery_zone_a = cls.env["partner.delivery.zone"].create(
2021
{"name": "Delivery Zone A", "code": "10"}
@@ -30,16 +31,14 @@ def setUpClass(cls):
3031
{
3132
"partner_id": cls.partner.id,
3233
"order_line": [
33-
(
34-
0,
35-
0,
34+
Command.create(
3635
{
3736
"name": cls.product.name,
3837
"product_id": cls.product.id,
3938
"product_uom_qty": 10.0,
4039
"product_uom": cls.product.uom_id.id,
4140
"price_unit": 1000.00,
42-
},
41+
}
4342
)
4443
],
4544
}
@@ -87,11 +86,11 @@ def test_default_line_discount_value(self):
8786
"type": "form",
8887
"model": "res.partner",
8988
"arch": """
90-
<data>
89+
<form>
9190
<field name='child_ids'
9291
context="{'default_name': 'test'}">
9392
</field>
94-
</data>
93+
</form>
9594
""",
9695
}
9796
)

‎partner_delivery_zone/views/partner_delivery_zone_view.xml

+2-16
Original file line numberDiff line numberDiff line change
@@ -2,25 +2,11 @@
22
<!-- Copyright 2018 Tecnativa - Sergio Teruel
33
License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). -->
44
<odoo>
5-
<record id="view_partner_delivery_zone_form" model="ir.ui.view">
6-
<field name="name">partner.delivery.zone.form</field>
7-
<field name="model">partner.delivery.zone</field>
8-
<field name="arch" type="xml">
9-
<form string="Delivery Zone">
10-
<sheet>
11-
<group>
12-
<field name="code" />
13-
<field name="name" />
14-
</group>
15-
</sheet>
16-
</form>
17-
</field>
18-
</record>
195
<record id="view_partner_delivery_zone_tree" model="ir.ui.view">
206
<field name="name">partner.delivery.zone.tree</field>
217
<field name="model">partner.delivery.zone</field>
228
<field name="arch" type="xml">
23-
<tree>
9+
<tree editable="top">
2410
<field name="code" />
2511
<field name="name" />
2612
</tree>
@@ -39,7 +25,7 @@
3925
<record id="partner_delivery_zone_action" model="ir.actions.act_window">
4026
<field name="name">Delivery Zones</field>
4127
<field name="res_model">partner.delivery.zone</field>
42-
<field name="view_mode">tree,form</field>
28+
<field name="view_mode">tree</field>
4329
<field name="view_id" ref="view_partner_delivery_zone_tree" />
4430
<field name="help" type="html">
4531
<p

‎partner_delivery_zone/views/report_shipping.xml

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@
44
<odoo>
55
<template id="report_picking" inherit_id="stock.report_picking">
66
<xpath expr="//div[@name='div_sched_date']" position="after">
7-
<div t-if="o.delivery_zone_id" class="col-auto">
7+
<div t-if="o.delivery_zone_id" class="col-auto col-3 mw-100 mb-2">
88
<strong>Zone</strong>
9-
<span t-field="o.delivery_zone_id" />
9+
<p t-field="o.delivery_zone_id" class="m-0" />
1010
</div>
1111
</xpath>
1212
</template>

‎partner_delivery_zone/views/res_partner_view.xml

+1-4
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,7 @@
1414
expr="//field[@name='child_ids']/form//field[@name='mobile']"
1515
position="after"
1616
>
17-
<field
18-
name="delivery_zone_id"
19-
attrs="{'invisible': [('type','!=', 'delivery')]}"
20-
/>
17+
<field name="delivery_zone_id" invisible="type != 'delivery'" />
2118
</xpath>
2219
</field>
2320
</record>

‎partner_delivery_zone/views/sale_order_view.xml

+2-5
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,7 @@
88
<field name="inherit_id" ref="sale.view_order_form" />
99
<field name="arch" type="xml">
1010
<xpath expr="//field[@name='payment_term_id']" position="after">
11-
<field
12-
name="delivery_zone_id"
13-
attrs="{'readonly':[('state','in', ['done', 'cancel'])]}"
14-
/>
11+
<field name="delivery_zone_id" readonly="state in ['sale', 'cancel']" />
1512
</xpath>
1613
</field>
1714
</record>
@@ -22,7 +19,7 @@
2219
<xpath expr="//field[@name='company_id']" position="before">
2320
<field
2421
name="delivery_zone_id"
25-
attrs="{'readonly':[('state','in', ['done', 'cancel'])]}"
22+
readonly="state in ['sale', 'cancel']"
2623
optional="hidden"
2724
/>
2825
</xpath>

‎partner_delivery_zone/views/stock_picking_view.xml

+2-10
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,7 @@
88
<field name="arch" type="xml">
99
<xpath expr="//page[@name='extra']" position="inside">
1010
<group name="delivery_zone">
11-
<field
12-
name="delivery_zone_id"
13-
attrs="{'readonly': [('state', '=', 'done')]}"
14-
/>
11+
<field name="delivery_zone_id" readonly="state == 'done'" />
1512
</group>
1613
</xpath>
1714
</field>
@@ -42,15 +39,10 @@
4239
<xpath expr="//field[@name='scheduled_date']" position="after">
4340
<field name="delivery_zone_id" />
4441
</xpath>
45-
<xpath expr="//div[hasclass('oe_kanban_bottom_left')]" position="replace">
46-
<div class="oe_kanban_bottom_left">
47-
<div>
48-
<t t-esc="record.partner_id.value" />
49-
</div>
42+
<xpath expr="//div[hasclass('oe_kanban_bottom_left')]" position="inside">
5043
<div t-if="record.delivery_zone_id">
5144
<t t-esc="record.delivery_zone_id.value" />
5245
</div>
53-
</div>
5446
</xpath>
5547
</field>
5648
</record>

0 commit comments

Comments
 (0)
Please sign in to comment.