| Index: trunk/phase3/includes/ImageQueryPage.php |
| — | — | @@ -15,7 +15,7 @@ |
| 16 | 16 | * |
| 17 | 17 | * @param $out OutputPage to print to |
| 18 | 18 | * @param $skin Skin: user skin to use [unused] |
| 19 | | - * @param $dbr Database (read) connection to use |
| | 19 | + * @param $dbr DatabaseBase (read) connection to use |
| 20 | 20 | * @param $res Integer: result pointer |
| 21 | 21 | * @param $num Integer: number of available result rows |
| 22 | 22 | * @param $offset Integer: paging offset |
| Index: trunk/phase3/includes/db/DatabaseOracle.php |
| — | — | @@ -34,6 +34,11 @@ |
| 35 | 35 | return $array_out; |
| 36 | 36 | } |
| 37 | 37 | |
| | 38 | + /** |
| | 39 | + * @param $db DatabaseBase |
| | 40 | + * @param $stmt |
| | 41 | + * @param bool $unique |
| | 42 | + */ |
| 38 | 43 | function __construct( &$db, $stmt, $unique = false ) { |
| 39 | 44 | $this->db =& $db; |
| 40 | 45 | |
| Index: trunk/phase3/includes/db/Database.php |
| — | — | @@ -217,7 +217,12 @@ |
| 218 | 218 | protected $mDoneWrites = false; |
| 219 | 219 | protected $mPHPError = false; |
| 220 | 220 | |
| 221 | | - protected $mServer, $mUser, $mPassword, $mConn = null, $mDBname; |
| | 221 | + protected $mServer, $mUser, $mPassword, $mDBname; |
| | 222 | + |
| | 223 | + /** |
| | 224 | + * @var DatabaseBase |
| | 225 | + */ |
| | 226 | + protected $mConn = null; |
| 222 | 227 | protected $mOpened = false; |
| 223 | 228 | |
| 224 | 229 | protected $mTablePrefix; |
| Index: trunk/phase3/includes/media/GIF.php |
| — | — | @@ -27,6 +27,10 @@ |
| 28 | 28 | return serialize($parsedGIFMetadata); |
| 29 | 29 | } |
| 30 | 30 | |
| | 31 | + /** |
| | 32 | + * @param $image File |
| | 33 | + * @return array|bool |
| | 34 | + */ |
| 31 | 35 | function formatMetadata( $image ) { |
| 32 | 36 | $meta = $image->getMetadata(); |
| 33 | 37 | |
| Index: trunk/phase3/includes/cache/LinkCache.php |
| — | — | @@ -85,6 +85,9 @@ |
| 86 | 86 | 'revision' => intval( $revision ) ); |
| 87 | 87 | } |
| 88 | 88 | |
| | 89 | + /** |
| | 90 | + * @param $title Title |
| | 91 | + */ |
| 89 | 92 | public function addBadLinkObj( $title ) { |
| 90 | 93 | $dbkey = $title->getPrefixedDbKey(); |
| 91 | 94 | if ( !$this->isBadLink( $dbkey ) ) { |
| — | — | @@ -96,6 +99,9 @@ |
| 97 | 100 | unset( $this->mBadLinks[$title] ); |
| 98 | 101 | } |
| 99 | 102 | |
| | 103 | + /** |
| | 104 | + * @param $title Title |
| | 105 | + */ |
| 100 | 106 | public function clearLink( $title ) { |
| 101 | 107 | $dbkey = $title->getPrefixedDbKey(); |
| 102 | 108 | if( isset($this->mBadLinks[$dbkey]) ) { |
| Index: trunk/phase3/includes/specials/SpecialRecentchangeslinked.php |
| — | — | @@ -220,6 +220,9 @@ |
| 221 | 221 | return $extraOpts; |
| 222 | 222 | } |
| 223 | 223 | |
| | 224 | + /** |
| | 225 | + * @return Title |
| | 226 | + */ |
| 224 | 227 | function getTargetTitle() { |
| 225 | 228 | if ( $this->rclTargetTitle === null ) { |
| 226 | 229 | $opts = $this->getOptions(); |
| Index: trunk/phase3/includes/specials/SpecialMostcategories.php |
| — | — | @@ -52,6 +52,11 @@ |
| 53 | 53 | ); |
| 54 | 54 | } |
| 55 | 55 | |
| | 56 | + /** |
| | 57 | + * @param $skin Skin |
| | 58 | + * @param $result |
| | 59 | + * @return string |
| | 60 | + */ |
| 56 | 61 | function formatResult( $skin, $result ) { |
| 57 | 62 | global $wgLang; |
| 58 | 63 | $title = Title::makeTitleSafe( $result->namespace, $result->title ); |
| Index: trunk/phase3/includes/specials/SpecialPopularpages.php |
| — | — | @@ -48,6 +48,11 @@ |
| 49 | 49 | 'page_namespace' => MWNamespace::getContentNamespaces() ) ); |
| 50 | 50 | } |
| 51 | 51 | |
| | 52 | + /** |
| | 53 | + * @param $skin Skin |
| | 54 | + * @param $result |
| | 55 | + * @return string |
| | 56 | + */ |
| 52 | 57 | function formatResult( $skin, $result ) { |
| 53 | 58 | global $wgLang, $wgContLang; |
| 54 | 59 | $title = Title::makeTitle( $result->namespace, $result->title ); |
| Index: trunk/phase3/includes/specials/SpecialUnwatchedpages.php |
| — | — | @@ -60,6 +60,11 @@ |
| 61 | 61 | return array( 'page_namespace', 'page_title' ); |
| 62 | 62 | } |
| 63 | 63 | |
| | 64 | + /** |
| | 65 | + * @param $skin Skin |
| | 66 | + * @param $result |
| | 67 | + * @return string |
| | 68 | + */ |
| 64 | 69 | function formatResult( $skin, $result ) { |
| 65 | 70 | global $wgContLang; |
| 66 | 71 | |
| Index: trunk/phase3/includes/specials/SpecialWhatlinkshere.php |
| — | — | @@ -28,9 +28,18 @@ |
| 29 | 29 | */ |
| 30 | 30 | class SpecialWhatLinksHere extends SpecialPage { |
| 31 | 31 | |
| 32 | | - // Stored objects |
| 33 | | - protected $opts, $target, $selfTitle; |
| | 32 | + /** |
| | 33 | + * @var FormOptions |
| | 34 | + */ |
| | 35 | + protected $opts; |
| 34 | 36 | |
| | 37 | + protected $selfTitle; |
| | 38 | + |
| | 39 | + /** |
| | 40 | + * @var Title |
| | 41 | + */ |
| | 42 | + protected $target; |
| | 43 | + |
| 35 | 44 | protected $limits = array( 20, 50, 100, 250, 500 ); |
| 36 | 45 | |
| 37 | 46 | public function __construct() { |
| Index: trunk/phase3/includes/specials/SpecialShortpages.php |
| — | — | @@ -61,6 +61,11 @@ |
| 62 | 62 | return array( 'page_len' ); |
| 63 | 63 | } |
| 64 | 64 | |
| | 65 | + /** |
| | 66 | + * @param $db DatabaseBase |
| | 67 | + * @param $res |
| | 68 | + * @return void |
| | 69 | + */ |
| 65 | 70 | function preprocessResults( $db, $res ) { |
| 66 | 71 | # There's no point doing a batch check if we aren't caching results; |
| 67 | 72 | # the page must exist for it to have been pulled out of the table |
| Index: trunk/phase3/includes/specials/SpecialUnusedtemplates.php |
| — | — | @@ -54,6 +54,11 @@ |
| 55 | 55 | ); |
| 56 | 56 | } |
| 57 | 57 | |
| | 58 | + /** |
| | 59 | + * @param $skin Skin |
| | 60 | + * @param $result |
| | 61 | + * @return string |
| | 62 | + */ |
| 58 | 63 | function formatResult( $skin, $result ) { |
| 59 | 64 | $title = Title::makeTitle( NS_TEMPLATE, $result->title ); |
| 60 | 65 | $pageLink = $skin->linkKnown( |
| Index: trunk/phase3/includes/specials/SpecialWantedcategories.php |
| — | — | @@ -48,6 +48,11 @@ |
| 49 | 49 | ); |
| 50 | 50 | } |
| 51 | 51 | |
| | 52 | + /** |
| | 53 | + * @param $skin Skin |
| | 54 | + * @param $result |
| | 55 | + * @return string |
| | 56 | + */ |
| 52 | 57 | function formatResult( $skin, $result ) { |
| 53 | 58 | global $wgLang, $wgContLang; |
| 54 | 59 | |
| Index: trunk/phase3/includes/specials/SpecialMostlinked.php |
| — | — | @@ -57,6 +57,9 @@ |
| 58 | 58 | |
| 59 | 59 | /** |
| 60 | 60 | * Pre-fill the link cache |
| | 61 | + * |
| | 62 | + * @param $db DatabaseBase |
| | 63 | + * @param $res |
| 61 | 64 | */ |
| 62 | 65 | function preprocessResults( $db, $res ) { |
| 63 | 66 | if( $db->numRows( $res ) > 0 ) { |
| Index: trunk/phase3/includes/specials/SpecialListredirects.php |
| — | — | @@ -63,6 +63,8 @@ |
| 64 | 64 | |
| 65 | 65 | /** |
| 66 | 66 | * Cache page existence for performance |
| | 67 | + * |
| | 68 | + * @param $db DatabaseBase |
| 67 | 69 | */ |
| 68 | 70 | function preprocessResults( $db, $res ) { |
| 69 | 71 | $batch = new LinkBatch; |
| Index: trunk/phase3/includes/specials/SpecialMergeHistory.php |
| — | — | @@ -35,6 +35,10 @@ |
| 36 | 36 | parent::__construct( 'MergeHistory', 'mergehistory' ); |
| 37 | 37 | } |
| 38 | 38 | |
| | 39 | + /** |
| | 40 | + * @param $request WebRequest |
| | 41 | + * @return void |
| | 42 | + */ |
| 39 | 43 | private function loadRequestParams( $request ) { |
| 40 | 44 | global $wgUser; |
| 41 | 45 | |
| Index: trunk/phase3/includes/LogPage.php |
| — | — | @@ -38,7 +38,18 @@ |
| 39 | 39 | const SUPPRESSED_USER = 12; |
| 40 | 40 | const SUPPRESSED_ACTION = 9; |
| 41 | 41 | /* @access private */ |
| 42 | | - var $type, $action, $comment, $params, $target, $doer; |
| | 42 | + var $type, $action, $comment, $params; |
| | 43 | + |
| | 44 | + /** |
| | 45 | + * @var User |
| | 46 | + */ |
| | 47 | + var $doer; |
| | 48 | + |
| | 49 | + /** |
| | 50 | + * @var Title |
| | 51 | + */ |
| | 52 | + var $target; |
| | 53 | + |
| 43 | 54 | /* @acess public */ |
| 44 | 55 | var $updateRecentChanges, $sendToUDP; |
| 45 | 56 | |