Merge pull request #23 from eclipxe13/version-3.2.1

Corregir el proceso de construcción (versión 3.2.1)

Author: eclipxe13

.github/workflows/build.yml

@@ -21,7 +21,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '8.3'
+          php-version: '8.4'
           coverage: none
           tools: phpcs, cs2pr
       - name: phpcs
@@ -51,7 +51,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '8.3'
+          php-version: '8.4'
           coverage: none
           tools: composer:v2, phpstan, cs2pr
       - name: Get composer cache directory
@@ -77,7 +77,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '8.3'
+          php-version: '8.4'
           coverage: none
           tools: psalm, cs2pr
       - name: Get composer cache directory
@@ -99,7 +99,7 @@ jobs:
     runs-on: "ubuntu-latest"
     strategy:
       matrix:
-        php-version: ['7.3', '7.4', '8.0', '8.1', '8.2', '8.3']
+        php-version: ['7.3', '7.4', '8.0', '8.1', '8.2', '8.3', '8.4']
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -134,7 +134,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '8.3'
+          php-version: '8.4'
           coverage: xdebug
           tools: composer:v2, infection
       - name: Get composer cache directory

.github/workflows/sonarcloud.yml

@@ -6,7 +6,7 @@ on:
 
 # Actions
 # shivammathur/setup-php@v2 https://github.com/marketplace/actions/setup-php-action
-# sonarsource/sonarcloud-github-action@master https://github.com/marketplace/actions/sonarcloud-scan
+# SonarSource/sonarqube-scan-action@v5 https://github.com/marketplace/actions/official-sonarqube-scan
 
 jobs:
 
@@ -19,7 +19,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '8.3'
+          php-version: '8.4'
           coverage: xdebug
           tools: composer:v2
         env:
@@ -79,7 +79,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '8.3'
+          php-version: '8.4'
           coverage: none
           tools: composer:v2
       - name: Get composer cache directory
@@ -103,7 +103,6 @@ jobs:
           sed 's#'$GITHUB_WORKSPACE'#/github/workspace#g' build/coverage/junit.xml > build/sonar-junit.xml
           sed 's#'$GITHUB_WORKSPACE'#/github/workspace#g' build/coverage/clover.xml > build/sonar-coverage.xml
       - name: SonarCloud Scan
-        uses: sonarsource/sonarcloud-github-action@master
+        uses: SonarSource/sonarqube-scan-action@v5
         env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}

.phive/phars.xml

@@ -1,9 +1,9 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phive xmlns="https://phar.io/phive">
-  <phar name="php-cs-fixer" version="^3.64.0" installed="3.64.0" location="./tools/php-cs-fixer" copy="false"/>
-  <phar name="phpcs" version="^3.10.2" installed="3.10.2" location="./tools/phpcs" copy="false"/>
-  <phar name="phpcbf" version="^3.10.2" installed="3.10.2" location="./tools/phpcbf" copy="false"/>
-  <phar name="phpstan" version="^1.12.3" installed="1.12.3" location="./tools/phpstan" copy="false"/>
-  <phar name="psalm" version="^5.26.1" installed="5.26.1" location="./tools/psalm" copy="false"/>
-  <phar name="infection" version="^0.29.6" installed="0.29.6" location="./tools/infection" copy="false"/>
+  <phar name="php-cs-fixer" version="^3.75.0" installed="3.75.0" location="./tools/php-cs-fixer" copy="false"/>
+  <phar name="phpcs" version="^3.12.2" installed="3.12.2" location="./tools/phpcs" copy="false"/>
+  <phar name="phpcbf" version="^3.12.2" installed="3.12.2" location="./tools/phpcbf" copy="false"/>
+  <phar name="phpstan" version="^2.1.11" installed="2.1.11" location="./tools/phpstan" copy="false"/>
+  <phar name="psalm" version="^6.10.0" installed="6.10.0" location="./tools/psalm" copy="false"/>
+  <phar name="infection" version="^0.29.14" installed="0.29.14" location="./tools/infection" copy="false"/>
 </phive>

docs/CHANGELOG.md

@@ -9,6 +9,20 @@ Usamos [Versionado Semántico 2.0.0](SEMVER.md) por lo que puedes usar esta libr
 Pueden aparecer cambios no liberados que se integran a la rama principal, pero no ameritan una nueva liberación de
 versión, aunque sí su incorporación en la rama principal de trabajo, generalmente se tratan de cambios en el desarrollo.
 
+## Listado de cambios
+
+### Version 3.2.1 2025-04-13
+
+- Se comprueba que el proyecto es compatible con PHP 8.4.
+- Se genera una excepción lógica cuando el método `DOMHelper#getAttribute()` es llamado sin argumentos.
+
+Los siguientes cambios aplican al entorno de desarrollo.
+
+- Se agrega PHP 8.4 a la matriz de pruebas.
+- Se ejecutan los flujos de trabajo con PHP 8.4.
+- Se actualizan las herramientas de desarrollo.
+- Se actualizó la herramienta para ejecutar *Sonarqube* en el flujo de trabajo *sonarcloud*.
+
 ### Mantenimiento 2024-09-09
 
 - Se remueven los paréntesis innecesarios de las llamadas `return`.
@@ -42,8 +56,6 @@ versión, aunque sí su incorporación en la rama principal de trabajo, generalm
 - Se cambian los badges en el archivo `README`.
 - Se corrigen las clases en las pruebas para que tengan alguno de los modificadores `final` o `abstract`.
 
-## Listado de cambios
-
 ### Version 3.2.0 2022-06-27
 
 #### Soporte de RET 2.0

psalm.xml.dist

@@ -1,6 +1,5 @@
 <?xml version="1.0"?>
 <psalm xmlns="https://getpsalm.org/schema/config"
-    resolveFromConfigFile="true"
     errorLevel="1">
     <projectFiles>
         <directory name="src" />
@@ -10,6 +9,10 @@
     </projectFiles>
 
     <issueHandlers>
+        <PossiblyUnusedMethod errorLevel="info" />
+        <UnusedClass errorLevel="info" />
+        <ClassMustBeFinal errorLevel="info" />
+        <MissingOverrideAttribute errorLevel="info" />
         <LessSpecificReturnType errorLevel="info" />
     </issueHandlers>
 </psalm>

src/Extractors/Standards/FormatSelloLast8.php

@@ -8,6 +8,6 @@ trait FormatSelloLast8
 {
     public function formatSello(string $sello): string
     {
-        return substr($sello, -8);
+        return (string) substr($sello, -8);
     }
 }

src/Internal/DOMHelper.php

@@ -34,6 +34,9 @@ public function rootElement(): DOMElement
 
     public function getAttribute(string ...$path): string
     {
+        if ([] === $path) {
+            throw new LogicException('Attrribute path cannot be empty');
+        }
         $value = $this->findAttribute(...$path);
         if (null === $value) {
             $attribute = array_pop($path);

tests/Unit/Internal/DOMHelperTest.php

@@ -162,4 +162,15 @@ public function testGetAttributeInDepth(): void
         $attribute = $helper->getAttribute('b:books', 'b:library', 't:topic', 'b:book', 'author');
         $this->assertSame('Carlos C Soto', $attribute);
     }
+
+    public function testGetAttributeWithNoArguments(): void
+    {
+        $document = new DOMDocument();
+        $helper = new DOMHelper($document);
+        $document->load($this->filePath('books.xml'));
+
+        $this->expectException(LogicException::class);
+        $this->expectExceptionMessage('Attrribute path cannot be empty');
+        $helper->getAttribute();
+    }
 }