Delete image on change #625

Merged
merged 7 commits into from May 30, 2014

Conversation

Projects
None yet
6 participants
@artragis
Contributor

artragis commented May 29, 2014

Retire le Todo qui demandait à ce que les images des articles soient libérées lorsqu'elles sont changées.

@SpaceFox

This comment has been minimized.

Show comment
Hide comment
@SpaceFox

SpaceFox May 29, 2014

Member

Erf, c'était une bonne idée mais ce code custom va sauter quand on va
utiliser un outil de miniatures.
Du coup ta modif ne devrait pas rester en place très longtemps :-(
Le 29 mai 2014 19:58, "artragis" notifications@github.com a écrit :

Retire le Todo qui demandait à ce que les images des articles soient

libérées lorsqu'elles sont changées.

You can merge this Pull Request by running

git pull https://github.com/artragis/zds-site delete_image_on_change

Or view, comment on, or merge it at:

#625
Commit Summary

  • Delete the old thumbnail when the author changes it
  • Add some dependencies about linux installation

File Changes

  • M README.mdhttps://github.com/zestedesavoir/zds-site/pull/625/files#diff-0(7)
  • M zds/article/models.pyhttps://github.com/zestedesavoir/zds-site/pull/625/files#diff-1(27)

Patch Links:


Reply to this email directly or view it on GitHubhttps://github.com/zestedesavoir/zds-site/pull/625
.

Member

SpaceFox commented May 29, 2014

Erf, c'était une bonne idée mais ce code custom va sauter quand on va
utiliser un outil de miniatures.
Du coup ta modif ne devrait pas rester en place très longtemps :-(
Le 29 mai 2014 19:58, "artragis" notifications@github.com a écrit :

Retire le Todo qui demandait à ce que les images des articles soient

libérées lorsqu'elles sont changées.

You can merge this Pull Request by running

git pull https://github.com/artragis/zds-site delete_image_on_change

Or view, comment on, or merge it at:

#625
Commit Summary

  • Delete the old thumbnail when the author changes it
  • Add some dependencies about linux installation

File Changes

  • M README.mdhttps://github.com/zestedesavoir/zds-site/pull/625/files#diff-0(7)
  • M zds/article/models.pyhttps://github.com/zestedesavoir/zds-site/pull/625/files#diff-1(27)

Patch Links:


Reply to this email directly or view it on GitHubhttps://github.com/zestedesavoir/zds-site/pull/625
.

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis May 30, 2014

Contributor

OK, noharm.
Je ne savais pas qu'un autre outil était prévu. Je voulais juste voir un peu à quoi ressemblait un code django et ce que je pouvais y faire. ^^

Contributor

artragis commented May 30, 2014

OK, noharm.
Je ne savais pas qu'un autre outil était prévu. Je voulais juste voir un peu à quoi ressemblait un code django et ce que je pouvais y faire. ^^

@firm1

This comment has been minimized.

Show comment
Hide comment
@firm1

firm1 May 30, 2014

Contributor

Je ne savais pas qu'un autre outil était prévu. Je voulais juste voir un peu à quoi ressemblait un code django et ce que je pouvais y faire. ^^

Pas de souci hein, c'est même un bon moyen d'appréhender django. Si t'arrive à réparer la faute de style pep8 elle sera mergée.

Contributor

firm1 commented May 30, 2014

Je ne savais pas qu'un autre outil était prévu. Je voulais juste voir un peu à quoi ressemblait un code django et ce que je pouvais y faire. ^^

Pas de souci hein, c'est même un bon moyen d'appréhender django. Si t'arrive à réparer la faute de style pep8 elle sera mergée.

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis May 30, 2014

Contributor

Ok pour pep8, bizarre, chez moi elle n'avait pas lieu... En plus elle se déclenche sur des fichiers que je n'avais pas touchés. Je vais voir si c'est pas un problème de stream mal synchronisés.

Contributor

artragis commented May 30, 2014

Ok pour pep8, bizarre, chez moi elle n'avait pas lieu... En plus elle se déclenche sur des fichiers que je n'avais pas touchés. Je vais voir si c'est pas un problème de stream mal synchronisés.

@Eskimon

This comment has been minimized.

Show comment
Hide comment
@Eskimon

Eskimon May 30, 2014

Member

Erf j'ai eu la meme et je suspect mon IDE d'avoir fait un clean auto des espaces (ce qui a donc modifie des fichier que j'ai ouvert mais pas touche plus que ca...).

Member

Eskimon commented May 30, 2014

Erf j'ai eu la meme et je suspect mon IDE d'avoir fait un clean auto des espaces (ce qui a donc modifie des fichier que j'ai ouvert mais pas touche plus que ca...).

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis May 30, 2014

Contributor

Même après resynchronisation, impossible de répliquer les erreurs du build.
par contre, j'ai un problème avec les tests unitaires : zds.gallery.test.ImageTest me renvoie une IOError parceque jpeg encoder is not available

Contributor

artragis commented May 30, 2014

Même après resynchronisation, impossible de répliquer les erreurs du build.
par contre, j'ai un problème avec les tests unitaires : zds.gallery.test.ImageTest me renvoie une IOError parceque jpeg encoder is not available

@cgabard

This comment has been minimized.

Show comment
Hide comment
@cgabard

cgabard May 30, 2014

Member

Pour ton prob de jpeg, je pense a un prob avec Pillow, c'est la dépendance qui nous pose le plus de soucis depuis le début. Il faut bien installer tous les paquet de dev necessaire, puis l'installer avec easy_install pour que ça passe correctement. Ça doit etre documenté quelque part. Mais va falloir faire quelque chose pour changer ça.

Member

cgabard commented May 30, 2014

Pour ton prob de jpeg, je pense a un prob avec Pillow, c'est la dépendance qui nous pose le plus de soucis depuis le début. Il faut bien installer tous les paquet de dev necessaire, puis l'installer avec easy_install pour que ça passe correctement. Ça doit etre documenté quelque part. Mais va falloir faire quelque chose pour changer ça.

@firm1

This comment has been minimized.

Show comment
Hide comment
@firm1

firm1 May 30, 2014

Contributor

par contre, j'ai un problème avec les tests unitaires

Si tu es sous linux et sous une debian-like, tu peut installer les libs d'images avec la commande suivante :
sudo apt-get install libjpeg8 libjpeg-dev libfreetype6 libfreetype6-dev zlib1g-dev

et une fois que tu es dans ton environnement python source ../bin/activate, il faut relancer l'installation de la lib pillow, avec la commande suivante : sudo pip install -I pillow

Même après resynchronisation, impossible de répliquer les erreurs du build.

C'est bizarre, bon je pense que je vais virer les vérification de PEP-8 sur les PR dans un premier temps.

Contributor

firm1 commented May 30, 2014

par contre, j'ai un problème avec les tests unitaires

Si tu es sous linux et sous une debian-like, tu peut installer les libs d'images avec la commande suivante :
sudo apt-get install libjpeg8 libjpeg-dev libfreetype6 libfreetype6-dev zlib1g-dev

et une fois que tu es dans ton environnement python source ../bin/activate, il faut relancer l'installation de la lib pillow, avec la commande suivante : sudo pip install -I pillow

Même après resynchronisation, impossible de répliquer les erreurs du build.

C'est bizarre, bon je pense que je vais virer les vérification de PEP-8 sur les PR dans un premier temps.

@firm1

This comment has been minimized.

Show comment
Hide comment
@firm1

firm1 May 30, 2014

Contributor

Bon j'ai désactivé temporairement les vérifications PEP8, c'est pas encore au point on dirait

Contributor

firm1 commented May 30, 2014

Bon j'ai désactivé temporairement les vérifications PEP8, c'est pas encore au point on dirait

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis May 30, 2014

Contributor

J'ai tout réinstallé (mais je n'utilise pas virtualenv, pour développer sur zds j'utilise une vm donc je m'en tappe total de comment elle est gérée, elle ne servira qu'à zds). Pour autant, j'ai toujours le problème de jpeg.

Contributor

artragis commented May 30, 2014

J'ai tout réinstallé (mais je n'utilise pas virtualenv, pour développer sur zds j'utilise une vm donc je m'en tappe total de comment elle est gérée, elle ne servira qu'à zds). Pour autant, j'ai toujours le problème de jpeg.

@firm1

This comment has been minimized.

Show comment
Hide comment
@firm1

firm1 May 30, 2014

Contributor

Pour autant, j'ai toujours le problème de jpeg.

Quand tu réinstalle pillow (sudo pip install -I pillow), il fait bien la compilation, et normalement à la fin du une table dans le style suivant :

*** TKINTER support not available
    --- JPEG support not available
    --- ZLIB (PNG/ZIP) support not available
    --- FREETYPE2 support not available
    *** LITTLECMS support not available

Peux-tu renvoyer ici le contenu de ton tableau à toi ?

Contributor

firm1 commented May 30, 2014

Pour autant, j'ai toujours le problème de jpeg.

Quand tu réinstalle pillow (sudo pip install -I pillow), il fait bien la compilation, et normalement à la fin du une table dans le style suivant :

*** TKINTER support not available
    --- JPEG support not available
    --- ZLIB (PNG/ZIP) support not available
    --- FREETYPE2 support not available
    *** LITTLECMS support not available

Peux-tu renvoyer ici le contenu de ton tableau à toi ?

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis May 30, 2014

Contributor
*** TKINTER support not available
--- JPEG support available
*** OPENJPEG(JPEG200) support available
--- ZLIB (PNG/ZIP) support available
*** LIBTIFF support not available
--- FREETYPE2 support available
*** LITTLECMS support not available
*** WEBP support not available
*** WEBPMUX support not available
Contributor

artragis commented May 30, 2014

*** TKINTER support not available
--- JPEG support available
*** OPENJPEG(JPEG200) support available
--- ZLIB (PNG/ZIP) support available
*** LIBTIFF support not available
--- FREETYPE2 support available
*** LITTLECMS support not available
*** WEBP support not available
*** WEBPMUX support not available
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls May 30, 2014

Coverage Status

Coverage decreased (-0.03%) when pulling 78b4e0e on artragis:delete_image_on_change into 17152a0 on zestedesavoir:dev.

Coverage Status

Coverage decreased (-0.03%) when pulling 78b4e0e on artragis:delete_image_on_change into 17152a0 on zestedesavoir:dev.

@firm1

This comment has been minimized.

Show comment
Hide comment
@firm1

firm1 May 30, 2014

Contributor

Une dernère chose.

Peux-tu installer apt-get install python-dev aussi ?

Désolé pour tout ce traca, mais on des problèmes de synchro avec cette lib sur les OS différents.

Contributor

firm1 commented May 30, 2014

Une dernère chose.

Peux-tu installer apt-get install python-dev aussi ?

Désolé pour tout ce traca, mais on des problèmes de synchro avec cette lib sur les OS différents.

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis May 30, 2014

Contributor

déjà installé depuis le début.
Par contre comme le dit travis, j'ai pas test unitaire car j'ai pas trop compris comment le faire. Entre les client.post, les factory etc, j'ai pas trouvé où insérer :

#old = la fonction pour créer l'image
#new = la fonction pour créer l'image
#publish article
#modify article
self.AssertFalse(os.path.isFile(os.path.join(settings.SITE_ROOT, 'media', old.name)))
Contributor

artragis commented May 30, 2014

déjà installé depuis le début.
Par contre comme le dit travis, j'ai pas test unitaire car j'ai pas trop compris comment le faire. Entre les client.post, les factory etc, j'ai pas trouvé où insérer :

#old = la fonction pour créer l'image
#new = la fonction pour créer l'image
#publish article
#modify article
self.AssertFalse(os.path.isFile(os.path.join(settings.SITE_ROOT, 'media', old.name)))
@firm1

This comment has been minimized.

Show comment
Hide comment
@firm1

firm1 May 30, 2014

Contributor

Par contre comme le dit travis, j'ai pas test unitaire car j'ai pas trop compris comment le faire. Entre les client.post, les factory etc, j'ai pas trouvé où insérer :

C'est le genre de truc qui n'est pas évident à faire je l'avoue. Et j'ai pas le temps d'expliquer tout ça. J'envisage de documenter un peu mieux la doc en ce sens. Mais t'inquiète, ta PR ma l'air cohérente, je mergerai une fois testé en local chez moi.

Contributor

firm1 commented May 30, 2014

Par contre comme le dit travis, j'ai pas test unitaire car j'ai pas trop compris comment le faire. Entre les client.post, les factory etc, j'ai pas trouvé où insérer :

C'est le genre de truc qui n'est pas évident à faire je l'avoue. Et j'ai pas le temps d'expliquer tout ça. J'envisage de documenter un peu mieux la doc en ce sens. Mais t'inquiète, ta PR ma l'air cohérente, je mergerai une fois testé en local chez moi.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls May 30, 2014

Coverage Status

Coverage decreased (-0.03%) when pulling 78b4e0e on artragis:delete_image_on_change into 17152a0 on zestedesavoir:dev.

Coverage Status

Coverage decreased (-0.03%) when pulling 78b4e0e on artragis:delete_image_on_change into 17152a0 on zestedesavoir:dev.

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis May 30, 2014

Contributor

Ne merge pas. J'ai réussi à faire un TU et j'ai trouvé une typo dans mon code.

Contributor

artragis commented May 30, 2014

Ne merge pas. J'ai réussi à faire un TU et j'ai trouvé une typo dans mon code.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls May 30, 2014

Coverage Status

Coverage increased (+0.22%) when pulling cab17e1 on artragis:delete_image_on_change into 17152a0 on zestedesavoir:dev.

Coverage Status

Coverage increased (+0.22%) when pulling cab17e1 on artragis:delete_image_on_change into 17152a0 on zestedesavoir:dev.

@firm1

This comment has been minimized.

Show comment
Hide comment
@firm1

firm1 May 30, 2014

Contributor

Naïsse Djobe

Contributor

firm1 commented May 30, 2014

Naïsse Djobe

firm1 pushed a commit that referenced this pull request May 30, 2014

@firm1 firm1 merged commit 6a5a788 into zestedesavoir:dev May 30, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@artragis artragis deleted the artragis:delete_image_on_change branch Apr 11, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment