Merge pull request #107 from DazedNConfused-/develop

Author: DazedNConfused-

.github/workflows/attachbinarytorelease.yml renamed to .github/workflows/attach_binary_on_manual_release.yml

@@ -1,4 +1,4 @@
-name: Attach binary to release
+name: Attach binary on manual release
 
 on:
   release:

.github/workflows/build.yml

@@ -1,4 +1,4 @@
-name: Java build & test
+name: Build & Test
 
 on: [push]
 

.github/workflows/pre_release_on_develop_merge.yml

@@ -0,0 +1,62 @@
+name: Pre-release on develop merge
+
+on:
+  workflow_run:
+    workflows: ["Build & Test"]
+    types:
+      - completed
+
+jobs:
+  build-and-release:
+    if: ${{ github.event.workflow_run.conclusion == 'success' && github.event.workflow_run.head_branch == 'develop' }}
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+
+      - name: Set up JDK 18
+        uses: actions/setup-java@v3
+        with:
+          java-version: '18'
+          distribution: 'adopt'
+          cache: maven
+
+      - name: Cache local Maven repository
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+          restore-keys: |
+            ${{ runner.os }}-maven-
+
+      - name: Build with Maven
+        run: mvn package
+
+      - name: Extract Version from POM
+        id: extract_version
+        run: echo "version=$(mvn help:evaluate -Dexpression=project.version -q -DforceStdout)" >> $GITHUB_OUTPUT
+
+      - name: Generate Timestamp
+        id: gen_timestamp
+        run: echo "timestamp=$(date +'%Y%m%d_%H%M%S')" >> $GITHUB_OUTPUT
+
+      - name: Create Pre-release
+        id: create_release
+        uses: actions/create-release@v1
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        with:
+          tag_name: prerelease-${{ steps.extract_version.outputs.version }}-${{ steps.gen_timestamp.outputs.timestamp }}
+          release_name: Pre-release ${{ steps.extract_version.outputs.version }} (${{ steps.gen_timestamp.outputs.timestamp }})
+          draft: false
+          prerelease: true
+
+      - name: Upload Release Asset
+        uses: actions/upload-release-asset@v1
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        with:
+          upload_url: ${{ steps.create_release.outputs.upload_url }}
+          asset_path: ./target/macatalauncher-${{ steps.extract_version.outputs.version }}.jar
+          asset_name: macatalauncher-${{ steps.extract_version.outputs.version }}-${{ steps.gen_timestamp.outputs.timestamp }}.jar
+          asset_content_type: application/java-archive

.github/workflows/sonarcloud.yml

@@ -39,4 +39,8 @@ jobs:
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}  # Needed to get PR information, if any
           SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
-        run: mvn -B verify org.sonarsource.scanner.maven:sonar-maven-plugin:sonar -Dsonar.projectKey=dazednconfused_macata-launcher
+        run: |
+          mvn -B verify sonar:sonar \
+            -Dsonar.projectKey=dazednconfused_macata-launcher \
+            -Dsonar.organization=dazednconfused \
+            -Dsonar.token=$SONAR_TOKEN

.java-version

@@ -1 +1 @@
-11.0.24
+19.0.2

README.md

@@ -35,7 +35,7 @@
 
 ## Requirements
 
-MaCata requires Java 11 to run.
+MaCata requires Java 19 to run.
 
 ## Installation
 
@@ -66,7 +66,8 @@ While not mandatory, if commits are not properly formatted, they may get rejecte
 - [x] Soundpack management
 - [x] Launcher autoupdater
 - [x] Mod management
-- [x] Bright Nights support
+- [x] [Bright Nights](https://github.com/cataclysmbnteam/Cataclysm-BN) support
+- [x] [The Last Generation](https://github.com/Cataclysm-TLG/Cataclysm-TLG) support
 
 ### Planned
 

checkstyle.xml

@@ -219,4 +219,10 @@
         </module>
         <module name="UnusedImports"/>
     </module>
+
+    <module name="RegexpMultiline">
+        <property name="format" value="(\n\s*){3,}"/>
+        <property name="message" value="Too many consecutive blank lines."/>
+        <property name="severity" value="error"/>
+    </module>
 </module>

pom.xml

@@ -6,16 +6,17 @@
 
     <groupId>com.dazednconfused</groupId>
     <artifactId>macatalauncher</artifactId>
-    <version>0.3.3</version>
+    <version>0.4.0</version>
 
     <properties>
         <maven.compiler.source>11</maven.compiler.source>
         <maven.compiler.target>11</maven.compiler.target>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
 
-        <sonar.organization>dazednconfused</sonar.organization>
+        <sonar.organization>dazednconfused</sonar.organization> <!-- should match .github/workflows/sonarcloud.yml -->
+        <sonar.projectKey>dazednconfused_macata-launcher</sonar.projectKey> <!-- ^ditto -->
         <sonar.host.url>https://sonarcloud.io</sonar.host.url>
-        <sonar.cpd.exclusions>**/MainWindow.java</sonar.cpd.exclusions>
+        <sonar.cpd.exclusions>**/gui/**</sonar.cpd.exclusions>
     </properties>
 
     <profiles>
@@ -102,16 +103,11 @@
                     </plugin>
 
                     <plugin>
-                        <groupId>org.apache.maven.plugins</groupId>
-                        <artifactId>maven-jar-plugin</artifactId>
-                        <version>3.3.0</version>
+                        <groupId>org.openjfx</groupId>
+                        <artifactId>javafx-maven-plugin</artifactId>
+                        <version>0.0.8</version>
                         <configuration>
-                            <archive>
-                                <manifest>
-                                    <addClasspath>true</addClasspath>
-                                    <mainClass>com.dazednconfused.catalauncher.Application</mainClass>
-                                </manifest>
-                            </archive>
+                            <mainClass>com.dazednconfused.catalauncher.Application</mainClass>
                         </configuration>
                     </plugin>
 
@@ -248,6 +244,24 @@
             <version>3.0</version>
         </dependency>
 
+        <dependency>
+            <groupId>li.flor</groupId>
+            <artifactId>native-j-file-chooser</artifactId>
+            <version>1.6.4</version>
+        </dependency>
+
+        <dependency>
+            <groupId>org.openjfx</groupId>
+            <artifactId>javafx-controls</artifactId>
+            <version>19.0.2.1</version>
+        </dependency>
+
+        <dependency>
+            <groupId>org.openjfx</groupId>
+            <artifactId>javafx-swing</artifactId>
+            <version>19.0.2.1</version>
+        </dependency>
+
         <dependency>
             <groupId>org.apache.xmlgraphics</groupId>
             <artifactId>batik-all</artifactId>

src/main/java/com/dazednconfused/catalauncher/database/base/BaseDAO.java

@@ -7,6 +7,7 @@
 import java.sql.Statement;
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collection;
 import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
@@ -27,6 +28,11 @@ public interface BaseDAO<T extends BaseEntity> {
      * */
     T insert(T t) throws DAOException;
 
+    /**
+     * Inserts the given {@link BaseEntity}(ies) into the table.
+     * */
+    int bulkInsert(Collection<T> t) throws DAOException;
+
     /**
      * Updates the given {@link BaseEntity}. It must have an ID set.
      * */

src/main/java/com/dazednconfused/catalauncher/database/h2/H2Database.java

@@ -104,7 +104,6 @@ protected static Result<Throwable, Connection> openConnection(String database) {
         ).map(Result::success).recover(Result::failure).get();
     }
 
-
     /**
      * Completely wipes this database of any and all data, <b>including</b> the underlying schema.
      *