Project

General

Profile

Bug #30865

mettre l'export ods avec les autres actions

Added by Frédéric Péters 5 months ago. Updated 4 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
25 Feb 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Aujourd'hui ce bouton se trouve en haut de la barre latérale, il devrait se trouver avec les autres actions de la visualisation (renommer/supprimer/url pour iframe).

0001-views-move-ods-export-next-to-other-visualisation-ac.patch View (5.83 KB) Frédéric Péters, 25 Feb 2019 09:43 AM

0001-views-move-ods-export-next-to-other-visualisation-ac.patch View (5.82 KB) Frédéric Péters, 28 Feb 2019 08:29 AM

Associated revisions

Revision 24943e31 (diff)
Added by Frédéric Péters 5 months ago

views: move ods export next to other visualisation actions (#30865)

Revision 85112b4e (diff)
Added by Frédéric Péters 5 months ago

tests: update for ods being a view (#30865)

History

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

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

Version du patch rebasée pour ne pas dépendre de #26847.

#3 Updated by Benjamin Dauvergne 5 months ago

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

Ack.

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

  • Status changed from Solution validée to Résolu (à déployer)
commit 24943e3136545086d3a41032c0eaa5efbc69e7dd
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Feb 25 09:39:02 2019 +0100

    views: move ods export next to other visualisation actions (#30865)

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

Et adaptation au test :

commit 85112b4ec8eb06010decddd4ee1c5e0042dda4d3
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Mar 1 16:05:55 2019 +0100

    tests: update for ods being a view (#30865)

À noter ici que je me suis fait avoir par ce qui était {% if visualization }bouton ods{ endif %}, prenant visualization pour une instance enregistrée, alors qu'il existe aussi une classe Visualization qui n'est pas le modèle Visualization, et que l'objet en question, c'était un objet de cette classe, pas une instance du modèle Visualization.

Cela fait que ce patch fait perdre la fonctionnalité d'export ODS de visualisation non enregistrées mais j'opte pour le laisser quand même.

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

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

Also available in: Atom PDF