Project

General

Profile

Bug #10798

les items des datasources avec text vides ne sont plus visibles

Added by Thomas Noël almost 5 years ago. Updated over 4 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Target version:
Start date:
02 May 2016
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Description

Dans #10447 il a été ajouté :

        # skip malformed items
        if not item.get('id') or not item.get('text'):
            continue

qui passe à la trappe les items où text="".

Bogue visible avec cette source :

{
    "data": [
        {
            "id": "0", 
            "text": "" 
        }, 
        {
            "id": "1", 
            "text": "Bis" 
        }, 
        {
            "id": "2", 
            "text": "Ter" 
        }, 
        {
            "id": "3", 
            "text": "Quater" 
        }
    ], 
    "err": 0
}

où le champ vide n'est plus visible dans la liste.


Files

Associated revisions

Revision 59d31901 (diff)
Added by Thomas Noël almost 5 years ago

data_source: dont ignore list items without text (#10798)

History

#1

Updated by Thomas Noël almost 5 years ago

Pour avis (j'ai déjà posé sur la recette, je le fais en prod si ok).

Pour le patch final, ajout de tests en cours.

#2

Updated by Thomas Noël almost 5 years ago

Ca ne passait pas les tests, voici donc une version où le data_source est mis dans la norme id+text au niveau de get_structured_items

#3

Updated by Frédéric Péters almost 5 years ago

ok.

#4

Updated by Thomas Noël almost 5 years ago

  • Status changed from En cours to Résolu (à déployer)
commit 59d3190118837dc661928c609fffb6beee302ffa
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Mon May 2 13:48:36 2016 +0200

    data_source: dont ignore list items without text (#10798)

#5

Updated by Thomas Noël almost 5 years ago

Comme c'est bloquant, je l'ai posé à la main sur prod et recette.

#6

Updated by Thomas Noël almost 5 years ago

  • Target version set to v1.42
#7

Updated by Frédéric Péters over 4 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF