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

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

    XMLWordPrintable

Details

    • Improvement
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • 4.0
    • None
    • Libraries, Report builder
    • MOODLE_400_STABLE

    Description

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

      One of the comments from 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

              Unassigned Unassigned
              pholden Paul Holden
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Sara Arjona (@sarjona), David Matamoros, Mikel Martín Corrales, Paul Holden
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated: