Page MenuHomePhabricator

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

Description

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

Details

Reference
bz55832

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:12 AM
bzimport set Reference to bz55832.

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

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.

awight removed awight as the assignee of this task.Feb 7 2016, 1:25 AM