https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342014-09-19T12:13:33ZRedmine Entr’ouvertMandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=203202014-09-19T12:13:33ZJérôme Schneiderjschneider@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/4970">0001-backends-add-last_connection_date-and-creation_date-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/4970/0001-backends-add-last_connection_date-and-creation_date-.patch">0001-backends-add-last_connection_date-and-creation_date-.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>En cours</i></li><li><strong>% réalisé</strong> changé de <i>0</i> à <i>60</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul> Mandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=203222014-09-19T12:25:55ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>On ne peut pas passer la valeur par défaut comme ça, démonstration avec ce script, qui affichera deux fois la même heure.</p>
<pre>
#! /usr/bin/env python
from datetime import datetime
import time
def print_time(t=datetime.utcnow()):
print t
print_time()
time.sleep(10)
print_time()
</pre> Mandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=203252014-09-19T13:17:03ZJérôme Schneiderjschneider@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/4972">0001-backends-add-last_connection_date-and-creation_date-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/4972/0001-backends-add-last_connection_date-and-creation_date-.patch">0001-backends-add-last_connection_date-and-creation_date-.patch</a> ajouté</li></ul><p>Zut je pensais que c'était exécuté à chaque appelle j'aurais du vérifier ! Une nouvelle version qui corrige le problème.</p> Mandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=203282014-09-19T13:43:06ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Perso je préfère de loin avoir les paramètres nommés et avoir "<code>if creation_date is None: creation_date = datetime.utcnow()</code>" dans le corps de la méthode, plutôt que passer par kwargs. (ou <code>creation_date = creation_date or datetime.utcnow()</code> si on veut ça sur une ligne sans "if").</p> Mandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=203352014-09-19T14:28:24ZJérôme Schneiderjschneider@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/4973">0001-backends-add-last_connection_date-and-creation_date-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/4973/0001-backends-add-last_connection_date-and-creation_date-.patch">0001-backends-add-last_connection_date-and-creation_date-.patch</a> ajouté</li></ul><p>Un nouveau patch sans kwargs.</p> Mandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=203362014-09-19T14:32:45ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Dans le backend sql tu utilises datetime.utcnow() dans le patch mais c'est datetime.now() qui est utilisé dans update_last_connection.</p> Mandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=203442014-09-19T14:50:36ZJérôme Schneiderjschneider@entrouvert.com
<ul></ul><p>Ca me semble mieux de migrer en utc surtout avec l'utilisation de plusieurs backends. Je pensais changer le now en utcnow dans le prochain commit. Après je ne suis pas sur d'avoir fait le bon choix.</p> Mandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=203452014-09-19T15:01:20ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Il y a aussi des datetime.now() dans le models.py.</p>
<pre>
creation_date = Column(DateTime, default=datetime.now(), nullable=False)
last_connection = Column(DateTime, default=datetime.now())
</pre>
<p>À lire rapidement la doc de sqlalchemy, par défaut il considère les colonnes de type datetime sans timezone, tout pourrait ainsi être passé à utcnow() (bien sûr il faudrait idéalement convertir les données existantes, mais passons); j'ai aussi regardé vite fait et jamais on n'utilise dans l'UI ces champs, tu confirmes ?</p> Mandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=203462014-09-19T15:09:31ZJérôme Schneiderjschneider@entrouvert.com
<ul></ul><p>On Fri, Sep 19, 2014 at 05:01:20PM +0200, <a class="email" href="mailto:redmine@entrouvert.com">redmine@entrouvert.com</a> wrote:</p>
<blockquote>
<p>À lire rapidement la doc de sqlalchemy, par défaut il considère les colonnes de type datetime sans timezone, tout pourrait ainsi être passé à utcnow() (bien sûr il faudrait idéalement convertir les données existantes, mais passons); j'ai aussi regardé vite fait et jamais on n'utilise dans l'UI ces champs, tu confirmes ?</p>
</blockquote>
<p>Oui je confirme. L'idée de base c'était de faire des statistiques pour<br />Isabelle mais finalement c'est tombé à l'eau.<br />Je ne l'utilise que pour débuger actuellement.</p> Mandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=203502014-09-19T15:56:46ZJérôme Schneiderjschneider@entrouvert.com
<ul></ul><p>C'est poussé. J'ouvre un autre ticket pour le soucis de date.</p>
<pre>
commit 4c755abe699f5f5cbf55c5299ed189c7724cee80
Author: Jérôme Schneider <jschneider@entrouvert.com>
Date: Fri Sep 19 16:24:53 2014 +0200
backends: add last_connection_date and creation_date to update_or_create
Closes #5542
</pre> Mandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=203512014-09-19T16:00:11ZJérôme Schneiderjschneider@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Résolu (à déployer)</i></li><li><strong>% réalisé</strong> changé de <i>60</i> à <i>100</i></li></ul><p>Appliqué par commit commit:4c755abe699f5f5cbf55c5299ed189c7724cee80.</p> Mandaye - Development #5542: Backends: ajouter l'option last_connection_date et creation_date dans le update_or_createhttps://dev.entrouvert.org/issues/5542?journal_id=208382014-10-02T08:31:33ZJérôme Schneiderjschneider@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Fermé</i></li></ul>