Projet

Général

Profil

Development #27476

brouillons, toujours, indépendamment du code de suivi

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
19 octobre 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

De #27425 :

En tant qu'administrateur fonctionnel, je veux que la fonctionnalité brouillon soit activée tout le temps, y compris si le code de suivi est désactivé, afin de permettre ne plus à avoir à expliquer cette fonctionnalité et ne plus être en collision avec l'activation ou non du code de suivi.


Fichiers

0001-forms-always-enable-drafts-27476.patch (14,7 ko) 0001-forms-always-enable-drafts-27476.patch Frédéric Péters, 19 octobre 2018 16:15
0001-scss-don-t-include-margin-before-single-remove-draft.patch (786 octets) 0001-scss-don-t-include-margin-before-single-remove-draft.patch côté intégration graphique, ajustement de marge Frédéric Péters, 19 octobre 2018 16:20

Demandes liées

Lié à Publik - Development #27425: Activation des brouillonsFermé19 octobre 2018

Actions

Révisions associées

Révision 08ea74c0 (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

forms: always enable drafts (#27476)

Historique

#1

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

#2

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

#4

Mis à jour par Thomas Noël il y a plus de 5 ans

On est conscient et ok que ça va générer des tas de choses inutiles dans nos tables, sur un logiciel qui est déjà assez lent ?

#5

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

Thomas Noël a écrit :

On est conscient et ok que ça va générer des tas de choses inutiles dans nos tables, sur un logiciel qui est déjà assez lent ?

Réduire la durée de conservation des brouillons à 30 jours ?

#6

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

On est conscient et ok que ça va générer des tas de choses inutiles dans nos tables, sur un logiciel qui est déjà assez lent ?

Réduire la durée de conservation des brouillons à 30 jours ?

Je ne pense pas que le nombre de codes de suivis, ou de brouillons, joue terriblement sur les performances.

Sur les codes de suivi, ça n'a clairement pas d'impact, ils n'y a jamais de .select() dessus, seulement des .get() sur l'id, et celui-ci est en primary key et indexé.

Sur les brouillons, il peut y avoir un impact sur les tableaux listant tous les formulaires, parce que le code est actuellement celui-ci :

            item_ids = formdata_class.keys()
            drafts = formdata_class.get_ids_with_indexed_value('status', 'draft')
            item_ids = [x for x in item_ids if x not in drafts]

On récupère tous les identifiants de demandes puis on en retire les brouillons via du Python. Au doigt mouillé je dirais que c'est négligeable mais c'est surtout très facile à améliorer, en donnant à .keys() la même gestion de critères qu'à .select().

Si vous voyez d'autres choses, dites.

#7

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

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

Ça me va.

#8

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 08ea74c03ccb8a78dedbef4b2ce4c881a0754e12
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Oct 19 16:03:16 2018 +0200

    forms: always enable drafts (#27476)
#9

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

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

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

+ le patch côté publik-base-theme que j'avais oublié, pushé aujourd'hui.

Formats disponibles : Atom PDF