Tâche A3 : Edition de message avec image

[jbghoul]

Il faudra se mettre d'accord avec Alexandre @alexandrejablon sur le protocole de téléchargement/upload de message

Cas d'utilisation associé :

3.3 Edition d’un message (+image)

A)

• L’utilisateur édite un message
• Il uploade une image depuis son téléphone dans le message
• Il valide l’envoie du message

B)

• Il ouvre un message de la carte qui contient une image
• Il voit l’image associé

[BBouchardon]

• Selection d'une image de la galerie ou via l'APN => OK
• Envoi via HTTP => OK ---> multipart avec un champ "json" et un champ "file"
• Tests d'envoi des images sur un serveur PHP => OK
• Ajout du champs "picture_url" dans la classe Message
• Affichage de l'image sur la carte => OK

Problème :

• Backstack à gérer (résolu si noHistory=false, mais changement dans toute l'appli)

Restant :

• Intégration avec l'API
• Quelle taille pour l'image à envoyer ? (pour l'instant 1024px max en longueur et jpg)
• Pouvoir zoomer sur clic de l'image

[jbghoul]

Pour Apiary, plutôt que de renvoyer "picture_url" = null, on peut utiliser des url d'images sur internet. un site pratique est http://placekitten.com/ car il permet de te renvoyer une image de la taille de ton choix. Exemple : http://placekitten.com/g/200/300

[alexjab]

@bbouchardon: tu pourrais me donner plus d’info sur l’upload d’images ? genre un JSON exemple ou une description plus détaillée.

Tu upload d’abord la photo à une url spécifique qui renvoie un lien que tu insère ensuite dans un message ?

Ou autre chose ?

Alex On 03 Mar 2014, at 17:23 , jbghoul notifications@github.com wrote:

Pour Apiary, plutôt que de renvoyer "picture_url" = null, on peut utiliser des url d'images sur internet. un site pratique est http://placekitten.com/ car il permet de te renvoyer une image de la taille de ton choix. Exemple : http://placekitten.com/200/300

— Reply to this email directly or view it on GitHub.

[BBouchardon]

Pour l'envoi au serveur : La requête http contient un multipart.

