F72X\Tools\QrGenerator

Complexity

Total Complexity

6

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getQrString()	0	9	1
A	image()	0	27	4
A	getQrContent()	0	11	1

<?php

/**
 * MÓDULO DE EMISIÓN ELECTRÓNICA F72X
 * UBL 2.1
 * Version 1.0
 * 
 * Copyright 2019, Jaime Cruz
 */

namespace F72X\Tools;

use F72X\F72X;
use F72X\Sunat\DataMap;
use F72X\Sunat\Operations;
use F72X\Company;
use Codelint\QRCode\QRCode;

class QrGenerator {

    public static function getQrString(DataMap $inv) {
        $documentName = $inv->getDocumentName();
        $qr = new QRCode();
        $qrContent = self::getQrContent($inv);
        $qrTempPath = F72X::getTempDir() . "/QR-$documentName.png";
        $qr->png($qrContent, $qrTempPath, 'Q', 8, 2);
        $qrs = base64_encode(file_get_contents($qrTempPath));
        unlink($qrTempPath);
        return $qrs;
    }

    //----------------------------------------------------------------------
    private static function image($frame, $pixelPerPoint = 4, $outerFrame = 4) {

The method image() is not used, and could be removed.

        $h = count($frame);
        $w = strlen($frame[0]);

        $imgW = $w + 2 * $outerFrame;
        $imgH = $h + 2 * $outerFrame;

        $base_image = ImageCreate($imgW, $imgH);

        $col[0] = ImageColorAllocate($base_image, 255, 255, 255);

It seems like $base_image can also be of type false; however, parameter $image of imagecolorallocate() does only seem to accept resource, maybe add an additional type check?

$col was never initialized. Although not strictly required by PHP, it is generally a good practice to add $col = array(); before regardless.

        $col[1] = ImageColorAllocate($base_image, 0, 0, 0);

        imagefill($base_image, 0, 0, $col[0]);

It seems like $base_image can also be of type false; however, parameter $image of imagefill() does only seem to accept resource, maybe add an additional type check?

        for ($y = 0; $y < $h; $y++) {
            for ($x = 0; $x < $w; $x++) {
                if ($frame[$y][$x] == '1') {
                    ImageSetPixel($base_image, $x + $outerFrame, $y + $outerFrame, $col[1]);

It seems like $base_image can also be of type false; however, parameter $image of imagesetpixel() does only seem to accept resource, maybe add an additional type check?

                }
            }
        }

        $target_image = ImageCreate($imgW * $pixelPerPoint, $imgH * $pixelPerPoint);
        ImageCopyResized($target_image, $base_image, 0, 0, 0, 0, $imgW * $pixelPerPoint, $imgH * $pixelPerPoint, $imgW, $imgH);

It seems like $base_image can also be of type false; however, parameter $src_image of imagecopyresized() does only seem to accept resource, maybe add an additional type check?

It seems like $target_image can also be of type false; however, parameter $dst_image of imagecopyresized() does only seem to accept resource, maybe add an additional type check?

        ImageDestroy($base_image);

It seems like $base_image can also be of type false; however, parameter $image of imagedestroy() does only seem to accept resource, maybe add an additional type check?

        return $target_image;
    }

    private static function getQrContent(DataMap $inv) {
        $ruc = Company::getRUC();
        $invoiveType = $inv->getDocumentType();
        $documentSeries = $inv->getDocumentSeries();
        $seriesNumber = $inv->getDocumentNumber();
        $igv = Operations::formatAmount($inv->getIGV());
        $payableAmount = Operations::formatAmount($inv->getPayableAmount());
        $issueDate = $inv->getIssueDate()->format('Y-m-d');
        $customerDocType = $inv->getCustomerDocType();
        $customerDocNumber = $inv->getCustomerDocNumber();
        return "$ruc|$invoiveType|$documentSeries|$seriesNumber|$igv|$payableAmount|$issueDate|$customerDocType|$customerDocNumber";
    }

}