Uploaded image for project: 'OpenOLAT'
  1. OpenOLAT
  2. OO-2591

Date enrollment: Subject of emails is calculated wrong

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 11.2.2
    • Fix Version/s: 11.3
    • Component/s: None
    • Labels:
      None

      Description

      The course owner can add and remove participants in date enrollment. When done, he has the possibility to send an email to the added or removed participant. The subject (and also a part of the body) of this email is always the subject of the first date in datelist (dateList.get(0).getSubject()). The correct subject should be the subject where the participant is enrolled or removed.
      Furthermore when selecting multiple dates in the participant table for removing the participants, the subject of the mail is also the subject of the first date. My suggestion is to set the short title of the course node as subject.

      This is the patch:

      diff --git a/src/main/java/de/bps/course/nodes/den/DENManageParticipantsController.java b/src/main/java/de/bps/course/nodes/den/DENManageParticipantsController.java
      --- a/src/main/java/de/bps/course/nodes/den/DENManageParticipantsController.java
      +++ b/src/main/java/de/bps/course/nodes/den/DENManageParticipantsController.java
      @@ -161,7 +161,7 @@
       					dateList = denManager.deleteParticipants(ores, courseNode, denManager.getSelectedEventIDs(dateList, selection));
       					listTableData.setObjects(dateList);
       					//send notification mail
      -					createRemovedNotificationMail(ureq, dateList.get(0).getSubject());
      +					createRemovedNotificationMail(ureq, courseNode.getShortTitle());
       				} else if(tmse.getAction().equals(DENListTableDataModel.MAIL_ACTION) && selection.cardinality() > 0) {
       					//send email to all users from the selected dates
       					List<Identity> participants = denManager.getSelectedEventParticipants(dateList, selection);
      @@ -201,7 +201,7 @@
       				userSearchCMC.deactivate();
       				if(added.size() > 0) {
       					//write notification mail
      -					createAddedNotificationMail(ureq, dateList.get(0).getSubject());
      +					createAddedNotificationMail(ureq, selectedEvent.getSubject());
       				}
       			}
       			
      @@ -217,7 +217,7 @@
       					else {
       						removed.clear();
       						removed.add(identity);
      -						createRemovedNotificationMail(ureq, dateList.get(0).getSubject());
      +						createRemovedNotificationMail(ureq, selectedEvent.getSubject());
       					}
       					refreshTables();
       				//write email to single user
      

        Attachments

          Activity

            People

            Assignee:
            srosse Stéphane Rossé
            Reporter:
            stephan Stephan Clemenz
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 20 minutes
                20m