https://bugzilla.wikimedia.org/show_bug.cgi?id=34744
Krinkle <krinklem...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Priority|Low |Lowest
Summary|Add a mw.Api.prototype.edit |Create mediawiki.api.edit
|function to some |module to abstract API
|mediawiki.api* module |action=edit
|(mediawiki.api.edit?) |
--- Comment #1 from Krinkle <krinklem...@gmail.com> ---
Yes, though it doens't make sense to create a separate resourceloader module
for every possible value of the "action" parameter.
Certain actions have dedicated modules to abstract certain logic. It isn't just
for the purpose of specifying the action.
mw.Api#post is not an internal method in anyway, so there's nothing wrong with:
mw.Api#post({
action: foo,
param: example
});
Creating a module for it only changes it to:
mw.Api#foo({
param: example
});
At the cost of having to specify a dependency on mediawiki.api.foo and the
extra code being downloaded, executed and increasing the function call stack
(user > foo > post > ajax > jquery.ajax > ...).
Unless this proposed module would do something useful beyond just passing
through the parameters and tacking .action = foo on it, recommending closing as
WONTFIX.
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
