From ba6f4cb0aaac9bdf04cbe33be314a5e369bad0fd Mon Sep 17 00:00:00 2001
From: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Date: Wed, 27 Aug 2025 11:34:03 -0300
Subject: [PATCH 1/2] feat: implement php-cs

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
---
 .github/workflows/lint-php-cs.yml        | 53 ++++++++++++++++++++++++
 .gitignore                               |  1 +
 .php-cs-fixer.dist.php                   | 18 ++++++++
 composer.json                            |  2 +
 vendor-bin/coding-standard/composer.json | 10 +++++
 5 files changed, 84 insertions(+)
 create mode 100644 .github/workflows/lint-php-cs.yml
 create mode 100644 .php-cs-fixer.dist.php
 create mode 100644 vendor-bin/coding-standard/composer.json

diff --git a/.github/workflows/lint-php-cs.yml b/.github/workflows/lint-php-cs.yml
new file mode 100644
index 0000000..b460796
--- /dev/null
+++ b/.github/workflows/lint-php-cs.yml
@@ -0,0 +1,53 @@
+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:
+  lint:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        php:
+          - "8.1"
+          - "8.2"
+          - "8.3"
+          - "8.4"
+
+    name: php-cs
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+        with:
+          persist-credentials: false
+
+      - name: Set up php${{ matrix.php }}
+        uses: shivammathur/setup-php@ec406be512d7077f68eed36e63f4d91bc006edc4 # v2.35.4
+        with:
+          php-version: ${{ matrix.php }}
+          coverage: none
+          ini-file: development
+
+      - name: Get composer cache directory
+        id: composer-cache
+        run: echo "::set-output name=dir::$(composer config cache-files-dir)"
+
+      - name: Cache composer cache
+        uses: actions/cache@v4
+        with:
+          path: ${{ steps.composer-cache.outputs.dir }}
+          key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.json') }}
+          restore-keys: ${{ runner.os }}-composer-
+
+      - name: Install dependencies
+        run: composer install
+
+      - name: Lint
+        run: composer cs:check || ( echo 'Please run `composer run cs:fix` to format your code' && exit 1 )
diff --git a/.gitignore b/.gitignore
index 0c2a5dd..f764d68 100644
--- a/.gitignore
+++ b/.gitignore
@@ -3,5 +3,6 @@ composer.lock
 /vendor/
 /vendor-bin/**/vendor/
 *.swp
