| Index: trunk/extensions/CentralNotice/SpecialBannerController.php |
| — | — | @@ -55,7 +55,8 @@ |
| 56 | 56 | 'fn': { |
| 57 | 57 | 'loadBanner': function( bannerName ) { |
| 58 | 58 | // Get the requested banner |
| 59 | | - var bannerPage = 'Special:BannerLoader?banner='+bannerName+'&userlang='+wgUserLanguage+'&db='+wgDBname+'&sitename='+wgSiteName+'&country='+Geo.country; |
| | 59 | + var bannerPageQuery = $.param( { 'banner': bannerName, 'userlang': wgUserLanguage, 'db': wgDBname, 'sitename': wgSiteName, 'country': Geo.country } ); |
| | 60 | + var bannerPage = 'Special:BannerLoader?' + bannerPageQuery; |
| 60 | 61 | EOT; |
| 61 | 62 | $js .= "\n\t\t\t\tvar bannerScript = '<script type=\"text/javascript\" src=\"".Xml::escapeJsString( $wgCentralPagePath )."' + bannerPage + '\"></script>';\n"; |
| 62 | 63 | $js .= <<<EOT |
| — | — | @@ -68,7 +69,8 @@ |
| 69 | 70 | } else { |
| 70 | 71 | var geoLocation = Geo.country; // pull the geo info |
| 71 | 72 | } |
| 72 | | - var bannerListPage = 'Special:BannerListLoader?language='+wgContentLanguage+'&project='+wgNoticeProject+'&country='+geoLocation; |
| | 73 | + var bannerListQuery = $.param( { 'language': wgContentLanguage, 'project': wgNoticeProject, 'country': geoLocation } ); |
| | 74 | + var bannerListPage = 'Special:BannerListLoader?' + bannerListQuery; |
| 73 | 75 | bannerListURL = wgArticlePath.replace( '$1', bannerListPage ); |
| 74 | 76 | var request = $.ajax( { |
| 75 | 77 | url: bannerListURL, |