Skip to content

Commit 1aec7d8

Browse files
committed
fix null progress bar "Updates" tab in Contributions Manager (resolves #567)
1 parent d930f32 commit 1aec7d8

File tree

2 files changed

+12
-5
lines changed

2 files changed

+12
-5
lines changed

app/src/processing/app/contrib/StatusDetail.java

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -166,11 +166,16 @@ protected void update() {
166166
// For the special "Updates" tab in the manager, there are no progress
167167
// bars, so if that's what we're doing, this will create a dummy bar.
168168
// TODO Not a good workaround [fry 220312]
169-
// if (progressBar == null) {
170-
// initProgressBar();
171-
// }
172-
progressBar.setVisible(true);
173-
progressBar.setIndeterminate(true);
169+
// TODO This is really, really gross [fry 221104]
170+
if (progressBar == null) {
171+
// This was removed in 4.x and brought back for 4.0.2 because
172+
// it broke the "Update All" option in the Contributions Manager.
173+
// https://github.com/processing/processing4/issues/567
174+
progressBar = new JProgressBar();
175+
} else {
176+
progressBar.setVisible(true);
177+
progressBar.setIndeterminate(true);
178+
}
174179

175180
ContribProgress progress = new ContribProgress(progressBar) {
176181
@Override

todo.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -357,6 +357,8 @@ _ when adding a new library, its examples aren't added to the list until restart
357357

358358
PDE / Manager (4.x notes)
359359

360+
_ get rid of dummy progress bar being created in StatusDetail.update()
361+
_ would be good to *add* a progress bar of some kind, but sheesh
360362
_ remove dorky loading.gif (used in ContributionTab and UpdateContributionTab)
361363
_ should be a better way to implement this
362364
_ if no internet available, install buttons disabled, but not clear why broken

0 commit comments

Comments
 (0)