New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Passage à Django 1.10 #4347

Merged
merged 1 commit into from Jun 11, 2017

Conversation

Projects
None yet
5 participants
@gustavi
Member

gustavi commented May 23, 2017

Q R
Type de modification évolution
Ticket(s) (issue(s)) concerné(s) #3898

QA

TOUT doit être testé.

Et merci à @vhf pour sa première PR qui m'a aidé et le bug magique parce que je sais par lire 😸

Show outdated Hide outdated zds/gallery/views.py Outdated
@@ -22,6 +22,6 @@ def contactable_members(self):
.exclude(user__groups__in=excluded_groups) \
.filter(Q(can_read=True) | Q(end_ban_read__lte=now)) \
.order_by('-user__date_joined') \
.select_related('user__username')
.select_related('user')

This comment has been minimized.

@artragis

artragis May 23, 2017

Contributor

Why?

@artragis

artragis May 23, 2017

Contributor

Why?

This comment has been minimized.

@gustavi

gustavi May 23, 2017

Member

This comment has been minimized.

@artragis

artragis May 24, 2017

Contributor

DOnc maintenant on est obligé de sélectionner le user complet?

@artragis

artragis May 24, 2017

Contributor

DOnc maintenant on est obligé de sélectionner le user complet?

This comment has been minimized.

@gustavi

gustavi May 27, 2017

Member

Comme répondu sur IRC, avec {prefetch,select}_related oui.

@gustavi

gustavi May 27, 2017

Member

Comme répondu sur IRC, avec {prefetch,select}_related oui.

Show outdated Hide outdated zds/settings.py Outdated

@gustavi gustavi changed the title from Passage à Django 1.10 to [WIP] Passage à Django 1.10 May 23, 2017

@gustavi

This comment has been minimized.

Show comment
Hide comment
Member

gustavi commented May 24, 2017

@gustavi

This comment has been minimized.

Show comment
Hide comment
@gustavi

gustavi May 28, 2017

Member

Si quelqu'un a l'explication ou arrive à reproduire en version de dev pour ça je prends !

Member

gustavi commented May 28, 2017

Si quelqu'un a l'explication ou arrive à reproduire en version de dev pour ça je prends !

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis May 29, 2017

Contributor

repproduit.

pas sûr de ça, mais ça ressemble à un bug de la debug toolbar si j'en crois ma stack de debug django.

Contributor

artragis commented May 29, 2017

repproduit.

pas sûr de ça, mais ça ressemble à un bug de la debug toolbar si j'en crois ma stack de debug django.

@gustavi

This comment has been minimized.

Show comment
Hide comment
@gustavi

gustavi May 29, 2017

Member

Après quelques recherches, pour reproduire le bug mystère :

  • DEBUG=False
  • Créer une galerie avec admin
  • Créer une galerie avec user (pour se faciliter la tâche)
  • Aller sur la page des galeries
  • Modifier l'HTML pour changer le PK de la galerie de user par cette d'admin
  • Essayer de supprimer
Member

gustavi commented May 29, 2017

Après quelques recherches, pour reproduire le bug mystère :

  • DEBUG=False
  • Créer une galerie avec admin
  • Créer une galerie avec user (pour se faciliter la tâche)
  • Aller sur la page des galeries
  • Modifier l'HTML pour changer le PK de la galerie de user par cette d'admin
  • Essayer de supprimer
