From fb013272b8bdc1e68fc3abe08dd3f7d5bc80f77e Mon Sep 17 00:00:00 2001
From: Renzo Lucioni <renzo@renzolucioni.com>
Date: Mon, 5 Jan 2015 15:34:32 -0500
Subject: [PATCH] Wrap full name placeholder in quotes to deal with spaces

Also fix failing JS test which was looking for a removed button.
---
 .../js/spec/verify_student/make_payment_step_view_spec.js | 8 +-------
 .../js/verify_student/views/make_payment_step_view.js     | 5 -----
 .../verify_student/review_photos_step.underscore          | 2 +-
 3 files changed, 2 insertions(+), 13 deletions(-)

diff --git a/lms/static/js/spec/verify_student/make_payment_step_view_spec.js b/lms/static/js/spec/verify_student/make_payment_step_view_spec.js
index 01abb15433..eb0fa26ecb 100644
--- a/lms/static/js/spec/verify_student/make_payment_step_view_spec.js
+++ b/lms/static/js/spec/verify_student/make_payment_step_view_spec.js
@@ -86,16 +86,10 @@ define([
             };
 
             var expectPaymentDisabledBecauseInactive = function() {
-                var payButton = $( '#pay_button'),
-                    activateButton = $( '#activate_button' );
+                var payButton = $( '#pay_button' );
 
                 // Payment button should be hidden
                 expect( payButton.length ).toEqual(0);
-
-                // Activate button should be displayed and disabled
-                expect( activateButton.length ).toEqual(1);
-                expect( activateButton.hasClass( 'is-disabled' ) ).toBe( true );
-                expect( activateButton.prop( 'disabled' ) ).toBe( true );
             };
 
             var goToPayment = function( requests, kwargs ) {
diff --git a/lms/static/js/verify_student/views/make_payment_step_view.js b/lms/static/js/verify_student/views/make_payment_step_view.js
index 641f3e15d4..225d2e6e93 100644
--- a/lms/static/js/verify_student/views/make_payment_step_view.js
+++ b/lms/static/js/verify_student/views/make_payment_step_view.js
@@ -38,11 +38,6 @@ var edx = edx || {};
             // Set the payment button to disabled by default
             this.setPaymentEnabled( false );
 
-            // The activate button is always disabled
-            $( '#activate_button' )
-                .addClass( 'is-disabled' )
-                .prop( 'disabled', true );
-
             // Update the contribution amount with the amount the user
             // selected in a previous screen.
             if ( templateContext.contributionAmount ) {
diff --git a/lms/templates/verify_student/review_photos_step.underscore b/lms/templates/verify_student/review_photos_step.underscore
index 351c5c0a0b..087de85b4d 100644
--- a/lms/templates/verify_student/review_photos_step.underscore
+++ b/lms/templates/verify_student/review_photos_step.underscore
@@ -35,7 +35,7 @@
 
                   <div class="copy expandable-area">
                     <p><%- gettext( "You should change the name on your account to match your ID." ) %></p>
-                    <input type="text" name="new-name" id="new-name" placeholder=<%= fullName %>>
+                    <input type="text" name="new-name" id="new-name" placeholder="<%= fullName %>">
                   </div>
                 </div>
               </li>