Last modified: 2011-09-14 23:53:55 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 T32877, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 30877 - Deleting an UploadCampaign probably shouldn't be an API method
Deleting an UploadCampaign probably shouldn't be an API method
Status: RESOLVED INVALID
Product: MediaWiki extensions
Classification: Unclassified
UploadWizard (Other open bugs)
unspecified
All All
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-09-12 18:27 UTC by Neil Kandalgaonkar
Modified: 2011-09-14 23:53 UTC (History)
2 users (show)

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


Attachments

Description Neil Kandalgaonkar 2011-09-12 18:27:21 UTC
We added an API method to delete Upload Campaigns just to accomplish certain functionality in a POST request -- this seems like a bad reason to have an API method. Before people rely on this we should withdraw the API method.
Comment 1 Brion Vibber 2011-09-12 19:22:29 UTC
IMO the best way to go these days is to design everything to use the API, and only add Special: page POST handling when needed for compatibility with non-JavaScript browsers and traditional form submission.

This gives the greatest flexibility for the frontend UI to act in a modern fashion and allows for alternate UIs and bot-based activity to proceed if/as needed.
Comment 2 Neil Kandalgaonkar 2011-09-13 16:06:38 UTC
I agree with you that an API-like system is best for frontend design, but my concern is that actual API methods are guarantees to 3rd parties.

Is there any way we can mark API methods as "internal", or otherwise not to be relied on?
Comment 3 Brion Vibber 2011-09-13 18:04:04 UTC
Per kitchen discussion, we're provisionally agreeing to mark the API method as 'provisional' or otherwise 'not fully stable' in its documentation that appears in API help.

Cleaning it up further and making the add/edit available through API as well in future would be nice at some point, but we don't have as much hurry as long as what's there is marked as incomplete so people know not to rely on it.

We should also consider adding an explicit 'stability level' marker in the API help?
Comment 4 Neil Kandalgaonkar 2011-09-14 23:53:55 UTC
Resolving this bug as invalid... we can solve the issue with documentation for now, and that is fixed in r97121

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links