r111386 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r111385‎ | r111386 | r111387 >
Date:16:35, 13 February 2012
Author:hashar
Status:ok
Tags:
Comment:
Remove backslash from @return types

Ping r111103
Modified paths:
  • /trunk/phase3/includes/ConfEditor.php (modified) (history)
  • /trunk/phase3/includes/Namespace.php (modified) (history)
  • /trunk/phase3/includes/QueryPage.php (modified) (history)
  • /trunk/phase3/includes/StringUtils.php (modified) (history)
  • /trunk/phase3/includes/db/LoadBalancer.php (modified) (history)
  • /trunk/phase3/includes/filerepo/file/LocalFile.php (modified) (history)
  • /trunk/phase3/includes/media/Generic.php (modified) (history)
  • /trunk/phase3/includes/parser/LinkHolderArray.php (modified) (history)
  • /trunk/phase3/includes/specials/SpecialAllmessages.php (modified) (history)
  • /trunk/phase3/includes/specials/SpecialUnblock.php (modified) (history)
  • /trunk/phase3/includes/upload/UploadFromChunks.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/upload/UploadFromChunks.php
@@ -242,7 +242,7 @@
243243 * Output the chunk to disk
244244 *
245245 * @param $chunkPath string
246 - * @return \FileRepoStatus
 246+ * @return FileRepoStatus
247247 */
248248 private function outputChunk( $chunkPath ){
249249 // Key is fileKey + chunk index
Index: trunk/phase3/includes/parser/LinkHolderArray.php
@@ -135,7 +135,7 @@
136136 /**
137137 * Get a subset of the current LinkHolderArray which is sufficient to
138138 * interpret the given text.
139 - * @return \LinkHolderArray
 139+ * @return LinkHolderArray
140140 */
141141 function getSubArray( $text ) {
142142 $sub = new LinkHolderArray( $this->parent );
Index: trunk/phase3/includes/db/LoadBalancer.php
@@ -91,7 +91,7 @@
9292 /**
9393 * Get or set arbitrary data used by the parent object, usually an LBFactory
9494 * @param $x
95 - * @return \Mixed
 95+ * @return Mixed
9696 */
9797 function parentInfo( $x = null ) {
9898 return wfSetVar( $this->mParentInfo, $x );
Index: trunk/phase3/includes/filerepo/file/LocalFile.php
@@ -1646,7 +1646,7 @@
16471647
16481648 /**
16491649 * Run the transaction
1650 - * @return \FileRepoStatus
 1650+ * @return FileRepoStatus
16511651 */
16521652 function execute() {
16531653 global $wgUseSquid;
@@ -1807,7 +1807,7 @@
18081808 * rows and there's no need to keep the image row locked while it's acquiring those locks
18091809 * The caller may have its own transaction open.
18101810 * So we save the batch and let the caller call cleanup()
1811 - * @return \FileRepoStatus
 1811+ * @return FileRepoStatus
18121812 */
18131813 function execute() {
18141814 global $wgLang;
@@ -2075,7 +2075,7 @@
20762076 /**
20772077 * Delete unused files in the deleted zone.
20782078 * This should be called from outside the transaction in which execute() was called.
2079 - * @return \FileRepoStatus|void
 2079+ * @return FileRepoStatus|void
20802080 */
20812081 function cleanup() {
20822082 if ( !$this->cleanupBatch ) {
@@ -2196,7 +2196,7 @@
21972197
21982198 /**
21992199 * Perform the move.
2200 - * @return \FileRepoStatus
 2200+ * @return FileRepoStatus
22012201 */
22022202 function execute() {
22032203 $repo = $this->file->repo;
Index: trunk/phase3/includes/media/Generic.php
@@ -188,7 +188,7 @@
189189 * @param $dstPath String: filesystem destination path
190190 * @param $dstUrl String: Destination URL to use in output HTML
191191 * @param $params Array: Arbitrary set of parameters validated by $this->validateParam()
192 - * @return \MediaTransformOutput
 192+ * @return MediaTransformOutput
193193 */
194194 final function getTransform( $image, $dstPath, $dstUrl, $params ) {
195195 return $this->doTransform( $image, $dstPath, $dstUrl, $params, self::TRANSFORM_LATER );
Index: trunk/phase3/includes/ConfEditor.php
@@ -731,7 +731,7 @@
732732
733733 /**
734734 * Create a ConfEditorToken from an element of token_get_all()
735 - * @return \ConfEditorToken
 735+ * @return ConfEditorToken
736736 */
737737 function newTokenObj( $internalToken ) {
738738 if ( is_array( $internalToken ) ) {
@@ -783,7 +783,7 @@
784784 /**
785785 * Get the token $offset steps ahead of the current position.
786786 * $offset may be negative, to get tokens behind the current position.
787 - * @return \ConfEditorToken
 787+ * @return ConfEditorToken
788788 */
789789 function getTokenAhead( $offset ) {
790790 $pos = $this->pos + $offset;
Index: trunk/phase3/includes/Namespace.php
@@ -186,7 +186,7 @@
187187 * Returns array of all defined namespaces with their canonical
188188 * (English) names.
189189 *
190 - * @return \array
 190+ * @return array
191191 * @since 1.17
192192 */
193193 public static function getCanonicalNamespaces() {
Index: trunk/phase3/includes/specials/SpecialUnblock.php
@@ -136,7 +136,7 @@
137137
138138 /**
139139 * Submit callback for an HTMLForm object
140 - * @return \Array( Array(message key, parameters)
 140+ * @return Array( Array(message key, parameters)
141141 */
142142 public static function processUIUnblock( array $data, HTMLForm $form ) {
143143 return self::processUnblock( $data, $form->getContext() );
Index: trunk/phase3/includes/specials/SpecialAllmessages.php
@@ -290,7 +290,7 @@
291291 /**
292292 * This function normally does a database query to get the results; we need
293293 * to make a pretend result using a FakeResultWrapper.
294 - * @return \FakeResultWrapper
 294+ * @return FakeResultWrapper
295295 */
296296 function reallyDoQuery( $offset, $limit, $descending ) {
297297 $result = new FakeResultWrapper( array() );
Index: trunk/phase3/includes/QueryPage.php
@@ -384,7 +384,7 @@
385385
386386 /**
387387 * Somewhat deprecated, you probably want to be using execute()
388 - * @return \ResultWrapper
 388+ * @return ResultWrapper
389389 */
390390 function doQuery( $offset = false, $limit = false ) {
391391 if ( $this->isCached() && $this->isCacheable() ) {
Index: trunk/phase3/includes/StringUtils.php
@@ -191,7 +191,7 @@
192192 * Returns an Iterator
193193 * @param $separator
194194 * @param $subject
195 - * @return \ArrayIterator|\ExplodeIterator
 195+ * @return ArrayIterator|\ExplodeIterator
196196 */
197197 static function explode( $separator, $subject ) {
198198 if ( substr_count( $subject, $separator ) > 1000 ) {

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r111103More return documentationreedy21:36, 9 February 2012

Status & tagging log