Merge pull request #290 from zuzana-kunckova/laravel-upgrade

Laravel version 11 upgrade

Author: JonoB

.github/workflows/ci.yml

@@ -0,0 +1,55 @@
+on: push
+name: Laravel CI
+jobs:
+  phpunit:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        container: [
+          "kirschbaumdevelopment/laravel-test-runner:8.2",
+          "kirschbaumdevelopment/laravel-test-runner:8.3"
+        ]
+
+    container:
+      image: ${{ matrix.container }}
+
+    name: ${{ matrix.container }}
+
+    services:
+      mysql:
+        image: mysql:5.7
+        env:
+          MYSQL_ROOT_PASSWORD: password
+          MYSQL_USER: homestead
+          MYSQL_PASSWORD: secret
+          MYSQL_DATABASE: sendportal_dev
+        ports:
+          - 3306:3306
+        options: --health-cmd="mysqladmin ping" --health-interval=10s --health-timeout=5s --health-retries=3
+
+      postgres:
+        image: postgres
+        env:
+          POSTGRES_USER: homestead
+          POSTGRES_PASSWORD: secret
+          POSTGRES_DB: sendportal_dev
+        ports:
+          - 5432:5432
+        options: --health-cmd="pg_isready" --health-interval=10s --health-timeout=5s --health-retries=3
+
+    steps:
+      - uses: actions/checkout@v3
+      - name: Install composer dependencies
+        run: composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
+      - name: Run Testsuite against MySQL
+        run: vendor/bin/phpunit
+        env:
+          DB_CONNECTION: mysql
+          DB_HOST: mysql
+          DB_PORT: 3306
+      - name: Run Testsuite against Postgres
+        run: vendor/bin/phpunit
+        env:
+          DB_CONNECTION: pgsql
+          DB_HOST: postgres
+          DB_PORT: 5432

.github/workflows/format.yml

@@ -0,0 +1,29 @@
+name: Format [PHP]
+
+on:
+  pull_request:
+    paths:
+      - '**.php'
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+  cancel-in-progress: true
+
+jobs:
+  php-cs-fixer:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          ref: ${{ github.event.pull_request.head.ref }}
+
+      - name: Run php-cs-fixer
+        uses: docker://oskarstark/php-cs-fixer-ga:3.13.0
+        with:
+          args: --config=.php-cs-fixer.dist.php
+
+      - uses: stefanzweifel/git-auto-commit-action@v4
+        with:
+          commit_message: Apply php-cs-fixer changes
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Sendportal
 
+## 3.0.0 - 2024-04-19
+- Update to Laravel 11 ([#290](https://github.com/mettle/sendportal/pull/290)
+
 ## 2.0.3 - 2021-07-05
 - Allow overwrite phpunit.xml only locally ([#148](https://github.com/mettle/sendportal/pull/148))
 

README.md

@@ -19,12 +19,16 @@ The [SendPortal](https://github.com/mettle/sendportal) application acts as a wra
 
 ## Installation
 
-As of Version 2, SendPortal can be installed as a stand-alone application (i.e. including everything you need to run SendPortal), or as a package inside an existing Laravel application.
-
 If you would like to install SendPortal as a stand-alone application, please follow the [installation guide](https://sendportal.io/docs/v2/getting-started/installation).
 
 If you would like to add SendPortal to an existing application, please follow the [package installation guide](https://sendportal.io/docs/v2/getting-started/package-installation).
 
-## Upgrading
+## Requirements
+SendPortal V3 requires:
+
+- PHP 8.2+
+- Laravel 10+
+- MySQL (≥ 5.7) or PostgreSQL (≥ 9.4)
+
+If you are on an earlier version of PHP (7.3+) or Laravel (8+), please use [SendPortal V2](https://github.com/mettle/sendportal/releases/tag/v2.0.4)
 
-If you already have a copy of SendPortal v1 installed, please follow the [upgrade guide](https://sendportal.io/docs/v2/getting-started/upgrade-guide).

app/Console/Commands/InstallApplication.php

@@ -90,7 +90,7 @@ protected function checkEnvironment(): void
      */
     protected function checkApplicationKey(): void
     {
-        if ( ! config('app.key')) {
+        if (! config('app.key')) {
             $this->call('key:generate');
         }
 
@@ -103,7 +103,7 @@ protected function checkApplicationKey(): void
     protected function checkAppUrl(): void
     {
         if (config('app.url') !== 'http://localhost') {
-            $this->info('✓ Application url set to ' . config('app.url'));
+            $this->info('✓ Application url set to '.config('app.url'));
 
             return;
         }
@@ -121,7 +121,7 @@ protected function checkDatabaseConnection(): void
             $this->info('✓ Database connection successful');
         } catch (Exception $e) {
             try {
-                if ( ! $this->createDatabaseCredentials()) {
+                if (! $this->createDatabaseCredentials()) {
                     $this->error(
                         'A database connection could not be established. Please update your configuration and try again.'
                     );
@@ -144,7 +144,7 @@ protected function createDatabaseCredentials(): bool
             true
         );
 
-        if ( ! $storeCredentials) {
+        if (! $storeCredentials) {
             return false;
         }
 
@@ -211,7 +211,7 @@ protected function getCompanyName(): string
         $this->line('Creating first admin user account and company/workspace');
         $companyName = $this->ask('Company/Workspace name');
 
-        if ( ! $companyName) {
+        if (! $companyName) {
             return $this->getCompanyName();
         }
 
@@ -271,7 +271,7 @@ protected function getUserParam(string $param): string
 
         if ($validator->fails()) {
             foreach ($validator->errors()->getMessages() as $error) {
-                $this->line((string)($error[0]));
+                $this->line((string) ($error[0]));
             }
 
             return $this->getUserParam($param);
@@ -312,7 +312,7 @@ protected function checkVendorAssets(): void
             [
                 '--provider' => SendportalBaseServiceProvider::class,
                 '--tag' => 'sendportal-assets',
-                '--force' => true
+                '--force' => true,
             ]
         );
 
@@ -329,11 +329,11 @@ protected function printDatabaseConfig(): void
         $this->line('');
         $this->info('Database Configuration:');
         $this->line("- Connection: {$connection}");
-        $this->line('- Host: ' . config("database.connections.{$connection}.host"));
-        $this->line('- Port: ' . config("database.connections.{$connection}.port"));
-        $this->line('- Database: ' . config("database.connections.{$connection}.database"));
-        $this->line('- Username: ' . config("database.connections.{$connection}.username"));
-        $this->line('- Password: ' . config("database.connections.{$connection}.password"));
+        $this->line('- Host: '.config("database.connections.{$connection}.host"));
+        $this->line('- Port: '.config("database.connections.{$connection}.port"));
+        $this->line('- Database: '.config("database.connections.{$connection}.database"));
+        $this->line('- Username: '.config("database.connections.{$connection}.username"));
+        $this->line('- Password: '.config("database.connections.{$connection}.password"));
     }
 
     /**
@@ -344,7 +344,7 @@ protected function persistVariables(array $connectionData): void
         $connection = $connectionData['DB_CONNECTION'];
 
         $configMap = [
-            'DB_CONNECTION' => "database.default",
+            'DB_CONNECTION' => 'database.default',
             'DB_HOST' => "database.connections.{$connection}.host",
             'DB_PORT' => "database.connections.{$connection}.port",
             'DB_DATABASE' => "database.connections.{$connection}.database",
@@ -374,7 +374,7 @@ protected function writeToEnvironmentFile(string $key, ?string $value): void
             )
         );
 
-        if ( ! $this->checkEnvValuePresent($key, $value)) {
+        if (! $this->checkEnvValuePresent($key, $value)) {
             throw new RuntimeException("Failed to persist environment variable value. {$key}={$value}");
         }
     }

app/Console/Commands/UpgradeApplication.php

@@ -34,8 +34,6 @@ class UpgradeApplication extends BaseCommand
 
     /**
      * Execute the console command.
-     *
-     * @return int
      */
     public function handle(): int
     {
@@ -62,7 +60,7 @@ protected function checkVendorAssets(): void
             'vendor:publish',
             [
                 '--provider' => SendportalBaseServiceProvider::class,
-                '--force' => true
+                '--force' => true,
             ]
         );
 

app/Console/Kernel.php

@@ -19,7 +19,6 @@ class Kernel extends ConsoleKernel
     /**
      * Define the application's command schedule.
      *
-     * @param  \Illuminate\Console\Scheduling\Schedule  $schedule
      * @return void
      */
     protected function schedule(Schedule $schedule)

app/Exceptions/Handler.php

@@ -29,7 +29,6 @@ class Handler extends ExceptionHandler
     /**
      * Report or log an exception.
      *
-     * @param  \Throwable  $exception
      * @return void
      *
      * @throws \Exception
@@ -43,7 +42,6 @@ public function report(Throwable $exception)
      * Render an exception into an HTTP response.
      *
      * @param  \Illuminate\Http\Request  $request
-     * @param  \Throwable  $exception
      * @return \Symfony\Component\HttpFoundation\Response
      *
      * @throws \Throwable

app/Http/Controllers/Auth/ChangePasswordController.php

@@ -20,7 +20,7 @@ public function edit(): View
     public function update(ChangePasswordRequest $request): RedirectResponse
     {
         $request->user()->update([
-            'password'=> Hash::make($request->password)
+            'password' => Hash::make($request->password),
         ]);
 
         return redirect()->route('profile.show')

app/Http/Controllers/Auth/ConfirmPasswordController.php

@@ -4,8 +4,8 @@
 
 namespace App\Http\Controllers\Auth;
 
-use Illuminate\Foundation\Auth\ConfirmsPasswords;
 use App\Http\Controllers\Controller;
+use Illuminate\Foundation\Auth\ConfirmsPasswords;
 
 class ConfirmPasswordController extends Controller
 {