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

Survey: coaches only see results if owners is configured

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 13.0, 13.1, 13.2, 13.2.1
    • Fix Version/s: 13.2.2
    • Component/s: Course
    • Labels:
      None

      Description

      If a survey is configured that coaches can see the results this doesn't have any effect. The results are shown if the owners are allowed to see them. This patch fixes that:

       

      diff --git a/src/main/java/org/olat/course/nodes/survey/SurveyRunSecurityCallback.java b/src/main/java/org/olat/course/nodes/survey/SurveyRunSecurityCallback.java
      index b508fbd..c201d68 100644
      --- a/src/main/java/org/olat/course/nodes/survey/SurveyRunSecurityCallback.java
      +++ b/src/main/java/org/olat/course/nodes/survey/SurveyRunSecurityCallback.java
      @@ -94,7 +94,7 @@
       			return true;
       		}
       		
      -		if (moduleConfiguration.getBooleanSafe(SurveyCourseNode.CONFIG_KEY_REPORT_FOR_OWNER)
      +		if (moduleConfiguration.getBooleanSafe(SurveyCourseNode.CONFIG_KEY_EXECUTION_BY_COACH)
       				&& userCourseEnv.isCoach()) {
       			return true;
       		}
      

        Attachments

          Activity

            People

            Assignee:
            uhensler Urs Hensler
            Reporter:
            stephan Stephan Clemenz
            Tester:
            Mandy Menzel
            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 - 30 minutes
                30m