AnaelMobilia / NextINpact-Unofficial

Application Android "NextINpact (Unofficial)"
GNU General Public License v3.0
16 stars 4 forks source link

Gestion des téléchargements des ressources #155

Closed ghost closed 9 years ago

ghost commented 9 years ago

La miniature pour la news sur la sacem (http://www.nextinpact.com/news/93019-la-sacem-veut-revoir-l-affectation-redevance-pour-copie-privee.htm) ne s'affiche pas

AnaelMobilia commented 9 years ago

As-tu activé le mode debug dans les options ? As-tu eu un message d'erreur ?

Sur ma tablette, la miniature s'affiche bien (je pense que c'est un problème de téléchargement). Du coup tu as bien une icône NXI ?

ghost commented 9 years ago

Oui mais j'ai pas fait attention si il y avait un message au 1er chargement et après il n'y en a pas.

Oui à la place j'ai l'icône de NXI.

AnaelMobilia commented 9 years ago

Parfait, le problème est correctement attrapé !

Il n'y a pas grand chose de plus à faire. Tout au plus je pourrais mettre l'icône dans une liste de trucs à dl ?

ghost commented 9 years ago

Aucune idée ^^ Après c'est pas vraiment un problème grave, si ça ne se reproduit pas pas besoin de s'embêter.

AnaelMobilia commented 9 years ago

Oui, le problème n'est pas grave puisqu'il ne fait ni planter l'application, nicause de réel souci à l'utilisation de l'application.

Je pourrais facilement lancer le dl de l'image lorsque je constate qu'elle manque (au moment de l'affichage, cf https://github.com/AnaelMobilia/NextINpact-Unofficial/blob/master/src/com/pcinpact/adapters/ItemsAdapter.java#L218 ) puisque j'affiche une image "par défaut".

Mon interrogation est sur le droit de lancer des DL à n'importe quel moment ? L'application est détaillée comme permettant de la consultation hors ligne => je m'attendrais à ce qu'elle ne fasse pas des DL sauvage à tout va. Mais j'ai peut-être une mauvaise vision du marché des forfaits d'aujourd'hui ^-^

ghost commented 9 years ago

J'ai trouvé pourquoi ça fait ça. En lançant le chargement de nouvelles news, si l'application est quitté avant que les nouvelles news s'affichent, lors du prochain chargement elles vont s'afficher sans la miniature. Ou un truc dans le genre, ça me l'a refait tout à l'heure quand j'ai essayé d'en charger des nouvelles mais comme je captais mal et que ça tournait pendant longtemps, j'ai quitté et en relançant l'appli plus tard j'avais les news sans les miniatures

C'est pas possible de mettre ce chargement des miniatures mal affichées en même temps que celui de nouvelles news pour éviter que l'appli ne se connecte n'importe quand ?

ghost commented 9 years ago

ça ne l'avait pas fait avec le 1er mais pour les 3 de ce matin dont la miniature ne s'affiche pas, il n'y a pas le contenu de l'article non plus.

AnaelMobilia commented 9 years ago

Hello,

Les miniatures sont chargées seulement et uniquement à la synchronisation (demandée par l'utilisateur). Si une erreur est rencontrée (mauvaise url, perte de connexion, ...), l'image du logo NXI sera utilisé en remplacement. (pas de téléchargement en dehors de la demande de l'utilisateur).

A noter, le souci du non téléchargement du contenu de l'article en cas de perte de connexion !

Point complémentaire : les seules images téléchargées à la volée sont aujourd'hui celles du contenu des articles (que je veux synchroniser en même temps que la liste des articles à terme !), et les émoticônes des commentaires. Ai-je intérêt à mettre les émoticônes dans l'appli directement ?

ghost commented 9 years ago

Salut,

C'est possible de relancer le téléchargement à la prochaine synchro si il y a une erreur entre deux ? Car pour l'instant si il y a une erreur, il n'y a pas moyen de récupérer quand même l'article.

Ha oui pourquoi pas, ça ne pourrait pas résoudre le problème d'affiche des smileys en même temps ?

AnaelMobilia commented 9 years ago

TODO : suppression des articles dont le contenu n'a pas été téléchargé de la liste des articles. Tenter un DL à nouveau avant de les supprimer

ghost commented 9 years ago

Ils vont être retéléchargés à la prochaine synchro une fois supprimés ou pas ? Sinon on risque de passer à coté d'article

AnaelMobilia commented 9 years ago

Yep !

ghost commented 9 years ago

Nickel alors ^^

AnaelMobilia commented 9 years ago

Le téléchargement des smileys au premier lancement fait crash l'application (128 threads en parallèle maximum, dl de ~300 smileys). Pas possible.

ghost commented 9 years ago

Ha mince, on fera avec alors ^^