Project

General

Profile

Bug #22651

recherche full texte et messages dans l'historique

Added by Thomas Noël over 1 year ago. Updated 11 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
20 Mar 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

sur un formdata, dans SQL :

    def store(self):
        ...
        if self._evolution:
            for evo in self._evolution:
                if evo.comment:
                    fts_strings.append(evo.comment)

Donc on indexe dans la recherche full texte les commentaires posés dans l'historique (evo.comment).

Mais on n'ajoute pas les messages qui seraient dans les evo.display_parts() provenant d'une action "enregistrer dans l'historique".

A l'usage, ça donne l'impression d'une incohérence dans la recherche, vu que ces contenus sont affichés de la même façon.

0001-misc-add-evolution-messages-to-fulltext-search-22651.patch View (4.43 KB) Frédéric Péters, 31 Dec 2018 02:46 PM

Associated revisions

Revision 4f6895a7 (diff)
Added by Frédéric Péters 11 months ago

misc: add evolution messages to fulltext search (#22651)

History

#2 Updated by Thomas Noël over 1 year ago

Avant de se lancer, là dedans, régler d'abord #22383 (séparation entre migration et re-indexation)

#3 Updated by Frédéric Péters 11 months ago

(j'ai hésité à ajouter le bump de migration, parce que #24430 utilise déjà ce numéro, mais c'est quand même moins lourd à changer que des migrations django, donc, voilà avec).

#4 Updated by Benjamin Dauvergne 11 months ago

  • Status changed from Solution proposée to Solution validée

Ack.

#5 Updated by Frédéric Péters 11 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 4f6895a7b137c4d50deb6b2ce35618d29f9e7ab6
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Dec 31 14:31:25 2018 +0100

    misc: add evolution messages to fulltext search (#22651)

#6 Updated by Frédéric Péters 11 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF