Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HTML being included in quiz results and response #304

Closed
caperneoignis opened this issue Nov 2, 2018 · 2 comments · Fixed by #315
Closed

HTML being included in quiz results and response #304

caperneoignis opened this issue Nov 2, 2018 · 2 comments · Fixed by #315
Labels

Comments

@caperneoignis
Copy link
Contributor

Description

  • HTML is being included in quiz results, when the answers and questions are put into the Moodle provided html editor.

Steps to reproduce the bug

  1. Create quiz with HTML editor selected in Moodle's editor settings, I forget which one at the moment I'll update when I check the Moodle instance, but I think it happens with either. Add formatting to text, like bold-ing or underlining.
  2. Take quiz as a normal non-privileged user.
  3. view results in LRS.

Expected behavior

  • mod_quiz events should be displaying in plain text, or as close to it as possible. Because pulling events for analytics research is more difficult with html in the response string and question string. Harder to find quiz questions when they all start with html tags.

Actual behaviour

  • HTML tagging is being included in the strings when the events are displaying in the LRS since the html is being included in the string.

Server information

  • MySQL with normal user signin.

Client information
not applicable for this case, since it is happening server to server communication.

Additional information

  • LRS is LearningLocker, Moodle is 3.5, PHP 7.2, Redis elestic cache, and GlusterFS since all instances are clustered behind ELB.
@caperneoignis
Copy link
Contributor Author

I'll be updating to strip html next week, so I can have this updated pretty quick. May also work on the ID issue depending on work load, since we are about to go to production with our version of Moodle and LearningLocker.

@HT2Bot
Copy link
Member

HT2Bot commented Nov 27, 2018

🎉 This issue has been resolved in version 3.18.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants