Skip to content

Commit 42f8411

Browse files
chenneschennes
committed
Addon Manager: Show versions in update all
1 parent 6241590 commit 42f8411

File tree

3 files changed

+58
-22
lines changed

3 files changed

+58
-22
lines changed

‎src/Mod/AddonManager/AddonManagerTest/gui/test_update_all_gui.py‎

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# SPDX-License-Identifier: LGPL-2.1-or-later
22
# ***************************************************************************
33
# * *
4-
# * Copyright (c) 2022 FreeCAD Project Association *
4+
# * Copyright (c) 2022-2025 FreeCAD project association AISBL *
55
# * *
66
# * This file is part of FreeCAD. *
77
# * *
@@ -73,6 +73,7 @@ def __init__(self, name):
7373
self.display_name = name
7474
self.name = name
7575
self.macro = None
76+
self.metadata = None
7677

7778
def status(self):
7879
return Addon.Status.UPDATE_AVAILABLE
@@ -151,22 +152,22 @@ def test_update_addon_status(self):
151152
self.test_object._setup_dialog()
152153
self.test_object._update_addon_status(0, AddonStatus.WAITING)
153154
self.assertEqual(
154-
self.test_object.dialog.tableWidget.item(0, 1).text(),
155+
self.test_object.dialog.tableWidget.item(0, 2).text(),
155156
AddonStatus.WAITING.ui_string(),
156157
)
157158
self.test_object._update_addon_status(0, AddonStatus.INSTALLING)
158159
self.assertEqual(
159-
self.test_object.dialog.tableWidget.item(0, 1).text(),
160+
self.test_object.dialog.tableWidget.item(0, 2).text(),
160161
AddonStatus.INSTALLING.ui_string(),
161162
)
162163
self.test_object._update_addon_status(0, AddonStatus.SUCCEEDED)
163164
self.assertEqual(
164-
self.test_object.dialog.tableWidget.item(0, 1).text(),
165+
self.test_object.dialog.tableWidget.item(0, 2).text(),
165166
AddonStatus.SUCCEEDED.ui_string(),
166167
)
167168
self.test_object._update_addon_status(0, AddonStatus.FAILED)
168169
self.assertEqual(
169-
self.test_object.dialog.tableWidget.item(0, 1).text(),
170+
self.test_object.dialog.tableWidget.item(0, 2).text(),
170171
AddonStatus.FAILED.ui_string(),
171172
)
172173

@@ -175,19 +176,19 @@ def test_process_next_update(self):
175176
self.test_object._launch_active_installer = lambda: None
176177
self.test_object._process_next_update()
177178
self.assertEqual(
178-
self.test_object.dialog.tableWidget.item(0, 1).text(),
179+
self.test_object.dialog.tableWidget.item(0, 2).text(),
179180
AddonStatus.INSTALLING.ui_string(),
180181
)
181182

182183
self.test_object._process_next_update()
183184
self.assertEqual(
184-
self.test_object.dialog.tableWidget.item(1, 1).text(),
185+
self.test_object.dialog.tableWidget.item(1, 2).text(),
185186
AddonStatus.INSTALLING.ui_string(),
186187
)
187188

188189
self.test_object._process_next_update()
189190
self.assertEqual(
190-
self.test_object.dialog.tableWidget.item(2, 1).text(),
191+
self.test_object.dialog.tableWidget.item(2, 2).text(),
191192
AddonStatus.INSTALLING.ui_string(),
192193
)
193194

@@ -208,15 +209,15 @@ def test_update_succeeded(self):
208209
self.test_object._setup_dialog()
209210
self.test_object._update_succeeded(self.addons[0])
210211
self.assertEqual(
211-
self.test_object.dialog.tableWidget.item(0, 1).text(),
212+
self.test_object.dialog.tableWidget.item(0, 2).text(),
212213
AddonStatus.SUCCEEDED.ui_string(),
213214
)
214215

215216
def test_update_failed(self):
216217
self.test_object._setup_dialog()
217218
self.test_object._update_failed(self.addons[0])
218219
self.assertEqual(
219-
self.test_object.dialog.tableWidget.item(0, 1).text(),
220+
self.test_object.dialog.tableWidget.item(0, 2).text(),
220221
AddonStatus.FAILED.ui_string(),
221222
)
222223

‎src/Mod/AddonManager/addonmanager_update_all_gui.py‎

Lines changed: 40 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# SPDX-License-Identifier: LGPL-2.1-or-later
22
# ***************************************************************************
33
# * *
4-
# * Copyright (c) 2022 FreeCAD Project Association *
4+
# * Copyright (c) 2022-2025 FreeCAD project association AISBL *
55
# * *
66
# * This file is part of FreeCAD. *
77
# * *
@@ -85,6 +85,8 @@ class UpdateAllGUI(QtCore.QObject):
8585
finished = QtCore.Signal()
8686
addon_updated = QtCore.Signal(object)
8787

88+
index_role = QtCore.Qt.UserRole + 1
89+
8890
def __init__(self, addons: List[Addon]):
8991
super().__init__()
9092
self.addons = addons
@@ -114,28 +116,58 @@ def _setup_dialog(self):
114116
self.dialog.tableWidget.clear()
115117
self.in_process_row = None
116118
self.row_map = {}
119+
self._setup_empty_table()
120+
counter = 0
117121
for addon in self.addons:
118122
if addon.status() == Addon.Status.UPDATE_AVAILABLE:
119-
self._add_addon_to_table(addon)
123+
self._add_addon_to_table(addon, counter)
120124
self.addons_with_update.append(addon)
125+
counter += 1
121126

122127
def _cancel_installation(self):
123128
self.cancelled = True
124129
if self.worker_thread and self.worker_thread.isRunning():
125130
self.worker_thread.requestInterruption()
126131

127-
def _add_addon_to_table(self, addon: Addon):
128-
"""Add the given addon to the list, with no icon in the first column"""
132+
def _setup_empty_table(self):
133+
self.dialog.tableWidget.setColumnCount(4)
134+
self.dialog.tableWidget.horizontalHeader().setSectionResizeMode(
135+
0, QtWidgets.QHeaderView.ResizeMode.ResizeToContents
136+
)
137+
self.dialog.tableWidget.horizontalHeader().setSectionResizeMode(
138+
1, QtWidgets.QHeaderView.ResizeMode.ResizeToContents
139+
)
140+
self.dialog.tableWidget.horizontalHeader().setSectionResizeMode(
141+
2, QtWidgets.QHeaderView.ResizeMode.ResizeToContents
142+
)
143+
self.dialog.tableWidget.horizontalHeader().setSectionResizeMode(
144+
3, QtWidgets.QHeaderView.ResizeMode.Stretch
145+
)
146+
147+
def _add_addon_to_table(self, addon: Addon, index: int):
148+
"""Add the given addon to the list, storing its index as user data in the first column"""
129149
new_row = self.dialog.tableWidget.rowCount()
130-
self.dialog.tableWidget.setColumnCount(2)
131150
self.dialog.tableWidget.setRowCount(new_row + 1)
132-
self.dialog.tableWidget.setItem(new_row, 0, QtWidgets.QTableWidgetItem(addon.display_name))
133-
self.dialog.tableWidget.setItem(new_row, 1, QtWidgets.QTableWidgetItem(""))
151+
new_item = QtWidgets.QTableWidgetItem(addon.display_name)
152+
new_item.setData(UpdateAllGUI.index_role, index) # Only first item in each row needs data()
153+
self.dialog.tableWidget.setItem(new_row, 0, new_item)
154+
if addon.installed_metadata and addon.installed_metadata.version:
155+
self.dialog.tableWidget.setItem(
156+
new_row, 1, QtWidgets.QTableWidgetItem(str(addon.installed_metadata.version))
157+
)
158+
self.dialog.tableWidget.setItem(new_row, 2, QtWidgets.QTableWidgetItem(""))
159+
self.dialog.tableWidget.setItem(new_row, 3, QtWidgets.QTableWidgetItem(""))
134160
self.row_map[addon.name] = new_row
135161

136162
def _update_addon_status(self, row: int, status: AddonStatus):
137163
"""Update the GUI to reflect this addon's new status."""
138-
self.dialog.tableWidget.item(row, 1).setText(status.ui_string())
164+
self.dialog.tableWidget.item(row, 2).setText(status.ui_string())
165+
if status == AddonStatus.SUCCEEDED and self.addons[row].metadata:
166+
self.dialog.tableWidget.item(row, 2).setText(status.ui_string() + " →")
167+
index = self.dialog.tableWidget.item(row, 0).data(UpdateAllGUI.index_role)
168+
addon = self.addons[index]
169+
if addon.metadata and addon.metadata.version:
170+
self.dialog.tableWidget.item(row, 3).setText(str(addon.metadata.version))
139171

140172
def _process_next_update(self):
141173
"""Grab the next addon in the list and start its updater."""

‎src/Mod/AddonManager/update_all.ui‎

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
<item>
2828
<widget class="QTableWidget" name="tableWidget">
2929
<property name="editTriggers">
30-
<set>QAbstractItemView::NoEditTriggers</set>
30+
<set>QAbstractItemView::EditTrigger::NoEditTriggers</set>
3131
</property>
3232
<property name="tabKeyNavigation">
3333
<bool>false</bool>
@@ -39,7 +39,10 @@
3939
<bool>false</bool>
4040
</property>
4141
<property name="selectionMode">
42-
<enum>QAbstractItemView::NoSelection</enum>
42+
<enum>QAbstractItemView::SelectionMode::NoSelection</enum>
43+
</property>
44+
<property name="selectionBehavior">
45+
<enum>QAbstractItemView::SelectionBehavior::SelectRows</enum>
4346
</property>
4447
<property name="showGrid">
4548
<bool>false</bool>
@@ -58,10 +61,10 @@
5861
<item>
5962
<widget class="QDialogButtonBox" name="buttonBox">
6063
<property name="orientation">
61-
<enum>Qt::Horizontal</enum>
64+
<enum>Qt::Orientation::Horizontal</enum>
6265
</property>
6366
<property name="standardButtons">
64-
<set>QDialogButtonBox::Cancel</set>
67+
<set>QDialogButtonBox::StandardButton::Cancel</set>
6568
</property>
6669
</widget>
6770
</item>

0 commit comments

Comments
 (0)