http://www.mediawiki.org/wiki/Special:Code/MediaWiki/65756

Revision: 65756
Author:   catrope
Date:     2010-05-01 19:13:38 +0000 (Sat, 01 May 2010)

Log Message:
-----------
CodeReview: (bug 23367) Deprecate ccprop=revision in favor of ccprop=status in 
list=codecomments . Old prop kept for backward compatibility

Modified Paths:
--------------
    trunk/extensions/CodeReview/api/ApiCodeComments.php

Modified: trunk/extensions/CodeReview/api/ApiCodeComments.php
===================================================================
--- trunk/extensions/CodeReview/api/ApiCodeComments.php 2010-05-01 19:09:33 UTC 
(rev 65755)
+++ trunk/extensions/CodeReview/api/ApiCodeComments.php 2010-05-01 19:13:38 UTC 
(rev 65756)
@@ -38,6 +38,8 @@
                if ( is_null( $params['repo'] ) )
                        $this->dieUsageMsg( array( 'missingparam', 'repo' ) );
                $this->props = array_flip( $params['prop'] );
+               if ( isset( $this->props['revision'] ) )
+                       $this->setWarning( 'ccprop=revision has been deprecated 
in favor of ccprop=status' );
 
                $listview = new CodeCommentsListView( $params['repo'] );
                if ( is_null( $listview->getRepo() ) )
@@ -82,7 +84,7 @@
                        $item['timestamp'] = wfTimestamp( TS_ISO_8601, 
$row->cc_timestamp );
                if ( isset( $this->props['user'] ) )
                        $item['user'] = $row->cc_user_text;
-               if ( isset( $this->props['revision'] ) )
+               if ( isset( $this->props['revision'] ) || isset( 
$this->props['status'] ) )
                        $item['status'] = $row->cr_status;
                if ( isset( $this->props['text'] ) )
                        ApiResult::setContent( $item, $row->cc_text );
@@ -104,13 +106,14 @@
                        ),
                        'prop' => array (
                                ApiBase :: PARAM_ISMULTI => true,
-                               ApiBase :: PARAM_DFLT => 
'timestamp|user|revision|revid',
+                               ApiBase :: PARAM_DFLT => 
'timestamp|user|status|revid',
                                ApiBase :: PARAM_TYPE => array (
                                        'timestamp',
                                        'user',
-                                       'revision',
+                                       'status',
                                        'text',
                                        'revid',
+                                       'revision',
                                ),
                        ),
                );
@@ -121,7 +124,7 @@
                        'repo' => 'Name of the repository',
                        'limit' => 'How many comments to return',
                        'start' => 'Timestamp to start listing at',
-                       'prop' => 'Which properties to return',
+                       'prop' => 'Which properties to return. revision is a 
deprecated alias for status',
                );
        }
 



_______________________________________________
MediaWiki-CVS mailing list
MediaWiki-CVS@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to