From b370f5b208244c9fb4e241c2a97add7ffe831daf Mon Sep 17 00:00:00 2001
From: "Christian P. MOMON" <christian.momon@devinsy.fr>
Date: Fri, 12 Jan 2018 02:23:26 +0100
Subject: [PATCH] Refactored raw review logs.

---
 src/org/april/hebdobot/model/Hebdobot.java | 27 +++++++++++-----------
 src/org/april/hebdobot/review/Review.java  |  1 -
 2 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/org/april/hebdobot/model/Hebdobot.java b/src/org/april/hebdobot/model/Hebdobot.java
index 3a0da64..1dba0e9 100644
--- a/src/org/april/hebdobot/model/Hebdobot.java
+++ b/src/org/april/hebdobot/model/Hebdobot.java
@@ -230,6 +230,11 @@ public class Hebdobot extends PircBot
         {
             String text = message.trim();
 
+            if (this.review != null)
+            {
+                this.review.addRaw(new Message(sender, text));
+            }
+
             if ((StringUtils.equalsIgnoreCase(text, "!aide")) || (StringUtils.equalsIgnoreCase(text, "!help")))
             {
                 logger.info("!help caught.");
@@ -239,6 +244,7 @@ public class Hebdobot extends PircBot
                 sendMessage(sender, "Je suis " + getName() + ", le robot de gestion des revues hebdomadaires de l'APRIL.");
                 sendMessage(sender, "Voici les commandes que je comprends :");
                 sendMessage(sender, " ");
+                sendMessage(sender, "  !aide         : afficher cette aide");
                 sendMessage(sender, "  !debut        : commencer une nouvelle revue");
                 sendMessage(sender, "  # titre       : démarrer un sujet individuel");
                 sendMessage(sender, "  ## titre      : démarrer un sujet collectif");
@@ -246,9 +252,9 @@ public class Hebdobot extends PircBot
                 sendMessage(sender, "  !courant      : afficher le sujet en cours");
                 sendMessage(sender, "  !manquants    : afficher les participants qui n'ont pas répondu sur le dernier sujet");
                 sendMessage(sender, "  !fin          : terminer la revue en cours");
-                sendMessage(sender, "  !aide         : afficher cette aide");
-                sendMessage(sender, "  !licence      : afficher la licence du logiciel Hebdobot et le lien vers ses sources");
                 sendMessage(sender, "  !stop         : abandonner la revue en cours");
+                sendMessage(sender, "  !licence      : afficher la licence du logiciel Hebdobot et le lien vers ses sources");
+                sendMessage(sender, "  !version      : afficher la version d'Hebdobot");
             }
             else if (StringUtils.equalsIgnoreCase(text, "!stop"))
             {
@@ -497,24 +503,19 @@ public class Hebdobot extends PircBot
             else if (StringsUtils.equalsAnyIgnoreCase(text, "!version"))
             {
                 logger.info("!version caught.");
-
                 sendMessage(new BuildInformation().toString());
             }
             else if (text.startsWith("%"))
             {
                 logger.info("% caught.");
 
-                // Store it in raw review part.
-                if (this.review != null)
-                {
-                    this.review.addRaw(new Message(sender, text));
-                }
+                // Ignore.
             }
             else if (StringsUtils.equalsAnyIgnoreCase(text, "!salut", "!bonjour", "!hello"))
             {
                 logger.info("!salut caught.");
 
-                // Command unknown.
+                // Salutation command.
                 sendMessage(sender + ", bonjour \\o/");
             }
             else if (text.startsWith("!"))
@@ -523,10 +524,6 @@ public class Hebdobot extends PircBot
 
                 // Command unknown.
                 sendMessage(sender + ", commande inconnue: " + text);
-                if (this.review != null)
-                {
-                    this.review.addRaw(new Message(sender, text));
-                }
             }
             else
             {
@@ -648,6 +645,10 @@ public class Hebdobot extends PircBot
     {
         logger.debug("Send message : {}", message);
         this.sendMessage(this.channel, message);
+        if (this.review != null)
+        {
+            this.review.addRaw(new Message("Hebdobot", message));
+        }
     }
 
     /**
diff --git a/src/org/april/hebdobot/review/Review.java b/src/org/april/hebdobot/review/Review.java
index 5117878..8783ff9 100644
--- a/src/org/april/hebdobot/review/Review.java
+++ b/src/org/april/hebdobot/review/Review.java
@@ -78,7 +78,6 @@ public class Review
             this.participants.add(message.getAuthor());
             this.currentTopic.add(message);
         }
-        this.addRaw(message);
     }
 
     /**