Last modified: 2013-10-09 15:16:25 UTC
According to documentation execute() should return a value, which it does not.
(In reply to comment #0) > According to documentation execute() should return a value, which it does not. Technically, the function in that class isn't documented ;)
We should change the documentation to remove the @return then. No need to make it return anything.
seems like this has been worked upon https://gerrit.wikimedia.org/r/#/c/47712/
I think execute() should be made abstract and its code moved to some other method 'setup'? or at least make it protected?
Patch in Gerrit needs rework...
Change 47712 abandoned by Siebrand: (bug 37713) Fix documentation for `MWHttpRequest::execute` Reason: Abandoning this change. Feel free to revive if there is reason to. https://gerrit.wikimedia.org/r/47712