diff --git a/src/main/java/org/olat/course/nodes/COCourseNode.java b/src/main/java/org/olat/course/nodes/COCourseNode.java index 60d578413ef7114a491e6831304fcbae3d5c40eb..e77cc77bd431b9c917848b4fe1d0274a08c36644 100755 --- a/src/main/java/org/olat/course/nodes/COCourseNode.java +++ b/src/main/java/org/olat/course/nodes/COCourseNode.java @@ -133,28 +133,26 @@ public class COCourseNode extends AbstractAccessableCourseNode { ModuleConfiguration mc = getModuleConfiguration(); String coachesGroupNames = (String)mc.get(COEditController.CONFIG_KEY_EMAILTOCOACHES_GROUP); String particpantsGroupNames = (String)mc.get(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_GROUP); - - @SuppressWarnings("unchecked") - List<Long> coachesGroupKeys = (List<Long>) mc.get(COEditController.CONFIG_KEY_EMAILTOCOACHES_GROUP_ID); - @SuppressWarnings("unchecked") - List<Long> participantsGroupKeys = (List<Long>) mc.get(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_GROUP_ID); + //remap group keys + List<Long> coachesGroupKeys = mc.getList(COEditController.CONFIG_KEY_EMAILTOCOACHES_GROUP_ID, Long.class); if(coachesGroupKeys == null) { coachesGroupKeys = envMapper.toGroupKeyFromOriginalNames(coachesGroupNames); } else { coachesGroupKeys = envMapper.toGroupKeyFromOriginalKeys(coachesGroupKeys); } - mc.set(COEditController.CONFIG_KEY_EMAILTOGROUP_IDS, coachesGroupKeys); + mc.set(COEditController.CONFIG_KEY_EMAILTOCOACHES_GROUP_ID, coachesGroupKeys); + + List<Long> participantsGroupKeys = mc.getList(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_GROUP_ID, Long.class); if(participantsGroupKeys == null) { participantsGroupKeys = envMapper.toGroupKeyFromOriginalNames(particpantsGroupNames); } else { participantsGroupKeys = envMapper.toGroupKeyFromOriginalKeys(participantsGroupKeys); } - mc.set(COEditController.CONFIG_KEY_EMAILTOGROUP_IDS, participantsGroupKeys); - + mc.set(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_GROUP_ID, participantsGroupKeys); + //remap area keys String coachesAreaNames = (String)mc.get(COEditController.CONFIG_KEY_EMAILTOCOACHES_AREA); - @SuppressWarnings("unchecked") - List<Long> coachesAreaKeys = (List<Long>) mc.get(COEditController.CONFIG_KEY_EMAILTOCOACHES_AREA_IDS); + List<Long> coachesAreaKeys = mc.getList(COEditController.CONFIG_KEY_EMAILTOCOACHES_AREA_IDS, Long.class); if(coachesAreaKeys == null) { coachesAreaKeys = envMapper.toAreaKeyFromOriginalNames(coachesAreaNames); } else { @@ -163,8 +161,7 @@ public class COCourseNode extends AbstractAccessableCourseNode { mc.set(COEditController.CONFIG_KEY_EMAILTOCOACHES_AREA_IDS, coachesAreaKeys); String participantsAreaNames = (String)mc.get(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_AREA); - @SuppressWarnings("unchecked") - List<Long> participantsAreaKeys = (List<Long>) mc.get(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_AREA_ID); + List<Long> participantsAreaKeys = mc.getList(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_AREA_ID, Long.class); if(participantsAreaKeys == null) { participantsAreaKeys = envMapper.toAreaKeyFromOriginalNames(participantsAreaNames); } else { diff --git a/src/main/java/org/olat/course/nodes/co/CORunController.java b/src/main/java/org/olat/course/nodes/co/CORunController.java index fe754b48ddf9f692087578db81e325e92e032b53..e529fb05cbb2fd6caba7b72061add024d0606968 100755 --- a/src/main/java/org/olat/course/nodes/co/CORunController.java +++ b/src/main/java/org/olat/course/nodes/co/CORunController.java @@ -95,8 +95,7 @@ public class CORunController extends BasicController { Translator fallback = Util.createPackageTranslator(ContactFormController.class, ureq.getLocale()); setTranslator(Util.createPackageTranslator(CORunController.class, ureq.getLocale(), fallback)); - @SuppressWarnings("unchecked") - List<String> emailListConfig = (List<String>) moduleConfiguration.get(COEditController.CONFIG_KEY_EMAILTOADRESSES); + List<String> emailListConfig = moduleConfiguration.getList(COEditController.CONFIG_KEY_EMAILTOADRESSES, String.class); String mSubject = (String) moduleConfiguration.get(COEditController.CONFIG_KEY_MSUBJECT_DEFAULT); String mBody = (String) moduleConfiguration.get(COEditController.CONFIG_KEY_MBODY_DEFAULT); @@ -110,29 +109,25 @@ public class CORunController extends BasicController { String participantGroupNames = (String)moduleConfiguration.get(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_GROUP); - @SuppressWarnings("unchecked") - List<Long> participantGroupKeys = (List<Long>) moduleConfiguration.get(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_GROUP_ID); + List<Long> participantGroupKeys = moduleConfiguration.getList(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_GROUP_ID, Long.class); if(participantGroupKeys == null && StringHelper.containsNonWhitespace(participantGroupNames)) { participantGroupKeys = businessGroupService.toGroupKeys(participantGroupNames, cgm.getCourseEntry()); } String participantAreaNames = (String)moduleConfiguration.get(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_AREA); - @SuppressWarnings("unchecked") - List<Long> participantAreaKeys = (List<Long>) moduleConfiguration.get(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_AREA_ID); + List<Long> participantAreaKeys = moduleConfiguration.getList(COEditController.CONFIG_KEY_EMAILTOPARTICIPANTS_AREA_ID, Long.class); if(participantAreaKeys == null && StringHelper.containsNonWhitespace(participantAreaNames)) { participantAreaKeys = businessGroupService.toGroupKeys(participantAreaNames, cgm.getCourseEntry()); } String coachGroupNames = (String)moduleConfiguration.get(COEditController.CONFIG_KEY_EMAILTOCOACHES_GROUP); - @SuppressWarnings("unchecked") - List<Long> coachGroupKeys = (List<Long>) moduleConfiguration.get(COEditController.CONFIG_KEY_EMAILTOCOACHES_GROUP_ID); + List<Long> coachGroupKeys = moduleConfiguration.getList(COEditController.CONFIG_KEY_EMAILTOCOACHES_GROUP_ID, Long.class); if(coachGroupKeys == null && StringHelper.containsNonWhitespace(coachGroupNames)) { coachGroupKeys = businessGroupService.toGroupKeys(coachGroupNames, cgm.getCourseEntry()); } String coachAreaNames = (String)moduleConfiguration.get(COEditController.CONFIG_KEY_EMAILTOCOACHES_AREA); - @SuppressWarnings("unchecked") - List<Long> coachAreaKeys = (List<Long>) moduleConfiguration.get(COEditController.CONFIG_KEY_EMAILTOCOACHES_AREA_IDS); + List<Long> coachAreaKeys = moduleConfiguration.getList(COEditController.CONFIG_KEY_EMAILTOCOACHES_AREA_IDS, Long.class); if(coachAreaKeys == null && StringHelper.containsNonWhitespace(coachAreaNames)) { coachAreaKeys = businessGroupService.toGroupKeys(coachAreaNames, cgm.getCourseEntry()); }