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

Feedback export includes unwanted html in item names

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.5.8, 3.6.6, 3.7.2, 3.8
    • Fix Version/s: 3.7.4, 3.8.1
    • Component/s: Feedback
    • Labels:
    • Testing Instructions:
      Hide

      Test instructions

      1. Open a course and turn editing on
      2. Create a new Feedback activity
      3. Open "Edit questions"
      4. Add a new "Numeric answer" question
      5. Give it a range from from 5 to 10.
      6. Open the Overview and choose "Answer the questions"
      7. Go to "Show responses" tab and export as CSV
      8. Open the CSV
        1. Confirm that there is no HTML in the name of the question
      Show
      Test instructions Open a course and turn editing on Create a new Feedback activity Open "Edit questions" Add a new "Numeric answer" question Give it a range from from 5 to 10. Open the Overview and choose "Answer the questions" Go to "Show responses" tab and export as CSV Open the CSV Confirm that there is no HTML in the name of the question
    • Affected Branches:
      MOODLE_35_STABLE, MOODLE_36_STABLE, MOODLE_37_STABLE, MOODLE_38_STABLE
    • Fixed Branches:
      MOODLE_37_STABLE, MOODLE_38_STABLE
    • Pull from Repository:
    • Pull 3.8 Branch:
    • Pull Master Branch:
      MDL-66938-master

      Description

      The export of responses includes the boundaries part of a numeric field name. This is not supposed to be so.

      To test you can do this:

      1. Create a feedback instance
      2. Create at least on item from type "Numeric answer" and a given range (from - to)
      3. Submit at lease one response
      4. Go to "Show responses" tab and do an export
      5. The field for the numeric field includes the html part of the boundaries.

        Attachments

          Activity

            People

            Assignee:
            grabs Andreas Grabs
            Reporter:
            grabs Andreas Grabs
            Peer reviewer:
            Simey Lameze
            Integrator:
            Andrew Nicols
            Tester:
            Anna Carissa Sadia
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
            Votes:
            0 Vote for this issue
            Watchers:
            7 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              13/Jan/20

                Time Tracking

                Estimated:
                Original Estimate - 0 minutes
                0m
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 54 minutes
                54m