Claudsonm\BoletoWinner\Converters\ConvenioConverter

Complexity

Total Complexity

5

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	toBarcode()	0	8	2
A	toWritableLine()	0	10	3

<?php

namespace Claudsonm\BoletoWinner\Converters;

use Claudsonm\BoletoWinner\Bill;
use Claudsonm\BoletoWinner\Convenio;

class ConvenioConverter implements Converter
{
    /**
     * {@inheritdoc}
     */
    public function toBarcode(Bill $bill): string
    {
        $blocks = str_split($bill->getWritableLine(), 12);
        foreach ($blocks as &$block) {
            $block = substr($block, 0, -1);
        }

        return implode('', $blocks);

It seems like $blocks can also be of type true; however, parameter $pieces of implode() does only seem to accept array, maybe add an additional type check?

    }

    /**
     * {@inheritdoc}
     */
    public function toWritableLine(Bill $bill): string
    {
        $barcode = $bill->getBarcode();
        $blocks = str_split($barcode, 11);
        $isModule10 = in_array($barcode[2], Convenio::USES_MODULE_10);
        foreach ($blocks as &$block) {
            $block .= $isModule10 ? module10($block) : module11($block);
        }

        return implode('', $blocks);

It seems like $blocks can also be of type true; however, parameter $pieces of implode() does only seem to accept array, maybe add an additional type check?

    }
}