mirror of
https://github.com/dcs-liberation/dcs_liberation.git
synced 2025-11-10 14:22:26 +00:00
Update UI when buy or sell raises exception
fixes #1562 cherry-pick from 34100d1c
This commit is contained in:
parent
bb1e314260
commit
0077b04698
@ -230,25 +230,23 @@ class UnitTransactionFrame(QFrame, Generic[TransactionItemType]):
|
|||||||
self.update_purchase_controls()
|
self.update_purchase_controls()
|
||||||
self.update_available_budget()
|
self.update_available_budget()
|
||||||
|
|
||||||
def buy(self, item: TransactionItemType, quantity: int) -> bool:
|
def buy(self, item: TransactionItemType, quantity: int) -> None:
|
||||||
try:
|
try:
|
||||||
self.purchase_adapter.buy(item, quantity)
|
self.purchase_adapter.buy(item, quantity)
|
||||||
except TransactionError as ex:
|
except TransactionError as ex:
|
||||||
logging.exception(f"Purchase of {self.display_name_of(item)} failed")
|
logging.exception(f"Purchase of {self.display_name_of(item)} failed")
|
||||||
QMessageBox.warning(self, "Purchase failed", str(ex), QMessageBox.Ok)
|
QMessageBox.warning(self, "Purchase failed", str(ex), QMessageBox.Ok)
|
||||||
return False
|
finally:
|
||||||
self.post_transaction_update()
|
self.post_transaction_update()
|
||||||
return True
|
|
||||||
|
|
||||||
def sell(self, item: TransactionItemType, quantity: int) -> bool:
|
def sell(self, item: TransactionItemType, quantity: int) -> None:
|
||||||
try:
|
try:
|
||||||
self.purchase_adapter.sell(item, quantity)
|
self.purchase_adapter.sell(item, quantity)
|
||||||
except TransactionError as ex:
|
except TransactionError as ex:
|
||||||
logging.exception(f"Sale of {self.display_name_of(item)} failed")
|
logging.exception(f"Sale of {self.display_name_of(item)} failed")
|
||||||
QMessageBox.warning(self, "Sale failed", str(ex), QMessageBox.Ok)
|
QMessageBox.warning(self, "Sale failed", str(ex), QMessageBox.Ok)
|
||||||
return False
|
finally:
|
||||||
self.post_transaction_update()
|
self.post_transaction_update()
|
||||||
return True
|
|
||||||
|
|
||||||
def update_purchase_controls(self) -> None:
|
def update_purchase_controls(self) -> None:
|
||||||
for group in self.purchase_groups.values():
|
for group in self.purchase_groups.values():
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user