Skip to content

Commit cbab6e7

Browse files
juancarlosonate-tecnativajuancarlosonate-tecnativa
committed
[MIG] partner_delivery_zone: Migration to 17.0
TT52339
1 parent 516944a commit cbab6e7

11 files changed

+19
-47
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

+6-6
Original file line numberDiff line numberDiff line change
@@ -2,19 +2,19 @@
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.tests import tagged
66

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

99

10-
class TestPartnerDeliveryZone(TransactionCase):
10+
@tagged("post_install", "-at_install")
11+
class TestPartnerDeliveryZone(BaseCommon):
1112
at_install = False
1213
post_install = True
1314

1415
@classmethod
1516
def setUpClass(cls):
1617
super().setUpClass()
17-
cls.env = cls.env(context=dict(cls.env.context, **DISABLED_MAIL_CONTEXT))
1818
cls.warehouse = cls.env.ref("stock.warehouse0")
1919
cls.delivery_zone_a = cls.env["partner.delivery.zone"].create(
2020
{"name": "Delivery Zone A", "code": "10"}
@@ -87,11 +87,11 @@ def test_default_line_discount_value(self):
8787
"type": "form",
8888
"model": "res.partner",
8989
"arch": """
90-
<data>
90+
<form>
9191
<field name='child_ids'
9292
context="{'default_name': 'test'}">
9393
</field>
94-
</data>
94+
</form>
9595
""",
9696
}
9797
)

‎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)