fix: fix 'duplicate-code' for pylint
This commit is contained in:
parent
4e7c8d393c
commit
21c037dc25
7 changed files with 27 additions and 4 deletions
|
@ -1,5 +1,4 @@
|
||||||
# -*- mode: conf; -*-
|
# -*- mode: conf; -*-
|
||||||
|
|
||||||
[MESSAGES CONTROL]
|
[MESSAGES CONTROL]
|
||||||
disable=fixme,
|
disable=fixme
|
||||||
duplicate-code,
|
|
||||||
|
|
|
@ -328,7 +328,9 @@ class OrderItemMixin: # pylint: disable=too-few-public-methods
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
class Order(OrderMixin, model.Base): # pylint: disable=too-few-public-methods
|
class Order( # pylint: disable=too-few-public-methods,duplicate-code
|
||||||
|
OrderMixin, model.Base
|
||||||
|
):
|
||||||
"""
|
"""
|
||||||
Represents an :term:`order` for a customer. Each order has one or
|
Represents an :term:`order` for a customer. Each order has one or
|
||||||
more :attr:`items`.
|
more :attr:`items`.
|
||||||
|
|
|
@ -34,7 +34,7 @@ from wuttjamaican.db import model
|
||||||
from sideshow.enum import PendingProductStatus
|
from sideshow.enum import PendingProductStatus
|
||||||
|
|
||||||
|
|
||||||
class ProductMixin:
|
class ProductMixin: # pylint: disable=duplicate-code
|
||||||
"""
|
"""
|
||||||
Base class for product tables. This has shared columns, used by e.g.:
|
Base class for product tables. This has shared columns, used by e.g.:
|
||||||
|
|
||||||
|
|
|
@ -84,6 +84,7 @@ class NewOrderBatchView(BatchMasterView): # pylint: disable=abstract-method
|
||||||
"executed": {"active": True, "verb": "is_null"},
|
"executed": {"active": True, "verb": "is_null"},
|
||||||
}
|
}
|
||||||
|
|
||||||
|
# pylint: disable=duplicate-code
|
||||||
form_fields = [
|
form_fields = [
|
||||||
"id",
|
"id",
|
||||||
"store_id",
|
"store_id",
|
||||||
|
@ -101,6 +102,7 @@ class NewOrderBatchView(BatchMasterView): # pylint: disable=abstract-method
|
||||||
"executed",
|
"executed",
|
||||||
"executed_by",
|
"executed_by",
|
||||||
]
|
]
|
||||||
|
# pylint: enable=duplicate-code
|
||||||
|
|
||||||
row_labels = {
|
row_labels = {
|
||||||
"product_scancode": "Scancode",
|
"product_scancode": "Scancode",
|
||||||
|
|
|
@ -56,6 +56,7 @@ class LocalCustomerView(MasterView): # pylint: disable=abstract-method
|
||||||
"external_id": "External ID",
|
"external_id": "External ID",
|
||||||
}
|
}
|
||||||
|
|
||||||
|
# pylint: disable=duplicate-code
|
||||||
grid_columns = [
|
grid_columns = [
|
||||||
"external_id",
|
"external_id",
|
||||||
"full_name",
|
"full_name",
|
||||||
|
@ -64,9 +65,11 @@ class LocalCustomerView(MasterView): # pylint: disable=abstract-method
|
||||||
"phone_number",
|
"phone_number",
|
||||||
"email_address",
|
"email_address",
|
||||||
]
|
]
|
||||||
|
# pylint: enable=duplicate-code
|
||||||
|
|
||||||
sort_defaults = "full_name"
|
sort_defaults = "full_name"
|
||||||
|
|
||||||
|
# pylint: disable=duplicate-code
|
||||||
form_fields = [
|
form_fields = [
|
||||||
"external_id",
|
"external_id",
|
||||||
"full_name",
|
"full_name",
|
||||||
|
@ -77,6 +80,7 @@ class LocalCustomerView(MasterView): # pylint: disable=abstract-method
|
||||||
"orders",
|
"orders",
|
||||||
"new_order_batches",
|
"new_order_batches",
|
||||||
]
|
]
|
||||||
|
# pylint: enable=duplicate-code
|
||||||
|
|
||||||
def configure_grid(self, grid): # pylint: disable=empty-docstring
|
def configure_grid(self, grid): # pylint: disable=empty-docstring
|
||||||
""" """
|
""" """
|
||||||
|
@ -171,6 +175,7 @@ class PendingCustomerView(PendingMixin, MasterView): # pylint: disable=abstract
|
||||||
"customer_id": "Customer ID",
|
"customer_id": "Customer ID",
|
||||||
}
|
}
|
||||||
|
|
||||||
|
# pylint: disable=duplicate-code
|
||||||
grid_columns = [
|
grid_columns = [
|
||||||
"full_name",
|
"full_name",
|
||||||
"first_name",
|
"first_name",
|
||||||
|
@ -182,9 +187,11 @@ class PendingCustomerView(PendingMixin, MasterView): # pylint: disable=abstract
|
||||||
"created",
|
"created",
|
||||||
"created_by",
|
"created_by",
|
||||||
]
|
]
|
||||||
|
# pylint: enable=duplicate-code
|
||||||
|
|
||||||
sort_defaults = "full_name"
|
sort_defaults = "full_name"
|
||||||
|
|
||||||
|
# pylint: disable=duplicate-code
|
||||||
form_fields = [
|
form_fields = [
|
||||||
"customer_id",
|
"customer_id",
|
||||||
"full_name",
|
"full_name",
|
||||||
|
@ -198,6 +205,7 @@ class PendingCustomerView(PendingMixin, MasterView): # pylint: disable=abstract
|
||||||
"orders",
|
"orders",
|
||||||
"new_order_batches",
|
"new_order_batches",
|
||||||
]
|
]
|
||||||
|
# pylint: enable=duplicate-code
|
||||||
|
|
||||||
def configure_grid(self, grid): # pylint: disable=empty-docstring
|
def configure_grid(self, grid): # pylint: disable=empty-docstring
|
||||||
""" """
|
""" """
|
||||||
|
|
|
@ -101,6 +101,7 @@ class OrderView(MasterView): # pylint: disable=too-many-public-methods
|
||||||
|
|
||||||
sort_defaults = ("order_id", "desc")
|
sort_defaults = ("order_id", "desc")
|
||||||
|
|
||||||
|
# pylint: disable=duplicate-code
|
||||||
form_fields = [
|
form_fields = [
|
||||||
"order_id",
|
"order_id",
|
||||||
"store_id",
|
"store_id",
|
||||||
|
@ -114,6 +115,7 @@ class OrderView(MasterView): # pylint: disable=too-many-public-methods
|
||||||
"created",
|
"created",
|
||||||
"created_by",
|
"created_by",
|
||||||
]
|
]
|
||||||
|
# pylint: enable=duplicate-code
|
||||||
|
|
||||||
has_rows = True
|
has_rows = True
|
||||||
row_model_class = OrderItem
|
row_model_class = OrderItem
|
||||||
|
@ -121,6 +123,7 @@ class OrderView(MasterView): # pylint: disable=too-many-public-methods
|
||||||
rows_sort_defaults = "sequence"
|
rows_sort_defaults = "sequence"
|
||||||
rows_viewable = True
|
rows_viewable = True
|
||||||
|
|
||||||
|
# pylint: disable=duplicate-code
|
||||||
row_labels = {
|
row_labels = {
|
||||||
"product_scancode": "Scancode",
|
"product_scancode": "Scancode",
|
||||||
"product_brand": "Brand",
|
"product_brand": "Brand",
|
||||||
|
@ -130,7 +133,9 @@ class OrderView(MasterView): # pylint: disable=too-many-public-methods
|
||||||
"order_uom": "Order UOM",
|
"order_uom": "Order UOM",
|
||||||
"status_code": "Status",
|
"status_code": "Status",
|
||||||
}
|
}
|
||||||
|
# pylint: enable=duplicate-code
|
||||||
|
|
||||||
|
# pylint: disable=duplicate-code
|
||||||
row_grid_columns = [
|
row_grid_columns = [
|
||||||
"sequence",
|
"sequence",
|
||||||
"product_scancode",
|
"product_scancode",
|
||||||
|
@ -145,7 +150,9 @@ class OrderView(MasterView): # pylint: disable=too-many-public-methods
|
||||||
"total_price",
|
"total_price",
|
||||||
"status_code",
|
"status_code",
|
||||||
]
|
]
|
||||||
|
# pylint: enable=duplicate-code
|
||||||
|
|
||||||
|
# pylint: disable=duplicate-code
|
||||||
PENDING_PRODUCT_ENTRY_FIELDS = [
|
PENDING_PRODUCT_ENTRY_FIELDS = [
|
||||||
"scancode",
|
"scancode",
|
||||||
"brand_name",
|
"brand_name",
|
||||||
|
@ -159,6 +166,7 @@ class OrderView(MasterView): # pylint: disable=too-many-public-methods
|
||||||
"unit_cost",
|
"unit_cost",
|
||||||
"unit_price_reg",
|
"unit_price_reg",
|
||||||
]
|
]
|
||||||
|
# pylint: enable=duplicate-code
|
||||||
|
|
||||||
def __init__(self, request, context=None):
|
def __init__(self, request, context=None):
|
||||||
super().__init__(request, context=context)
|
super().__init__(request, context=context)
|
||||||
|
@ -1311,6 +1319,7 @@ class OrderItemView(MasterView): # pylint: disable=abstract-method
|
||||||
|
|
||||||
sort_defaults = ("order_id", "desc")
|
sort_defaults = ("order_id", "desc")
|
||||||
|
|
||||||
|
# pylint: disable=duplicate-code
|
||||||
form_fields = [
|
form_fields = [
|
||||||
"order",
|
"order",
|
||||||
# 'customer_name',
|
# 'customer_name',
|
||||||
|
@ -1341,6 +1350,7 @@ class OrderItemView(MasterView): # pylint: disable=abstract-method
|
||||||
"paid_amount",
|
"paid_amount",
|
||||||
"payment_transaction_number",
|
"payment_transaction_number",
|
||||||
]
|
]
|
||||||
|
# pylint: enable=duplicate-code
|
||||||
|
|
||||||
def __init__(self, request, context=None):
|
def __init__(self, request, context=None):
|
||||||
super().__init__(request, context=context)
|
super().__init__(request, context=context)
|
||||||
|
|
|
@ -71,6 +71,7 @@ class LocalProductView(MasterView): # pylint: disable=abstract-method
|
||||||
|
|
||||||
sort_defaults = "scancode"
|
sort_defaults = "scancode"
|
||||||
|
|
||||||
|
# pylint: disable=duplicate-code
|
||||||
form_fields = [
|
form_fields = [
|
||||||
"external_id",
|
"external_id",
|
||||||
"scancode",
|
"scancode",
|
||||||
|
@ -89,6 +90,7 @@ class LocalProductView(MasterView): # pylint: disable=abstract-method
|
||||||
"orders",
|
"orders",
|
||||||
"new_order_batches",
|
"new_order_batches",
|
||||||
]
|
]
|
||||||
|
# pylint: enable=duplicate-code
|
||||||
|
|
||||||
def configure_grid(self, grid): # pylint: disable=empty-docstring
|
def configure_grid(self, grid): # pylint: disable=empty-docstring
|
||||||
""" """
|
""" """
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue