Index: trunk/phase3/includes/api/ApiQueryCategoryMembers.php |
— | — | @@ -72,7 +72,7 @@ |
73 | 73 | $this->addFields(array('cl_from', 'cl_sortkey')); |
74 | 74 | } |
75 | 75 | |
76 | | - $this->addFieldsIf('cl_timestamp', $fld_timestamp); |
| 76 | + $this->addFieldsIf('cl_timestamp', $fld_timestamp || $params['sort'] == 'timestamp'); |
77 | 77 | $this->addTables(array('page','categorylinks')); // must be in this order for 'USE INDEX' |
78 | 78 | // Not needed after bug 10280 is applied to servers |
79 | 79 | if($params['sort'] == 'timestamp') |
— | — | @@ -105,7 +105,10 @@ |
106 | 106 | if (++ $count > $limit) { |
107 | 107 | // We've reached the one extra which shows that there are additional pages to be had. Stop here... |
108 | 108 | // TODO: Security issue - if the user has no right to view next title, it will still be shown |
109 | | - $this->setContinueEnumParameter('continue', $this->getContinueStr($row, $lastSortKey)); |
| 109 | + if ($params['sort'] == 'timestamp') |
| 110 | + $this->setContinueEnumParameter('start', $row->cl_timestamp); |
| 111 | + else |
| 112 | + $this->setContinueEnumParameter('continue', $this->getContinueStr($row, $lastSortKey)); |
110 | 113 | break; |
111 | 114 | } |
112 | 115 | |
Index: trunk/phase3/RELEASE-NOTES |
— | — | @@ -224,6 +224,7 @@ |
225 | 225 | * (bug 11673) Return error 'unknown_action' in specified format |
226 | 226 | * (bug 13618) Added rcprop=redirect and rcshow=redirect to list=recentchanges |
227 | 227 | * (bug 13544) Added oldid parameter to action=parse to allow for parsing of old revisions |
| 228 | +* (bug 13718) Return the proper continue parameter for cmsort=timestamp |
228 | 229 | |
229 | 230 | === Languages updated in 1.13 === |
230 | 231 | |