https://bugzilla.wikimedia.org/show_bug.cgi?id=28070

--- Comment #5 from Dan Nessett <[email protected]> 2011-03-23 00:42:28 UTC ---
After thinking about it a bit more, a variation of the first possibility is to
encapsulate the code currently in the case "timestamp' arm of the switch
statement into a protected method and then override it in ApiQueryWatchlist.
This minimizes code changes and doesn't mess up the method signatures with
bandaids. Furthermore, it probably changes the code even less than the first
option, since the latter requires modifying the method signatures of both
extractRequestParams() and getParameterFromSettings().

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to