Gather conversation methods first to prevent these from using as event

methods

Author: angdev

jbot/src/main/java/me/ramswaroop/jbot/core/common/BaseBot.java

@@ -42,17 +42,26 @@ public abstract class BaseBot {
      */
     public BaseBot() {
         Method[] methods = this.getClass().getMethods();
+
+        // Gather conversation methods first to prevent these from using as event methods
         for (Method method : methods) {
             if (method.isAnnotationPresent(Controller.class)) {
                 Controller controller = method.getAnnotation(Controller.class);
-                EventType[] eventTypes = controller.events();
-                String pattern = controller.pattern();
-                int patternFlags = controller.patternFlags();
                 String next = controller.next();
 
                 if (!StringUtils.isEmpty(next)) {
                     conversationMethodNames.add(next);
                 }
+            }
+        }
+
+        for (Method method : methods) {
+            if (method.isAnnotationPresent(Controller.class)) {
+                Controller controller = method.getAnnotation(Controller.class);
+                EventType[] eventTypes = controller.events();
+                String pattern = controller.pattern();
+                int patternFlags = controller.patternFlags();
+                String next = controller.next();
 
                 MethodWrapper methodWrapper = new MethodWrapper();
                 methodWrapper.setMethod(method);
@@ -75,6 +84,8 @@ public BaseBot() {
                 methodNameMap.put(method.getName(), methodWrapper);
             }
         }
+
+
     }
 
     protected void startConversation(String id, String methodName) {