wutta/sideshow
3
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: fix 'consider-using-dict-comprehension' for pylint

Browse source
This commit is contained in:
Lance Edgar 2025-09-01 15:13:59 -05:00
parent 3ba57e3640
commit c3e262804c
3 changed files with 6 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

1
.pylintrc
View file

@ -8,4 +8,3 @@ disable=fixme,
arguments-renamed,
attribute-defined-outside-init,
broad-exception-caught,
consider-using-dict-comprehension,

4
src/sideshow/batch/neworder.py
View file

@ -1255,7 +1255,7 @@ class NewOrderBatchHandler(BatchHandler): # pylint: disable=too-many-public-met
]
# make order
kw = dict([(field, getattr(batch, field)) for field in batch_fields])
kw = {field: getattr(batch, field) for field in batch_fields}
kw["order_id"] = batch.id
kw["created_by"] = user
order = model.Order(**kw)
@ -1265,7 +1265,7 @@ class NewOrderBatchHandler(BatchHandler): # pylint: disable=too-many-public-met
def convert(row, i): # pylint: disable=unused-argument
# make order item
kw = dict([(field, getattr(row, field)) for field in row_fields])
kw = {field: getattr(row, field) for field in row_fields}
item = model.OrderItem(**kw)
order.items.append(item)

10
src/sideshow/web/views/orders.py
View file

@ -308,12 +308,10 @@ class OrderView(MasterView): # pylint: disable=too-many-public-methods
context["default_item_discount"] = self.app.render_quantity(
self.batch_handler.get_default_item_discount()
)
context["dept_item_discounts"] = dict(
[
(d["department_id"], d["default_item_discount"])
for d in self.get_dept_item_discounts()
]
)
context["dept_item_discounts"] = {
d["department_id"]: d["default_item_discount"]
for d in self.get_dept_item_discounts()
}
return self.render_to_response("create", context)