Produits Entr'ouvert » w.c.s.

Pour un tableau dans le backoffice, on fait :

                field_value = filled.get_field_view_value(f,
                        max_length=min(120/len(fields), 30))

Et l'idée derrière doit être de caler un max de colonnes à l'écran, tout en leur donnant des largeurs plutôt similaires (ça date de 2005 pour la forme précise de cette formule…). Mais quand il y a plus de 120 champs ça fait que min(0, 30), ça fait 0, et ça arrive sur :

        if max_length:
            # if max_length is set the target is a backoffice listing/table,
            # return an html value, appropriately shortened.

Mais ne prend du coup pas cette branche; ce qui donne alors une valeur qui n'est pas assurée d'être une chaine.

On pourra dire qu'avoir ce mélange de comportement dans get_field_view_value n'était pas une bonne idée.

Fix immédiat, taper "if max_length is not None". Ça passe mais le résultat ne sera pas celui attendu, parce qu'ellipsize() avec une chaine < 5 caractères, coupe mal la chaine (le code fait [:max_length-5]).

Plus généralement, je reviendrais bien sur ce comportement pour toujours faire max_length=30 et hop.