From e92a615fd5d1c2690008a18b7f06b9a0852f6f6a Mon Sep 17 00:00:00 2001
From: Dani Palou <dani@moodle.com>
Date: Tue, 17 Dec 2019 11:41:19 +0100
Subject: [PATCH] MOBILE-3213 h5p: Fix closing tag in content validator

---
 src/core/h5p/classes/content-validator.ts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/core/h5p/classes/content-validator.ts b/src/core/h5p/classes/content-validator.ts
index 3d4baac9f..146fb10c1 100644
--- a/src/core/h5p/classes/content-validator.ts
+++ b/src/core/h5p/classes/content-validator.ts
@@ -503,7 +503,7 @@ export class CoreH5PContentValidator {
                     continue;
                 }
 
-                // Find semantics for name=$key
+                // Find semantics for name=key.
                 let found = false,
                     fn = null,
                     field = null;
@@ -729,7 +729,7 @@ export class CoreH5PContentValidator {
         }
 
         if (slash != '') {
-            return '</$elem>';
+            return '</' + elem + '>';
         }
 
         // Is there a closing XHTML slash at the end of the attributes?
@@ -871,8 +871,8 @@ export class CoreH5PContentValidator {
      * Processes an HTML attribute value and strips dangerous protocols from URLs.
      *
      * @param str The string with the attribute value.
-     * @param decode Whether to decode entities in the $string.
-     * @return Cleaned up and HTML-escaped version of $string.
+     * @param decode Whether to decode entities in the str.
+     * @return Cleaned up and HTML-escaped version of str.
      */
     filterXssBadProtocol(str: string, decode: boolean = true): string {
         // Get the plain text representation of the attribute value (i.e. its meaning).