Projet

Général

Profil

Development #31139

python3: utiliser les fonctions de parsing d'url de django.utils.six

Ajouté par Paul Marillonnet il y a environ 5 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
06 mars 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

qui simplifient beaucoup la chose


Fichiers


Demandes liées

Lié à Authentic 2 - Development #28276: Fonctionner avec Python3 pour Django1.11Fermé23 novembre 2018

Actions

Révisions associées

Révision 934d619f (diff)
Ajouté par Paul Marillonnet il y a environ 5 ans

python3: use django.utils.six url-parsing functions (#31139)

Historique

#1

Mis à jour par Paul Marillonnet il y a environ 5 ans

  • Statut changé de Nouveau à Rejeté

oups...

#2

Mis à jour par Paul Marillonnet il y a environ 5 ans

  • Projet changé de ldaptools à Authentic 2
  • Statut changé de Rejeté à Nouveau

Je ré-ouvre ici.

#3

Mis à jour par Paul Marillonnet il y a environ 5 ans

#4

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Solution proposée à Solution validée
  • Assigné à mis à Paul Marillonnet

Ack.

#5

Mis à jour par Paul Marillonnet il y a environ 5 ans

Aïe, dans ma branche globale de migration python3 je fais des patches sur mes patches, et je me vautre.
Un nouveau patch, qui corrige le précédent comme ci-dessous :

diff --git a/tests/utils.py b/tests/utils.py
index 21891afb..404247b2 100644
--- a/tests/utils.py
+++ b/tests/utils.py
@@ -7,7 +7,7 @@ import pytest
 from django.test import TestCase
 from django.core.urlresolvers import reverse
 from django.conf import settings
-from django.utils.six.moves.urllib.parse import urlparse
+from django.utils.six.moves.urllib import parse as urlparse
 from six import string_types

 from authentic2 import utils

#7

Mis à jour par Paul Marillonnet il y a environ 5 ans

#8

Mis à jour par Paul Marillonnet il y a environ 5 ans

Erreur de namespace dans le précédent patch. En voici un nouveau qui devrait corriger le build.

#9

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Solution proposée à Solution validée

Ack mais ça m'irait qu'on ait toujours urlparse et pas parse que je trouve un peu obscure.

#10

Mis à jour par Paul Marillonnet il y a environ 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)

Ok oui c'est vrai, corrigé ici.

commit 934d619f6bff2ccbd903d495c33a9c8d065fe324
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Thu Jan 24 11:40:48 2019 +0100

    python3: use django.utils.six url-parsing functions (#31139)

#11

Mis à jour par Frédéric Péters il y a environ 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF