From a9057039c420fdbd893a752fa9ee2d2927e6453e Mon Sep 17 00:00:00 2001
From: "Christian P. MOMON" <christian.momon@devinsy.fr>
Date: Fri, 3 May 2019 16:18:27 +0200
Subject: [PATCH] Restricted the use of !debut command when review is already
 started (#3689).

---
 .../hebdobot/bot/hooks/StartReviewHook.java   | 39 +++++++++++--------
 1 file changed, 23 insertions(+), 16 deletions(-)

diff --git a/src/org/april/hebdobot/bot/hooks/StartReviewHook.java b/src/org/april/hebdobot/bot/hooks/StartReviewHook.java
index a88e01f..93d26c5 100644
--- a/src/org/april/hebdobot/bot/hooks/StartReviewHook.java
+++ b/src/org/april/hebdobot/bot/hooks/StartReviewHook.java
@@ -54,24 +54,31 @@ public class StartReviewHook extends Hook
             else
             {
                 // Start.
-                if (bot.getCronManager() != null)
+                if (bot.getReview() != null)
                 {
-                    try
-                    {
-                        bot.getCronManager().shutdown();
-                    }
-                    catch (SchedulerException exception)
-                    {
-                        logger.warn("Scheduler shutdown failed.", exception);
-                    }
+                    bot.sendMessage("% " + sender + ", une revue est déjà en cours.");
+                }
+                else
+                {
+                    if (bot.getCronManager() != null)
+                    {
+                        try
+                        {
+                            bot.getCronManager().shutdown();
+                        }
+                        catch (SchedulerException exception)
+                        {
+                            logger.warn("Scheduler shutdown failed.", exception);
+                        }
+                    }
+                    bot.setReview(new Review(sender, bot.getAliases()));
+                    bot.sendMessage(sender, "Bonjour " + sender + ", vous êtes le conducteur de réunion.");
+                    bot.sendMessage(sender, "Pour terminer la réunion, tapez \"!fin\"");
+                    bot.sendMessage("% Début de la réunion hebdomadaire");
+                    bot.sendMessage(
+                            "% rappel : toute ligne commençant par % sera considérée comme un commentaire et non prise en compte dans la synthèse");
+                    bot.sendMessage("% pour connaître le point courant, taper !courant");
                 }
-                bot.setReview(new Review(sender, bot.getAliases()));
-                bot.sendMessage(sender, "Bonjour " + sender + ", vous êtes le conducteur de réunion.");
-                bot.sendMessage(sender, "Pour terminer la réunion, tapez \"!fin\"");
-                bot.sendMessage("% Début de la réunion hebdomadaire");
-                bot.sendMessage(
-                        "% rappel : toute ligne commençant par % sera considérée comme un commentaire et non prise en compte dans la synthèse");
-                bot.sendMessage("% pour connaître le point courant, taper !courant");
             }
 
             result = true;