feat: php8.3

.drone.star

@@ -26,7 +26,7 @@ SONARSOURCE_SONAR_SCANNER_CLI = "sonarsource/sonar-scanner-cli:5"
 TOOLHIPPIE_CALENS = "toolhippie/calens:latest"
 WEBHIPPIE_REDIS = "webhippie/redis:latest"
 
-DEFAULT_PHP_VERSION = "7.4"
+DEFAULT_PHP_VERSION = "8.3"
 DEFAULT_NODEJS_VERSION = "14"
 
 # minio mc environment variables
@@ -85,18 +85,6 @@ config = {
                 "oracle",
             ],
         },
-        "ubuntu22": {
-            "phpVersions": [
-                "7.4-ubuntu22.04",
-            ],
-            # These pipelines are run just to help avoid any obscure regression
-            # on Ubuntu 22.04. We do not need coverage for this.
-            "coverage": False,
-            "databases": [
-                "mariadb:10.6",
-                "mariadb:10.11",
-            ],
-        },
         "external-samba": {
             "phpVersions": [
                 DEFAULT_PHP_VERSION,
@@ -251,10 +239,6 @@ config = {
             "testingRemoteSystem": False,
         },
         "cliEncryption": {
-            "phpVersions": [
-                DEFAULT_PHP_VERSION,
-                "7.4-ubuntu22.04",
-            ],
             "suites": [
                 "cliEncryption",
             ],
@@ -420,18 +404,6 @@ config = {
             "runAllSuites": True,
             "numberOfParts": 8,
         },
-        "apiUbuntu22": {
-            "phpVersions": [
-                "7.4-ubuntu22.04",
-            ],
-            "suites": {
-                "apiUbuntu22": "apiUbuntu22",
-            },
-            "useHttps": False,
-            "filterTags": "@smokeTest&&~@notifications-app-required&&~@local_storage&&~@files_external-app-required",
-            "runAllSuites": True,
-            "numberOfParts": 8,
-        },
         "apiOnSqlite": {
             "suites": {
                 "apiOnSqlite": "apiOnSqlite",
@@ -1153,15 +1125,18 @@ def phpTests(ctx, testType, withCoverage):
             else:
                 command = "unknown tbd"
 
-            # Shorten PHP docker tags that have longer names like 7.4-ubuntu22.04
-            phpVersionString = phpVersion.replace("-ubuntu", "-u")
+            # Get the first 3 characters of the PHP version (7.4 or 8.0 etc)
+            # And use that for constructing the pipeline name
+            # That helps shorten pipeline names when using owncloud-ci images
+            # that have longer names like 7.4-ubuntu20.04
+            phpMinorVersion = phpVersion[0:3]
 
             for db in params["databases"]:
                 for externalType in params["externalTypes"]:
                     keyString = "-" + category if params["includeKeyInMatrixName"] else ""
                     filesExternalType = externalType if externalType != "none" else ""
                     externalNameString = "-" + externalType if externalType != "none" else ""
-                    name = "%s%s-php%s-%s%s" % (testType, keyString, phpVersionString, getShortDbNameAndVersion(db), externalNameString)
+                    name = "%s%s-php%s-%s%s" % (testType, keyString, phpMinorVersion, getShortDbNameAndVersion(db), externalNameString)
                     maxLength = 50
                     nameLength = len(name)
                     if nameLength > maxLength:
@@ -1426,8 +1401,11 @@ def acceptance(ctx):
             for federatedServerVersion in params["federatedServerVersions"]:
                 for browser in params["browsers"]:
                     for phpVersion in params["phpVersions"]:
-                        # Shorten PHP docker tags that have longer names like 7.4-ubuntu22.04
-                        phpVersionString = phpVersion.replace("-ubuntu", "-u")
+                        # Get the first 3 characters of the PHP version (7.4 or 8.0 etc)
+                        # And use that for constructing the pipeline name
+                        # That helps shorten pipeline names when using owncloud-ci images
+                        # that have longer names like 7.4-ubuntu20.04
+                        phpMinorVersion = phpVersion[0:3]
                         for db in params["databases"]:
                             for runPart in range(1, params["numberOfParts"] + 1):
                                 debugPartsEnabled = (len(params["skipExceptParts"]) != 0)
@@ -1449,7 +1427,7 @@ def acceptance(ctx):
                                     keyString = "-" + category if params["includeKeyInMatrixName"] else ""
                                     partString = "" if params["numberOfParts"] == 1 else "-%d-%d" % (params["numberOfParts"], runPart)
                                     federatedServerVersionString = "-" + federatedServerVersion.replace("daily-", "").replace("-qa", "") if (federatedServerVersion != "") else ""
-                                    name = "%s%s%s%s%s-%s-php%s" % (alternateSuiteName, keyString, partString, federatedServerVersionString, browserString, getShortDbNameAndVersion(db), phpVersionString)
+                                    name = "%s%s%s%s%s-%s-php%s" % (alternateSuiteName, keyString, partString, federatedServerVersionString, browserString, getShortDbNameAndVersion(db), phpMinorVersion)
                                     maxLength = 50
                                     nameLength = len(name)
                                     if nameLength > maxLength:

.github/workflows/ci.yml

@@ -26,4 +26,4 @@ jobs:
     name: PHP Unit
     uses: ./.github/workflows/php-unit.yml
     with:
-      php-versions: '["7.4"]'
+      php-versions: '["8.3"]'

.github/workflows/lint-and-codestyle.yml

@@ -24,7 +24,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php-version: ["7.4"]
+        php-version: ["8.3"]
     steps:
       - name: Checkout code
         uses: actions/checkout@v4

.github/workflows/php-unit.yml

@@ -19,9 +19,9 @@ jobs:
         php: ${{ fromJSON(inputs.php-versions) }}
         database: [sqlite]
         include:
-          - php: "7.4"
+          - php: "8.3"
             database: "mysql:8.0"
-          - php: "7.4"
+          - php: "8.3"
             database: "postgres:10.21"
 
     services:

apps/dav/lib/CalDAV/Schedule/IMipPlugin.php

@@ -41,21 +41,12 @@
  * @license http://sabre.io/license/ Modified BSD License
  */
 class IMipPlugin extends SabreIMipPlugin {
-	/** @var IMailer */
-	private $mailer;
-
-	/** @var ILogger */
-	private $logger;
-
-	/** @var IRequest */
-	private $request;
+	private IMailer $mailer;
+	private ILogger $logger;
+	private IRequest $request;
 
 	/**
 	 * Creates the email handler.
-	 *
-	 * @param IMailer $mailer
-	 * @param ILogger $logger
-	 * @param IRequest $request
 	 */
 	public function __construct(IMailer $mailer, ILogger $logger, IRequest $request) {
 		parent::__construct('');
@@ -123,14 +114,10 @@ public function schedule(ITip\Message $iTipMessage) {
 			->setFrom([$sender => $senderName])
 			->setTo([$recipient => $recipientName])
 			->setSubject($subject)
-			->setBody($iTipMessage->message->serialize(), $contentType);
+			->attach($iTipMessage->message->serialize(), "event.ics", $contentType);
 		try {
-			$failed = $this->mailer->send($message);
+			$this->mailer->send($message);
 			$iTipMessage->scheduleStatus = '1.1; Scheduling message is sent via iMip';
-			if ($failed) {
-				$this->logger->error('Unable to deliver message to {failed}', ['app' => 'dav', 'failed' =>  \implode(', ', $failed)]);
-				$iTipMessage->scheduleStatus = '5.0; EMail delivery failed';
-			}
 		} catch (\Exception $ex) {
 			$this->logger->logException($ex, ['app' => 'dav']);
 			$iTipMessage->scheduleStatus = '5.0; EMail delivery failed';

apps/dav/lib/Connector/Sabre/Principal.php

@@ -97,10 +97,10 @@ public function getPrincipalsByPrefix($prefixPath) {
 	 * getPrincipalsByPrefix.
 	 *
 	 * @param string $path
-	 * @return array
+	 * @return array|null
 	 */
-	public function getPrincipalByPath($path) {
-		list($prefix, $name) = \Sabre\Uri\split($path);
+	public function getPrincipalByPath($path): ?array {
+		[$prefix, $name] = \Sabre\Uri\split($path);
 
 		if ($prefix === $this->principalPrefix) {
 			$user = $this->userManager->get($name);
@@ -219,12 +219,12 @@ public function findByUri($uri, $principalPrefix) {
 	 * @param IUser $user
 	 * @return array
 	 */
-	protected function userToPrincipal($user) {
+	protected function userToPrincipal(IUser $user): array {
 		$userId = $user->getUID();
 		$displayName = $user->getDisplayName();
 		$principal = [
 				'uri' => $this->principalPrefix . '/' . $userId,
-				'{DAV:}displayname' => $displayName === null ? $userId : $displayName,
+				'{DAV:}displayname' => $displayName,
 		];
 
 		$email = $user->getEMailAddress();

apps/dav/lib/Server.php

@@ -179,7 +179,6 @@ public function __construct(IRequest $request, $baseUri) {
 			$acl->principalCollectionSet = [
 				'principals/users', 'principals/groups'
 			];
-			$acl->defaultUsernamePath = 'principals/users';
 			$this->server->addPlugin($acl);
 		}
 

apps/dav/lib/Upload/AssemblyStream.php

@@ -248,9 +248,12 @@ public static function wrap(array $nodes) {
 			'assembly' => [
 				'nodes' => $nodes]
 		]);
-		\stream_wrapper_register('assembly', '\OCA\DAV\Upload\AssemblyStream');
+		$existed = \in_array("assembly", stream_get_wrappers());
+		if (!$existed) {
+			\stream_wrapper_register('assembly', __CLASS__);
+		}
 		try {
-			$wrapped = \fopen('assembly://', 'r', null, $context);
+			$wrapped = \fopen('assembly://', 'r', false, $context);
 		} catch (\BadMethodCallException $e) {
 			\stream_wrapper_unregister('assembly');
 			throw $e;

apps/dav/tests/unit/CalDAV/CalendarTest.php

@@ -42,9 +42,9 @@ public function setUp(): void {
 		$this->l10n
 			->expects($this->any())
 			->method('t')
-			->will($this->returnCallback(function ($text, $parameters = []) {
+			->willReturnCallback(function ($text, $parameters = []) {
 				return \vsprintf($text, $parameters);
-			}));
+			});
 	}
 
 	public function testDelete() {