From 7a1570885e7310e5b1e15ab798c9eb66694a96fb Mon Sep 17 00:00:00 2001
From: toasted-nutbread <toasted-nutbread@users.noreply.github.com>
Date: Fri, 9 Jul 2021 20:14:05 -0400
Subject: Simplify get media structure (#1817)

* Remove format argument of getMedia

* Implement escaping
---
 ext/js/templates/template-renderer-media-provider.js | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

(limited to 'ext/js/templates/template-renderer-media-provider.js')

diff --git a/ext/js/templates/template-renderer-media-provider.js b/ext/js/templates/template-renderer-media-provider.js
index 604b5331..114d3387 100644
--- a/ext/js/templates/template-renderer-media-provider.js
+++ b/ext/js/templates/template-renderer-media-provider.js
@@ -15,6 +15,10 @@
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
 
+/* global
+ * Handlebars
+ */
+
 class TemplateRendererMediaProvider {
     constructor() {
         this._requirements = null;
@@ -38,8 +42,7 @@ class TemplateRendererMediaProvider {
         const {media} = root;
         const data = this._getMediaData(media, args, namedArgs);
         if (data !== null) {
-            const {format} = namedArgs;
-            const result = this._getFormattedValue(data, format);
+            const result = this._getFormattedValue(data, namedArgs);
             if (typeof result === 'string') { return result; }
         }
         const defaultValue = namedArgs.default;
@@ -53,8 +56,13 @@ class TemplateRendererMediaProvider {
         this._requirements.push(value);
     }
 
-    _getFormattedValue(data, format) {
-        return Object.prototype.hasOwnProperty.call(data, format) ? data[format] : null;
+    _getFormattedValue(data, namedArgs) {
+        let {value} = data;
+        const {escape=true} = namedArgs;
+        if (escape) {
+            value = Handlebars.Utils.escapeExpression(value);
+        }
+        return value;
     }
 
     _getMediaData(media, args, namedArgs) {
-- 
cgit v1.2.3