| Index: trunk/extensions/Collection/Collection.body.php |
| — | — | @@ -805,6 +805,11 @@ |
| 806 | 806 | return $collection; |
| 807 | 807 | } |
| 808 | 808 | |
| | 809 | + /** |
| | 810 | + * @param $title Title |
| | 811 | + * @param $forceOverwrite bool |
| | 812 | + * @return bool |
| | 813 | + */ |
| 809 | 814 | function saveCollection( $title, $forceOverwrite = false ) { |
| 810 | 815 | global $wgUser; |
| 811 | 816 | |
| Index: trunk/extensions/CodeReview/CodeReview.php |
| — | — | @@ -377,6 +377,10 @@ |
| 378 | 378 | # Unit tests |
| 379 | 379 | $wgHooks['UnitTestsList'][] = 'efCodeReviewUnitTests'; |
| 380 | 380 | |
| | 381 | +/** |
| | 382 | + * @param $files array |
| | 383 | + * @return bool |
| | 384 | + */ |
| 381 | 385 | function efCodeReviewUnitTests( &$files ) { |
| 382 | 386 | $files[] = dirname( __FILE__ ) . '/tests/CodeReviewApiTest.php'; |
| 383 | 387 | $files[] = dirname( __FILE__ ) . '/tests/CodeReviewTest.php'; |
| — | — | @@ -387,6 +391,10 @@ |
| 388 | 392 | # Add global JS vars |
| 389 | 393 | $wgHooks['MakeGlobalVariablesScript'][] = 'efCodeReviewResourceLoaderGlobals'; |
| 390 | 394 | |
| | 395 | +/** |
| | 396 | + * @param $values array |
| | 397 | + * @return bool |
| | 398 | + */ |
| 391 | 399 | function efCodeReviewResourceLoaderGlobals( &$values ){ |
| 392 | 400 | # Bleugh, this is horrible |
| 393 | 401 | global $wgTitle; |
| Index: trunk/extensions/LocalisationUpdate/QuickArrayReader.php |
| — | — | @@ -10,6 +10,9 @@ |
| 11 | 11 | class QuickArrayReader { |
| 12 | 12 | var $vars = array(); |
| 13 | 13 | |
| | 14 | + /** |
| | 15 | + * @param $string string |
| | 16 | + */ |
| 14 | 17 | function __construct( $string ) { |
| 15 | 18 | $scalarTypes = array( |
| 16 | 19 | T_LNUMBER => true, |
| — | — | @@ -118,6 +121,11 @@ |
| 119 | 122 | } |
| 120 | 123 | } |
| 121 | 124 | |
| | 125 | + /** |
| | 126 | + * @param $got string |
| | 127 | + * @param $expected string |
| | 128 | + * @return Exception |
| | 129 | + */ |
| 122 | 130 | private function except( $got, $expected ) { |
| 123 | 131 | if( is_array( $got ) ) { |
| 124 | 132 | $got = token_name( $got[0] ) . " ('" . $got[1] . "')"; |
| — | — | @@ -129,6 +137,9 @@ |
| 130 | 138 | |
| 131 | 139 | /** |
| 132 | 140 | * Parse a scalar value in PHP |
| | 141 | + * |
| | 142 | + * @param $token string |
| | 143 | + * |
| 133 | 144 | * @return mixed Parsed value |
| 134 | 145 | */ |
| 135 | 146 | function parseScalar( $token ) { |
| — | — | @@ -161,6 +172,10 @@ |
| 162 | 173 | return $str; |
| 163 | 174 | } |
| 164 | 175 | |
| | 176 | + /** |
| | 177 | + * @param $varname string |
| | 178 | + * @return null|string |
| | 179 | + */ |
| 165 | 180 | function getVar( $varname ) { |
| 166 | 181 | if( isset( $this->vars[$varname] ) ) { |
| 167 | 182 | return $this->vars[$varname]; |
| Index: trunk/extensions/SwiftMedia/SwiftMedia.body.php |
| — | — | @@ -334,6 +334,7 @@ |
| 335 | 335 | * We KNOW the container should exist, so puke if it doesn't. |
| 336 | 336 | * |
| 337 | 337 | * @param $conn CF_Connection |
| | 338 | + * @param $cont string |
| 338 | 339 | * |
| 339 | 340 | * @return CF_Container |
| 340 | 341 | */ |
| — | — | @@ -583,7 +584,11 @@ |
| 584 | 585 | return $status; |
| 585 | 586 | } |
| 586 | 587 | |
| 587 | | - |
| | 588 | + /** |
| | 589 | + * @param $title |
| | 590 | + * @param $archiveName |
| | 591 | + * @return OldSwiftFile |
| | 592 | + */ |
| 588 | 593 | function newFromArchiveName( $title, $archiveName ) { |
| 589 | 594 | return OldSwiftFile::newFromArchiveName( $title, $this, $archiveName ); |
| 590 | 595 | } |
| — | — | @@ -612,7 +617,7 @@ |
| 613 | 618 | list ( $cont, $rel ) = $rvu; |
| 614 | 619 | $container = $this->get_container( $conn, $cont ); |
| 615 | 620 | try { |
| 616 | | - $obj = $container->get_object( $rel ); |
| | 621 | + $container->get_object( $rel ); |
| 617 | 622 | $result[$key] = true; |
| 618 | 623 | } catch ( NoSuchObjectException $e ) { |
| 619 | 624 | $result[$key] = false; |
| — | — | @@ -622,7 +627,6 @@ |
| 623 | 628 | return $result; |
| 624 | 629 | } |
| 625 | 630 | |
| 626 | | - |
| 627 | 631 | // FIXME: do we really need to reject empty titles? |
| 628 | 632 | function newFile( $title, $time = false ) { |
| 629 | 633 | if ( empty( $title ) ) { |