Last modified: 2011-03-13 00:15:31 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T29182, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 27182 - API: Add filter by prefix for meta=allmessages
API: Add filter by prefix for meta=allmessages
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
1.18.x
All All
: Normal enhancement (vote)
: ---
Assigned To: Roan Kattouw
: patch, patch-reviewed
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2011-02-05 18:42 UTC by db [inactive,noenotif]
Modified: 2011-03-13 00:15 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Attachments
Non-functional patch (2.47 KB, patch)
2011-03-10 20:09 UTC, Jarry1250 		Details
Add an attachment (proposed patch, testcase, etc.)

Description db [inactive,noenotif] 2011-02-05 18:42:43 UTC 
The special page Special:AllMessages can filtered by prefix. I have not found a option for the api.

Thanks.
Comment 1 Jarry1250 2011-03-10 20:09:28 UTC 
Created attachment 8279 [details]
Non-functional patch

Patch to add a new amprefix option. In all honesty this replaces most of the uses of amfilter, including the one in the examples

Note that this patch is compiled against r83664, and is, AFAICT, broken. For context, see my comments on r81469. Once that issue is resolved this patch should work fine.
Comment 2 Jarry1250 2011-03-10 22:03:23 UTC 
r83664 now working: marking patch as need-review. (For all the good it'll do :) )
Comment 3 Sam Reed (reedy) 2011-03-10 22:06:56 UTC 
API stuff usually gets well triaged ;)

I should get time to review properly over the weekend :)
Comment 4 Sam Reed (reedy) 2011-03-13 00:15:31 UTC 
Looks good. Minor logic tweak made (you'd inverted it which made it read a little strange)

Otherwise good, and applied in r83800. Thanks! :)
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links