diff options
Diffstat (limited to 'dozentenmodul/src/main/java/org/openslx/dozmod/thrift/ThriftActions.java')
| -rw-r--r-- | dozentenmodul/src/main/java/org/openslx/dozmod/thrift/ThriftActions.java | 29 |
1 files changed, 12 insertions, 17 deletions
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/thrift/ThriftActions.java b/dozentenmodul/src/main/java/org/openslx/dozmod/thrift/ThriftActions.java index 9022534c..29a977d4 100644 --- a/dozentenmodul/src/main/java/org/openslx/dozmod/thrift/ThriftActions.java +++ b/dozentenmodul/src/main/java/org/openslx/dozmod/thrift/ThriftActions.java @@ -742,7 +742,7 @@ public class ThriftActions { List<LectureSummary> lectureList = ThriftManager.getSatClient().getLectureList( Session.getSatelliteToken(), 100); for (LectureSummary lecture : lectureList) { - if (lecture.getImageBaseId().equals(imageBaseId)) { + if (imageBaseId.equals(lecture.getImageBaseId())) { if (lecturesToBeDeleted == null) lecturesToBeDeleted = new ArrayList<LectureSummary>(); lecturesToBeDeleted.add(lecture); @@ -761,28 +761,24 @@ public class ThriftActions { "Fehler beim Holen der Versionen/Veranstaltung zu folgendes Image: " + imageBaseId); return; } - String questionText = ""; - if (versionToBeDeleted != null) { + String questionText; + if (versionToBeDeleted != null && !versionToBeDeleted.isEmpty()) { questionText = "Dieses Image hat folgende gültige Versionen:\n"; for (ImageVersionDetails version : versionToBeDeleted) { questionText += version.getVersionId() + "\n"; } + questionText += "\n"; } else { - questionText = "Dieses Image hat keine gültige Version.\n"; + questionText = ""; } - if (lecturesToBeDeleted != null) { - if (!questionText.isEmpty()) - questionText += "\n"; - questionText += "Dieses Image ist zu folgende Veranstaltungen verknüpft:\n"; + if (lecturesToBeDeleted != null && !lecturesToBeDeleted.isEmpty()) { + questionText += "Dieses Image ist mit folgenden Veranstaltungen verknüpft:\n"; for (LectureSummary lecture : lecturesToBeDeleted) { questionText += lecture.getLectureName() + "\n"; } - } else { - if (!questionText.isEmpty()) - questionText += "\n"; - questionText += "Dieses Image ist zu keine Veranstaltungen verknüpft.\n"; + questionText += "\n"; } - questionText += "\nWollen Sie dieses Image wirklich löschen?\n"; + questionText += "Wollen Sie dieses Image wirklich löschen?"; if (!userConfirmed(frame, questionText)) return; try { @@ -827,7 +823,7 @@ public class ThriftActions { boolean matches = false; if (lectureList != null && !lectureList.isEmpty()) { for (LectureSummary lecture : lectureList) { - if (lecture.getImageVersionId().equals(imageVersionId)) { + if (imageVersionId.equals(lecture.getImageVersionId())) { if (!matches) questionText = "Diese Version ist zu folgende Veranstaltungen verknüpft:\n"; matches = true; @@ -838,8 +834,7 @@ public class ThriftActions { questionText += "\nWollen Sie diese Version samt Veranstaltungen löschen?\n"; } if (!matches) - questionText = "Diese Version ist zu keine Veranstaltungen verknüpft.\n" - + "Wollen Sie sie wirklich löschen?"; + questionText = "Wollen Sie diese Image-Version wirklich löschen?"; if (!userConfirmed(frame, questionText)) return; @@ -1096,7 +1091,7 @@ public class ThriftActions { * @return true if the user confirmed (clicked yes), false otherwise */ private static boolean userConfirmed(final Frame frame, final String message) { - return Gui.showMessageBox(frame, message, MessageType.QUESTION_YESNO, LOGGER, null); + return Gui.showMessageBox(frame, message, MessageType.QUESTION_YESNO, null, null); } /** |