• le champ nommé "request" qui contient le JSON (exactement le même JSON qu'avant)
• le champ nommé "file" qui contient un tableau de byte de l'image (s'il n'y a pas d'image ce champ n'est pas envoyé)

Pour la récupération (avec le get_message) :

• nouveau champ "picture_url" qui peut contenir n'importe quelle URL, c'est au serveur de retourner la bonne.

[alexjab]

Quand tu dis “le même JSON qu’avant”, tu veux dire quelle méthode à quelle url ? post_message ? On 04 Mar 2014, at 15:43 , BBouchardon notifications@github.com wrote:

Pour l'envoi au serveur : La requête http contient un multipart.

le champ nommé "request" qui contient le JSON (exactement le même JSON qu'avant) le champ nommé "file" qui contient un tableau de byte de l'image (s'il n'y a pas d'image ce champ n'est pas envoyé) Pour la récupération (avec le get_message) :

nouveau champ "picture_url" qui peut contenir n'importe quelle URL, c'est au serveur de retourner la bonne. — Reply to this email directly or view it on GitHub.

[BBouchardon]

Le multipart il est que pour le post_message. Pour tous les autres POST, j'ai laissé comme avant pour le moment mais on peut tout unifier et mettre le champ "request" pour tous les POST. Dis-moi ce que tu préfères.

[alexjab]

je suis pas sûr de voir à 100% la structure de ta requête;

si tu pouvais lancer une de tes requêtes d’upload au serveur, j’ai fait une motif sur le serveur live pour afficher les requêtes et voir en détail ce que tu envoies.

On 04 Mar 2014, at 15:57 , BBouchardon notifications@github.com wrote:

Le multipart il est que pour le post_message. Pour tous les autres POST, j'ai laissé comme avant pour le moment mais on peut tout unifier et mettre le champ "request" pour tous les POST. Dis-moi ce que tu préfères.

— Reply to this email directly or view it on GitHub.

[BBouchardon]

j'en ai envoyé une (le retour est "missing 'lat' param" à cause du multipart)

[alexjab]

J’ai bien la requête affichée sur le serveur donc c’est bon; je vais la regarder en détail et je t’enverrai un autre email si j’ai besoin de précisions !

On 04 Mar 2014, at 16:10 , BBouchardon notifications@github.com wrote:

j'en ai envoyé une (le retour est "missing 'lat' param" à cause du multipart)

— Reply to this email directly or view it on GitHub.

[alexjab]

C’est bon, je pense avoir cerné le truc ! Je vais ajouter la fonctionnalité vite fait, et je te dis quand c’est déployé pour que tu puisses tester !

On 04 Mar 2014, at 16:12 , Alexandre Jablon alexandre.jablon@gmail.com wrote:

J’ai bien la requête affichée sur le serveur donc c’est bon; je vais la regarder en détail et je t’enverrai un autre email si j’ai besoin de précisions !

On 04 Mar 2014, at 16:10 , BBouchardon notifications@github.com wrote:

j'en ai envoyé une (le retour est "missing 'lat' param" à cause du multipart)

— Reply to this email directly or view it on GitHub.

[alexjab]

J’ai fait un quick fix sur le serveur pour tester. Tu peux refaire une requête avec une image ?

On 04 Mar 2014, at 16:23 , Alexandre Jablon alexandre.jablon@gmail.com wrote:

C’est bon, je pense avoir cerné le truc ! Je vais ajouter la fonctionnalité vite fait, et je te dis quand c’est déployé pour que tu puisses tester !

On 04 Mar 2014, at 16:12 , Alexandre Jablon alexandre.jablon@gmail.com wrote:

J’ai bien la requête affichée sur le serveur donc c’est bon; je vais la regarder en détail et je t’enverrai un autre email si j’ai besoin de précisions !

On 04 Mar 2014, at 16:10 , BBouchardon notifications@github.com wrote:

j'en ai envoyé une (le retour est "missing 'lat' param" à cause du multipart)

— Reply to this email directly or view it on GitHub.

[BBouchardon]

j'ai refait une requête et même erreur ...

[alexjab]

okay, je vais regarder, il a un fichier qui a été uploadé en tous cas ! On 04 Mar 2014, at 16:37 , BBouchardon notifications@github.com wrote:

j'ai refait une requête et même erreur ...

— Reply to this email directly or view it on GitHub.

[alexjab]

réessaie ?

On 04 Mar 2014, at 16:39 , Alexandre Jablon alexandre.jablon@gmail.com wrote:

okay, je vais regarder, il a un fichier qui a été uploadé en tous cas ! On 04 Mar 2014, at 16:37 , BBouchardon notifications@github.com wrote:

j'ai refait une requête et même erreur ...

— Reply to this email directly or view it on GitHub.

[BBouchardon]

serveur down en ce moment ...

[alexjab]

yep, my bad, j’ai oublié une ligne de requirements ! back online now !

On 04 Mar 2014, at 16:47 , BBouchardon notifications@github.com wrote:

serveur down en ce moment ...

— Reply to this email directly or view it on GitHub.

[BBouchardon]

le message est parti, il manque plus que le champ de l'url dans le get_message

[alexjab]

okay, au moins on est d’accord sur comment ça marche

On 04 Mar 2014, at 17:02 , BBouchardon notifications@github.com wrote:

le message est parti, il manque plus que le champ de l'url dans le get_message

— Reply to this email directly or view it on GitHub.

[BBouchardon]

par contre j'ai pas testé les autres POST, mais tu as gardé le même fonctionnement qu'avant ou ça utilise pour tous un champ "request" (pour le json) ?

[alexjab]

non à priori je vais mettre ce champ request pour post_message v1.3 seulement et le mode normal pour tout le reste. On 04 Mar 2014, at 17:04 , BBouchardon notifications@github.com wrote:

par contre j'ai pas tester les autres POST, mais tu as gardé le même fonctionnement qu'avant ou ça utilise pour tous un champ "request" (pour le json) ?

— Reply to this email directly or view it on GitHub.

[BBouchardon]

Résolu :

• Backstack à gérer
• Intégration avec l'API
• Taille à 1024px de large + compression jpeg 85%

Restant :

• Pouvoir zoomer sur clic de l'image ?
• Progressbar sur LoadPictureTask ?

[BBouchardon]

Intégration OK pour l'envoi et l'affichage de bout en bout.

[jbghoul]

tout à l'heure, une image trop grande bouchait tout l'écran et empêchait de voir les commentaires et like buttons

[BBouchardon]

Oui j'ai vu le problème aussi, le bloc contenant le message n'est pas scrollable ....

[BBouchardon]

Problème corrigé. Reste à voir pour la rotation des images.

[BBouchardon]

J'ai rajouté un vérification sur l'orientation via le exif au moment où on prend la photo. Sur l'emulateur, l'orientation est toujours la même (même en changeant avec le bouton). Quelqu'un aurait un device pour tester @jbghoul @veve90 ? Dans le onActivityResult du EditMessageFragment il y a un debug avec EXIF TAG_ORIENTATION (chez moi il retourne toujours 0 ...).

[jbghoul]

Non je n'ai pas de device :/

[veve90]

Hello, je viens de tester :) Dsl de prendre plus de temps, mais c'est due au fait qu'il a des soucis téléphone/tablette.

Sur mon wiko tout marche bien (rotation, prendre image, prendre image de la galery...) .

Cependant sur la tablette quand on veut prendre une image le code ne s’exécute pas (il rentre pas dans le if if (resultCode == Activity.RESULT_OK)) Du coup, il set même pas l'image. J'ai enlevé ce if, et après la rotation il la fait bien; cependant il y a des éléments qui doivent être vérifié de pas être null si on enlève ce if..

J'ai fait un checkout sur la version tagué et, sur la tablette pour prendre des images ça fait pareil que mnt (ça rentre pas dans le premier if). Sur wiko tout bien :)

Note: sur user profile je n'ai pas mis ce "if "et la prise d'une image marche sur la tablette également.

Egalement le code de rotation est mis seulement sur le 'take image' . Je pense qu'il faudra le mettre également pr les images qu'on prend de la gallery

De plus, pense tu qu'il peut être ajouté dans les utils (CameraUtils)? afin que je l'utilise egalement dans le profil de l'utilisateur.

[BBouchardon]

Pour la tablette c'est bizarre car mardi JB arrivait bien à prendre des photos non ? (et il y avait le if RESULT_OK).

Tu as pu regardé ce que te retournes le TAG_ORIENTATION (si c'est différent de 0 ?)

Je suis d'accord, il faut le mettre dans un utils si tu veux l'utiliser pour le profil.

[veve90]

Bah JB a tj utilisé ma tablette et la j'ai la tablette que @xiaoyufeng avait..du coup cela peut être intéressant de demander à xiao-yu de tester également.

sur mon wiko le TAG_ORIENTATION me retourne tjs 1 :)) Quand j'ai testé la rotation j'ai pris une photo en landscape et il l'as bien affiché en landscape...

