add code linting

.github/workflows/lint-php-cs.yml

@@ -0,0 +1,88 @@
+# This workflow is provided via the organization template repository
+#
+# https://github.com/nextcloud/.github
+# https://docs.github.com/en/actions/learn-github-actions/sharing-workflows-with-your-organization
+#
+# SPDX-FileCopyrightText: 2021-2024 Nextcloud GmbH and Nextcloud contributors
+# SPDX-License-Identifier: MIT
+
+name: Lint php-cs
+
+on:
+    pull_request:
+
+permissions:
+  contents: read
+
+concurrency:
+  group: lint-php-cs-${{ github.head_ref || github.run_id }}
+  cancel-in-progress: true
+
+jobs:
+  changes:
+    runs-on: ubuntu-latest
+    outputs:
+      src: ${{ steps.changes.outputs.src}}
+    steps:
+      - uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3.0.2
+        id: changes
+        continue-on-error: true
+        with:
+          filters: |
+            src:
+              - '.github/workflows/**'
+              - 'configs/**'
+              - '.php-cs-fixer.dist.php'
+              - 'composer.json'
+              - 'composer.lock'
+              - '**.php'
+
+  lint:
+    runs-on: ubuntu-latest
+
+    name: PHP CS fixer lint
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
+
+      - name: Set up php8.1
+        uses: shivammathur/setup-php@c541c155eee45413f5b09a52248675b1a2575231 #v2.31.1
+        with:
+          php-version: 8.1
+          extensions: bz2, ctype, curl, dom, fileinfo, gd, iconv, intl, json, libxml, mbstring, openssl, pcntl, posix, session, simplexml, xmlreader, xmlwriter, zip, zlib, sqlite, pdo_sqlite
+          coverage: none
+          ini-file: development
+
+      - name: Install dependencies
+        run: composer i
+
+      -   name: Install PHP CS fixer
+          run: composer require --dev friendsofphp/php-cs-fixer
+
+      - name: Lint
+        run: composer run cs:check || ( echo 'Please run `composer run cs:fix` to format your code' && exit 1 )
+
+  summary:
+    permissions:
+      contents: none
+    runs-on: ubuntu-latest
+    needs: [changes, lint]
+
+    if: always()
+
+    # This is the summary, we just avoid to rename it so that branch protection rules still match
+    name: PHP CS Summary
+
+    steps:
+      - name: Summary status
+        run: |
+          if [[ "${{ needs.changes.outputs.src }}" == "false" ]]; then
+            echo "No relevant changes detected. Nothing to lint."
+          elif [[ "${{ needs.lint.result }}" != "success" ]]; then
+            echo "Check the logs for details of 'PHP CS fixer lint' job."
+            echo "Linting failed. Please run \`composer run cs:fix\` to fix the issues."
+            exit 1
+          else
+            echo "All checks passed successfully."
+          fi

.github/workflows/lint-php.yml

@@ -0,0 +1,77 @@
+# This workflow is provided via the organization template repository
+#
+# https://github.com/nextcloud/.github
+# https://docs.github.com/en/actions/learn-github-actions/sharing-workflows-with-your-organization
+#
+# SPDX-FileCopyrightText: 2021-2024 Nextcloud GmbH and Nextcloud contributors
+# SPDX-License-Identifier: MIT
+
+name: Lint php
+
+on:
+    pull_request:
+
+permissions:
+  contents: read
+
+concurrency:
+  group: lint-php-${{ github.head_ref || github.run_id }}
+  cancel-in-progress: true
+
+jobs:
+  changes:
+    runs-on: ubuntu-latest
+    outputs:
+      src: ${{ steps.changes.outputs.src}}
+    steps:
+      - uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3.0.2
+        id: changes
+        continue-on-error: true
+        with:
+          filters: |
+            src:
+              - '.github/workflows/**'
+              - 'configs/**'
+              - 'composer.json'
+              - 'composer.lock'
+              - '**.php'
+
+  lint:
+    runs-on: ubuntu-latest
+
+    needs: changes
+    if: needs.changes.outputs.src != 'false'
+
+    strategy:
+      matrix:
+        php-versions: [ '8.1', '8.2', '8.3' ]
+
+    name: php-lint
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
+
+      - name: Set up php ${{ matrix.php-versions }}
+        uses: shivammathur/setup-php@c541c155eee45413f5b09a52248675b1a2575231 #v2.31.1
+        with:
+          php-version: ${{ matrix.php-versions }}
+          coverage: none
+          ini-file: development
+
+      - name: Lint
+        run: composer run lint
+
+  summary:
+    permissions:
+      contents: none
+    runs-on: ubuntu-latest
+    needs: [changes, lint]
+
+    if: always()
+
+    name: php-lint Summary
+
+    steps:
+      - name: Summary status
+        run: if ${{ needs.changes.outputs.src != 'false' && needs.lint.result != 'success' }}; then exit 1; fi

.gitignore

@@ -0,0 +1,14 @@
+### PHP-CS-Fixer template
+# Covers PHP CS Fixer
+# Reference: https://cs.symfony.com/
+
+# Generated files
+.php-cs-fixer.cache
+
+# Local config See: https://cs.symfony.com/doc/config.html
+.php-cs-fixer.php
+
+# Composer related files
+composer.phar
+composer.lock
+/vendor/

.php-cs-fixer.dist.php

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+require_once './vendor/autoload.php';
+
+use PhpCsFixer\Runner\Parallel\ParallelConfigFactory;
+use Nextcloud\CodingStandard\Config;
+
+$config = new Config();
+$config
+	->setParallelConfig(ParallelConfigFactory::detect())
+	->getFinder()
+	->ignoreVCSIgnored(true)
+	->notPath('composer')
+	->notPath('node_modules')
+	->notPath('vendor')
+	->in('configs')
+	->in(__DIR__);
+
+return $config;

composer.json

@@ -0,0 +1,13 @@
+{
+  "name": "ionos-productivity/nc-config",
+  "description": "HiDrive Next config partials and configuration scripts",
+  "scripts": {
+    "cs:fix": "./vendor/bin/php-cs-fixer fix",
+    "cs:check": "./vendor/bin/php-cs-fixer fix --dry-run --diff",
+    "lint": "find . -name \\*.php -print0 | xargs -0 -n1 php -l"
+  },
+  "require-dev": {
+    "friendsofphp/php-cs-fixer": "^3.75",
+    "nextcloud/coding-standard": "^1.3"
+  }
+}

configs/admin_audit.config.php

@@ -4,7 +4,7 @@
 	'log.condition' => [
 		'apps' => ['admin_audit'],
 	],
-	"log_type_audit" => "errorlog",
-	"syslog_tag_audit" => "nextcloud",
-	"logfile_audit" => "",
+	'log_type_audit' => 'errorlog',
+	'syslog_tag_audit' => 'nextcloud',
+	'logfile_audit' => '',
 ];