From 27f853d6052ead9e505fdba391291ddb11f0ebbc Mon Sep 17 00:00:00 2001
From: Clinton Blackburn <cblackburn@edx.org>
Date: Fri, 11 Dec 2015 14:51:03 -0500
Subject: [PATCH] Followup to Credit Patch

- Removed unused variable
- Updated message class and text
- HTML-escaping relevant fields

ECOM-2893
---
 common/djangoapps/student/tests/test_credit.py      | 3 +--
 lms/templates/dashboard/_dashboard_credit_info.html | 7 ++-----
 2 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/common/djangoapps/student/tests/test_credit.py b/common/djangoapps/student/tests/test_credit.py
index 054aafed25..7e571b592b 100644
--- a/common/djangoapps/student/tests/test_credit.py
+++ b/common/djangoapps/student/tests/test_credit.py
@@ -116,9 +116,8 @@ class CreditCourseDashboardTest(ModuleStoreTestCase):
         self._make_eligible()
         self._purchase_credit()
 
-        # Expect that the user's status is "pending"
         response = self._load_dashboard()
-        self.assertContains(response, "credit-request-pending-msg")
+        self.assertContains(response, "credit-request-not-started-msg")
 
     def test_purchased_credit_and_request_pending(self):
         # Simulate that the user has purchased credit and initiated a request,
diff --git a/lms/templates/dashboard/_dashboard_credit_info.html b/lms/templates/dashboard/_dashboard_credit_info.html
index 07d31c5b23..4e0c3ad7e7 100644
--- a/lms/templates/dashboard/_dashboard_credit_info.html
+++ b/lms/templates/dashboard/_dashboard_credit_info.html
@@ -13,9 +13,6 @@
 
         error = credit_status['error']
 
-
-        status = 'eligible'
-
         # Translators: provider_name is the name of a credit provider or university (e.g. State University)
         credit_msg = _("You have completed this course and are eligible to purchase course credit. Select <strong>Get Credit</strong> to get started.")
         credit_msg_class = "credit-eligibility-msg"
@@ -35,7 +32,7 @@
                 "at the {link_to_provider_site} website. Select <b>Request Credit</b> to get started.").format(
                     link_to_provider_site=provider_link,
                 )
-                credit_msg_class = "credit-request-pending-msg"
+                credit_msg_class = "credit-request-not-started-msg"
                 credit_btn_label = _("Request Credit")
                 credit_btn_class = 'pending-credit-btn'
             elif request_status == 'pending':
@@ -77,7 +74,7 @@
         </p>
         <div class="credit-action">
             % if credit_btn_label:
-                <a class="btn credit-btn ${credit_btn_class}" href="${credit_btn_href}" target="_blank" data-course-key="${credit_status['course_key']}" data-user="${user.username}" data-provider="${credit_status['provider_id']}">
+                <a class="btn credit-btn ${credit_btn_class}" href="${credit_btn_href | h}" target="_blank" data-course-key="${credit_status['course_key'] | h}" data-user="${user.username | h}" data-provider="${credit_status['provider_id'] | h}">
                     ${credit_btn_label}
                 </a>
             % endif