2017-07-04
01:44 Fixed ticket [1c44dd7d62]: Possibilité d'ajouter des extensions Skriv plus 4 other changes artifact: b7ae444772 user: bohwaz
01:44 Fixed ticket [b7f19b5b01]: Extensions Skriv plus 5 other changes artifact: 655fdacf99 user: bohwaz
01:43
Ajout signal plugin à l'initialisation de SkrivML, cf. [b7f19b5b01d3a531c0d9a4800c508f092815a816] check-in: 3d5e15e89f user: bohwaz tags: dev
2017-06-28
16:37 New ticket [b7f19b5b01] Extensions Skriv. artifact: f4aa8a263f user: sylvain.philip

Ticket Hash: b7f19b5b01d3a531c0d9a4800c508f092815a816
Title: Extensions Skriv
Status: Fixed Type: Feature_Request
Severity: Cosmetic Priority: Immediate
Subsystem: Resolution: Fixed
Last Modified: 2017-07-04 01:44:28
Version Found In: 0.8.0
User Comments:

sylvain.philip added on 2017-06-28 14:37:20:

La possibilité d'ajouter des extensions Skriv serait super sympa pour les développeurs.

J'avais déjà ouvert un ticket en 2015 mais la solution que je proposais n'était pas clean.

Après mûre réflexion, il me semble que l'utilisation d'un plugin serait bien adaptée pour ajouter des extensions Skriv.

Il suffirait simplement d'appliquer ce patch à la version de dev :

diff --git a/src/include/lib/Garradin/Utils.php b/src/include/lib/Garradin/Utils.php
index 8ba1d57..e4fc507 100644
--- a/src/include/lib/Garradin/Utils.php
+++ b/src/include/lib/Garradin/Utils.php
@@ -381,6 +381,7 @@
             self::$skriv = new \KD2\SkrivLite;
             self::$skriv->registerExtension('fichier', ['\\Garradin\\Fichiers', 'SkrivFichier']);
             self::$skriv->registerExtension('image', ['\\Garradin\\Fichiers', 'SkrivImage']);
+            Plugin::fireSignal('skriv.init', array('skriv' => self::$skriv));
         }
 
         $skriv =& self::$skriv;

bohwaz added on 2017-07-03 23:44:28:

Bonne idée, implémenté dans [3d5e15e89fc10bfb5099c3d4818bde5001b7e654] :)