Fix objective filter not reacting dynamically (PGMDev#1050)

Signed-off-by: Pablete1234 <[email protected]>
Signed-off-by: Samuel Roach <[email protected]>

core/src/main/java/tc/oc/pgm/filters/FilterMatchModule.java

@@ -188,6 +188,16 @@ private <F extends Filterable<?>> void register(
       throw new IllegalStateException("Cannot register filter listener after match has started");
     }
 
+    // Ideally this should never happen. All features that try to register a dynamic filter should
+    // have ensured that the filter can respond dynamically to the scope, at XML parse time.
+    // However, that does not occur, nothing is validating dynamic filters' scopes at all.
+    // The proper fix is to have a validation, using FeatureValidation and all features
+    // that will use a filter dynamically to validate the scope.
+    if (!filter.respondsTo(scope)) {
+      throw new IllegalStateException(
+          "Filter " + filter + " does not respond to " + scope.getSimpleName() + " scope");
+    }
+
     final ListenerSet listenerSet =
         this.listeners.row(filter).computeIfAbsent(scope, s -> new ListenerSet());
 

core/src/main/java/tc/oc/pgm/filters/operator/TeamFilterAdapter.java

@@ -1,6 +1,8 @@
 package tc.oc.pgm.filters.operator;
 
+import java.util.Collection;
 import java.util.Optional;
+import org.bukkit.event.Event;
 import tc.oc.pgm.api.filter.query.MatchQuery;
 import tc.oc.pgm.features.XMLFeatureReference;
 import tc.oc.pgm.filters.matcher.TypedFilter;
@@ -41,6 +43,11 @@ public boolean isDynamic() {
     return filter.isDynamic();
   }
 
+  @Override
+  public Collection<Class<? extends Event>> getRelevantEvents() {
+    return filter.getRelevantEvents();
+  }
+
   @Override
   public Class<? extends MatchQuery> queryType() {
     return MatchQuery.class;

core/src/main/java/tc/oc/pgm/modes/ObjectiveModesMatchModule.java

@@ -52,6 +52,7 @@ public void load() throws ModuleLoadException {
       if (mode.getFilter() != null) {
         // if filter returns ALLOW at any time in the match, start countdown for mode change
         fmm.onRise(
+            Match.class,
             mode.getFilter(),
             listener -> {
               if (!this.countdownContext.isRunning(countdown) && match.isRunning()) {