Merge pull request #801 from matiasdelellis/nc31

Bump to NC31

Author: matiasdelellis

.github/workflows/coverage.yml

@@ -13,9 +13,9 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php-versions: ['8.2']
+        php-versions: ['8.3']
         databases: ['mysql']
-        server-versions: ['stable29']
+        server-versions: ['stable31']
 
     name: php${{ matrix.php-versions }}-${{ matrix.databases }}-${{ matrix.server-versions }}
 

.github/workflows/lint.yml

@@ -24,7 +24,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        php-versions: ['8.0', '8.1', '8.2', '8.3']
+        php-versions: ['8.1', '8.2', '8.3', '8.4']
     name: php${{ matrix.php-versions }} lint
     steps:
       - name: Checkout

.github/workflows/phpunit.yml

@@ -17,9 +17,9 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php-versions: ['8.1', '8.2', '8.3']
+        php-versions: ['8.1', '8.2', '8.3', '8.4']
         databases: ['sqlite', 'mysql', 'pgsql']
-        server-versions: ['stable29', 'stable30']
+        server-versions: ['stable31']
 
     name: php${{ matrix.php-versions }}-${{ matrix.databases }}-${{ matrix.server-versions }}
 

.github/workflows/static-analysis.yml

@@ -11,7 +11,7 @@ jobs:
       runs-on: ubuntu-latest
       strategy:
           matrix:
-              ocp-version: ['dev-stable29']
+              ocp-version: ['dev-stable31']
       name: Nextcloud ${{ matrix.ocp-version }}
       steps:
           - name: Checkout

CHANGELOG.md

@@ -1,6 +1,11 @@
 # Changelog
 All notable changes to this project will be documented in this file.
 
+## [0.9.70] - 2025-04-22
+- Support Nextcloud 31 and deprectate NC29 and NC30. Issue #797
+- Support PHP 8.4 and deprecate PHP 8.0 in line with NC31. Issue #795
+- Fix clustering on Oracle database. Issue #786
+
 ## [0.9.60] - 2024-11-19
 - Enable NC30, and deprecte NC28. Issue #776
 - Fix review people found does not work on Nextcloud Hub 9 (30.0.0) #779

appinfo/info.xml

@@ -16,7 +16,7 @@
 - **🚀 Build your own thing:** FaceRecognition app is just a basic building block. Through FaceRecognition API, you can build your advanced scenarios - automatically add tags to images, connect contacts and persons, share images from specific person… We want to hear your ideas!
 ]]>
 	</description>
-	<version>0.9.60</version>
+	<version>0.9.70</version>
 	<licence>agpl</licence>
 	<author>Matias De lellis</author>
 	<author>Branko Kokanovic</author>
@@ -33,8 +33,8 @@
 	<screenshot>https://matiasdelellis.github.io/img/facerecognition/facerecognition-photos-integration.jpeg</screenshot>
 	<screenshot>https://matiasdelellis.github.io/img/facerecognition/facerecognition-assign-initial-name.jpeg</screenshot>
 	<dependencies>
-		<php min-version="8.0" max-version="8.3" />
-		<nextcloud min-version="29" max-version="30"/>
+		<php min-version="8.1" max-version="8.4" />
+		<nextcloud min-version="31" max-version="31"/>
 	</dependencies>
 	<repair-steps>
 		<uninstall>

lib/Service/SettingsService.php

@@ -316,6 +316,8 @@ public function getDefaultUserEnabled (): bool {
 	}
 
 	public function getClusterigBatchSize(): int {
+		if ($this->config->getSystemValue('dbtype', 'sqlite') === 'oci')
+			return 1000;
 		return intval($this->config->getAppValue(Application::APP_NAME, self::CLUSTERING_BATCH_SIZE_KEY, self::DEFAULT_CLUSTERING_BATCH_SIZE));
 	}