do not extend `String.prototype` with non-standard methods

diff --git a/public/js/app.js b/public/js/app.js
index b5156e8c1c139f02fe7a2ec3b12bbfb558bb382f..c5c307c40d4953be4b5d8f8c89b83db4ba1abefa 100644 (file)
--- a/public/js/app.js
+++ b/public/js/app.js
@@ -96,10 +96,6 @@
     , 'Yes, you guessed the title. Who is the artist?'
   ];
 
-  String.prototype.encodeEntities = function() {
-    return this.replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;');
-  };
-
   var addCassetteBackdrop = function() {
     var html = [
       '<div id="touch-backdrop">'
@@ -540,6 +536,10 @@
     $users.empty();
   };
 
+  var encodeEntities = function(str) {
+    return str.replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;');
+  };
+
   var gameOver = function(podium) {
     var html = [
       '<div class="modal-header">'
@@ -1077,7 +1077,7 @@
         , splits = text.split(urlregex);
 
       for (var i = 0; i < splits.length; i++) {
-        var escapedsplit = splits[i].encodeEntities();
+        var escapedsplit = encodeEntities(splits[i]);
 
         if (urlregex.test(splits[i])) {
           html += '<a target="_blank" href="' + escapedsplit + '">' +
@@ -1091,7 +1091,7 @@
       return html;
     }
 
-    return text.encodeEntities();
+    return encodeEntities(text);
   };
 
   // A new player has joined the game