Produits Entr'ouvert » w.c.s.

Faut pas utiliser les séquences postgres, elles ne sont pas intégré aux transactions (en gros quand tu fais nextval('masequence') et que la transaction échoue le compteur n'est pas diminué de un, parce que sinon ça implique un lock sur le compteur et ça fait des contentions entre transactions qui font juste des insertions, mais ça veut dire que les primary key dans postgres n'ont rien à voir avec l'ordre de commit des lignes dans la table (on peut avoir t.id < t.id et t.time > t.time même si t.time à pour valeur par défaut now()).

Donc la bonne façon de faire c'est une table séquence à deux colonnes 'name' et 'value' et on fait des

update sequence set value = value + 1 where sequence = 'masequence' return value

Note : on parle ici de utils. mais on est passé en Django, on pourrait donc partir sur un tag counter disponible dans dans {% load qommon %}

• {% counter "foo" %} : renvoie 1, 2, 3, ... incrémenté à chaque appel sur "foo"
• {% counter "foo" get %} : renvoie la valeur actuelle du compteur
• {% counter "foo" silent %} : incrémente mais ne dit rien
• {% counter "foo" as bar %} : incrémente et stocke la valeur dans {{bar}}
• {% counter "foo" set 10 %} : remet le compteur à 10 (pourrait être une variable)
• {% counter "foo" add 5 %} : ajoute 5 au compteur (peut être un nombre négatif ou une variable)

• {% counter form_slug %} : et hop, un compteur sur le formulaire, quel qu'il soit

• {% counter form_slug add 3 silent as counter %}On a encore vendu 3 pépitos, ça en fait {{ counter }} au total

Thomas Noël a écrit :

Note : on parle ici de utils. mais on est passé en Django, on pourrait donc partir sur un tag counter disponible dans dans @{% load qommon ?
[...]
Et si c'est bien fait on peut cumuler tout ça, genre :

• {% counter form_slug add 3 silent as counter %}On a encore vendu 3 pépitos, ça en fait {{ counter }} au total

Je ne suis pas fan des trucs qui ont un effet de bord au milieu d'un template, c'est mon coté Haskell mais sauf si c'est fait transactionellement et tout et tout je préfère encore une action de workflow explicite "Obtenir un nouveau numéro et affecter à une variable de traitement", là on multiplie les états cachés.

Benjamin Dauvergne a écrit :

Je ne suis pas fan des trucs qui ont un effet de bord au milieu d'un template, c'est mon coté Haskell mais sauf si c'est fait transactionellement et tout et tout je préfère encore une action de workflow explicite "Obtenir un nouveau numéro et affecter à une variable de traitement", là on multiplie les états cachés.

• le nom du compteur (qui peut être une variable via l'usage de template Django)
• éventuellement la valeur à ajouter (là aussi possibilité d'expression souple via template) ; par défaut c'est « 1 »
• la donnée de traitement cible, à choisir parmi les champs de type texte disponibles

Et ne rendre cette action visible dans la liste des choix d'action que s'il y a au moins une donné de traitement texte, et qu'on est en mode SQL, ça me semble une bonne idée.

A voir si ça répond à Meyzieu.

Bonjour,

A voir si ça répond à Meyzieu.

cela peut répondre si, comme je l'ai compris, il s'agit de la mise à disposition d'une variable associée au formulaire (et non au téléservice usager) qui s'incrémente à chaque appel de l'action "compteur" dans le workflow associé. Sa valeur est alors stockée dans une donnée de traitement cible pour chaque téléservice usager auquel il s'applique.

Aude MERITZA-BOZON a écrit :

Bonjour,

A voir si ça répond à Meyzieu.

cela peut répondre si, comme je l'ai compris, il s'agit de la mise à disposition d'une variable associée au formulaire (et non au téléservice usager) qui s'incrémente à chaque appel de l'action "compteur" dans le workflow associé. Sa valeur est alors stockée dans une donnée de traitement cible pour chaque téléservice usager auquel il s'applique.

Le plan qui est imaginé ici est d'avoir un compteur lié à "un nom" (par exemple "inscription-base-nautique-2018"), qui est transversal à toute la plateforme. Ce nom serait à préciser dans l'action. Au passage dans l'action, quand l'inscription est validée, la demande concernée gagne un nouveau numéro d'inscription à la base nautique. On peut même imaginer que ça soit utilisé dans des workflows différents, ça restera un compteur unique.

Cela permet de générer des numéros d'inscription dans l'ordre, sans trou ; c'est je crois ce qui était attendu (dans la note numéro 4 du ticket #17420 https://dev.entrouvert.org/issues/17420#note-4)

J'espère avoir réussi à être clair...?

C'est tout à fait ça. :-)

Pour inspiration seulement je transfère ma note privée du ticket Meyzieu:

Benjamin Dauvergne a écrit :

Thomas Noël a écrit :

Frédéric Péters a écrit :

Je ne sais pas pourquoi tu veux attendre lundi, ce n'est pas avec trois minutes par personne sur un sujet que tout le monde découvrira qu'il y aura une réponse pertinente, permettant de fixer qui/quand/comment.

J'ai posé une idée de syntaxe sur https://dev.entrouvert.org/issues/17437 ; Benjamin a donné la technique derrière (une simple table SQL clé/valeur "bête et méchante").

Si personne a le temps on peut aussi poser un script dans teleservics.meyzieu.fr/scripts/counter.py

from wcs.sql import get_connection()
name = args[0]

conn = get_connection(new=True)
with conn.cursor() as cur:
    cur.execute('CREATE TABLE IF NOT EXISTS meyzieu_counter (name varchar primary key, value int)')
    conn.commit()
    while True:
        try:
             cur.execute('update meyzieu_counter set value = value + 1 where name = %s returning value', name)
             r = cur.fetchone()
             if r:
                 result = r[0]
             else:
                 cur.execute('insert into meyzieu_counter values (%s, 0)', name)
                 result = 0
        exception conn.Error:
             conn.rollback()
        break
    conn.commit()

Je propose ça à Aude, car je ne suis qu'à un brouillon de patch.
Je viens de tester en local le code proposé par Benj avec quelques corrections mineures:

from wcs.sql import get_connection

name = args[0]

conn = get_connection(new=True)
with conn.cursor() as cur:
    cur.execute('CREATE TABLE IF NOT EXISTS meyzieu_counter (name varchar primary key, value int)')
    conn.commit()
    while True:
        try:
             cur.execute('update meyzieu_counter set value=value+1 where name=%s returning value', (name,))
             r = cur.fetchone()
             if r:
                 result = str(r[0])
             else:
                 cur.execute('insert into meyzieu_counter values (%s, 0)', (name,))
                 result = '0'
        except conn.Error:
            conn.rollback()
        break
    conn.commit()

Ce ticket est pour avoir la fonctionnalité en standard dans w.c.s.