help='The prefix for users. Default: user.')
parser.add_argument('size', action='store', nargs='?', default='low', choices=['low', 'medium', 'high'],
type=str, help='Size level: low (x1), medium (x2) or high (x3). Default: low.')
parser.add_argument('type', action='store', nargs='*', type=str, default=self.zds_modules,

This comment has been minimized.

@artragis

artragis May 31, 2017

Contributor

en fait j'aurai pas fait comme ça :

parser.add_argument('-m', action='append_const', const='member', dest='type')
parser.add_argument('-s', action='append_const', const='staff', help='create staff members', dest='type')
parser.add_argument('--cf', action='append_const', const='category_forum', help='create_forum categories', dest='type')
parser.add_argument('--cc', action='append_const', const='category_content', help='create_forum categories', dest='type')
# etc.
@artragis

artragis May 31, 2017

Contributor

en fait j'aurai pas fait comme ça :

parser.add_argument('-m', action='append_const', const='member', dest='type')
parser.add_argument('-s', action='append_const', const='staff', help='create staff members', dest='type')
parser.add_argument('--cf', action='append_const', const='category_forum', help='create_forum categories', dest='type')
parser.add_argument('--cc', action='append_const', const='category_content', help='create_forum categories', dest='type')
# etc.

This comment has been minimized.

@gustavi

gustavi Jun 2, 2017

Member

J'ai fait en sorte que le comportement soit identique à celui actuel (pas de modification des TU). Je suis d'accord sur le fait que ça DOIT être amélioré.

@gustavi

gustavi Jun 2, 2017

Member

J'ai fait en sorte que le comportement soit identique à celui actuel (pas de modification des TU). Je suis d'accord sur le fait que ça DOIT être amélioré.

default_root = options[1].split(',')[0]
default_size = options.get('size', 'low')
default_root = options.get('racine', 'user')
default_module = options.get('type', self.zds_modules)

This comment has been minimized.

@artragis

artragis May 31, 2017

Contributor

si tu utilises ma méthode, va falloir mettre options.get('type') or self.zds_modules

@artragis

artragis May 31, 2017

Contributor

si tu utilises ma méthode, va falloir mettre options.get('type') or self.zds_modules

# create "media" folder if not existing
if not os.path.exists(MEDIA_ROOT):
os.mkdir(MEDIA_ROOT)
for filename in glob.glob(' '.join(args)):
for filename in glob.glob(files):

This comment has been minimized.

@artragis

artragis May 31, 2017

Contributor

+1

@artragis

artragis May 31, 2017

Contributor

+1

@@ -14,14 +14,19 @@ class Command(BaseCommand):
args = 'files'
help = 'Load complex fixtures for ZdS'
def add_arguments(self, parser):
parser.add_argument('files', nargs='?', type=str)

This comment has been minimized.

@artragis

artragis May 31, 2017

Contributor

un petit help pour dire que :

  • on gère le glob
  • on s'attend à des fichier yaml
@artragis

artragis May 31, 2017

Contributor

un petit help pour dire que :

  • on gère le glob
  • on s'attend à des fichier yaml

This comment has been minimized.

@gustavi

gustavi Jun 8, 2017

Member

Pour le coup il va falloir faire une PR propre pour revoir TOUTES les commandes. Je ferai ça à ce moment.

@gustavi

gustavi Jun 8, 2017

Member

Pour le coup il va falloir faire une PR propre pour revoir TOUTES les commandes. Je ferai ça à ce moment.

@vhf

This comment has been minimized.

Show comment
Hide comment
@vhf

vhf Jun 6, 2017

Member

FYI https://code.djangoproject.com/ticket/28276#comment:1

On saute direct à 1.11 ou on renomme une petite dizaine de clés à cause de ce bug Django. Tu choisis.

Member

vhf commented Jun 6, 2017

FYI https://code.djangoproject.com/ticket/28276#comment:1

On saute direct à 1.11 ou on renomme une petite dizaine de clés à cause de ce bug Django. Tu choisis.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jun 6, 2017

Coverage Status

Coverage increased (+0.2%) to 73.347% when pulling 1c19121 on gustavi:django-update into 96c0063 on zestedesavoir:dev.

coveralls commented Jun 6, 2017

Coverage Status

Coverage increased (+0.2%) to 73.347% when pulling 1c19121 on gustavi:django-update into 96c0063 on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jun 7, 2017

Coverage Status

Coverage increased (+0.2%) to 73.341% when pulling fe50a14 on gustavi:django-update into 96c0063 on zestedesavoir:dev.

coveralls commented Jun 7, 2017

Coverage Status

Coverage increased (+0.2%) to 73.341% when pulling fe50a14 on gustavi:django-update into 96c0063 on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jun 8, 2017

Coverage Status

Coverage increased (+0.04%) to 89.075% when pulling 28cb164 on gustavi:django-update into 4f73279 on zestedesavoir:dev.

coveralls commented Jun 8, 2017

Coverage Status

Coverage increased (+0.04%) to 89.075% when pulling 28cb164 on gustavi:django-update into 4f73279 on zestedesavoir:dev.

@gustavi gustavi added the QA svp label Jun 8, 2017

@gustavi gustavi changed the title from [WIP] Passage à Django 1.10 to Passage à Django 1.10 Jun 8, 2017

@gustavi

This comment has been minimized.

Show comment
Hide comment
@gustavi

gustavi Jun 8, 2017

Member

QA GO GO GO !

Member

gustavi commented Jun 8, 2017

QA GO GO GO !

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jun 8, 2017

Coverage Status

Coverage increased (+0.03%) to 89.064% when pulling a493c3e on gustavi:django-update into d68f8c5 on zestedesavoir:dev.

coveralls commented Jun 8, 2017

Coverage Status

Coverage increased (+0.03%) to 89.064% when pulling a493c3e on gustavi:django-update into d68f8c5 on zestedesavoir:dev.

Show outdated Hide outdated zds/gallery/tests/tests_views.py Outdated
Show outdated Hide outdated zds/gallery/tests/tests_views.py Outdated
@gcodeur

This comment has been minimized.

Show comment
Hide comment
@gcodeur

gcodeur Jun 10, 2017

Member

J'ai fait un petit tour du site avec Django 1.10 et tout est OK !

J'ai testé :

  • la génération des fixtures (j'ai fait un make restart_db qui a marché parfaitement)
  • les galeries (ajout d'images notamment)
  • les contenus (création et publication, import et export d'archives, dont certaines venant de Django 1.8)
  • les forums (création de sujets, mise en résolu, déplacement, alertes de modération)
  • l'API (plusieurs vues testées via la documentation automatique)
  • la création d'un MP
  • l'admin Django
Member

gcodeur commented Jun 10, 2017

J'ai fait un petit tour du site avec Django 1.10 et tout est OK !

J'ai testé :

  • la génération des fixtures (j'ai fait un make restart_db qui a marché parfaitement)
  • les galeries (ajout d'images notamment)
  • les contenus (création et publication, import et export d'archives, dont certaines venant de Django 1.8)
  • les forums (création de sujets, mise en résolu, déplacement, alertes de modération)
  • l'API (plusieurs vues testées via la documentation automatique)
  • la création d'un MP
  • l'admin Django
@gustavi

This comment has been minimized.

Show comment
Hide comment
@gustavi

gustavi Jun 10, 2017

Member

C'est mis à jour avec les dernières retouches et rebase. Je pense que si bug ultra spécifique il y a il sera trouvé en bêta.

@artragis je te laisse merge quand c'est bon pour toi !

Member

gustavi commented Jun 10, 2017

C'est mis à jour avec les dernières retouches et rebase. Je pense que si bug ultra spécifique il y a il sera trouvé en bêta.

@artragis je te laisse merge quand c'est bon pour toi !

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jun 10, 2017

Coverage Status

Coverage increased (+0.03%) to 89.064% when pulling 1052351 on gustavi:django-update into ab085bd on zestedesavoir:dev.

coveralls commented Jun 10, 2017

Coverage Status

Coverage increased (+0.03%) to 89.064% when pulling 1052351 on gustavi:django-update into ab085bd on zestedesavoir:dev.

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Jun 11, 2017

Contributor

Je merge, on verra en béta.

Contributor

artragis commented Jun 11, 2017

Je merge, on verra en béta.

@artragis artragis merged commit 1c22d82 into zestedesavoir:dev Jun 11, 2017

1 check passed

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

@artragis artragis added this to the Version de développement milestone Jun 11, 2017

@gcodeur gcodeur removed the QA svp label Aug 12, 2017

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