QrGeneratorService::buildQrContent necesita cambios en los parámetros

[way2ecommerce]

Descripción del error
El servicio QrGeneratorService::buildQrContent no genera el QR con los parámetros correctos. Actualmente utiliza "num" en lugar de "numserie" y no incluye el campo "importe", lo que provoca que el enlace generado sea erróneo y el QR no se construya adecuadamente.

Cómo reproducirlo
Pasos para reproducir el error:

1. Ir a la funcionalidad que genera el QR.
2. Ejecutar la generación de un QR con los parámetros actuales.
3. Observar que el link generado no es válido.
4. Ver que el QR no se genera con los parámetros correctos.

Comportamiento esperado
El QR debería generarse con los parámetros correctos:

$params = http_build_query([
    'nif' => $nif,
    'numserie' => $series,
    'fecha' => $date,
    'importe' => $amount,
    'huella' => $hash,
]);

Capturas de pantalla

Contexto adicional
El bug se soluciona reemplazando "num" por "numserie" y añadiendo el campo "importe" en el array de parámetros.

[israelgs]

Cambia esto:

En src/services/QrGeneratorService.php, donde dice:

protected static function buildQrContent(InvoiceRecord $record, $baseVerificationUrl): string
    {
        $invoiceId = $record->getInvoiceId();
        $nif = $invoiceId->issuerNif;
        $series = $invoiceId->seriesNumber;
        $date = $invoiceId->issueDate;
        $hash = $record->hash;

        $params = http_build_query([
            'nif' => $nif,
            'num' => $series,
            'fecha' => $date,
            'huella' => $hash,
        ]);

        return rtrim($baseVerificationUrl, '?') . '?' . $params;
    }

Sustituye por:

protected static function buildQrContent(InvoiceRecord $record, $baseVerificationUrl): string
    {
        $invoiceId = $record->getInvoiceId();
        $nif = $invoiceId->issuerNif;
        $series = $invoiceId->seriesNumber;
        $date = $invoiceId->issueDate;
        $hash = $record->hash;

        $params = http_build_query([
            'nif' => $nif,
            'numserie' => $series,
            'fecha' => $date,
            'huella' => $hash,
            'importe' => $record->totalAmount
        ]);

        return rtrim($baseVerificationUrl, '?') . '?' . $params;
    }

[taiwanleaftea]

Regarding the last version of Verifactu documentation par. 6 you don't need to add "huella" to the QR-code.