diff options
| author | Simon Rettberg | 2019-10-15 14:34:25 +0200 |
|---|---|---|
| committer | Simon Rettberg | 2019-10-15 14:34:25 +0200 |
| commit | 0c54b3b868e4434d69be41de73fd74fc4685a298 (patch) | |
| tree | a11fef681c5e60b1666cd436cd965cd882215acd | |
| parent | [client] Add update check button to login window (diff) | |
| download | tutor-module-0c54b3b868e4434d69be41de73fd74fc4685a298.tar.gz tutor-module-0c54b3b868e4434d69be41de73fd74fc4685a298.tar.xz tutor-module-0c54b3b868e4434d69be41de73fd74fc4685a298.zip | |
[client] THINK before commiting
Setting variable before checking its value is a bad idea.
| -rw-r--r-- | dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ImageUploadSummaryPage.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ImageUploadSummaryPage.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ImageUploadSummaryPage.java index 55615a00..9b564979 100644 --- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ImageUploadSummaryPage.java +++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ImageUploadSummaryPage.java @@ -142,8 +142,10 @@ public class ImageUploadSummaryPage extends ImageUploadSummaryPageLayout { @Override protected void onPageLeave() { + if (!pageIsVisible) + return; pageIsVisible = false; - if (!pageIsVisible || state == null) + if (state == null) return; // optional lectureWizard |