mbk/pesterchum
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Select and remove servers based on index instead of 'server'

Browse source
This commit is contained in:
Dpeta 2022-12-23 17:10:31 +01:00
parent afac304821
commit 8aa6a90f55
No known key found for this signature in database
GPG key ID: EC2D51B3B383C731
1 changed files with 24 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
pesterchum.py
View file

@ -3836,9 +3836,18 @@ class PesterWindow(MovingWindow):
selected_entry = None selected_entry = None
try:
assert (
server_list_obj[self.removeServerBox.currentIndex()]["server"]
== self.removeServerBox.currentText()
)
selected_entry = self.removeServerBox.currentIndex()
except (IndexError, AssertionError) as e:
PchumLog.warning(e)
for i in range(len(server_list_obj)): for i in range(len(server_list_obj)):
if server_list_obj[i]["server"] == self.removeServerBox.currentText(): if server_list_obj[i]["server"] == self.removeServerBox.currentText():
selected_entry = i selected_entry = i
if selected_entry is not None: if selected_entry is not None:
server_list_obj.pop(selected_entry) server_list_obj.pop(selected_entry)
@ -3927,7 +3936,7 @@ class PesterWindow(MovingWindow):
self.resetServerlist() self.resetServerlist()
return 1 return 1
PchumLog.info("server_list_items: " + str(server_list_items)) PchumLog.info(f"server_list_items: {server_list_items}")
# Widget 1 # Widget 1
self.chooseRemoveServerWidged = QtWidgets.QDialog() self.chooseRemoveServerWidged = QtWidgets.QDialog()
@ -3982,6 +3991,14 @@ class PesterWindow(MovingWindow):
selected_entry = None selected_entry = None
try:
selected_entry = self.serverBox.currentIndex()
assert (
server_obj[selected_entry]["server"] == self.serverBox.currentText()
)
except (IndexError, AssertionError) as e:
# fallback using 'server' as primary key
PchumLog.warning(e)
for i in range(len(server_obj)): for i in range(len(server_obj)):
if server_obj[i]["server"] == self.serverBox.currentText(): if server_obj[i]["server"] == self.serverBox.currentText():
selected_entry = i selected_entry = i