Bump version to 5.1.2 for release

Boy, and I thought having to make the first commit was embarrassing, wait till people see that I realized that was missing and didn't think about the counterpart. At least I did think about it though, I wasn't even testing on 1.20 where it would break things.

addon/togglepersist/pom.xml

@@ -8,7 +8,7 @@
         <artifactId>openinvparent</artifactId>
         <groupId>com.lishid</groupId>
         <relativePath>../../pom.xml</relativePath>
-        <version>5.1.0</version>
+        <version>5.1.2</version>
     </parent>
 
     <artifactId>openinvtogglepersist</artifactId>

api/pom.xml

@@ -21,7 +21,7 @@
   <parent>
     <artifactId>openinvparent</artifactId>
     <groupId>com.lishid</groupId>
-    <version>5.1.0</version>
+    <version>5.1.2</version>
   </parent>
 
   <artifactId>openinvapi</artifactId>

common/pom.xml

@@ -7,7 +7,7 @@
     <parent>
         <groupId>com.lishid</groupId>
         <artifactId>openinvparent</artifactId>
-        <version>5.1.0</version>
+        <version>5.1.2</version>
     </parent>
 
     <artifactId>openinvcommon</artifactId>

internal/v1_20_R3/pom.xml

@@ -23,7 +23,7 @@
     <artifactId>openinvparent</artifactId>
     <groupId>com.lishid</groupId>
     <relativePath>../../pom.xml</relativePath>
-    <version>5.1.0</version>
+    <version>5.1.2</version>
   </parent>
 
   <artifactId>openinvadapter1_20_R3</artifactId>

internal/v1_20_R4/pom.xml

@@ -23,7 +23,7 @@
     <artifactId>openinvparent</artifactId>
     <groupId>com.lishid</groupId>
     <relativePath>../../pom.xml</relativePath>
-    <version>5.1.0</version>
+    <version>5.1.2</version>
   </parent>
 
   <artifactId>openinvadapter1_20_R4</artifactId>

internal/v1_21_R1/pom.xml

@@ -23,7 +23,7 @@
     <artifactId>openinvparent</artifactId>
     <groupId>com.lishid</groupId>
     <relativePath>../../pom.xml</relativePath>
-    <version>5.1.0</version>
+    <version>5.1.2</version>
   </parent>
 
   <artifactId>openinvadapter1_21_R1</artifactId>
@@ -32,7 +32,7 @@
   <properties>
     <maven.compiler.system>21</maven.compiler.system>
     <maven.compiler.release>21</maven.compiler.release>
-    <spigot.version>1.21-R0.1-SNAPSHOT</spigot.version>
+    <spigot.version>1.21.1-R0.1-SNAPSHOT</spigot.version>
   </properties>
 
   <dependencies>

jitpack.yml

@@ -0,0 +1,4 @@
+before_install:
+  - sdk install java 21.0.4-tem
+  - sdk use java 21.0.4-tem
+  - mvn wrapper:wrapper -Dmaven=3.9.8

plugin/pom.xml

@@ -21,7 +21,7 @@
   <parent>
     <artifactId>openinvparent</artifactId>
     <groupId>com.lishid</groupId>
-    <version>5.1.0</version>
+    <version>5.1.2</version>
   </parent>
 
   <artifactId>openinvplugin</artifactId>
@@ -47,19 +47,19 @@
     <dependency>
       <groupId>com.lishid</groupId>
       <artifactId>openinvadapter1_21_R1</artifactId>
-      <version>5.1.0</version>
+      <version>5.1.2</version>
       <scope>compile</scope>
     </dependency>
     <dependency>
       <groupId>com.lishid</groupId>
       <artifactId>openinvadapter1_20_R4</artifactId>
-      <version>5.1.0</version>
+      <version>5.1.2</version>
       <scope>compile</scope>
     </dependency>
     <dependency>
       <groupId>com.lishid</groupId>
       <artifactId>openinvadapter1_20_R3</artifactId>
-      <version>5.1.0</version>
+      <version>5.1.2</version>
       <scope>compile</scope>
     </dependency>
     <dependency>

plugin/src/main/java/com/lishid/openinv/util/InternalAccessor.java

@@ -40,7 +40,8 @@ public class InternalAccessor {
     public InternalAccessor(@NotNull Logger logger, @NotNull LanguageManager lang) {
 
         try {
-            if (BukkitVersions.MINECRAFT.equals(Version.of(1, 21))) {
+            if (BukkitVersions.MINECRAFT.equals(Version.of(1, 21, 1))
+                    || BukkitVersions.MINECRAFT.equals(Version.of(1, 21))) {
                 internal = new com.lishid.openinv.internal.v1_21_R1.InternalAccessor(logger, lang);
             } else if (BukkitVersions.MINECRAFT.equals(Version.of(1, 20, 4))) {
                 internal = new com.lishid.openinv.internal.v1_20_R3.InternalAccessor(logger, lang);

plugin/src/main/java/com/lishid/openinv/util/InventoryManager.java

@@ -111,13 +111,19 @@ public class InventoryManager implements Listener {
   @Keep
   @EventHandler(priority = EventPriority.LOWEST)
   private void onPlayerJoin(@NotNull PlayerJoinEvent event) {
-    consumeLoaded(event.getPlayer().getUniqueId(), inventory -> checkViewerAccess(inventory, true));
+    consumeLoaded(event.getPlayer().getUniqueId(), inventory -> {
+      inventory.setPlayerOnline(event.getPlayer());
+      checkViewerAccess(inventory, true);
+    });
   }
 
   @Keep
   @EventHandler(priority = EventPriority.MONITOR)
   private void onPlayerQuit(@NotNull PlayerQuitEvent event) {
-    consumeLoaded(event.getPlayer().getUniqueId(), inventory -> checkViewerAccess(inventory, false));
+    consumeLoaded(event.getPlayer().getUniqueId(), inventory -> {
+      inventory.setPlayerOffline();
+      checkViewerAccess(inventory, false);
+    });
   }
 
   @Keep

pom.xml

@@ -24,7 +24,7 @@
   <modules>
     <module>common</module>
   </modules>
-  <version>5.1.0</version>
+  <version>5.1.2</version>
 
   <packaging>pom</packaging>
 
@@ -117,7 +117,7 @@
             -->
           <artifactId>maven-dependency-plugin</artifactId>
           <groupId>org.apache.maven.plugins</groupId>
-          <version>3.7.1</version>
+          <version>3.8.0</version>
         </plugin>
 
         <plugin>
@@ -152,7 +152,7 @@
               <path>
                 <groupId>com.google.errorprone</groupId>
                 <artifactId>error_prone_core</artifactId>
-                <version>2.29.2</version>
+                <version>2.31.0</version>
               </path>
             </annotationProcessorPaths>
             <compilerArgs>