@@ -113,7 +113,7 @@ discard block |
||
113 | 113 | /** |
114 | 114 | * @since 2.5 |
115 | 115 | * |
116 | - * @param array|string $errorMsg |
|
116 | + * @param array|string $error |
|
117 | 117 | * @param DIProperty|null $property |
118 | 118 | * |
119 | 119 | * @return DIContainer|null |
@@ -171,6 +171,11 @@ discard block |
||
171 | 171 | return new DIContainer( $containerSemanticData ); |
172 | 172 | } |
173 | 173 | |
174 | + /** |
|
175 | + * @param ContainerSemanticData $containerSemanticData |
|
176 | + * @param DIProperty|null $property |
|
177 | + * @param string $error |
|
178 | + */ |
|
174 | 179 | private function addToContainerSemanticData( $containerSemanticData, $property, $error ) { |
175 | 180 | |
176 | 181 | if ( $property !== null ) { |
@@ -186,6 +191,9 @@ discard block |
||
186 | 191 | ); |
187 | 192 | } |
188 | 193 | |
194 | + /** |
|
195 | + * @param string $hash |
|
196 | + */ |
|
189 | 197 | private function newContainerSemanticData( $hash ) { |
190 | 198 | |
191 | 199 | if ( $this->subject === null ) { |
@@ -86,7 +86,7 @@ discard block |
||
86 | 86 | * |
87 | 87 | * @since 2.5 |
88 | 88 | * |
89 | - * @return boolean |
|
89 | + * @return null|boolean |
|
90 | 90 | */ |
91 | 91 | public function run() { |
92 | 92 | |
@@ -160,6 +160,9 @@ discard block |
||
160 | 160 | $this->doRebuildFromRows( $searchTable, $table, $pid, $rows ); |
161 | 161 | } |
162 | 162 | |
163 | + /** |
|
164 | + * @param SearchTable $searchTable |
|
165 | + */ |
|
163 | 166 | private function doRebuildFromRows( $searchTable, $table, $pid, $rows ) { |
164 | 167 | |
165 | 168 | $i = 0; |
@@ -203,6 +206,9 @@ discard block |
||
203 | 206 | $this->reportMessage( "\n" ); |
204 | 207 | } |
205 | 208 | |
209 | + /** |
|
210 | + * @param string $message |
|
211 | + */ |
|
206 | 212 | private function reportMessage( $message, $verbose = true ) { |
207 | 213 | if ( $verbose ) { |
208 | 214 | $this->messageReporter->reportMessage( $message ); |
@@ -119,6 +119,9 @@ |
||
119 | 119 | return $reference === false || $reference === null ? false : $reference; |
120 | 120 | } |
121 | 121 | |
122 | + /** |
|
123 | + * @param integer $id |
|
124 | + */ |
|
122 | 125 | private function findReferenceByPropertyTable( $proptable, $id ) { |
123 | 126 | |
124 | 127 | $row = false; |
@@ -186,14 +186,17 @@ discard block |
||
186 | 186 | * The function sets the provided call-by-ref values to the respective |
187 | 187 | * values. If errors are encountered, they are added to the objects |
188 | 188 | * error list and false is returned. Otherwise, true is returned. |
189 | - * @param $string string input time representation, e.g. "12 May 2007 13:45:23-3:30" |
|
189 | + * @param string $string string input time representation, e.g. "12 May 2007 13:45:23-3:30" |
|
190 | 190 | * @param $datecomponents array of strings that might belong to the specification of a date |
191 | 191 | * @param $calendarmodesl string if model was set in input, otherwise false |
192 | - * @param $era string '+' or '-' if provided, otherwise false |
|
193 | - * @param $hours integer set to a value between 0 and 24 |
|
194 | - * @param $minutes integer set to a value between 0 and 59 |
|
195 | - * @param $seconds integer set to a value between 0 and 59, or false if not given |
|
196 | - * @param $timeoffset double set to a value for time offset (e.g. 3.5), or false if not given |
|
192 | + * @param boolean $era string '+' or '-' if provided, otherwise false |
|
193 | + * @param boolean $hours integer set to a value between 0 and 24 |
|
194 | + * @param boolean $minutes integer set to a value between 0 and 59 |
|
195 | + * @param boolean $seconds integer set to a value between 0 and 59, or false if not given |
|
196 | + * @param boolean $timeoffset double set to a value for time offset (e.g. 3.5), or false if not given |
|
197 | + * @param boolean $calendarmodel |
|
198 | + * @param boolean $microseconds |
|
199 | + * @param boolean $timezone |
|
197 | 200 | * @return boolean stating if the parsing succeeded |
198 | 201 | * @todo This method in principle allows date parsing to be internationalized further. Should be done. |
199 | 202 | */ |
@@ -409,7 +412,7 @@ discard block |
||
409 | 412 | * messages are added to the objects list of errors, and false is |
410 | 413 | * returned. Otherwise, true is returned. |
411 | 414 | * @param $datecomponents array of strings that might belong to the specification of a date |
412 | - * @param $date array set to result |
|
415 | + * @param boolean $date array set to result |
|
413 | 416 | * @return boolean stating if successful |
414 | 417 | */ |
415 | 418 | protected function interpretDateComponents( $datecomponents, &$date ) { |
@@ -807,10 +810,16 @@ discard block |
||
807 | 810 | } |
808 | 811 | } |
809 | 812 | |
813 | + /** |
|
814 | + * @param string $value |
|
815 | + */ |
|
810 | 816 | private function isInterpretableAsYearOnly( $value ) { |
811 | 817 | return strpos( $value, ' ' ) === false && is_numeric( strval( $value ) ) && ( strval( $value ) < 0 || strlen( $value ) < 6 ); |
812 | 818 | } |
813 | 819 | |
820 | + /** |
|
821 | + * @param string $value |
|
822 | + */ |
|
814 | 823 | private function isInterpretableAsTimestamp( $value ) { |
815 | 824 | // 1200-11-02T12:03:25 or 20120320055913 |
816 | 825 | return ( ( strlen( $value ) > 4 && substr( $value, 10, 1 ) === 'T' ) || strlen( $value ) == 14 ) && wfTimestamp( TS_MW, $value ) !== false; |
@@ -770,6 +770,9 @@ discard block |
||
770 | 770 | return '(' . $downloadLinks . ')'; |
771 | 771 | } |
772 | 772 | |
773 | + /** |
|
774 | + * @param string $html |
|
775 | + */ |
|
773 | 776 | private function doFinalModificationsOnBorrowedOutput( &$html, &$searchInfoText ) { |
774 | 777 | |
775 | 778 | if ( !$this->getRequest()->getCheck( 'bTitle' ) ) { |
@@ -790,6 +793,7 @@ discard block |
||
790 | 793 | |
791 | 794 | /** |
792 | 795 | * FIXME MW 1.25 |
796 | + * @param string $key |
|
793 | 797 | */ |
794 | 798 | private function addExternalHelpLinkFor( $key ) { |
795 | 799 |
@@ -94,6 +94,9 @@ discard block |
||
94 | 94 | } |
95 | 95 | } |
96 | 96 | |
97 | + /** |
|
98 | + * @param DIProperty $property |
|
99 | + */ |
|
97 | 100 | protected function getResourceElementForProperty( $property ) { |
98 | 101 | |
99 | 102 | $key = 'resource:builder:' . $property->getKey(); |
@@ -112,6 +115,9 @@ discard block |
||
112 | 115 | return $resourceElement; |
113 | 116 | } |
114 | 117 | |
118 | + /** |
|
119 | + * @param DIProperty $property |
|
120 | + */ |
|
115 | 121 | protected function getResourceElementHelperForProperty( $property ) { |
116 | 122 | |
117 | 123 | $key = 'resource:builder:aux:' . $property->getKey(); |
@@ -40,7 +40,6 @@ discard block |
||
40 | 40 | /** |
41 | 41 | * @see DataValue::getShortWikiText |
42 | 42 | * |
43 | - * @param string $value |
|
44 | 43 | */ |
45 | 44 | public function getShortWikiText( $linker = null ) { |
46 | 45 | |
@@ -137,6 +136,9 @@ discard block |
||
137 | 136 | return $dataValue->getDataItem(); |
138 | 137 | } |
139 | 138 | |
139 | + /** |
|
140 | + * @return boolean |
|
141 | + */ |
|
140 | 142 | private function getExternalFormattedUri( $value ) { |
141 | 143 | |
142 | 144 | if ( $this->externalFormattedUri !== null ) { |
@@ -41,6 +41,7 @@ discard block |
||
41 | 41 | * |
42 | 42 | * @param string $tableName |
43 | 43 | * @param array|null $tableOptions |
44 | + * @return void |
|
44 | 45 | */ |
45 | 46 | public function createTable( $tableName, array $tableOptions = null ); |
46 | 47 | |
@@ -50,6 +51,7 @@ discard block |
||
50 | 51 | * @since 2.5 |
51 | 52 | * |
52 | 53 | * @param string $tableName |
54 | + * @return void |
|
53 | 55 | */ |
54 | 56 | public function dropTable( $tableName ); |
55 | 57 | |
@@ -66,8 +68,8 @@ discard block |
||
66 | 68 | * |
67 | 69 | * @since 2.5 |
68 | 70 | * |
69 | - * @param string $indexName |
|
70 | 71 | * @param array|null $indexOptions |
72 | + * @return void |
|
71 | 73 | */ |
72 | 74 | public function createIndex( $tableName, array $indexOptions = null ); |
73 | 75 |
@@ -131,6 +131,9 @@ discard block |
||
131 | 131 | $this->reportMessage( "done.\n" ); |
132 | 132 | } |
133 | 133 | |
134 | + /** |
|
135 | + * @param string $tableName |
|
136 | + */ |
|
134 | 137 | private function getIndexInfo( $tableName ) { |
135 | 138 | |
136 | 139 | $tableName = $this->connection->tableName( $tableName ); |
@@ -154,6 +157,9 @@ discard block |
||
154 | 157 | return $indices; |
155 | 158 | } |
156 | 159 | |
160 | + /** |
|
161 | + * @param string $tableName |
|
162 | + */ |
|
157 | 163 | private function doDropIndex( $tableName, $indexName, $columns ) { |
158 | 164 | $this->reportMessage( " ... removing index $columns ..." ); |
159 | 165 | $this->connection->query( 'DROP INDEX ' . $indexName, __METHOD__ ); |