| Index: trunk/extensions/ConfirmAccount/business/AccountRequestSubmission.php |
| — | — | @@ -44,6 +44,9 @@ |
| 45 | 45 | return $this->attachmentDidNotForget; |
| 46 | 46 | } |
| 47 | 47 | |
| | 48 | + /** |
| | 49 | + * @return string |
| | 50 | + */ |
| 48 | 51 | public function getAttachtmentPrevName() { |
| 49 | 52 | return $this->attachmentPrevName; |
| 50 | 53 | } |
| Index: trunk/extensions/ConfirmAccount/presentation/ConfirmAccountUI.hooks.php |
| — | — | @@ -6,6 +6,10 @@ |
| 7 | 7 | /** |
| 8 | 8 | * Register ConfirmAccount special pages as needed. |
| 9 | 9 | * Also sets $wgSpecialPages just to be consistent. |
| | 10 | + * |
| | 11 | + * @param $list array |
| | 12 | + * |
| | 13 | + * @return true |
| 10 | 14 | */ |
| 11 | 15 | public static function defineSpecialPages( array &$list ) { |
| 12 | 16 | global $wgSpecialPages, $wgSpecialPageGroups, $wgConfirmAccountSaveInfo; |
| Index: trunk/extensions/ConfirmAccount/presentation/specialpages/actions/ConfirmAccount_body.php |
| — | — | @@ -223,7 +223,10 @@ |
| 224 | 224 | $out->addHTML( '</ul>' ); |
| 225 | 225 | } |
| 226 | 226 | |
| 227 | | - protected function showAccountConfirmForm( $msg='' ) { |
| | 227 | + /** |
| | 228 | + * @param $msg string |
| | 229 | + */ |
| | 230 | + protected function showAccountConfirmForm( $msg = '' ) { |
| 228 | 231 | global $wgAccountRequestTypes; |
| 229 | 232 | $reqUser = $this->getUser(); |
| 230 | 233 | $out = $this->getOutput(); |
| — | — | @@ -435,6 +438,9 @@ |
| 436 | 439 | StreamFile::stream( $path ); |
| 437 | 440 | } |
| 438 | 441 | |
| | 442 | + /** |
| | 443 | + * @return bool |
| | 444 | + */ |
| 439 | 445 | protected function doAccountConfirmSubmit() { |
| 440 | 446 | $reqUser = $this->getUser(); |
| 441 | 447 | $out = $this->getOutput(); |
| — | — | @@ -779,8 +785,10 @@ |
| 780 | 786 | return true; |
| 781 | 787 | } |
| 782 | 788 | |
| 783 | | - /* |
| | 789 | + /** |
| 784 | 790 | * Get requested account request row and load some fields |
| | 791 | + * |
| | 792 | + * @param $forUpdate bool |
| 785 | 793 | */ |
| 786 | 794 | function getAccountRequest( $forUpdate = false ) { |
| 787 | 795 | if( !$this->acrID ) return false; |
| — | — | @@ -846,6 +854,11 @@ |
| 847 | 855 | return $linkList; |
| 848 | 856 | } |
| 849 | 857 | |
| | 858 | + /** |
| | 859 | + * @param $titleObj Title |
| | 860 | + * @param $name string |
| | 861 | + * @param $errors array |
| | 862 | + */ |
| 850 | 863 | protected function showSuccess( $titleObj, $name = null, $errors = array() ) { |
| 851 | 864 | $out = $this->getOutput(); |
| 852 | 865 | |
| — | — | @@ -903,6 +916,10 @@ |
| 904 | 917 | } |
| 905 | 918 | } |
| 906 | 919 | |
| | 920 | + /** |
| | 921 | + * @param $row |
| | 922 | + * @return string |
| | 923 | + */ |
| 907 | 924 | public function formatRow( $row ) { |
| 908 | 925 | global $wgUseRealNamesOnly, $wgAllowRealName, $wgMemc; |
| 909 | 926 | |
| — | — | @@ -1000,14 +1017,24 @@ |
| 1001 | 1018 | $this->mLimit = $urlLimit ? $urlLimit : 20; |
| 1002 | 1019 | } |
| 1003 | 1020 | |
| | 1021 | + /** |
| | 1022 | + * @return Title |
| | 1023 | + */ |
| 1004 | 1024 | function getTitle() { |
| 1005 | 1025 | return SpecialPage::getTitleFor( 'ConfirmAccounts', $this->mForm->specialPageParameter ); |
| 1006 | 1026 | } |
| 1007 | 1027 | |
| | 1028 | + /** |
| | 1029 | + * @param $row |
| | 1030 | + * @return string |
| | 1031 | + */ |
| 1008 | 1032 | function formatRow( $row ) { |
| 1009 | 1033 | return $this->mForm->formatRow( $row ); |
| 1010 | 1034 | } |
| 1011 | 1035 | |
| | 1036 | + /** |
| | 1037 | + * @return string |
| | 1038 | + */ |
| 1012 | 1039 | function getStartBody() { |
| 1013 | 1040 | if ( $this->getNumRows() ) { |
| 1014 | 1041 | return '<ul>'; |
| — | — | @@ -1016,6 +1043,9 @@ |
| 1017 | 1044 | } |
| 1018 | 1045 | } |
| 1019 | 1046 | |
| | 1047 | + /** |
| | 1048 | + * @return string |
| | 1049 | + */ |
| 1020 | 1050 | function getEndBody() { |
| 1021 | 1051 | if ( $this->getNumRows() ) { |
| 1022 | 1052 | return '</ul>'; |
| — | — | @@ -1024,6 +1054,9 @@ |
| 1025 | 1055 | } |
| 1026 | 1056 | } |
| 1027 | 1057 | |
| | 1058 | + /** |
| | 1059 | + * @return array |
| | 1060 | + */ |
| 1028 | 1061 | function getQueryInfo() { |
| 1029 | 1062 | $conds = $this->mConds; |
| 1030 | 1063 | $tables = array( 'account_requests' ); |
| — | — | @@ -1046,6 +1079,9 @@ |
| 1047 | 1080 | ); |
| 1048 | 1081 | } |
| 1049 | 1082 | |
| | 1083 | + /** |
| | 1084 | + * @return string |
| | 1085 | + */ |
| 1050 | 1086 | function getIndexField() { |
| 1051 | 1087 | return 'acr_registration'; |
| 1052 | 1088 | } |