Page MenuHomePhabricator

api parameter for list=centralnoticelogs are not prefixed
Open, MediumPublic


action=query&list=centralnoticelogs is not using a prefix which would cause throuble if any other module would do so, too.

Currently existing api paramters are campain, user, limit, offset, start, end. They should be renamed to e.g. cnlcampain, cnluser, cnllimit, cnloffset, cnlstart, cnlend.

This would be a breaking change.

Version: unspecified
Severity: normal



Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:12 AM
bzimport set Reference to bz55832.
Merl created this task.Oct 17 2013, 9:16 AM

Agreed, this should happen. Thank you for the report!

atgo set Security to None.

Noting that the WMF currently has nothing that relies on the CentralNotice API, but we should check the API logs just in case third parties are using it.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 21 2015, 7:13 AM
awight removed awight as the assignee of this task.Feb 7 2016, 1:25 AM
Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptAug 5 2016, 8:49 PM
mmodell removed a subscriber: awight.Jun 22 2017, 9:39 PM