[veve90]

Pfff...comment est ce qu'on va faire pour la démo? Est-tu en train de faire de modifications? De travaillé sur ça? Si oui, dit moi qd tester....Ou, veut tu qu'on se voit demain à la BU ou quelque part à coté de l'INSA afin que je te donne la tablette? D’ailleurs peut tu ajouter une tablette sur l'emulateur?

[BBouchardon]

Je vais mettre la rotation dans un utils et pusher.

Par contre pour le (if RESULT_OK) qui n'est pas bon je ne vois pas comment faire ... Ce bug il est aussi présent quand tu choisis ta photo de profil ?

(Je ne suis pas à Lyon ce weekend, je vais installer une tablette dans l'émulateur, mais j'ai bien peur que le bug ne soit pas présent ...) C'est le même modèle les 2 tablettes et même version ?

[veve90]

Pour les tablettes ce n'est pas la même version (Ceci est en android 4.2.2). Je ne sais plus la version de l'autre tablette. Cependant c'est sur que ce n'est pas la même car je n'efface pas les applications de la même manière, je n'ai pas les mêmes options dans les options développeur...

Les 2 tablettes sont des Samsung. Ceci est un Galaxy Tab 2, l'autre je ne sais pas..

Puis quand je choisi la photo profil, moi je n'ai pas mis du tout la vérification sur Activity.RESULT_OK, mais je pense que si je la mettrais le bug serais..si tu veux je peux essayer..

[veve90]

Egalement, il faudrait penser à ajouter la rotation dans la vue avec le message et ses commentaires..

[BBouchardon]

J'ai mis en commentaire le if (resultCode == Activity.RESULT_OK). J'ai pushé.

[veve90]

J'ai testé avec une image en landscape et une portrait et ça marchait bien ..

Le code sur la tablette est tjs 1...du coup je me demande si je teste bien ou pas :))

Comment je teste: Je prend une photo et si je la prend en photo en portrait je m'attend qu'elle soit en portrait sur le message. C'est bien ça non? Car étant donné que le code est tjs 1..ça rentre pas dans les if de rotation..

[BBouchardon]

1 ça correspond à ORIENTATION_NORMAL On dirait que la tablette ne met pas l'orientation dans le exif ...

[jbghoul]

étant donné que la présentation est Lundi à 13h et au vu du peu de temps qu'il reste, je rappelle l'importance d'avoir quelque chose de suffisamment bon c'est à dire pas forcément dénuée de toute anomalie d'orientation, mais qui fonctionne dans la plupart des cas en étant stable (pas de crash)

[BBouchardon]

Bon je pense qu'on va laisser sans la rotation (étant donné que je ne peux pas tester).