Mwalker has uploaded a new change for review.

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


Change subject: FR #1113 Treat a subpage like it's a language
......................................................................

FR #1113 Treat a subpage like it's a language

This will allow thank you pages to be treated like landing
pages.

Change-Id: I64d407e226beb32e977219764fd3bee7433f66f0
---
M SpecialLandingCheck.php
1 file changed, 12 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/LandingCheck 
refs/changes/13/99613/1

diff --git a/SpecialLandingCheck.php b/SpecialLandingCheck.php
index 63d4e48..2d2be0d 100644
--- a/SpecialLandingCheck.php
+++ b/SpecialLandingCheck.php
@@ -26,14 +26,21 @@
        
        public function execute( $sub ) {
                global $wgRequest, $wgPriorityCountries;
-               
+
+               // If we have a subpage; assume it's a language like an 
internationalized page
+               $language = 'en';
+               $path = explode( '/', $sub );
+               if ( Language::isValidCode( $path[count($path) - 1] ) ) {
+                       $language = $sub;
+               }
+
                // Pull in query string parameters
-               $language = $wgRequest->getVal( 'language', 'en' );
+               $language = $wgRequest->getVal( 'language', $language );
                $this->basic = $wgRequest->getBool( 'basic' );
                $country = $wgRequest->getVal( 'country' );
 
                // if the language is false-ish, set to default
-               if( !$language ){
+               if( !$language ) {
                        $language = 'en';
                }
 
@@ -165,8 +172,8 @@
                        'utm_medium' => $wgRequest->getVal( 'utm_medium' ),
                        'utm_campaign' => $wgRequest->getVal( 'utm_campaign' ),
                        'utm_key' => $wgRequest->getVal( 'utm_key' ),
-                       'language' => $wgRequest->getVal( 'language', 'en'),
-                       'uselang' => $wgRequest->getVal( 'language', 'en'), // 
for {{int:xxx}} rendering
+                       'language' => $language,
+                       'uselang' => $language, // for {{int:xxx}} rendering
                        'country' => $country,
                        'referrer' => $wgRequest->getHeader( 'referer' )
                ) );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I64d407e226beb32e977219764fd3bee7433f66f0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/LandingCheck
Gerrit-Branch: master
Gerrit-Owner: Mwalker <[email protected]>

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

Reply via email to