search

hhkaos / idealista-arcgis

Buscador de pisos que permite buscar además por características de la zona
http://proyectos.geodevelopers.org/idealista-arcgis/
GNU Affero General Public License v3.0
9 stars 11 forks source link

Sacar objetos de esri del $scope #12

Closed ivanmarques closed 9 years ago

ivanmarques commented 9 years ago

Es algo que comentaste en el video de presentación del proyecto, que no sabías si estabas haciendo bien guardando en el scope los objetos de la API arcgis.

No digo que esté mal, pues funciona correctamente, pero tal como lo entiendo yo, en el scope sólo metería objetos o funciones que interactúen directamente con las vistas.

Es algo similar a hacer "privados" esos objetos para que solo los use el controlador en este caso.

Otra modificación que podría estar bien, pero en este caso no he atrevido a hacerla directamente es encapsular toda la interacción con el mapa en un servicio para separar un poco responsabilidades, en otras ocasiones lo he hecho, pero también es cierto que no utilizaba la directiva de esri-angular, y seguramente la implementación sea distinta.

hhkaos commented 9 years ago

Hi crack!, perdona que no vi el pull-request. Ya he visto lo cambios. Thx!! Mergeo!