r99124 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r99123‎ | r99124 | r99125 >
Date:17:55, 6 October 2011
Author:awjrichards
Status:deferred
Tags:
Comment:
MFT r99044 through r99121
Modified paths:
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/extras/minfraud/ccfd/CreditCardFraudDetection.php (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/extras/recaptcha/recaptcha-php/recaptchalib.php (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/Form.php (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms/rapidhtml/html (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms/rapidhtml/html/globalcollect_test.html (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms/rapidhtml/html/globalcollect_test_2.html (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms/rapidhtml/html/webitects_2_3step.html (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.alias.php (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.body.php (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.i18n.php (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.php (modified) (history)
  • /branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/validate_input.js (modified) (history)

Diff [purge]

Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.alias.php
___________________________________________________________________
Modified: svn:mergeinfo
11 Merged /trunk/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.alias.php:r99045-99121
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/validate_input.js
___________________________________________________________________
Modified: svn:mergeinfo
22 Merged /trunk/extensions/DonationInterface/payflowpro_gateway/validate_input.js:r99045-99121
Index: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.body.php
@@ -54,15 +54,6 @@
5555 $wgPayFlowProGatewayCSSVersion,
5656 $wgPayflowGatewaySalt;
5757
58 - $wgOut->addExtensionStyle(
59 - "{$wgScriptPath}/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.css?284" .
60 - $wgPayFlowProGatewayCSSVersion );
61 -
62 - // Hide unneeded interface elements
63 - $wgOut->addExtensionStyle(
64 - "{$wgScriptPath}/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.css?284" .
65 - $wgPayFlowProGatewayCSSVersion );
66 -
6758 $scriptVars = array(
6859 'payflowproGatewayErrorMsgJs' => wfMsg( 'payflowpro_gateway-error-msg-js' ),
6960 'payflowproGatewayErrorMsgEmail' => wfMsg( 'payflowpro_gateway-error-msg-email' ),
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/extras/minfraud/ccfd/CreditCardFraudDetection.php
___________________________________________________________________
Modified: svn:mergeinfo
7061 Merged /trunk/extensions/DonationInterface/payflowpro_gateway/extras/minfraud/ccfd/CreditCardFraudDetection.php:r99045-99121
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/extras/recaptcha/recaptcha-php/recaptchalib.php
___________________________________________________________________
Modified: svn:mergeinfo
7162 Merged /trunk/extensions/DonationInterface/payflowpro_gateway/extras/recaptcha/recaptcha-php/recaptchalib.php:r99045-99121
Index: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms/rapidhtml/html/globalcollect_test.html
@@ -1,6 +1,6 @@
2 -<link rel="stylesheet" href="@script_path/skins/vector/main-ltr.css" />
3 -<link rel="stylesheet" href="@script_path/extensions/DonationInterface/payflowpro_gateway/forms/css/TwoStepTwoColumnLetter3.css" />
4 -<link rel="stylesheet" href="@script_path/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.css" />
 2+<script type="text/javascript">
 3+mw.loader.load('pfp.form.rapidhtml.globalcollect_test');
 4+</script>
55 <style>
66 .step2hidden{ display:none; }
77 #footer-places { display:none; }
Index: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms/rapidhtml/html/globalcollect_test_2.html
@@ -1,6 +1,6 @@
2 -<link rel="stylesheet" href="@script_path/skins/vector/main-ltr.css" />
3 -<link rel="stylesheet" href="@script_path/extensions/DonationInterface/payflowpro_gateway/forms/css/TwoStepTwoColumnLetter3.css" />
4 -<link rel="stylesheet" href="@script_path/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.css" />
 2+<script type="text/javascript">
 3+mw.loader.load('pfp.form.rapidhtml.globalcollect_test');
 4+</script>
55 <style>
66 .step2hidden{ display:none; }
77 #footer-places { display:none; }
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms/rapidhtml/html/webitects_2_3step.html
___________________________________________________________________
Modified: svn:mergeinfo
88 Merged /trunk/extensions/DonationInterface/payflowpro_gateway/forms/rapidhtml/html/webitects_2_3step.html:r99045-99121
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms/rapidhtml/html
___________________________________________________________________
Modified: svn:mergeinfo
99 Merged /trunk/extensions/DonationInterface/payflowpro_gateway/forms/rapidhtml/html:r99045-99121
Index: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms/Form.php
@@ -652,10 +652,8 @@
653653 }
654654
655655 protected function loadValidateJs() {
656 - global $wgOut, $wgScriptPath;
657 - $wgOut->addHeadItem( 'validatescript', '<script type="text/javascript" src="' .
658 - $wgScriptPath .
659 - '/extensions/DonationInterface/payflowpro_gateway/validate_input.js?284"></script>' );
 656+ global $wgOut;
 657+ $wgOut->addModules( 'pfp.form.core.validate' );
660658 }
661659
662660 protected function loadApiJs() {
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/forms
___________________________________________________________________
Modified: svn:mergeinfo
663661 Merged /trunk/extensions/DonationInterface/payflowpro_gateway/forms:r99045-99121
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.i18n.php
___________________________________________________________________
Modified: svn:mergeinfo
664662 Merged /trunk/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.i18n.php:r99045-99121
Index: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway/payflowpro_gateway.php
@@ -183,6 +183,37 @@
184184 'remoteExtPath' => 'DonationInterface/payflowpro_gateway'
185185 );
186186
 187+// RapidHtml globalcollect_test form resources
 188+$wgResourceModules[ 'pfp.form.rapidhtml.globalcollect_test' ] = array(
 189+ 'styles' => array(
 190+ 'forms/css/TwoStepTwoColumnLetter3.css',
 191+ 'payflowpro_gateway.css',
 192+ ),
 193+ 'scripts' => array(),
 194+ 'dependencies' => array(),
 195+ 'localBasePath' => dirname( __FILE__ ),
 196+ 'remoteExtPath' => 'DonationInterface/payflowpro_gateway'
 197+);
 198+
 199+// form validation resource
 200+$wgResourceModules[ 'pfp.form.core.validate' ] = array(
 201+ 'styles' => array(),
 202+ 'scripts' => 'validate_input.js',
 203+ 'dependencies' => 'pfp.form.core.pfp_css',
 204+ 'localBasePath' => dirname( __FILE__ ),
 205+ 'remoteExtPath' => 'DonationInterface/payflowpro_gateway'
 206+);
 207+
 208+// general PFP css
 209+$wgResourceModules[ 'pfp.form.core.pfp_css' ] = array(
 210+ 'styles' => 'payflowpro_gateway.css',
 211+ 'scripts' => array(),
 212+ 'dependencies' => array(),
 213+ 'localBasePath' => dirname( __FILE__ ),
 214+ 'remoteExtPath' => 'DonationInterface/payflowpro_gateway'
 215+);
 216+
 217+// RapidHtml lightbox form resources
187218 $wgResourceModules[ 'pfp.form.rapidhtml.lightbox.js' ] = array(
188219 'scripts' => array(
189220 'forms/rapidhtml/js/lightbox1.js',
@@ -202,6 +233,9 @@
203234 'position' => 'top',
204235 );
205236
 237+// RapidHtml lightbox form css resources (these are separate from the js
 238+// resources for a good reason but I forget what - I believe to facilitate
 239+// ensuring proper load order?
206240 $wgResourceModules[ 'pfp.form.rapidhtml.lightbox.css' ] = array(
207241 'styles' => array(
208242 'forms/rapidhtml/css/lightbox1.css',
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface/payflowpro_gateway
___________________________________________________________________
Modified: svn:mergeinfo
209243 Merged /trunk/extensions/DonationInterface/payflowpro_gateway:r99045-99121
Property changes on: branches/fundraising/deployment/payments_1.17/extensions/DonationInterface
___________________________________________________________________
Modified: svn:mergeinfo
210244 Merged /trunk/extensions/DonationInterface:r99045-99121

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r99044MFT r97944 through r98891awjrichards20:16, 5 October 2011
r99121Localization update for he.rotem17:05, 6 October 2011

Status & tagging log