Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-36056

Emailing test takers in Chrome or Safari causes stacktrace

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: CLOSED
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.7.1, 2.8.0
    • Fix Version/s: None
    • Labels:
      None
    • Previous Issue Keys:
      SAM-1130

      Description

      The feature that allows instructors to email test-takers in Tests & Quizzes causes a stacktrace in Chrome or Safari. To reproduce:

      In Chrome or Safari, go to Tests and Quizzes and view the scores of a published test. (At least one user must complete the test first.)
      Click on the "Email" link under one of the users' names.
      Click "Send"

      This will cause a stacktrace.

      I did some poking around and found the problem is caused by the "clickEmailLink()" javascript function on the totalScores.jsp (and some other pages as well). The problem boils down to the fact that the script tries to submit a form and then also load a different web page in the same script. Firefox and Internet Explorer seem to be able to handle this, but Chrome and Safari won't. They'll either load the page or submit the form, but not both. In the current configuration, Chrome/Safari don't submit the form. Since the form isn't submitted, Sakai doesn't know who the recipient of the email is supposed to be. When it tries to send it, it finds a null email address which causes a stacktrace.

      I tried to reshuffle the script to make it work in Chrome/Safari, but almost any changes caused the script to break in Firefox. Firefox would tend to stop navigating back to the totalScores page with any change. I'm not familiar enough with the inner workings of Tests and Quizzes to know what other solution to attempt.

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  jamai Jacqueline Mai (Inactive)
                  Reporter:
                  ansorgej John Ansorge
                  Votes:
                  2 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration