Re: Agenda et pull request

pour la modifi à l'origine du présent message, ce serait cela

<diff>
--- a/action/editer_evenement.php
+++ b/action/editer_evenement.php
@@ -198,12 +198,10 @@ function
agenda_action_update_repetitions($id_evenement, $repetitions) {
                                // TODO : prendre en charge la mise a
jour uniquement si conforme a l'original
                                $c['date_debut'] = $update_date_debut;
                                $c['date_fin'] = $update_date_fin;
-
                                // mettre a jour l'evenement
-                               sql_updateq(
-                                       'spip_evenements',
-                                       $c,
-                                       'id_evenement =
'.$row['id_evenement']
+                               evenement_modifier(
+                                       $row['id_evenement'],
+                                       $c
                                );
                        } else {
                                // il est supprime
</diff>
comme tu vois, rien de bien méchant, mais cela serait utile

Le vendredi 31 janvier 2020 à 15:57 +0100, Cerdic a écrit :
> Hello Maieul,
> 
> désolé j’ai pas eu le temps d’aller remettre le nez dans le plugin
> agenda, et j’ai un peu peur de ce que je vais y trouver.
> J’ai l’impression (peut-être à tort) que ce ne soit en train de
> devenir une usine à gaz sur le mode « faut pouvoir tout faire avec »,
> ce qui ne m’emballe pas du tout.
> 
> Si tu peux attendre un peu, je vais aller me replonger dedans semaine
> prochaine, je te répondrai sur le fond…
> 
> 
> 
> --
> Cédric
> Le 31 janv. 2020 à 14:00 +0100, Maïeul Rouquette <[email protected]>,
> a écrit :
> > Oups,
> > sur la bonne liste, spip-zone n'existe plus
> > Le vendredi 31 janvier 2020 à 13:50 +0100, Maïeul Rouquette a écrit
> > :
> > > Salut Cédric,
> > > 
> > > j'ai une modification que j'aimerais soumettre (comme tu l'a
> > > demandé)
> > > sous forme de pull request pour le plugin agenda. La modification
> > > vise
> > > simplement à ce que les modifications des repetitions d'evenement
> > > soient
> > > aussi l'objet de révision.
> > > 
> > > J'ai voulu faire une bifurcation personnelle du dépot, mais il
> > > n'est
> > > pas
> > > possible de le faire car je dispose des droits. Du coup j'hésite à
> > > faire
> > > une branche puis un pull request depuis la branche, parce que cela
> > > va
> > > créer aussi la branche sur la zone pour pas longtemps. Qu'en pense
> > > tu
> > > ?
> > > 
> > > Par ailleurs, le fait qu'une modification d'un evenement source se
> > > repercute systématiquement sur les evenements enfants peut être
> > > problématique dans certain cas, bien qu'utile dans la majorité des
> > > cas.
> > > 
> > > 
> > > Par exemple sur un de mes sites j'ai un champ extra "evenement
> > > annulé".
> > > Si je l'applique sur la première occurence, cela l'applique
> > > partout,
> > > ce
> > > qui n'est pas le but souhaité.
> > > 
> > > J'aimerais bien trouver une solution pour avoir le choix, mais ce
> > > n'est
> > > pas évident de trouver le mieux en terme d'UX. En gros je vois
> > > trois
> > > solutions, mais aucune vraiment satisfaisante
> > > 
> > > 
> > > 
> > > 1. Pouvoir "rompre" le lien entre une evenement et sa source
> > > (variante
> > > :pouvoir créer une repetition d'evenement sans que le lien ne soit
> > > gardé
> > > en base)
> > > 2. Une option lors de la modif d'evenement pour que les
> > > repetitions ne
> > > soient pas impactées, mais j'ai peur que cela fasse un formulaire
> > > pas
> > > clair.
> > > 3. Une option globale dans la configuration de l'agenda.
> > > 
> > > Qu'en penses-tu ?
> > > 
> > > Amicalement
> > > 
> > > Maïeul
> > > 
> >

diff --git a/action/editer_evenement.php b/action/editer_evenement.php
index b9ee975..69a81b6 100644
--- a/action/editer_evenement.php
+++ b/action/editer_evenement.php
@@ -198,12 +198,10 @@ function agenda_action_update_repetitions($id_evenement, $repetitions) {
 				// TODO : prendre en charge la mise a jour uniquement si conforme a l'original
 				$c['date_debut'] = $update_date_debut;
 				$c['date_fin'] = $update_date_fin;
-
 				// mettre a jour l'evenement
-				sql_updateq(
-					'spip_evenements',
-					$c,
-					'id_evenement = '.$row['id_evenement']
+				evenement_modifier(
+					$row['id_evenement'],
+					$c
 				);
 			} else {
 				// il est supprime