Add config option to ignore maintenance for hidePlayersHover

Reviewed-on: https://codeberg.org/Andre601/AdvancedServerList/pulls/397

core/pom.xml

@@ -41,7 +41,7 @@
     <properties>
         <maven.compiler.source>16</maven.compiler.source>
         <maven.compiler.target>16</maven.compiler.target>
-        <plugin.config.version>7</plugin.config.version>
+        <plugin.config.version>8</plugin.config.version>
     </properties>
     
     <repositories>

core/src/main/java/ch/andre601/advancedserverlist/core/events/PingEventHandler.java

@@ -158,7 +158,7 @@ public class PingEventHandler{
         
         boolean hidePlayers = ProfileManager.checkOption(entry.hidePlayersEnabled());
         
-        if(hidePlayers && ignoreMaintenance(event, "hidePlayers")){
+        if(hidePlayers && ignoreMaintenance(event, "hidePlayers") && ignoreMaintenance(event, "hidePlayersHover")){
             logger.debug(PingEventHandler.class, "'playerCount -> hidePlayers' enabled. Hiding player count...");
             
             event.hidePlayers();

core/src/main/java/ch/andre601/advancedserverlist/core/file/ConfigMigrator.java

@@ -46,7 +46,8 @@ public class ConfigMigrator{
     public static ConfigurationTransformation.Versioned create(){
         return ConfigurationTransformation.versionedBuilder()
             .versionKey("configVersion")
-            .addVersion(LATEST, sixToSeven())
+            .addVersion(LATEST, sevenToEight())
+            .addVersion(7, sixToSeven())
             .addVersion(6, fiveToSix())
             .addVersion(5, fourToFive())
             .addVersion(4, threeToFour())
@@ -55,6 +56,16 @@ public class ConfigMigrator{
             .build();
     }
     
+    public static ConfigurationTransformation sevenToEight(){
+        return ConfigurationTransformation.builder()
+            .addAction(NodePath.path("disableDuringMaintenance"), (path, value) -> {
+                value.node("hidePlayersHover").set(true);
+                
+                return null;
+            })
+            .build();
+    }
+    
     public static ConfigurationTransformation sixToSeven(){
         return ConfigurationTransformation.builder()
             .addAction(NodePath.path(), (path, value) -> {

core/src/main/resources/config.yml

@@ -92,6 +92,12 @@ disableDuringMaintenance:
   #
   hidePlayers: true
   #
+  # Whether hiding player list on hover should be disabled while Maintenance mode is active.
+  #
+  # Default: true
+  #
+  hidePlayersHover: true
+  #
   # Whether player count text changing should be disabled while Maintenance mode is active.
   #
   # Default: true

core/src/main/resources/profiles/default.yml

@@ -95,6 +95,15 @@ playerCount:
   #
   hidePlayers: false
   #
+  # Set whether the List of players is shown when hovering over the player count.
+  # Setting this to true will also make the plugin ignore the hover option below.
+  #
+  # If not present, defaults to false.
+  #
+  # Read more: https://asl.andre601.ch/profiles/#hideplayershover
+  #
+  hidePlayersHover: false
+  #
   # Set the lines shown when the player hovers over the player count.
   # Unlike the MOTD option does this one only support basic color and formatting codes. Any RGB colors will be
   # downsampled to the nearest matching color.

docs/getting-started/index.md

@@ -244,6 +244,15 @@ playerCount:
   #
   hidePlayers: false
   #
+  # Set whether the List of players is shown when hovering over the player count.
+  # Setting this to true will also make the plugin ignore the hover option below.
+  #
+  # If not present, defaults to false.
+  #
+  # Read more: https://asl.andre601.ch/profiles/#hideplayershover
+  #
+  hidePlayersHover: false
+  #
   # Set the lines shown when the player hovers over the player count.
   # Unlike the MOTD option does this one only support basic color and formatting codes. Any RGB colors will be
   # downsampled to the nearest matching color.
@@ -465,6 +474,12 @@ disableDuringMaintenance:
   #
   hidePlayers: true
   #
+  # Whether hiding player list on hover should be disabled while Maintenance mode is active.
+  #
+  # Default: true
+  #
+  hidePlayersHover: true
+  #
   # Whether player count text changing should be disabled while Maintenance mode is active.
   #
   # Default: true

docs/profiles/index.md

@@ -198,6 +198,13 @@ playerCount:
 </div>
 ///
 
+### HidePlayersHover
+
+The `playerCount -> hidePlayersHover` option takes a boolean (`true` or `false`) to set whether the hover displayed on the player count should be hidden or not.
+
+When set to true will hovering over the player count show nothing.  
+In addition will [`playerCount -> hover`](#hover) not be used.
+
 ### Hover
 
 The `playerCount -> hover` option allows you to define lines of text to show when a player hovers over the player count with their cursor.

pom.xml

@@ -37,7 +37,7 @@
     <properties>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         
-        <plugin.version>5.4.0</plugin.version>
+        <plugin.version>5.4.1</plugin.version>
         <plugin.description>Create multiple Server lists based on conditions.</plugin.description>
         
         <api.version>4.1.0</api.version>

renovate.json

@@ -8,16 +8,6 @@
     "ch.andre601.advancedserverlist:core"
   ],
   "packageRules": [
-    {
-      "description": "Automerge non-major updates",
-      "matchUpdateTypes": [
-        "minor",
-        "patch",
-        "pin",
-        "digest"
-      ],
-      "automerge": true
-    },
     {
       "description": "Use Jitpack registry for various Libs",
       "matchPackageNames": [

version-uploader/pom.xml

@@ -52,7 +52,7 @@
         <dependency>
             <groupId>ch.qos.logback</groupId>
             <artifactId>logback-classic</artifactId>
-            <version>1.5.15</version>
+            <version>1.5.16</version>
             <scope>compile</scope>
         </dependency>
         <dependency>