Awight has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/176037

Change subject: remote unused stylepath cruft
......................................................................

remote unused stylepath cruft

We're using ResourceLoader now.  There are many reasons we want to do that,
most importantly, transparent RTL conversion.

Change-Id: Ie3e234f9d1d218655d46efd945d3996afacaf9d2
---
M gateway_forms/Form.php
D gateway_forms/css/Form.css
2 files changed, 0 insertions(+), 39 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/DonationInterface 
refs/changes/37/176037/1

diff --git a/gateway_forms/Form.php b/gateway_forms/Form.php
index a5b2770..b70dbb1 100644
--- a/gateway_forms/Form.php
+++ b/gateway_forms/Form.php
@@ -77,20 +77,6 @@
                $this->form_errors = array_merge( 
DataValidator::getEmptyErrorArray(), $gateway_errors );
                $this->paypal = $wgRequest->getBool( 'paypal', false );
 
-               /**
-                *  add form-specific css - the path can be set in child classes
-                *  using $this->setStylePath, which should be called before
-                *  calling parent::__construct()
-                *
-                *  @TODO ditch this and start using ResourceLoader. Perhaps do 
something
-                *  similar to how resources are getting loaded in 
TwoStepTwoColumn and
-                *  its children.
-                */
-               if ( !strlen( $this->getStylePath() ) ) {
-                       $this->setStylePath();
-               }
-               $wgOut->addExtensionStyle( $this->getStylePath() );
-
                // This method should be overridden in the child class
                $this->init();
        }
@@ -101,31 +87,6 @@
         * overridden where necessary in child classes.
         */
        protected function init() {
-       }
-
-       /**
-        * Set the path to the CSS file for the form
-        *
-        * This should be a full path, perhaps taking advantage of 
$wgScriptPath.
-        * If you do not pass the path to the method, the style path will 
default
-        * to the default css in css/Form.css
-        * @param string $style_path
-        */
-       public function setStylePath( $style_path = null ) {
-               global $wgExtensionAssetsPath;
-               if ( !$style_path ) {
-                       // load the default form CSS if the style path not 
explicitly set
-                       $style_path = $wgExtensionAssetsPath . 
'/DonationInterface/gateway_forms/css/Form.css';
-               }
-               $this->style_path = $style_path;
-       }
-
-       /**
-        * Get the path to CSS
-        * @return String
-        */
-       public function getStylePath() {
-               return $this->style_path;
        }
 
        /**
diff --git a/gateway_forms/css/Form.css b/gateway_forms/css/Form.css
deleted file mode 100644
index e69de29..0000000
--- a/gateway_forms/css/Form.css
+++ /dev/null

-- 
To view, visit https://gerrit.wikimedia.org/r/176037
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie3e234f9d1d218655d46efd945d3996afacaf9d2
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/DonationInterface
Gerrit-Branch: master
Gerrit-Owner: Awight <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to