TransactionLines::writeItem() - Code Metrics - Ecodev/my-ichtus - Measure and Improve Code Quality continuously with Scrutinizer

TransactionLines::writeItem() C
last analyzed 2025-10-31 05:07 UTC

↳ Parent: TransactionLines

Complexity

Conditions	12
Paths	4

Size

Total Lines	75
Code Lines	42

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	44
CRAP Score	12

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	42
c	1
b	0
f	0
dl	0
loc	75
ccs	44
cts	44
cp	1
rs	6.9666
cc	12
nc	4
nop	1
crap	12

How to fix Long Method Complexity

<?php

declare(strict_types=1);

namespace Application\Service\Exporter;

use Application\Model\TransactionLine;
use PhpOffice\PhpSpreadsheet\Cell\Coordinate;

class TransactionLines extends AbstractExcel
{
    public function __construct(string $hostname)
    {
        parent::__construct($hostname);

        $this->sheet->setTitle('Compta Écritures');
    }

    private int $currentTransactionRowStart = 0;

    private ?int $currentTransactionId = null;

    protected function getTitleForFilename(): string
    {
        return 'compta_ecritures_' . date('Y-m-d-H:i:s');
    }

    protected function getHeaders(): array
    {
        return [
            ['label' => 'Date', 'width' => 12, 'formats' => [self::$headerFormat, self::$dateFormat], 'autofilter' => true],
            ['label' => 'ID', 'width' => 7, 'formats' => [self::$headerFormat, self::$centerFormat], 'autofilter' => true],
            ['label' => 'Transaction', 'width' => 25, 'formats' => [self::$headerFormat], 'autofilter' => true],
            ['label' => 'Remarques', 'width' => 25, 'formats' => [self::$headerFormat], 'autofilter' => true],
            ['label' => 'Remarques internes', 'width' => 25, 'formats' => [self::$headerFormat], 'autofilter' => true],
            ['label' => 'Écriture', 'width' => 25, 'formats' => [self::$headerFormat], 'autofilter' => true],
            ['label' => 'Remarques', 'width' => 25, 'formats' => [self::$headerFormat], 'autofilter' => true],
            ['label' => 'Réservable', 'width' => 30, 'formats' => [self::$headerFormat], 'autofilter' => true],
            ['label' => 'Compte débit', 'width' => 30, 'formats' => [self::$headerFormat], 'autofilter' => true],
            ['label' => 'Compte crédit', 'width' => 30, 'formats' => [self::$headerFormat], 'autofilter' => true],
            ['label' => 'Montant débit', 'width' => 20, 'formats' => [self::$headerFormat], 'autofilter' => true],
            ['label' => 'Montant crédit', 'width' => 20, 'formats' => [self::$headerFormat], 'autofilter' => true],
            ['label' => 'Pointé', 'width' => 12, 'formats' => [self::$headerFormat, self::$centerFormat], 'autofilter' => true],
            ['label' => 'Tag', 'width' => 30, 'formats' => [self::$headerFormat], 'autofilter' => true],
        ];
    }

    protected function initialize(string $path): void
    {
        parent::initialize($path);
        $this->writeHeaders($this->getHeaders());
        $this->firstDataRow = ++$this->row;
        $this->column = $this->firstDataColumn;
        $this->sheet->setShowGridlines(false);
    }

    /**
     * @param TransactionLine $line
     */
    protected function writeItem($line): void
    {
        $mergeRowsFromColumns = [
            $this->firstDataColumn + 1, // Transaction.ID
            $this->firstDataColumn + 2, // Transaction.name
            $this->firstDataColumn + 3, // Transaction.remarks
            $this->firstDataColumn + 4, // Transaction.internalRemarks
        ];

        $transaction = $line->getTransaction();
        $transactionId = $transaction->getId();

        if (!$this->currentTransactionId) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
            $this->currentTransactionId = $transactionId;
            $this->currentTransactionRowStart = $this->row;
        } elseif ($transactionId !== $this->currentTransactionId) {
            // Current line starts a new transaction
            if ($this->currentTransactionRowStart < $this->row - 1) {
                // Multi-line transaction
                // Merge cells that have the same value because from the same transaction
                foreach ($mergeRowsFromColumns as $column) {
                    $this->sheet->mergeCells([$column, $this->currentTransactionRowStart, $column, $this->row - 1]);
                }
            }
            $this->currentTransactionRowStart = $this->row;
            $this->currentTransactionId = $transactionId;
        }

        // Date
        $this->write($line->getTransactionDate()->format('d.m.Y'));

        // Transaction.ID
        $this->write($transactionId);
        $url = 'https://' . $this->hostname . '/admin/transaction/%u';
        $this->sheet->getCell([$this->column - 1, $this->row])->getHyperlink()->setUrl(sprintf($url, $transactionId));
        // Transaction.name
        $this->write($transaction->getName());
        // Transaction.remarks
        $this->write($transaction->getRemarks());
        // Transaction.internalRemarks
        $this->write($transaction->getInternalRemarks());

        // TransactionLine.name
        $this->write($line->getName());

        // TransactionLine.remarks
        $this->write($line->getRemarks());
        // Bookable.name
        $bookable = $line->getBookable();
        $this->write($bookable ? $bookable->getName() : '');
        // Debit account
        $debit = $line->getDebit();
        $this->write($debit ? implode(' ', [$debit->getCode(), $debit->getName()]) : '');
        // Credit account
        $credit = $line->getCredit();
        $this->write($credit ? implode(' ', [$credit->getCode(), $credit->getName()]) : '');
        // Debit amount
        $this->write($debit ? $this->moneyFormatter->format($line->getBalance()) : '');
        // Credit amount
        $this->write($credit ? $this->moneyFormatter->format($line->getBalance()) : '');
        // Reconciled
        $this->sheet->getStyle([$this->column, $this->row])->applyFromArray(self::$centerFormat);
        $this->write($line->isReconciled() ? '✔️' : '');
        // Tag
        $tag = $line->getTransactionTag();
        $this->write($tag ? $tag->getName() : '');

        $this->lastDataColumn = $this->column - 1;

        $range = Coordinate::stringFromColumnIndex($this->firstDataColumn) . $this->row . ':' . Coordinate::stringFromColumnIndex($this->lastDataColumn) . $this->row;
        // Thicker horizontal delimiter between lines of different transactions
        $this->sheet->getStyle($range)->applyFromArray(self::$bordersBottomLight);

