fergalmoran/picard
1
0
Fork 0
mirror of https://github.com/fergalmoran/picard.git synced 2026-02-15 20:24:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

Avoid confusing reuse of variable names in the same method

Browse Source
This commit is contained in:
Bob Swift
2021-07-13 10:14:28 -06:00
parent bb76532dcf
commit 73e11bbf5c
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
picard/ui/options/dialog.py
View File

@@ -408,17 +408,17 @@ class AttachedProfilesDialog(PicardDialog):
self.setWindowTitle(window_title)
for name, title in group_options:
item = QtGui.QStandardItem(_(title))
item.setEditable(False)
row = [item]
option_item = QtGui.QStandardItem(_(title))
option_item.setEditable(False)
row = [option_item]
attached = []
for profile in profiles:
if name in settings[profile["id"]]:
attached.append("{0}{1}".format(profile["title"], _(" [Enabled]") if profile["enabled"] else "",))
attached_profiles = "\n".join(attached) if attached else _("None")
item = QtGui.QStandardItem(attached_profiles)
item.setEditable(False)
row.append(item)
profile_item = QtGui.QStandardItem(attached_profiles)
profile_item.setEditable(False)
row.append(profile_item)
model.appendRow(row)
self.ui.options_list.setModel(model)