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

          Zeplin

            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 Integration