Produits Entr'ouvert » django-mellon

Ticket sans rapport ?

Si je vais avoir besoin d'isodate pour parser l'attribut cacheDuration et de toute façon isodate m'inspire plus confiance que dateutil pour parser des dates iso.

Mais ça veut dire ne plus builder pour jessie alors. (moi ça me va)

(ou taper ce paquet dans notre dépôt jessie, sans doute jouable)

(ou taper ce paquet dans notre dépôt jessie, sans doute jouable)

fait.

Je reformulerais: All other keys are override of generic settings -> All other keys override generic settings.

La logique et l'implémentation me semblent ok. Est-ce qu'on anticipe qu'il y'aura éventuellement des installations qui ne fonctionneront plus parce que URL distante absente et durée de cache dépassée?

URL distante absente et durée de cache dépassée

Le cache était dans la mémoire des processus, qui s'éteignaient de toute façon avec le temps etc.

Concernant 0001-code-style-10196.patch, sur mellon/utils.py je suis curieux, pourquoi passer par un xml_encoding[0] au lieu du simple xml_encoding précédent ?

Dans le même patch, concernant le expect après le isodate.parse_datetime, on couvre actuellement toutes les exceptions, tu as restreint ici à ValueError, TypError mais on pourrait en avoir d'autres si date_string n'est pas une string par exemple. Je serais pour être bête et laisser un "except Exception:" général. (Parce qu'on traite ici des bouts d'objets réponses de lasso, et je suis pas à l'aise pour savoir si c'est toujours des strings)

(0002 non relu encore par moi, je laisse Christophe finir ou je peux venir en rerelecteur si besoin)

Thomas Noël a écrit :

Concernant 0001-code-style-10196.patch, sur mellon/utils.py je suis curieux, pourquoi passer par un xml_encoding[0] au lieu du simple xml_encoding précédent ?

Dans le même patch, concernant le expect après le isodate.parse_datetime, on couvre actuellement toutes les exceptions, tu as restreint ici à ValueError, TypError mais on pourrait en avoir d'autres si date_string n'est pas une string par exemple. Je serais pour être bête et laisser un "except Exception:" général. (Parce qu'on traite ici des bouts d'objets réponses de lasso, et je suis pas à l'aise pour savoir si c'est toujours des strings)

Pour moi si c'est pas une string ça fera forcément TypeError (derrière ça utilise re je pense qui balance du TypeError dès qu'on a pas une string); et en vrai lasso renvoie juste une string ou None ou vraiment au pire un LassoNode (un objet Python tout con). Même pas peur.

(0002 non relu encore par moi, je laisse Christophe finir ou je peux venir en rerelecteur si besoin)

Benjamin Dauvergne a écrit :

Thomas Noël a écrit :

Concernant 0001-code-style-10196.patch, sur mellon/utils.py je suis curieux, pourquoi passer par un xml_encoding[0] au lieu du simple xml_encoding précédent ?

Pourquoi t'as pas répondu à ma question metaphysique d'abord hein ?

Dans le même patch, concernant le expect après le isodate.parse_datetime, on couvre actuellement toutes les exceptions, tu as restreint ici à ValueError, TypError mais on pourrait en avoir d'autres si date_string n'est pas une string par exemple. Je serais pour être bête et laisser un "except Exception:" général. (Parce qu'on traite ici des bouts d'objets réponses de lasso, et je suis pas à l'aise pour savoir si c'est toujours des strings)

Pour moi si c'est pas une string ça fera forcément TypeError (derrière ça utilise re je pense qui balance du TypeError dès qu'on a pas une string); et en vrai lasso renvoie juste une string ou None ou vraiment au pire un LassoNode (un objet Python tout con). Même pas peur.

Je disais ça parce que justement, None :

isodate.parse_datetime(None) -> AttributeError: 'NoneType' object has no attribute 'split'

Thomas Noël a écrit :

Benjamin Dauvergne a écrit :

Thomas Noël a écrit :

Concernant 0001-code-style-10196.patch, sur mellon/utils.py je suis curieux, pourquoi passer par un xml_encoding[0] au lieu du simple xml_encoding précédent ?

Pourquoi t'as pas répondu à ma question metaphysique d'abord hein ?

Ah.. ma réponse s'est perdue; en python tu ne peux pas modifier dans une closure une variable du scope supérieur, elle devient automatiquement locale; le simple fait d'avoir une affectation dans un scope rend la variable locale. Mais j'aurai pu faire un global xml_encoding aussi, je ne sais pas pourquoi je n'aime pas beaucoup.

Je disais ça parce que justement, None :

isodate.parse_datetime(None) -> AttributeError: 'NoneType' object has no attribute 'split'

Ok alors.