wip

Author: enricodelazzari

.github/workflows/fix-php-code-style-issues.yml

@@ -0,0 +1,28 @@
+name: Fix PHP code style issues
+
+on:
+  push:
+    paths:
+      - '**.php'
+
+permissions:
+  contents: write
+
+jobs:
+  php-code-styling:
+    runs-on: ubuntu-latest
+    timeout-minutes: 5
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v5
+        with:
+          ref: ${{ github.head_ref }}
+
+      - name: Fix PHP code style issues
+        uses: aglipanci/[email protected]
+
+      - name: Commit changes
+        uses: stefanzweifel/git-auto-commit-action@v6
+        with:
+          commit_message: Fix styling

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

@@ -29,12 +29,11 @@
         "illuminate/database": "^11.0|^12.0",
         "illuminate/support": "^11.0|^12.0",
         "illuminate/validation": "^11.0|^12.0",
-        "laravel/framework": "11.*",
-        "orchestra/testbench": "9.*",
         "spatie/laravel-package-tools": "^1.14.1"
     },
     "require-dev": {
-        "friendsofphp/php-cs-fixer": "^3.4",
+        "laravel/pint": "^1.25",
+        "orchestra/testbench": "^9.0|^10.0",
         "pestphp/pest": "^3.8",
         "pestphp/pest-plugin-laravel": "^3.2",
         "phpunit/phpunit": "^10.5|^11.0",
@@ -51,7 +50,7 @@
         }
     },
     "scripts": {
-        "format": "vendor/bin/php-cs-fixer fix --allow-risky=yes",
+        "format": "vendor/bin/pint",
         "psalm": "vendor/bin/psalm",
         "test": "vendor/bin/pest",
         "test-coverage": "vendor/bin/pest --coverage"

.php-cs-fixer.dist.php

@@ -18,7 +18,7 @@ protected function getEncryptionKey(): string
         $key = config('encryptable.key');
 
         if (empty($key)) {
-            throw new MissingEncryptionKeyException();
+            throw new MissingEncryptionKeyException;
         }
 
         return (string) $key;
@@ -29,7 +29,7 @@ protected function getEncryptionCipher(): string
         $cipher = config('encryptable.cipher');
 
         if (empty($cipher)) {
-            throw new MissingEncryptionCipherException();
+            throw new MissingEncryptionCipherException;
         }
 
         return (string) $cipher;

composer.json

@@ -19,7 +19,7 @@ public static function serialize($value): string
         $valueType = gettype($value);
 
         if (! in_array($valueType, self::SUPPORTED_TYPES)) {
-            throw new SerializationException();
+            throw new SerializationException;
         }
 
         $value = strval($value);
@@ -32,13 +32,13 @@ public static function unserialize(string $payload)
         $payload = explode(':', $payload, 2);
 
         if (count($payload) !== 2) {
-            throw new UnserializationException();
+            throw new UnserializationException;
         }
 
         [$valueType, $value] = $payload;
 
         if (! settype($value, $valueType)) {
-            throw new UnserializationException();
+            throw new UnserializationException;
         }
 
         return $value;

src/Encrypter.php

@@ -3,7 +3,6 @@
 use Illuminate\Contracts\Encryption\EncryptException;
 use Maize\Encryptable\Encryption;
 
-
 it('should throw exception on db encrypt', function () {
     $this->expectException(EncryptException::class);
 

src/Utils/Serializer.php

@@ -4,7 +4,6 @@
 use Maize\Encryptable\Encryption;
 use Maize\Encryptable\Tests\Models\User;
 
-
 it('should encrypt data when saving model instance', function () {
     $user = $this->createUser();
 

tests/DBEncrypterTest.php

@@ -4,7 +4,6 @@
 use Maize\Encryptable\Exceptions\MissingEncryptionCipherException;
 use Maize\Encryptable\Exceptions\MissingEncryptionKeyException;
 
-
 it('should throw exception when encryption key is missing', function () {
     config()->set('encryptable.key', null);
 

tests/EncryptableTest.php

@@ -4,7 +4,6 @@
 use Illuminate\Validation\Rule;
 use Maize\Encryptable\Rules\ExistsEncrypted;
 
-
 it('should validate encrypted data with custom exists rule', function () {
     $user = $this->createUser();