From: Evan Broder <broder@mit.edu>
Date: Sun, 27 Dec 2009 17:21:03 +0000 (-0500)
Subject: Don't over-escape Javascript in the web templates.
X-Git-Tag: 0.1.5~22
X-Git-Url: http://xvm.mit.edu/gitweb/invirt/packages/invirt-web.git/commitdiff_plain/bd07973cb4926fa3418a3e44d539c379f3e5a85d?hp=b4105367ae5fb5a6a70ab0a9e50ca77ccbec83fe

Don't over-escape Javascript in the web templates.

$ doesn't need to be escaped in Mako templates when it's followed by
an open parenthesis.

svn path=/trunk/packages/invirt-web/; revision=2790
---

diff --git a/code/templates/list.mako b/code/templates/list.mako
index d513763..288e44f 100644
--- a/code/templates/list.mako
+++ b/code/templates/list.mako
@@ -54,7 +54,7 @@ VM List
 	<tr>
 	  <td>Autoinstall${self.fn.helppopup('Autoinstalls')}</td>
 	  <td><input type="radio" name="cd_or_auto" id="cd_or_auto_auto"
-                 onchange="\$('cdromlist').value = ''; \$('vmtype-linux').checked = true">
+                 onchange="$('cdromlist').value = ''; $('vmtype-linux').checked = true">
 ${self.fn.autoList(defaults.cdrom, "$('cd_or_auto_auto').checked = true;$('cdromlist').value = '';$('vmtype-linux').checked = true")}
 	      (experimental; 2-3 minutes, and you have a machine with empty root password.)
 	  </input>
@@ -62,7 +62,7 @@ ${self.fn.autoList(defaults.cdrom, "$('cd_or_auto_auto').checked = true;$('cdrom
 	<tr>
 	  <td>Boot CD</td>
 	  <td><input type="radio" name="cd_or_auto" id="cd_or_auto_cd" checked="checked"
-                onchange="\$('autoinstalllist').value = ''; \$('vmtype-linux-hvm').checked = true">
+                onchange="$('autoinstalllist').value = ''; $('vmtype-linux-hvm').checked = true">
 ${self.fn.cdromList(defaults.cdrom, "$('cd_or_auto_cd').checked = true;$('autoinstalllist').value = '';$('vmtype-linux-hvm').checked = true")}
 </td>
 	  </input>
@@ -147,7 +147,7 @@ ${has_vnc[machine]}
     <script type="text/javascript" src="/static/stripe.js"></script>
     <script type="text/javascript">
         document.observe("dom:loaded", function() {
-            stripe(\$('machinelist').getElementsByTagName('table')[0],
+            stripe($('machinelist').getElementsByTagName('table')[0],
                    'stripedrow');
         });
     </script>