Page MenuHomePhabricator

Port Networking code to use Volley
Closed, DeclinedPublic

Description

Volley is a new Networking library from a bunch of Google Engineers. I spent today working with it on a few things, and came away impressed, mostly on how much it simplifies the *entire* codebase, rather than just the networking part.

(https://developers.google.com/events/io/sessions/325304728 is so far the only documentation available, but the code is pretty sweet and short too)

It can replace not just UIL (Image Loader library), but also the Mediawiki API Wrapper I wrote a while ago. We can also use this opportunity to switch to JSON from XML.

This will lead to much cleaner code (a lot of AsyncTasks and assorted other inner classes will just disappear), better performance and proper caching!


Version: unspecified
Severity: normal

Details

Reference
bz49042

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 1:41 AM
bzimport set Reference to bz49042.
bzimport added a subscriber: Unknown Object (MLST).

We've mostly switched over the Image display code. Will need to start thinking of switching over the rest of it soon.

As the Mobile Apps Team is focussing on the new, native Wikipedia app, the Commons app is no longer being maintained by the Wikimedia Foundation. I am WONTFIXing all open Commons app bugs to reflect that we will not be spending time fixing them.

For the full story, see this thread on mobile-l: https://lists.wikimedia.org/pipermail/mobile-l/2014-September/thread.html#7974