NFePHP\DA\CTe\Dacce::pBuildDACCE()

Complexity

Conditions	4
Paths	4

Size

Total Lines	56
Code Lines	36

Duplication

Lines	56
Ratio	100 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	1
Bugs	0	Features	0

<?php
    private function pBuildDACCE()

This method seems to be duplicated in your project.

    {
        $this->pdf = new Pdf($this->orientacao, 'mm', $this->papel);
        if ($this->orientacao == 'P') {
            // margens do PDF
            $margSup = 2;
            $margEsq = 2;
            $margDir = 2;
            // posição inicial do relatorio
            $xInic = 1;
            $yInic = 1;
            if ($this->papel == 'A4') { // A4 210x297mm
                $maxW = 210;
                $maxH = 297;
            }
        } else {
            // margens do PDF
            $margSup = 3;
            $margEsq = 3;
            $margDir = 3;
            // posição inicial do relatorio
            $xInic = 5;
            $yInic = 5;
            if ($this->papel == 'A4') { // A4 210x297mm
                $maxH = 210;
                $maxW = 297;
            }
        } // orientação

        // largura imprimivel em mm
        $this->wPrint = $maxW - ($margEsq + $xInic);

The variable $maxW does not seem to be defined for all execution paths leading up to this point.

        // comprimento imprimivel em mm
        $this->hPrint = $maxH - ($margSup + $yInic);

The variable $maxH does not seem to be defined for all execution paths leading up to this point.

        // estabelece contagem de paginas
        $this->pdf->aliasNbPages();
        // fixa as margens
        $this->pdf->setMargins($margEsq, $margSup, $margDir);
        $this->pdf->setDrawColor(0, 0, 0);
        $this->pdf->setFillColor(255, 255, 255);
        // inicia o documento
        $this->pdf->open();
        // adiciona a primeira página
        $this->pdf->addPage($this->orientacao, $this->papel);
        $this->pdf->setLineWidth(0.1);
        $this->pdf->setTextColor(0, 0, 0);
        // montagem da página
        $pag = 1;
        $x = $xInic;
        $y = $yInic;
        // coloca o cabeçalho
        $y = $this->pHeader($x, $y, $pag);
        // coloca os dados da CCe
        $y = $this->pBody($x, $y + 15);

Are you sure the assignment to $y is correct as $this->pBody($x, $y + 15) (which targets NFePHP\DA\CTe\Dacce::pBody()) seems to always return null.

        // coloca o rodapé
        $y = $this->pFooter($x, $y + $this->hPrint - 20);

Are you sure the assignment to $y is correct as $this->pFooter($x, $y + $this->hPrint - 20) (which targets NFePHP\DA\CTe\Dacce::pFooter()) seems to always return null.

$y is not used, you could remove the assignment.

    }