Cambios en la función searchTracks

[voxsoftware]

Buenas tardes,

Los cambios hechos en el proyecto son pocos.
Permitir cancelar la búsqueda, devolviendo en la función searchTracks un objeto 'request'
Solucionar error en el callback de service.searchTracks cuando no se obtenía información de la búsqueda salía error de operación sobre null
Se modificó service.searchTracks para que en los resultados cada objeto 'track' tenga también las siguientes propiedades: picture, href

[PaquitoSoft]

Gracias por el pull request corrigiendo el error detectado.
Aún no he encontrado tiempo para bajarme el pull request y examinarlo, pero viendo el código veo que no has modificadolos tests del módulo.
Deberías añadir los tests que validan la corrección detectada así como las mejoras incluidas.

Un saludo.

[voxsoftware]

Verá, si nota en la línea 80 del archivo service.js ejecuta, callback(error), lo que hace que el segundo argumento sea undefined, y ese segundo argumento se usa en el archivo goear_api.js, en la línea 89 utiliza ese segundo argumento que es innerData, pero ese argumento no se manda en el archivo service.js. Ese es el error que se corrigió.

Cordial saludo

[PaquitoSoft]

Hola,
Según veo en su commit, los cambios hechos son los siguientes:
1.- Devolver el objeto request para poder cancelar peticiones en curso
2.- Corrección del uso de la variable innerData en la función search del componente goear_api.js para gestionar correctamente los casos de error.
3.- Adición de la cabecera User-Agent en las peticiones al servicio externo
4.- Adición de los atributos href y picture a cada track del resultado de las búsquedas

Como le decía en mi anterior comentario, para aceptar un pull-request se deben incluir los tests que validen los cambios introducidos.
Yo voy a revisar el error en la gestión de la variable innerData que ha detectado y crearé un nuevo test para comprobar que su corrección es válida.
En cuanto al tercer punto, no veo cuál es la necesidad de establecer el User-Agent; en mis pruebas no encontré que su no existencia fuese un factor determinante.

Si además, de las modificaciones que ha hecho incluye los tests que validen las mismas así como la actualización de la documentación para indicar las nuevas propiedades de este módulo, no tendré problema en aceptar su pull request.

Reciba un cordial saludo.

[PaquitoSoft]

@voxsoftware, he publicado una nueva versión de la librería con la gestión del posible error en las peticiones de paginación tal y como usted destacó en su pull request.

Saludos.