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

Make the returns from FormattedText content scans configurable (silent, quiet, noisy) to allow parity with all other major LMSs

    XMLWordPrintable

    Details

    • Type: Feature Request
    • Status: Verified
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.9.x
    • Fix Version/s: 10.0
    • Component/s: Kernel
    • Labels:
    • Property addition/change required:
      Yes
    • CLE Team Issue:
      Yes
    • Previous Issue Keys:
      KNL-1075

      Description

      Currently the FormattedText processing always returns a very wordy list of errors that are dumped on screen (sometimes). This is actually different from literally every other LMS. The rest of them silently strip dangerous content without even a warning.

      The suggestion is that there be 3 levels of output. Tools would simply dump whatever was returned from the method as before but the method would return nothing when in silent mode.
      (1) silent - just clean it
      (2) quiet - small notice which can be optionally expanded to show all the errors
      (3) noisy - all errors in a big ol box

      Noisy would be the the same as the current operation in announcements.
      Silent would be the same as the current operation in Forums.
      NOTE: Tools in Sakai are NOT consistent in the way they handle this so there should be another ticket to handle this consistently.

      IMPL notes:
      content.cleaner.errors.handling = none|logged|return|notify|display (Default: notify)

      • none - errors are completely ignored and not even stored at all
      • logged - errors are output in the logs only
      • return - errors are returned to the tool (legacy behavior)
      • notify - user notified about errors using a non-blocking JS popup
      • display - errors are displayed to the user using the new and fancy JS popup
        content.cleaner.errors.display = true|false (false by default)

        Gliffy Diagrams

          Attachments

          1. kernel_KNL-1075.patch.txt
            19 kB
          2. kernel_KNL-1075.patch.txt
            19 kB
          3. kernel_KNL-1075.patch.txt
            17 kB
          4. kernel_KNL-1075.patch.txt
            19 kB
          5. kernel_KNL-1075.patch.txt
            19 kB
          6. KNL-1075.patch
            18 kB
          7. KNL-1075.patch
            30 kB
          8. KNL-1075.patch
            28 kB
          9. KNL-1075.patch
            5 kB
          10. portal_KNL-1075.patch.txt
            2 kB

            Issue Links

              Activity

                People

                • Assignee:
                  aaronz Aaron Zeckoski (Inactive)
                  Reporter:
                  aaronz Aaron Zeckoski (Inactive)
                • Votes:
                  2 Vote for this issue
                  Watchers:
                  9 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Git Source Code