r102245 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r102244‎ | r102245 | r102246 >
Date:03:33, 7 November 2011
Author:awjrichards
Status:deferred
Tags:
Comment:
Modified paths:
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/globalcollect_gateway/forms/js/lightbox1.js (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms/js/lightbox1.js (modified) (history)

Diff [purge]

Index: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms/js/lightbox1.js
@@ -150,7 +150,6 @@
151151 )
152152
153153 /* Submit the form */
154 - /* TODO: Replace this with AJAX request */
155154 var sendData = {
156155 'action': 'donate',
157156 'gateway': 'payflowpro',
@@ -160,10 +159,10 @@
161160 'lname': $( "input[name='lname']" ).val(),
162161 'street': $( "input[name='street']" ).val(),
163162 'city': $( "input[name='city']" ).val(),
164 - 'state': $( "input[name='state']" ).val(),
 163+ 'state': $( 'select#state option:selected' ).val(),
165164 'zip': $( "input[name='zip']" ).val(),
166165 'emailAdd': $( "input[name='emailAdd']" ).val(),
167 - 'country': $( "input[name='country']" ).val(),
 166+ 'country': 'US',
168167 'payment_method': 'cc',
169168 'language': 'en',
170169
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms
___________________________________________________________________
Modified: svn:mergeinfo
171170 Merged /trunk/extensions/DonationInterface/payflowpro_gateway/forms:r102242
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway
___________________________________________________________________
Modified: svn:mergeinfo
172171 Merged /trunk/extensions/DonationInterface/payflowpro_gateway:r102242
Index: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/globalcollect_gateway/forms/js/lightbox1.js
@@ -141,7 +141,6 @@
142142 $( "input[name='country']" ).val('US' );
143143
144144 /* Submit the form */
145 - /* TODO: Replace this with AJAX request */
146145 var sendData = {
147146 'action': 'donate',
148147 'gateway': 'globalcollect',
@@ -151,10 +150,10 @@
152151 'lname': $( "input[name='lname']" ).val(),
153152 'street': $( "input[name='street']" ).val(),
154153 'city': $( "input[name='city']" ).val(),
155 - 'state': $( "input[name='state']" ).val(),
 154+ 'state': $( 'select#state option:selected' ).val(),
156155 'zip': $( "input[name='zip']" ).val(),
157156 'emailAdd': $( "input[name='emailAdd']" ).val(),
158 - 'country': $( "input[name='country']" ).val(),
 157+ 'country': 'US',
159158 'payment_method': 'cc',
160159 'language': 'en',
161160 'card_type': 'visa',
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface
___________________________________________________________________
Modified: svn:mergeinfo
162161 Merged /trunk/extensions/DonationInterface:r102242

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r102241follow-up to r102239, correct id for currency select listkaldari03:27, 7 November 2011
r102242hardcoding country for lightbox forms, using correct selector for statekaldari03:30, 7 November 2011

Status & tagging log