removed custom condition from IntervalReward because it makes no sense there

Author: jandie1505

PlayerLevels-API/src/main/java/net/jandie1505/playerlevels/core/rewards/IntervalRewardData.java

@@ -10,7 +10,6 @@
  * This data is normally created by the reward creator for the specific reward type, like {@link CommandReward#createInterval(String, boolean, CommandReward.SenderType, Component, int, int, int)}.<br/>
  * It can be used to create events using {@link net.jandie1505.playerlevels.api.core.reward.RewardsManager#addIntervalReward(RewardConfig, IntervalRewardData)}.
  * @param executor the code that will be executed when the reward is applied
- * @param customCondition a custom condition when the reward counts as applied (apart from the level condition)
  * @param descriptionProvider Provides a description for the reward.
  * @param requiresPlayerOnline if the player is required to be online that the reward is applied
  * @param start the level where from which the interval calculation starts
@@ -19,7 +18,6 @@
  */
 public record IntervalRewardData(
         @NotNull RewardExecutor executor,
-        @Nullable RewardCondition customCondition,
         @Nullable RewardDescriptionProvider descriptionProvider,
         boolean requiresPlayerOnline,
         int start,

PlayerLevels-API/src/main/java/net/jandie1505/playerlevels/core/rewards/types/CommandReward.java

@@ -155,7 +155,7 @@ public static IntervalRewardData createInterval(
             int limit
     ) {
         CommandReward reward = new CommandReward(command, senderType, description);
-        return new IntervalRewardData(reward, null, reward, requiresOnlinePlayer, start, interval, limit);
+        return new IntervalRewardData(reward, reward, requiresOnlinePlayer, start, interval, limit);
     }
 
     // ----- CREATOR -----

PlayerLevels-API/src/main/java/net/jandie1505/playerlevels/core/rewards/types/PlayerPointsReward.java

@@ -137,7 +137,7 @@ public static IntervalRewardData createInterval(
             int limit
     ) {
         PlayerPointsReward reward = new PlayerPointsReward(formula, description);
-        return new IntervalRewardData(reward, null, reward, true, start, interval, limit);
+        return new IntervalRewardData(reward, reward, true, start, interval, limit);
     }
 
     // ----- OTHER STUFF -----

PlayerLevels-Core/src/main/java/net/jandie1505/playerlevels/core/rewards/IntervalReward.java

@@ -13,18 +13,16 @@ public class IntervalReward extends Reward implements net.jandie1505.playerlevel
     private final int start;
     private final int interval;
     private final int limit;
-    @NotNull private final RewardCondition customCondition;
 
-    public IntervalReward(@NotNull RewardsManager manager, @NotNull String id, @Nullable String serverId, int start, int interval, int limit, @NotNull RewardExecutor executor, @Nullable RewardCondition customCondition, boolean requireOnlinePlayer, @NotNull String name, @Nullable RewardDescriptionProvider descriptionProvider) {
+    public IntervalReward(@NotNull RewardsManager manager, @NotNull String id, @Nullable String serverId, int start, int interval, int limit, @NotNull RewardExecutor executor, boolean requireOnlinePlayer, @NotNull String name, @Nullable RewardDescriptionProvider descriptionProvider) {
         super(manager, id, serverId, executor, requireOnlinePlayer, limit, name, descriptionProvider);
         this.start = start > 0 ? start : 1;
         this.interval = interval > 0 ? interval : 1;
         this.limit = limit;
-        this.customCondition = customCondition != null ? customCondition : net.jandie1505.playerlevels.api.core.reward.IntervalReward.DEFAULT_CONDITION;
     }
 
     public IntervalReward(@NotNull RewardsManager manager, @NotNull RewardConfig config, @NotNull IntervalRewardData data) {
-        this(manager, config.id(), config.serverId(), data.start(), data.interval(), data.limit(), data.executor(), data.customCondition(), data.requiresPlayerOnline(), config.name(), data.descriptionProvider());
+        this(manager, config.id(), config.serverId(), data.start(), data.interval(), data.limit(), data.executor(), data.requiresPlayerOnline(), config.name(), data.descriptionProvider());
     }
 
     // ----- CONDITIONS -----
@@ -37,26 +35,8 @@ public boolean checkApplyCondition(@NotNull Leveler leveler, int checkedLevel) {
             return false;
         }
 
-        // Condition is not met when the reward has already been applied for the level
-        if (leveler.getData().getOrCreateReceivedReward(this.getId()).level() >= checkedLevel) {
-            return false;
-        }
-
-        // Check if the reward has already been applied with the custom condition
-        try {
-            return !this.customCondition.isApplied(this, leveler, checkedLevel);
-        } catch (Exception e) {
-            IntervalReward.this.getManager().getPlugin().getLogger().log(Level.WARNING, "Exception while applying reward " + IntervalReward.this.getId() + " to player " + leveler.getPlayerUUID(), e);
-            return false;
-        } catch (Throwable t) {
-            IntervalReward.this.setEnabled(false);
-            IntervalReward.this.getManager().getPlugin().getLogger().log(Level.SEVERE,
-                    "A throwable which is not an exception has been thrown in isApplied from reward " + IntervalReward.this.getId() + " to player " + leveler.getPlayerUUID() + " " +
-                            "The reward has been disabled for safety reasons. DO NOT IGNORE THIS!",
-                    t
-            );
-            return false;
-        }
+        // The Condition is not met when the reward has already been applied for the level
+        return leveler.getData().getOrCreateReceivedReward(this.getId()).level() < checkedLevel;
     }
 
     @Override