+.php-cs-fixer.cache
 .phpunit.result.cache
 /tests/logs
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
new file mode 100644
index 0000000..4b9869d
--- /dev/null
+++ b/.php-cs-fixer.dist.php
@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+require_once './vendor-bin/coding-standard/vendor/autoload.php';
+
+use PhpCsFixer\Config;
+use PhpCsFixer\Runner\Parallel\ParallelConfigFactory;
+
+$config = new Config();
+$config
+	->setParallelConfig(ParallelConfigFactory::detect())
+	->getFinder()
+	->ignoreVCSIgnored(true)
+	->notPath('vendor')
+	->notPath('vendor-bin')
+	->in(__DIR__);
+return $config;
diff --git a/composer.json b/composer.json
index d043eca..9e8b5e9 100644
--- a/composer.json
+++ b/composer.json
@@ -31,6 +31,8 @@
     "scripts": {
         "bin": "echo 'bin not installed'",
         "lint": "find . -name \\*.php -not -path './vendor/*' -not -path './vendor-bin/*' -print0 | xargs -0 -n1 php -l",
+        "cs:check": "php-cs-fixer fix --dry-run --diff",
+        "cs:fix": "php-cs-fixer fix",
         "post-install-cmd": [
             "@composer bin all install --ansi",
             "composer dump-autoload"
diff --git a/vendor-bin/coding-standard/composer.json b/vendor-bin/coding-standard/composer.json
new file mode 100644
index 0000000..4e36e32
--- /dev/null
+++ b/vendor-bin/coding-standard/composer.json
@@ -0,0 +1,10 @@
+{
+    "require-dev": {
+        "friendsofphp/php-cs-fixer": "^3.86"
+    },
+    "config": {
+        "platform": {
+            "php": "8.1"
+        }
+    }
+}

From 395cc0da12dd6f0b551fccd30768b87dca3a6d23 Mon Sep 17 00:00:00 2001
From: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Date: Wed, 27 Aug 2025 11:34:22 -0300
Subject: [PATCH 2/2] chore: apply php-cs rules

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
---
 src/Command.php               | 1 +
 src/FdfFile.php               | 5 +++--
 src/InfoFields.php            | 1 +
 src/InfoFile.php              | 1 +
 src/Pdf.php                   | 5 +++--
 src/XfdfFile.php              | 9 +++++----
 tests/unit/CommandTest.php    | 9 +++++----
 tests/unit/DataFieldsTest.php | 1 +
 tests/unit/FdfFileTest.php    | 1 +
 tests/unit/InfoFieldsTest.php | 1 +
 tests/unit/InfoFileTest.php   | 1 +
 tests/unit/PdfTest.php        | 5 +++--
 tests/unit/XfdfFileTest.php   | 1 +
 13 files changed, 27 insertions(+), 14 deletions(-)

diff --git a/src/Command.php b/src/Command.php
index 0919e6b..aa9f886 100644
--- a/src/Command.php
+++ b/src/Command.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace mikehaertl\pdftk;
 
 use mikehaertl\shellcommand\Command as BaseCommand;
diff --git a/src/FdfFile.php b/src/FdfFile.php
index 269ec0a..3081bf3 100644
--- a/src/FdfFile.php
+++ b/src/FdfFile.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace mikehaertl\pdftk;
 
 use mikehaertl\tmp\File;
@@ -15,13 +16,13 @@
 class FdfFile extends File
 {
     // FDF file header
-    const FDF_HEADER = <<<FDF
+    private const FDF_HEADER = <<<FDF
 %FDF-1.2
 1 0 obj<</FDF<< /Fields[
 FDF;
 
     // FDF file footer
-    const FDF_FOOTER = <<<FDF
+    private const FDF_FOOTER = <<<FDF
 ] >> >>
 endobj
 trailer
diff --git a/src/InfoFields.php b/src/InfoFields.php
index 09f4655..9599e14 100644
--- a/src/InfoFields.php
+++ b/src/InfoFields.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace mikehaertl\pdftk;
 
 use ArrayObject;
diff --git a/src/InfoFile.php b/src/InfoFile.php
index 0325dfa..7315a6f 100644
--- a/src/InfoFile.php
+++ b/src/InfoFile.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace mikehaertl\pdftk;
 
 use Exception;
diff --git a/src/Pdf.php b/src/Pdf.php
index f50ed8a..f7c02f0 100644
--- a/src/Pdf.php
+++ b/src/Pdf.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace mikehaertl\pdftk;
 
 use mikehaertl\tmp\File;
@@ -18,7 +19,7 @@
 class Pdf
 {
     // The prefix for temporary files
-    const TMP_PREFIX = 'tmp_php_pdftk_';
+    private const TMP_PREFIX = 'tmp_php_pdftk_';
 
     /**
      * @var bool whether to ignore any errors if some non-empty output file was
@@ -669,7 +670,7 @@ public function toString()
     public function getCommand()
     {
         if ($this->_command === null) {
-            $this->_command = new Command;
+            $this->_command = new Command();
         }
         return $this->_command;
     }
diff --git a/src/XfdfFile.php b/src/XfdfFile.php
index c1be2d9..ff75a16 100644
--- a/src/XfdfFile.php
+++ b/src/XfdfFile.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace mikehaertl\pdftk;
 
 use mikehaertl\tmp\File;
@@ -52,7 +53,7 @@
 class XfdfFile extends File
 {
     // XFDF file header
-    const XFDF_HEADER = <<<FDF
+    private const XFDF_HEADER = <<<FDF
 <?xml version="1.0" encoding="UTF-8"?>
 <xfdf xmlns="http://ns.adobe.com/xfdf/" xml:space="preserve">
 <fields>
@@ -60,7 +61,7 @@ class XfdfFile extends File
 FDF;
 
     // XFDF file footer
-    const XFDF_FOOTER = <<<FDF
+    private const XFDF_FOOTER = <<<FDF
 </fields>
 </xfdf>
 
@@ -198,14 +199,14 @@ protected function writeXml($fields)
     protected function writeFields($fp, $fields)
     {
         foreach ($fields as $key => $value) {
-            $key = $this->xmlEncode(substr($key,1));
+            $key = $this->xmlEncode(substr($key, 1));
             fwrite($fp, "<field name=\"$key\">\n");
             if (!is_array($value)) {
                 $value = array($value);
             }
             if (array_key_exists(0, $value)) {
                 // Numeric keys: single or multi-value field
-                foreach($value as $val) {
+                foreach ($value as $val) {
                     $val = $this->xmlEncode($val);
                     fwrite($fp, "<value>$val</value>\n");
                 }
diff --git a/tests/unit/CommandTest.php b/tests/unit/CommandTest.php
index c87069b..3a9fa62 100644
--- a/tests/unit/CommandTest.php
+++ b/tests/unit/CommandTest.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace tests;
 
 use PHPUnit\Framework\TestCase;
@@ -21,7 +22,7 @@ public function testCanAddFiles()
         $document2 = $this->getDocument2();
         $file = $this->getOutFile();
 
-        $command = new Command;
+        $command = new Command();
         $this->assertEquals(0, $command->getFileCount());
         $this->assertInstanceOf('mikehaertl\pdftk\Command', $command->addFile($document1, 'A'));
         $this->assertInstanceOf('mikehaertl\pdftk\Command', $command->addFile($document2, 'B', 'complex\'"password'));
@@ -37,7 +38,7 @@ public function testCanAddOptions()
         $document1 = $this->getDocument1();
         $file = $this->getOutFile();
 
-        $command = new Command;
+        $command = new Command();
         $this->assertInstanceOf('mikehaertl\pdftk\Command', $command->addFile($document1, 'A'));
         $this->assertInstanceOf('mikehaertl\pdftk\Command', $command->addOption('encrypt_40bit'));
         $this->assertInstanceOf('mikehaertl\pdftk\Command', $command->addOption('allow', 'Printing', false));
@@ -53,7 +54,7 @@ public function testCanSetAndGetOperationAndArgument()
         $document1 = $this->getDocument1();
         $file = $this->getOutFile();
 
-        $command = new Command;
+        $command = new Command();
         $this->assertInstanceOf('mikehaertl\pdftk\Command', $command->addFile($document1, 'A'));
         $this->assertInstanceOf('mikehaertl\pdftk\Command', $command->setOperation('cat'));
         $this->assertInstanceOf('mikehaertl\pdftk\Command', $command->setOperationArgument('A'));
@@ -70,7 +71,7 @@ public function testCanAddPageRanges()
         $document1 = $this->getDocument1();
         $file = $this->getOutFile();
 
-        $command = new Command;
+        $command = new Command();
         $this->assertInstanceOf('mikehaertl\pdftk\Command', $command->addFile($document1, 'A'));
         $this->assertInstanceOf('mikehaertl\pdftk\Command', $command->setOperation('cat'));
         $this->assertInstanceOf('mikehaertl\pdftk\Command', $command->addPageRange(1));
diff --git a/tests/unit/DataFieldsTest.php b/tests/unit/DataFieldsTest.php
index 727fed6..0333087 100644
--- a/tests/unit/DataFieldsTest.php
+++ b/tests/unit/DataFieldsTest.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace tests;
 
 use PHPUnit\Framework\TestCase;
diff --git a/tests/unit/FdfFileTest.php b/tests/unit/FdfFileTest.php
index 8c46af0..8171ded 100644
--- a/tests/unit/FdfFileTest.php
+++ b/tests/unit/FdfFileTest.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace tests;
 
 use PHPUnit\Framework\TestCase;
diff --git a/tests/unit/InfoFieldsTest.php b/tests/unit/InfoFieldsTest.php
index 16d3fc7..b985054 100644
--- a/tests/unit/InfoFieldsTest.php
+++ b/tests/unit/InfoFieldsTest.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace tests;
 
 use PHPUnit\Framework\TestCase;
diff --git a/tests/unit/InfoFileTest.php b/tests/unit/InfoFileTest.php
index c607d21..edc2529 100644
--- a/tests/unit/InfoFileTest.php
+++ b/tests/unit/InfoFileTest.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace tests;
 
 use PHPUnit\Framework\TestCase;
diff --git a/tests/unit/PdfTest.php b/tests/unit/PdfTest.php
index a99d6ee..7af8177 100644
--- a/tests/unit/PdfTest.php
+++ b/tests/unit/PdfTest.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace tests;
 
 use PHPUnit\Framework\TestCase;
@@ -118,7 +119,7 @@ public function testCanAddFiles()
         $document2 = $this->getDocument2();
         $file = $this->getOutFile();
 
-        $pdf = new Pdf;
+        $pdf = new Pdf();
         $this->assertInstanceOf('mikehaertl\pdftk\Pdf', $pdf->addFile($document1, null, 'complex\'"password'));
         $this->assertInstanceOf('mikehaertl\pdftk\Pdf', $pdf->addFile($document2, 'D'));
         $this->assertTrue($pdf->saveAs($file));
@@ -136,7 +137,7 @@ public function testCanPerformEmptyOperation()
         $document1 = $this->getDocument1();
         $file = $this->getOutFile();
 
-        $pdf = new Pdf;
+        $pdf = new Pdf();
         $this->assertInstanceOf('mikehaertl\pdftk\Pdf', $pdf->addFile($document1, null, 'complex\'"password'));
         $this->assertTrue($pdf->saveAs($file));
         $this->assertFileExists($file);
diff --git a/tests/unit/XfdfFileTest.php b/tests/unit/XfdfFileTest.php
index e899e52..430bc50 100644
--- a/tests/unit/XfdfFileTest.php
+++ b/tests/unit/XfdfFileTest.php
@@ -1,4 +1,5 @@
 <?php
+
 namespace tests;
 
 use PHPUnit\Framework\TestCase;