Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-51410

Missing notifysucess and hardcoded strings on survey report

XMLWordPrintable

    • MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE, MOODLE_30_STABLE, MOODLE_31_STABLE, MOODLE_32_STABLE
    • MOODLE_30_STABLE, MOODLE_31_STABLE
    • MDL-51410_m32v2
    • Easy
    • Hide

      Manual testing:
      (Requires a course with a student and a teacher)

      1. Create a survey activity using the ATTLS survey type.
      2. Have the student complete the survey
      3. As the teacher view the "Participants" Report (View the survey, Administration block -> Survey Administration -> Response reports -> Participants .
      4. Click on the student's name
      5. Enter some text into "Your private analysis and notes:" and press "Save changes"
      6. Verify the save successful notification is "green"
      7. Edit the text into "Your private analysis and notes:" and trigger the error message on save (The easiest way to edit the test on line 398 of mod/survey/report.php by removing the !)
      8. Verify the error message is displayed in Red.
      Show
      Manual testing: (Requires a course with a student and a teacher) Create a survey activity using the ATTLS survey type. Have the student complete the survey As the teacher view the "Participants" Report (View the survey, Administration block -> Survey Administration -> Response reports -> Participants . Click on the student's name Enter some text into "Your private analysis and notes:" and press "Save changes" Verify the save successful notification is "green" Edit the text into "Your private analysis and notes:" and trigger the error message on save (The easiest way to edit the test on line 398 of mod/survey/report.php by removing the !) Verify the error message is displayed in Red.

      When you browse an user survey report and add a private analysis and notes, the report displays the Your notes were saved message in red background, at first glance seems there was an error while saving the notes.

      It's just a matter of adding notifiysucess parameter on the 

      $OUTPUT->notification(get_string("savednotes", "survey"), 'notifysuccess');

      There also hardocoded strings, that I think could be adress here too:

      echo $OUTPUT->notification("An error occurred while saving your notes.  Sorry.");

        1. missingnotify.png
          missingnotify.png
          105 kB

            sbourget Stephen Bourget
            lameze Simey Lameze
            Simey Lameze Simey Lameze
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Adrian Greeve Adrian Greeve
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.