mozilla-mobile / prox

[INACTIVE] A search and discovery app for the "here & now". We're experimenting with ideas on mobile that can better surface content from the open web.
https://wiki.mozilla.org/New_Mobile_Experience
Mozilla Public License 2.0
22 stars 12 forks source link

No issue: slightly clean up PlacesController #537

Closed mcomella closed 7 years ago

mcomella commented 7 years ago

From commit:

It's no longer a priority for me to do the refactoring, but here are the
next steps:
* Make a file of generic place transformations (filter, sort)
  * Keep in mind we need somewhere to cache travel times
* PlaceDetailController should keep an array of places, fetched via
  getPlaces(forLocation:withRadius:). This removes the need for the accessor
  methods in PlacesController. Fetching is also simplified, eliminating the
  PlacesController entirely.
* Next step: the details view passes the places array to the map view, which
  makes a copy, manipulating it (e.g. future planning), and returning the result
  to the places detail.