r57383 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r57382‎ | r57383 | r57384 >
Date:13:53, 5 October 2009
Author:werdna
Status:deferred
Tags:
Comment:
Fix LiquidThreads API documentation (bug 20951)
* prefix was not updated when changed.
* new fields were not documented
Modified paths:
  • /trunk/extensions/LiquidThreads/api/ApiQueryLQTThreads.php (modified) (history)

Diff [purge]

Index: trunk/extensions/LiquidThreads/api/ApiQueryLQTThreads.php
@@ -311,7 +311,6 @@
312312 'startid' => 'The thread id to start enumerating from',
313313 'endid' => 'The thread id to stop enumerating at',
314314 'dir' => 'The direction in which to enumerate',
315 - 'show' => 'Also show threads which meet these criteria',
316315 'limit' => 'The maximum number of threads to list',
317316 'prop' => 'Which properties to get',
318317 'page' => 'Limit results to threads on a particular page(s)',
@@ -319,6 +318,18 @@
320319 'root' => 'Limit results to threads with the given root(s)',
321320 'summary' => 'Limit results to threads corresponding to the given summary page(s)',
322321 'id' => 'Get threads with the given ID(s)',
 322+ 'showdeleted' => 'Whether or not to show deleted threads',
 323+ 'render' => 'Whether or not to include the rendered thread in the results',
 324+ 'renderlevel' => 'When rendering, the level at which to start (for the sake of depth limits, etc)',
 325+ 'renderthreadpos' => 'When rendering, the position of the thread in the group of '.
 326+ 'threads being rendered at that level (affects display somewhat)',
 327+ 'renderthreadcount' => 'When rendering, the number of threads in that level group',
 328+ 'rendermaxthreadcount' => 'When rendering, the maximum number of replies to show '.
 329+ 'before adding a "Show more replies" link',
 330+ 'rendermaxdepth' => 'When rendering, the maximum depth of replies to show before '.
 331+ 'showing a "Show X replies" link instead of replies',
 332+ 'renderstartrepliesat' => 'When rendering, the point at which to start showing replies '.
 333+ '(used internally to load extra replies)',
323334 );
324335 }
325336
@@ -328,8 +339,8 @@
329340
330341 protected function getExamples() {
331342 return array(
332 - 'api.php?action=query&list=threads&lqtpage=Talk:Main_Page',
333 - 'api.php?action=query&list=threads&lqtid=1|2|3|4&lqtprop=id|subject|modified'
 343+ 'api.php?action=query&list=threads&thpage=Talk:Main_Page',
 344+ 'api.php?action=query&list=threads&thid=1|2|3|4&thprop=id|subject|modified'
334345 );
335346 }
336347

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r57381Remove unnecessary explicit "thread.", which breaks when we have a table pref...werdna13:46, 5 October 2009

Status & tagging log