Produits Entr'ouvert » w.c.s.

0001: ok

• factoriser evo_author = evo.get_author_name()
• et j'ai pas compris le calcul de evo_author_more mais c'est la fatigue

et j'ai pas compris le calcul de evo_author_more mais c'est la fatigue

J'avais regardé et ce truc remontait à loin, à l'utilisation de "user_hash", et donc un tas de trucs compliqués qui avaient aujourd'hui comme seul résultat d'afficher "(Original Submitter)" derrière le nom de l'usager.

Tout ça est bien beau mais il y a un troisième commit à ce ticket, visible dans la branche :
https://git.entrouvert.org/wcs.git/commit/?h=wip/django&id=6a451dcf35452005a770a246701dd50119a4058b

Et il supprime de toute façon ce code python. (en fait même pour 0002 vaut mieux regarder la branche).

Au final cette partie donne :

    def get_author_qualification(self):
        if self.who == '_submitter' and not self.formdata.is_submitter(get_request().user):
             return _('Original Submitter')
        return None

Et donc on affiche "original submitter" derrière le nom de l'usager, quand ce n'est pas lui qui consulte.

Je pense que sur l'algo qui évite d'afficher 10 fois un status, il y a une différence de logique entre le if not (parts_value or evo.status != previous_status): dans l'ancien code et le if evolution.who or evolution.is_status_change or display_parts du template.

Dans le premier, le previous_status est bien celui qu'on a affiché en dernier, en fonction de l'usager (s'il y a le droit de le voir ou pas). Dans le template, is_status_change ne prend pas en compte l'usager, il regarde juste si le statut est nouveau dans la liste des évolutions.

Dans le cadre d'une évolution genre "en attente","relance interne","en attente" où la relance n'est pas visible de l'usager, on doit juste afficher "en attente" une seule fois.

Je me demande dans quelle mesure on ne pourrait pas juste utiliser le "ifchanged" de Django (https://docs.djangoproject.com/en/1.10/ref/templates/builtins/#ifchanged) -- ah non, y'a l'heure...

C'est très juste, patch revu https://git.entrouvert.org/wcs.git/commit/?h=wip/django&id=7d24ec44ee86480dd42239a6f53a7108f6f54750

Aussi, ce qui n'était pas fait avant (même avant django il me semble), ça prend en compte l'auteur, comme ça deux interventions dans un même statut par deux personnes différentes, même s'il n'y a ni commentaire ni rien d'autre à afficher, seront visibles.

Et yield, parce que 2017.

Ack