From adc7c13f45a1958e5ed7e2904ee11e37992a8a75 Mon Sep 17 00:00:00 2001
From: Kevin Luo <kevkluo@gmail.com>
Date: Mon, 14 Jul 2014 17:04:42 -0700
Subject: [PATCH] Fix pending name changes template

- Strings marked for translation need to be wrapped in quotes to be
string type.
- Update old post function call
- Remove extra close brackets in function definitions
- Wrap href attribute in quotes
- Move square bracket outside of translated text
---
 lms/templates/name_changes.html | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/lms/templates/name_changes.html b/lms/templates/name_changes.html
index 58ea7db9be..72d55d3ef0 100644
--- a/lms/templates/name_changes.html
+++ b/lms/templates/name_changes.html
@@ -3,24 +3,24 @@
 <%inherit file="main.html" />
 <script>
 function name_confirm(id) {
-  postJSON('/accept_name_change',{"id":id},
+  $.post('/accept_name_change',{"id":id},
             function(data){
           if(data.success){
-            $("#div"+id).html(${_("Accepted")});
+            $("#div"+id).html("${_('Accepted')}");
           } else {
-            alert(${_('Error')});
-}          }
+            alert("${_('Error')}");
+          }
      });
 }
 
 function name_deny(id) {
-  postJSON('/reject_name_change',{"id":id},
+  $.post('/reject_name_change',{"id":id},
             function(data){
           if(data.success){
-            $("#div"+id).html(${_("Rejected")});
+            $("#div"+id).html("${_('Rejected')}");
           } else {
-            alert(${_('Error')});
-}          }
+            alert("${_('Error')}");
+          }
      });
 }
 </script>
@@ -32,12 +32,12 @@ function name_deny(id) {
   <table>
   % for s in students:
   <tr>
-    <td><a href=/profile/${s['uid']}/>${s['old_name']}</td>
+    <td><a href="/profile/${s['uid']}"/>${s['old_name']}</td>
     <td>${s['new_name']|h}</td>
     <td>${s['email']|h}</td>
     <td>${s['rationale']|h}</td>
     <td><span id="div${s['cid']}"><span onclick="name_confirm(${s['cid']});">[${_("Confirm")}]</span>
-    <span onclick="name_deny(${s['cid']});">${_("[Reject]")}</span></span></td></tr>
+    <span onclick="name_deny(${s['cid']});">[${_("Reject")}]</span></span></td></tr>
   % endfor
   </table>
   </section>