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

Adobe Connect: ampersand is not allowed in meeting name

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 14.2.2
    • Fix Version/s: 14.2.3
    • Component/s: None
    • Labels:
      None

      Description

      If the name of an Adobe Connect meeting contains an ampersand an error is thrown by Connect. This patch avoids that:

      diff --git a/src/main/java/org/olat/modules/adobeconnect/manager/AdobeConnectManagerImpl.java b/src/main/java/org/olat/modules/adobeconnect/manager/AdobeConnectManagerImpl.java
      index e413491..9c12411 100644
      diff --git a/src/main/java/org/olat/modules/adobeconnect/ui/EditAdobeConnectMeetingController.java b/src/main/java/org/olat/modules/adobeconnect/ui/EditAdobeConnectMeetingController.java
      index cc6168b..92af411 100644
      --- a/src/main/java/org/olat/modules/adobeconnect/ui/EditAdobeConnectMeetingController.java
      +++ b/src/main/java/org/olat/modules/adobeconnect/ui/EditAdobeConnectMeetingController.java
      @@ -179,6 +183,10 @@
       			nameEl.setErrorKey("form.legende.mandatory", null);
       			allOk &= false;
       		}
      +		if (nameEl.getValue().contains("&")) {
      +			nameEl.setErrorKey("form.invalidchar.noamp", null);
      +			allOk &= false;
      +		}
       		
       		startDateEl.clearError();
       		endDateEl.clearError();
      diff --git a/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_de.properties b/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_de.properties
      index 0b6f002..f0db324 100644
      --- a/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_de.properties
      +++ b/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_de.properties
      @@ -36,6 +36,7 @@
       error.rangeError=Datumsbereich ist nicht g\u00FCltig.
       error.start.after.end=Das Datum f\u00FCr das Ende des Meetings darf nicht vor dem Beginn Datum sein.
       error.unkown=Unerwartete Fehler
      +form.invalidchar.noamp=Das Und-Zeichen ist nicht erlaubt.
       meeting.configure.button=In Raum gehen und konfigurieren
       meeting.create.intro=Das Meeting wurde noch nicht er\u00f6ffnet. Teilnehmer k\u00F6nnen den Raum f\u00fcr ein geplantes Meeting ggf. nicht betreten.
       meeting.create.button=Meeting er\u00f6ffnen
      diff --git a/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_en.properties b/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_en.properties
      index 8e1d611..5112ae5 100644
      --- a/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_en.properties
      +++ b/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_en.properties
      @@ -36,6 +36,7 @@
       error.rangeError=The range is not valid.
       error.start.after.end=The end date of the meeting must not be before the start date.
       error.unkown=Unkown error
      +form.invalidchar.noamp=The ampersand is not allowed.
       meeting.configure.button=Go and configure the room
       meeting.create.button=Open the meeting
       meeting.create.intro=The meeting has not been opened, yet. Participants are not able to enter the classroom for a meeting.
      diff --git a/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_fr.properties b/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_fr.properties
      index c93b933..2b451eb 100644
      --- a/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_fr.properties
      +++ b/src/main/java/org/olat/modules/adobeconnect/ui/_i18n/LocalStrings_fr.properties
      @@ -87,3 +87,5 @@
       meetings.past=Meetings anciens
       content.open=Ouvrir
       meeting.name=Nom
      +form.invalidchar.noamp=L'esperluette n'est pas permis.
      +
      

       

        Figma for Jira By 42nd

          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: