Projet

Général

Profil

Development #8293

Ajouter MediaDefiningClass à la métaclasse de CellBase

Ajouté par Benjamin Dauvergne il y a plus de 8 ans. Mis à jour il y a plus de 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
18 septembre 2015
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour pouvoir faire des sum(cell.media for cell in cells).


Fichiers


Demandes liées

Lié à Combo - Development #8167: possibilité d'ouvrir des popups dans les pagesFermé03 septembre 2015

Actions

Révisions associées

Révision 52aa9794 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 8 ans

data: add support for Media class to CellBase (fixes #8293)

You can produce media for a whole page with

media = sum((cell.media for cell in cells), Media())

The `Media()` is ugly but sum is not able to find itself the base case
for the folding, it defaults to 0 which is not addable to Media.

Historique

#2

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

#3

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

Les tests passent, par contre la ligne pour utiliser ça est un poil plus longue

ctx['media'] = sum((cell.media for cell in cells), Media())

#4

Mis à jour par Serghei Mihai il y a plus de 8 ans

It works.
Sinon, la declaration de requests dans le setup.py sert à quelquechose?

#5

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

Ah oui désolé c'est partie dans le même patch, je l'enlèverai. Oui combo ne démarre pas en local sans cela, je fais un pip install -e . dans combo et j'ai une erreur sur le import requests dans le fichier models.py. Je vais ouvrir un autre ticket.

#6

Mis à jour par Serghei Mihai il y a plus de 8 ans

En tout cas ACK pour moi

#8

Mis à jour par Serghei Mihai il y a plus de 8 ans

Ack

#9

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

  • Statut changé de Nouveau à Résolu (à déployer)
  • % réalisé changé de 0 à 100
#10

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

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

Formats disponibles : Atom PDF