| Index: trunk/extensions/AssertEdit/AssertEdit.php |
| — | — | @@ -33,6 +33,10 @@ |
| 34 | 34 | $wgHooks['APIGetAllowedParams'][] = 'efAssertApiEditGetAllowedParams'; |
| 35 | 35 | $wgHooks['APIGetParamDescription'][] = 'efAssertApiEditGetParamDescription'; |
| 36 | 36 | |
| | 37 | +/** |
| | 38 | + * @param $editpage EditPage |
| | 39 | + * @return bool |
| | 40 | + */ |
| 37 | 41 | function efAssertEditHook( $editpage ) { |
| 38 | 42 | global $wgOut, $wgRequest; |
| 39 | 43 | |
| — | — | @@ -70,6 +74,12 @@ |
| 71 | 75 | } |
| 72 | 76 | } |
| 73 | 77 | |
| | 78 | +/** |
| | 79 | + * @param $editPage EditPage |
| | 80 | + * @param $textBox |
| | 81 | + * @param $result array |
| | 82 | + * @return bool|mixed |
| | 83 | + */ |
| 74 | 84 | function efAssertApiEditHook( $editPage, $textBox, &$result ) { |
| 75 | 85 | global $wgRequest; |
| 76 | 86 | |
| — | — | @@ -93,10 +103,15 @@ |
| 94 | 104 | $result['nassert'] = $assertName; |
| 95 | 105 | } |
| 96 | 106 | } |
| 97 | | - |
| | 107 | + |
| 98 | 108 | return $pass; |
| 99 | 109 | } |
| 100 | | - |
| | 110 | + |
| | 111 | +/** |
| | 112 | + * @param $module ApiBase |
| | 113 | + * @param $params array |
| | 114 | + * @return bool |
| | 115 | + */ |
| 101 | 116 | function efAssertApiEditGetAllowedParams( &$module, &$params ) { |
| 102 | 117 | if ( !$module instanceof ApiEditPage ) { |
| 103 | 118 | return true; |
| — | — | @@ -108,7 +123,12 @@ |
| 109 | 124 | |
| 110 | 125 | return true; |
| 111 | 126 | } |
| 112 | | - |
| | 127 | + |
| | 128 | +/** |
| | 129 | + * @param $module ApiBase |
| | 130 | + * @param $desc array |
| | 131 | + * @return bool |
| | 132 | + */ |
| 113 | 133 | function efAssertApiEditGetParamDescription( &$module, &$desc ) { |
| 114 | 134 | if ( !$module instanceof ApiEditPage ) { |
| 115 | 135 | return true; |
| Index: trunk/extensions/Video/SpecialAddVideo.php |
| — | — | @@ -53,7 +53,6 @@ |
| 54 | 54 | // If user is blocked, s/he doesn't need to access this page |
| 55 | 55 | if( $this->getUser()->isBlocked() ) { |
| 56 | 56 | throw new UserBlockedError( $this->getUser()->mBlock ); |
| 57 | | - return false; |
| 58 | 57 | } |
| 59 | 58 | |
| 60 | 59 | $this->setHeaders(); |
| Index: trunk/extensions/ApiSandbox/ext.apiSandbox.js |
| — | — | @@ -41,7 +41,7 @@ |
| 42 | 42 | setHelp: function( $container ) { |
| 43 | 43 | var desc = smartEscape( this.info.description ); |
| 44 | 44 | if ( isset( this.info.helpurls ) && isset( this.info.helpurls[0] ) && this.info.helpurls[0] ) { |
| 45 | | - desc = desc.replace( /^([^\r\n\.]*)/, |
| | 45 | + desc = desc.replace( /^([^\r\n\.]*)/, |
| 46 | 46 | '<a target="_blank" href="' + mw.html.escape( this.info.helpurls[0] ) + '">$1</a>' |
| 47 | 47 | ); |
| 48 | 48 | } |
| — | — | @@ -138,7 +138,7 @@ |
| 139 | 139 | if ( !$content.length ) { |
| 140 | 140 | return; |
| 141 | 141 | } |
| 142 | | - $content.show(); |
| | 142 | + $content.show(); |
| 143 | 143 | |
| 144 | 144 | // page elements |
| 145 | 145 | var $format = $( '#api-sandbox-format' ), |
| — | — | @@ -205,7 +205,7 @@ |
| 206 | 206 | |
| 207 | 207 | $action.change( updateUI ); |
| 208 | 208 | $query.change( updateUI ); |
| 209 | | - |
| | 209 | + |
| 210 | 210 | $( '#param-generator' ).live( 'change', function() { |
| 211 | 211 | var generator = $( '#param-generator' ).val(); |
| 212 | 212 | if ( generator == '' ) { |
| — | — | @@ -294,7 +294,7 @@ |
| 295 | 295 | |
| 296 | 296 | function showLoading( $element ) { |
| 297 | 297 | $element.html( |
| 298 | | - mw.html.element( 'img', |
| | 298 | + mw.html.element( 'img', |
| 299 | 299 | { src: mw.config.get( 'stylepath' ) + '/common/images/spinner.gif', alt: '' } ) |
| 300 | 300 | + mw.html.escape( mw.msg( 'apisb-loading' ) ) ); |
| 301 | 301 | } |
| Index: trunk/extensions/APC/SpecialAPC.php |
| — | — | @@ -5,7 +5,16 @@ |
| 6 | 6 | |
| 7 | 7 | // Stored objects |
| 8 | 8 | |
| 9 | | - protected $opts, $title; |
| | 9 | + /** |
| | 10 | + * @var FormOptions |
| | 11 | + */ |
| | 12 | + protected $opts; |
| | 13 | + |
| | 14 | + /** |
| | 15 | + * @var Title |
| | 16 | + */ |
| | 17 | + protected $title; |
| | 18 | + |
| 10 | 19 | function __construct() { |
| 11 | 20 | parent::__construct( 'APC' ); |
| 12 | 21 | $this->title = $this->getTitle(); |
| Index: trunk/extensions/ShortUrl/ShortUrl.php |
| — | — | @@ -15,12 +15,14 @@ |
| 16 | 16 | die( 1 ); |
| 17 | 17 | } |
| 18 | 18 | |
| 19 | | -// Configuration variables |
| 20 | | -// Prefix to use for the shortened URL. mod_rewrite (or equivalent) needs to be setup |
| 21 | | -// to produce a shorter URL |
| 22 | | -// Default is 'null' which just uses the (not so short) URL that all Special Pages get |
| 23 | | -// Eg: http://en.wikipedia.org/wiki/Special:ShortUrl/5234 |
| 24 | | -$wgShortUrlPrefix = null; |
| | 19 | +/** |
| | 20 | + * Configuration variables |
| | 21 | + * Prefix to use for the shortened URL. mod_rewrite (or equivalent) needs to be setup |
| | 22 | + * to produce a shorter URL |
| | 23 | + * Default is 'null' which just uses the (not so short) URL that all Special Pages get |
| | 24 | + * Eg: http://en.wikipedia.org/wiki/Special:ShortUrl/5234 |
| | 25 | + */ |
| | 26 | +$wgShortUrlPrefix = null; |
| 25 | 27 | |
| 26 | 28 | // Extension credits that will show up on Special:Version |
| 27 | 29 | $wgExtensionCredits['specialpage'][] = array( |
| — | — | @@ -52,4 +54,3 @@ |
| 53 | 55 | 'localBasePath' => dirname( __FILE__ ), |
| 54 | 56 | 'remoteExtPath' => 'ShortUrl' |
| 55 | 57 | ); |
| 56 | | - |
| Index: trunk/extensions/ShortUrl/ShortUrl.hooks.php |
| — | — | @@ -47,12 +47,13 @@ |
| 48 | 48 | /** |
| 49 | 49 | * @param $out OutputPage |
| 50 | 50 | * @param $text string the HTML text to be added |
| | 51 | + * @return bool |
| 51 | 52 | */ |
| 52 | 53 | public static function onOutputPageBeforeHTML( &$out, &$text ) { |
| 53 | 54 | global $wgOut; |
| 54 | 55 | $title = $wgOut->getTitle(); |
| 55 | 56 | if ( ShortUrlUtils::needsShortUrl( $title ) ) { |
| 56 | | - $wgOut->addModules( 'ext.shortUrl' ); |
| | 57 | + $wgOut->addModules( 'ext.shortUrl' ); |
| 57 | 58 | } |
| 58 | 59 | return true; |
| 59 | 60 | } |