Index: src/include/class.rappels.php
==================================================================
--- src/include/class.rappels.php
+++ src/include/class.rappels.php
@@ -19,11 +19,11 @@
throw new UserException('Cotisation inconnue.');
}
$data['id_cotisation'] = (int) $data['id_cotisation'];
- if (empty($data['delai']) || !is_numeric($data['delai']))
+ if ((trim($data['delai']) === '') || !is_numeric($data['delai']))
{
throw new UserException('Délai avant rappel invalide : doit être indiqué en nombre de jours.');
}
$data['delai'] = (int) $data['delai'];
Index: src/templates/admin/membres/cotisations/gestion/rappel_modifier.tpl
==================================================================
--- src/templates/admin/membres/cotisations/gestion/rappel_modifier.tpl
+++ src/templates/admin/membres/cotisations/gestion/rappel_modifier.tpl
@@ -37,13 +37,21 @@
obligatoire
obligatoire
- jours
-
-
+
+
+
+
+
+
+
+
+
+
+
obligatoire
Astuce : pour inclure dans le contenu du mail le nom du membre, utilisez #IDENTITE, pour inclure le délai de l'envoi utilisez #NB_JOURS.
Index: src/templates/admin/membres/cotisations/gestion/rappels.tpl
==================================================================
--- src/templates/admin/membres/cotisations/gestion/rappels.tpl
+++ src/templates/admin/membres/cotisations/gestion/rappels.tpl
@@ -1,6 +1,6 @@
-{include file="admin/_head.tpl" title="Gestion des rappels automatiques" current="membres/cotisations"}
+{include file="admin/_head.tpl" title="Gestion des rappels automatiques" current="membres/cotisations" js=1}
- Cotisations
- Saisie d'une cotisation
- État des rappels
@@ -33,11 +33,19 @@
du {$rappel.debut|format_sqlite_date_to_french} au {$rappel.fin|format_sqlite_date_to_french}
{else}
ponctuelle
{/if}
- {$rappel.delai|abs|escape} jours {if $rappel.delai > 0}après{else}avant{/if} expiration |
+
+ {if $rappel.delai == 0}le jour de l'expiration
+ {else}
+ {$rappel.delai|abs|escape}
+ {if abs($rappel.delai) > 1}jours{else}jour{/if}
+ {if $rappel.delai > 0}après{else}avant{/if}
+ expiration
+ {/if}
+ |
{$rappel.sujet|escape} |
✎
✘
|
@@ -77,13 +85,21 @@
- obligatoire
- obligatoire
- - jours
-
-
+
+ -
+
+
+
+
+ -
+
+
+
+
- obligatoire
- Astuce : pour inclure dans le contenu du mail le nom du membre, utilisez #IDENTITE, pour inclure le délai de l'envoi utilisez #NB_JOURS.
@@ -92,7 +108,20 @@
{csrf_field key="new_rappel"}
+
+
{include file="admin/_foot.tpl"}
Index: src/www/admin/membres/cotisations/gestion/rappel_modifier.php
==================================================================
--- src/www/admin/membres/cotisations/gestion/rappel_modifier.php
+++ src/www/admin/membres/cotisations/gestion/rappel_modifier.php
@@ -33,12 +33,16 @@
$error = 'Une erreur est survenue, merci de renvoyer le formulaire.';
}
else
{
try {
- $delai = (int) utils::post('delai');
- $delai = utils::post('delai_pre') ? -$delai : $delai;
+ if (utils::post('delai_choix') == 0)
+ $delai = 0;
+ elseif (utils::post('delai_choix') > 0)
+ $delai = (int) utils::post('delai_post');
+ else
+ $delai = -(int) utils::post('delai_pre');
$rappels->edit($rappel['id'], [
'sujet' => utils::post('sujet'),
'texte' => utils::post('texte'),
'delai' => $delai,
@@ -54,14 +58,14 @@
}
}
$tpl->assign('error', $error);
-$rappel['delai_pre'] = $rappel['delai'] > 0 ? 0 : 1;
-$rappel['delai'] = abs($rappel['delai']);
+$rappel['delai_pre'] = $rappel['delai_post'] = abs($rappel['delai']) ?: 30;
+$rappel['delai_choix'] = $rappel['delai'] == 0 ? 0 : ($rappel['delai'] > 0 ? 1 : -1);
$tpl->assign('rappel', $rappel);
$tpl->assign('cotisations', $cotisations->listCurrent());
$tpl->display('admin/membres/cotisations/gestion/rappel_modifier.tpl');
?>
Index: src/www/admin/membres/cotisations/gestion/rappels.php
==================================================================
--- src/www/admin/membres/cotisations/gestion/rappels.php
+++ src/www/admin/membres/cotisations/gestion/rappels.php
@@ -20,12 +20,16 @@
$error = 'Une erreur est survenue, merci de renvoyer le formulaire.';
}
else
{
try {
- $delai = (int) utils::post('delai');
- $delai = utils::post('delai_pre') ? -$delai : $delai;
+ if (utils::post('delai_choix') == 0)
+ $delai = 0;
+ elseif (utils::post('delai_choix') > 0)
+ $delai = (int) utils::post('delai_post');
+ else
+ $delai = -(int) utils::post('delai_pre');
$rappels->add([
'sujet' => utils::post('sujet'),
'texte' => utils::post('texte'),
'delai' => $delai,