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

Remove HTML tags when exporting to Dataformats that don't support it

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 4.0
    • Fix Version/s: None
    • Component/s: Libraries
    • Labels:
      None
    • Affected Branches:
      MOODLE_400_STABLE

      Description

      Since MDL-67547 dataformat writer classes can declare whether they support HTML rendering or not

      One of the comments from Sara Arjona (@sarjona) in her review of MDL-70794 regarding this:

      In reportbuilder/classes/output/dataformat_export_format.php there is a comment "Maybe this change should actually go in core.". Looking at the code in add_data method in lib/tablelib.php, it makes sense (although it seems that may impact in several places). I'm not an expert in table_dataformat_export_format so I can't evaluate how it will affect other places using it (probably third-party plugins). Thoughts?

      It makes sense in terms of consistency that a decision about this is taken for the whole of Moodle, rather than in just the Reportbuilder component, hence creating this issue

      Should dataformat writers that don't support rendering of HTML remove all HTML tags?

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              pholden Paul Holden
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated: