Laravel 11.15+ and SQLite compatibility fix

- Added PHP 8.4 to the CI
- Bump Laravel to v11.35 in the CI

.github/workflows/tests.yml

@@ -8,13 +8,13 @@ jobs:
     timeout-minutes: 10
     strategy:
       matrix:
-        php: ['8.1', '8.2', '8.3']
-        laravel: ['10.0', '10.48', '11.0', '11.5']
+        php: ['8.1', '8.2', '8.3', '8.4']
+        laravel: ['10.0', '10.48', '11.0', '11.35']
         exclude:
           - php: '8.1'
             laravel: '11.0'
           - php: '8.1'
-            laravel: '11.5'
+            laravel: '11.35'
     name: PHP ${{ matrix.php }} Laravel ${{ matrix.laravel }}
     steps:
       - name: Checkout

Changelog.md

@@ -1,5 +1,11 @@
 # Customer Module Changelog
 
+## Unreleased
+##### 2024-XX-YY
+
+- Fixed an SQLite migration error with Laravel 11.15+
+- Added PHP 8.4 support
+
 ## 3.1.0
 ##### 2024-11-25
 

src/resources/database/migrations/2023_10_18_173544_add_default_address_fields_to_the_customer_table.php

@@ -13,8 +13,10 @@ public function up(): void
             $table->intOrBigIntBasedOnRelated('default_billing_address_id', Schema::connection(null), 'addresses.id')->nullable();
             $table->intOrBigIntBasedOnRelated('default_shipping_address_id', Schema::connection(null), 'addresses.id')->nullable();
 
-            $table->foreign('default_billing_address_id')->references('id')->on('addresses')->nullOnDelete();
-            $table->foreign('default_shipping_address_id')->references('id')->on('addresses')->nullOnDelete();
+            if (!$this->isSqlite()) {
+                $table->foreign('default_billing_address_id')->references('id')->on('addresses')->nullOnDelete();
+                $table->foreign('default_shipping_address_id')->references('id')->on('addresses')->nullOnDelete();
+            }
         });
     }