        $this->column = $this->firstDataColumn;
        ++$this->row;
    }

    protected function writeFooter(): void
    {
        // Date
        $this->write('');
        // ID
        $this->write('');
        // Transaction.name
        $this->write('');
        // Transaction.remarks
        $this->write('');
        // Internal remarks
        $this->write('');
        // TransactionLine.name
        $this->write('');
        // Remarks
        $this->write('');
        // Bookable.name
        $this->write('');
        // Debit account
        $this->write('');
        // Credit account
        $this->write('');
        // Debit amount
        $this->write('=SUBTOTAL(9,' . Coordinate::stringFromColumnIndex($this->column) . '2:' . Coordinate::stringFromColumnIndex($this->column) . ($this->row - 1) . ')');
        // Credit amount
        $this->write('=SUBTOTAL(9,' . Coordinate::stringFromColumnIndex($this->column) . '2:' . Coordinate::stringFromColumnIndex($this->column) . ($this->row - 1) . ')');
        // Reconciled
        $this->write('');

        // Apply style
        $range = Coordinate::stringFromColumnIndex($this->firstDataColumn) . $this->row . ':' . Coordinate::stringFromColumnIndex($this->column - 1) . $this->row;
        $this->sheet->getStyle($range)->applyFromArray(self::$totalFormat);
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Application\Service\Exporter;
6
7		use Application\Model\TransactionLine;
8		use PhpOffice\PhpSpreadsheet\Cell\Coordinate;
9
10		class TransactionLines extends AbstractExcel
11		{
12	1	public function __construct(string $hostname)
13		{
14	1	parent::__construct($hostname);
15
16	1	$this->sheet->setTitle('Compta Écritures');
17		}
18
19		private int $currentTransactionRowStart = 0;
20
21		private ?int $currentTransactionId = null;
22
23	1	protected function getTitleForFilename(): string
24		{
25	1	return 'compta_ecritures_' . date('Y-m-d-H:i:s');
26		}
27
28	1	protected function getHeaders(): array
29		{
30	1	return [
31	1	['label' => 'Date', 'width' => 12, 'formats' => [self::$headerFormat, self::$dateFormat], 'autofilter' => true],
32	1	['label' => 'ID', 'width' => 7, 'formats' => [self::$headerFormat, self::$centerFormat], 'autofilter' => true],
33	1	['label' => 'Transaction', 'width' => 25, 'formats' => [self::$headerFormat], 'autofilter' => true],
34	1	['label' => 'Remarques', 'width' => 25, 'formats' => [self::$headerFormat], 'autofilter' => true],
35	1	['label' => 'Remarques internes', 'width' => 25, 'formats' => [self::$headerFormat], 'autofilter' => true],
36	1	['label' => 'Écriture', 'width' => 25, 'formats' => [self::$headerFormat], 'autofilter' => true],
37	1	['label' => 'Remarques', 'width' => 25, 'formats' => [self::$headerFormat], 'autofilter' => true],
38	1	['label' => 'Réservable', 'width' => 30, 'formats' => [self::$headerFormat], 'autofilter' => true],
39	1	['label' => 'Compte débit', 'width' => 30, 'formats' => [self::$headerFormat], 'autofilter' => true],
40	1	['label' => 'Compte crédit', 'width' => 30, 'formats' => [self::$headerFormat], 'autofilter' => true],
41	1	['label' => 'Montant débit', 'width' => 20, 'formats' => [self::$headerFormat], 'autofilter' => true],
42	1	['label' => 'Montant crédit', 'width' => 20, 'formats' => [self::$headerFormat], 'autofilter' => true],
43	1	['label' => 'Pointé', 'width' => 12, 'formats' => [self::$headerFormat, self::$centerFormat], 'autofilter' => true],
44	1	['label' => 'Tag', 'width' => 30, 'formats' => [self::$headerFormat], 'autofilter' => true],
45	1	];
46		}
47
48	1	protected function initialize(string $path): void
49		{
50	1	parent::initialize($path);
51	1	$this->writeHeaders($this->getHeaders());
52	1	$this->firstDataRow = ++$this->row;
53	1	$this->column = $this->firstDataColumn;
54	1	$this->sheet->setShowGridlines(false);
55		}
56
57		/**
58		* @param TransactionLine $line
59		*/
60	1	protected function writeItem($line): void
61		{
62	1	$mergeRowsFromColumns = [
63	1	$this->firstDataColumn + 1, // Transaction.ID
64	1	$this->firstDataColumn + 2, // Transaction.name
65	1	$this->firstDataColumn + 3, // Transaction.remarks
66	1	$this->firstDataColumn + 4, // Transaction.internalRemarks
67	1	];
68
69	1	$transaction = $line->getTransaction();
70	1	$transactionId = $transaction->getId();
71
72	1	if (!$this->currentTransactionId) {
		0 ignored issues – show Bug Best Practice introduced 2023-10-09 09:33 UTC by Report Bug Copy Issue Report The expression `$this->currentTransactionId` of type `integer\|null` is loosely compared to `false`; this is ambiguous if the integer can be 0. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
73	1	$this->currentTransactionId = $transactionId;
74	1	$this->currentTransactionRowStart = $this->row;
75	1	} elseif ($transactionId !== $this->currentTransactionId) {
76		// Current line starts a new transaction
77	1	if ($this->currentTransactionRowStart < $this->row - 1) {
78		// Multi-line transaction
79		// Merge cells that have the same value because from the same transaction
80	1	foreach ($mergeRowsFromColumns as $column) {
81	1	$this->sheet->mergeCells([$column, $this->currentTransactionRowStart, $column, $this->row - 1]);
82		}
83		}
84	1	$this->currentTransactionRowStart = $this->row;
85	1	$this->currentTransactionId = $transactionId;
86		}
87
88		// Date
89	1	$this->write($line->getTransactionDate()->format('d.m.Y'));
90
91		// Transaction.ID
92	1	$this->write($transactionId);
93	1	$url = 'https://' . $this->hostname . '/admin/transaction/%u';
94	1	$this->sheet->getCell([$this->column - 1, $this->row])->getHyperlink()->setUrl(sprintf($url, $transactionId));
95		// Transaction.name
96	1	$this->write($transaction->getName());
97		// Transaction.remarks
98	1	$this->write($transaction->getRemarks());
99		// Transaction.internalRemarks
100	1	$this->write($transaction->getInternalRemarks());
101
102		// TransactionLine.name
103	1	$this->write($line->getName());
104
105		// TransactionLine.remarks
106	1	$this->write($line->getRemarks());
107		// Bookable.name
108	1	$bookable = $line->getBookable();
109	1	$this->write($bookable ? $bookable->getName() : '');
110		// Debit account
111	1	$debit = $line->getDebit();
112	1	$this->write($debit ? implode(' ', [$debit->getCode(), $debit->getName()]) : '');
113		// Credit account
114	1	$credit = $line->getCredit();
115	1	$this->write($credit ? implode(' ', [$credit->getCode(), $credit->getName()]) : '');
116		// Debit amount
117	1	$this->write($debit ? $this->moneyFormatter->format($line->getBalance()) : '');
118		// Credit amount
119	1	$this->write($credit ? $this->moneyFormatter->format($line->getBalance()) : '');
120		// Reconciled
121	1	$this->sheet->getStyle([$this->column, $this->row])->applyFromArray(self::$centerFormat);
122	1	$this->write($line->isReconciled() ? '✔️' : '');
123		// Tag
124	1	$tag = $line->getTransactionTag();
125	1	$this->write($tag ? $tag->getName() : '');
126
127	1	$this->lastDataColumn = $this->column - 1;
128
129	1	$range = Coordinate::stringFromColumnIndex($this->firstDataColumn) . $this->row . ':' . Coordinate::stringFromColumnIndex($this->lastDataColumn) . $this->row;
130		// Thicker horizontal delimiter between lines of different transactions
131	1	$this->sheet->getStyle($range)->applyFromArray(self::$bordersBottomLight);
132
133	1	$this->column = $this->firstDataColumn;
134	1	++$this->row;
135		}
136
137	1	protected function writeFooter(): void
138		{
139		// Date
140	1	$this->write('');
141		// ID
142	1	$this->write('');
143		// Transaction.name
144	1	$this->write('');
145		// Transaction.remarks
146	1	$this->write('');
147		// Internal remarks
148	1	$this->write('');
149		// TransactionLine.name
150	1	$this->write('');
151		// Remarks
152	1	$this->write('');
153		// Bookable.name
154	1	$this->write('');
155		// Debit account
156	1	$this->write('');
157		// Credit account
158	1	$this->write('');
159		// Debit amount
160	1	$this->write('=SUBTOTAL(9,' . Coordinate::stringFromColumnIndex($this->column) . '2:' . Coordinate::stringFromColumnIndex($this->column) . ($this->row - 1) . ')');
161		// Credit amount
162	1	$this->write('=SUBTOTAL(9,' . Coordinate::stringFromColumnIndex($this->column) . '2:' . Coordinate::stringFromColumnIndex($this->column) . ($this->row - 1) . ')');
163		// Reconciled
164	1	$this->write('');
165
166		// Apply style
167	1	$range = Coordinate::stringFromColumnIndex($this->firstDataColumn) . $this->row . ':' . Coordinate::stringFromColumnIndex($this->column - 1) . $this->row;
168	1	$this->sheet->getStyle($range)->applyFromArray(self::$totalFormat);
169		}
170		}
171

Ecodev / my-ichtus

TransactionLines::writeItem() C last analyzed 2025-10-31 05:07 UTC

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

TransactionLines::writeItem() C
last analyzed 2025-10-31 05:07 UTC