ApplETS / ETSMobile-iOS

Apache License 2.0
8 stars 3 forks source link

Modifier ETSSynchronisation pour gérer les échecs d'accès au réseau #123

Open ttauveron opened 8 years ago

ttauveron commented 8 years ago

Actuellement, un échec d'accès au réseau ne charge aucune erreur et laisse l'utilisateur sans information. Il faudrait détecter le NSError de l'appel réseau, et l'envoyer au Delegate pour le gérer dans la vue.

Voilà ce que Jean-Philippe Martin en dit : "Tu me fais réaliser que j’ai mis un NSError dans la fonction synchronize mais qu’en aucun cas il va retourner l’erreur. Pour corriger le problème, il faudrait renommer la fonction du protocole synchronizationDidFinishLoading, pour y inclure le NSError en paramètre, et à la réponse de la requête HTTP, s’il y a erreur, appeler cette fonction delegate avec l’erreur retourné par le dataTaskWithRequest. Il y a d’ailleurs un // FIXME à cet endroit. Toutes les erreurs HTTP sont présentement ignorées."