changeset 515d978c7f4e in modules/purchase:default
details: https://hg.tryton.org/modules/purchase?cmd=changeset;node=515d978c7f4e
description:
Rename purchase state from cancel to cancelled
issue8927
review305681002
diffstat:
CHANGELOG | 1 +
party.py | 2 +-
party.xml | 2 +-
purchase.py | 47 +++++++++++++++++++++++++++--------------------
4 files changed, 30 insertions(+), 22 deletions(-)
diffs (200 lines):
diff -r 4923c4ebd179 -r 515d978c7f4e CHANGELOG
--- a/CHANGELOG Mon Jun 01 10:08:02 2020 +0100
+++ b/CHANGELOG Sat Jun 06 10:14:13 2020 +0100
@@ -1,3 +1,4 @@
+* Rename purchase state from cancel to cancelled
* Add window tab for exception states
Version 5.6.0 - 2020-05-04
diff -r 4923c4ebd179 -r 515d978c7f4e party.py
--- a/party.py Mon Jun 01 10:08:02 2020 +0100
+++ b/party.py Sat Jun 06 10:14:13 2020 +0100
@@ -56,7 +56,7 @@
purchases = Purchase.search([
('party', '=', party.id),
('company', '=', company.id),
- ('state', 'not in', ['done', 'cancel']),
+ ('state', 'not in', ['done', 'cancelled']),
])
if purchases:
raise EraseError(
diff -r 4923c4ebd179 -r 515d978c7f4e party.xml
--- a/party.xml Mon Jun 01 10:08:02 2020 +0100
+++ b/party.xml Sat Jun 06 10:14:13 2020 +0100
@@ -14,7 +14,7 @@
<field name="domain"
eval="[If(Eval('active_ids', []) == [Eval('active_id')],
('party', '=', Eval('active_id')), ('party', 'in', Eval('active_ids')))]"
pyson="1"/>
- <field name="search_value" eval="[('state', 'not in', ['done',
'cancel'])]" pyson="1"/>
+ <field name="search_value" eval="[('state', 'not in', ['done',
'cancelled'])]" pyson="1"/>
</record>
<record model="ir.action.keyword"
id="act_open_purchase_keyword1">
diff -r 4923c4ebd179 -r 515d978c7f4e purchase.py
--- a/purchase.py Mon Jun 01 10:08:02 2020 +0100
+++ b/purchase.py Sat Jun 06 10:14:13 2020 +0100
@@ -89,7 +89,8 @@
purchase_date = fields.Date('Purchase Date',
states={
'readonly': ~Eval('state').in_(['draft', 'quotation']),
- 'required': ~Eval('state').in_(['draft', 'quotation', 'cancel']),
+ 'required': ~Eval('state').in_(
+ ['draft', 'quotation', 'cancelled']),
},
depends=['state'])
payment_term = fields.Many2One('account.invoice.payment_term',
@@ -127,7 +128,8 @@
],
states={
'readonly': Eval('state') != 'draft',
- 'required': ~Eval('state').in_(['draft', 'quotation', 'cancel']),
+ 'required': ~Eval('state').in_(
+ ['draft', 'quotation', 'cancelled']),
},
depends=['party', 'invoice_party', 'state'])
warehouse = fields.Many2One('stock.location', 'Warehouse',
@@ -187,7 +189,7 @@
"Delivery Date",
states={
'readonly': Eval('state').in_([
- 'processing', 'done', 'cancel']),
+ 'processing', 'done', 'cancelled']),
},
depends=['state'],
help="The default delivery date for each line.")
@@ -207,17 +209,17 @@
quoted_by = employee_field(
"Quoted By",
- states=['quotation', 'confirmed', 'processing', 'done', 'cancel'])
+ states=['quotation', 'confirmed', 'processing', 'done', 'cancelled'])
confirmed_by = employee_field(
"Confirmed By",
- states=['confirmed', 'processing', 'done', 'cancel'])
+ states=['confirmed', 'processing', 'done', 'cancelled'])
state = fields.Selection([
('draft', "Draft"),
('quotation', "Quotation"),
('confirmed', "Confirmed"),
('processing', "Processing"),
('done', "Done"),
- ('cancel', "Canceled"),
+ ('cancelled', "Cancelled"),
], "State", readonly=True, required=True)
del _states, _depends
@@ -237,10 +239,10 @@
('processing', 'processing'),
('processing', 'done'),
('done', 'processing'),
- ('draft', 'cancel'),
- ('quotation', 'cancel'),
+ ('draft', 'cancelled'),
+ ('quotation', 'cancelled'),
('quotation', 'draft'),
- ('cancel', 'draft'),
+ ('cancelled', 'draft'),
))
cls._buttons.update({
'cancel': {
@@ -249,8 +251,8 @@
},
'draft': {
'invisible': ~Eval('state').in_(
- ['cancel', 'quotation', 'confirmed']),
- 'icon': If(Eval('state') == 'cancel', 'tryton-undo',
+ ['cancelled', 'quotation', 'confirmed']),
+ 'icon': If(Eval('state') == 'cancelled', 'tryton-undo',
'tryton-back'),
'depends': ['state'],
},
@@ -280,12 +282,12 @@
},
'handle_invoice_exception': {
'invisible': ((Eval('invoice_state') != 'exception')
- | (Eval('state') == 'cancel')),
+ | (Eval('state') == 'cancelled')),
'depends': ['state', 'invoice_state'],
},
'handle_shipment_exception': {
'invisible': ((Eval('shipment_state') != 'exception')
- | (Eval('state') == 'cancel')),
+ | (Eval('state') == 'cancelled')),
'depends': ['state', 'shipment_state'],
},
'modify_header': {
@@ -295,7 +297,7 @@
},
})
# The states where amounts are cached
- cls._states_cached = ['confirmed', 'done', 'cancel']
+ cls._states_cached = ['confirmed', 'done', 'cancelled']
@classmethod
def __register__(cls, module_name):
@@ -348,6 +350,11 @@
# Add index on create_date
table.index_action('create_date', action='add')
+ # Migration from 5.6: rename state cancel to cancelled
+ cursor.execute(*sql_table.update(
+ [sql_table.state], ['cancelled'],
+ where=sql_table.state == 'cancel'))
+
@classmethod
def default_payment_term(cls):
PaymentTerm = Pool().get('account.invoice.payment_term')
@@ -658,7 +665,7 @@
def view_attributes(cls):
attributes = [
('/form//field[@name="comment"]', 'spell', Eval('party_lang')),
- ('/tree', 'visual', If(Eval('state') == 'cancel', 'muted', '')),
+ ('/tree', 'visual', If(Eval('state') == 'cancelled', 'muted', '')),
('/tree/field[@name="invoice_state"]', 'visual',
If(Eval('invoice_state') == 'exception', 'danger', '')),
('/tree/field[@name="shipment_state"]', 'visual',
@@ -846,7 +853,7 @@
# Cancel before delete
cls.cancel(purchases)
for purchase in purchases:
- if purchase.state != 'cancel':
+ if purchase.state != 'cancelled':
raise AccessError(
gettext('purchase.msg_purchase_delete_cancel',
purchase=purchase.rec_name))
@@ -854,7 +861,7 @@
@classmethod
@ModelView.button
- @Workflow.transition('cancel')
+ @Workflow.transition('cancelled')
def cancel(cls, purchases):
cls.store_cache(purchases)
@@ -1131,7 +1138,7 @@
states={
'invisible': Eval('type') != 'line',
'readonly': Eval('purchase_state').in_([
- 'processing', 'done', 'cancel']),
+ 'processing', 'done', 'cancelled']),
},
depends=['type', 'purchase_state'],
help="Check to edit the delivery date.")
@@ -1141,7 +1148,7 @@
'invisible': ((Eval('type') != 'line')
| ~Eval('delivery_date_edit', False)),
'readonly': Eval('purchase_state').in_([
- 'processing', 'done', 'cancel']),
+ 'processing', 'done', 'cancelled']),
},
depends=['type', 'delivery_date_edit', 'purchase_state'])
purchase_state = fields.Function(
@@ -1675,7 +1682,7 @@
@classmethod
def delete(cls, lines):
for line in lines:
- if line.purchase_state not in {'cancel', 'draft'}:
+ if line.purchase_state not in {'cancelled', 'draft'}:
raise AccessError(
gettext('purchase.msg_purchase_line_delete_cancel_draft',
line=line.rec_name,
