From a79e21b0d15cdf6fe271b9e7a10f6b74f0548921 Mon Sep 17 00:00:00 2001
From: Zach Borboa <zachborboa@gmail.com>
Date: Mon, 15 Sep 2025 21:24:22 -0400
Subject: [PATCH] Fix PHP_CodeSniffer coding standard errors (#993)

* Fix PHP_CodeSniffer errors

Error message:
Header blocks must be separated by a single blank line (PSR12.Files.FileHeader.SpacingAfterTagBlock)

* Fix PHP_CodeSniffer errors

Error message:
Opening brace should be on a new line (Squiz.Functions.MultiLineFunctionDeclaration.BraceOnSameLine)

* Fix PHP_CodeSniffer warnings

Error message:
A file should declare new symbols (classes, functions, constants, etc.)
and cause no other side effects, or it should execute logic with side
effects, but should not do both. The first symbol is defined on line 6
and the first side effect is on line 14.
(PSR1.Files.SideEffects.FoundWithSymbols)
---
 examples/post_multiple_values_same_key_without_indexes.php | 1 +
 examples/put_large_file_chunked.php                        | 1 +
 examples/receive_large_file_chunked.php                    | 1 +
 tests/ruleset.xml                                          | 7 +++++++
 4 files changed, 10 insertions(+)

diff --git a/examples/post_multiple_values_same_key_without_indexes.php b/examples/post_multiple_values_same_key_without_indexes.php
index 39950cd..5e45ab5 100644
--- a/examples/post_multiple_values_same_key_without_indexes.php
+++ b/examples/post_multiple_values_same_key_without_indexes.php
@@ -1,4 +1,5 @@
 <?php
+
 // keywords:
 //   Django: MultiValueDict, QueryDict, request.GET.getlist(), request.POST.getlist()
 //   Python: urllib.urlencode, parse.urlencode
diff --git a/examples/put_large_file_chunked.php b/examples/put_large_file_chunked.php
index 6368e90..020b818 100644
--- a/examples/put_large_file_chunked.php
+++ b/examples/put_large_file_chunked.php
@@ -1,4 +1,5 @@
 <?php
+
 // PUT a file using chunked data.
 // See also "examples/receive_large_file_chunked.php".
 
diff --git a/examples/receive_large_file_chunked.php b/examples/receive_large_file_chunked.php
index d5fff18..df1c045 100644
--- a/examples/receive_large_file_chunked.php
+++ b/examples/receive_large_file_chunked.php
@@ -1,4 +1,5 @@
 <?php
+
 // Receive PUT file.
 // See also "examples/put_large_file_chunked.php".
 
diff --git a/tests/ruleset.xml b/tests/ruleset.xml
index a52783b..10ddbed 100644
--- a/tests/ruleset.xml
+++ b/tests/ruleset.xml
@@ -13,6 +13,13 @@
 
         <!-- TODO: Remove the following exclusion when ArrayUtil functions not in camel caps format like ArrayUtil::is_array_assoc() are removed. -->
         <exclude name="PSR1.Methods.CamelCapsMethodName.NotCamelCaps" />
+
+        <exclude name="Squiz.Functions.MultiLineFunctionDeclaration.BraceOnSameLine" />
+    </rule>
+
+    <!-- Allow side effects and symbols within files in the examples directory. -->
+    <rule ref="PSR1.Files.SideEffects.FoundWithSymbols">
+        <exclude-pattern>./examples/*</exclude-pattern>
     </rule>
 
     <!-- Prohibit long lines in php files. -->