[MDL-66938] Feedback export includes unwanted html in item names - Moodle Tracker

XML

Word

Printable

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:
- ci
- triaged

Testing Instructions:
Hide

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
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:
https://github.com/grabs/moodle
Pull 3.7 Branch:
~~MDL-66938~~-37
Pull 3.7 Diff URL:
https://github.com/moodle/moodle/compare/MOODLE_37_STABLE...grabs:MDL-66938-37
Pull 3.8 Branch:
~~MDL-66938~~-38
Pull 3.8 Diff URL:
https://github.com/moodle/moodle/compare/MOODLE_38_STABLE...grabs:MDL-66938-38
Pull Master Branch:
~~MDL-66938~~-master
Pull Master Diff URL:
https://github.com/moodle/moodle/compare/master...grabs: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:

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

Attachments

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

Attachments

export-with-html-fragments.png
7 kB
17/Oct/19 3:52 PM
MDL-66938.jpg
8 kB
08/Jan/20 9:30 AM

Issue Links

mentioned on

Commit - Merge branch 'MDL-66938-38' of https://github.com/grabs/moodle into MOODLE_38_STABLE

Activity

People

Assignee:: Andreas Grabs

Reporter:: Andreas Grabs

Peer reviewer:: Simey Lameze

Integrator:: Andrew Nicols

Tester:: Anna Carissa Sadia

Participants:: Andreas Grabs, Andrew Nicols, Anna Carissa Sadia, Carlos Escobedo, CiBoT, Gitlab MoodleCloud, Simey Lameze

Component watchers:: Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón

Votes:: 0 Vote for this issue

Watchers:: 7 Start watching this issue

Dates

Created:: 17/Oct/19 3:30 PM

Updated:: 14/Jan/20 10:41 AM

Resolved:: 09/Jan/20 11:31 PM

Fix Release Date:: 13/Jan/20

Time Tracking

Estimated:

0m

Remaining:

0m

Logged:

54m