Skip to content

Update tcpdf.php for undefined array key#860

Open
medarob wants to merge 1 commit intotecnickcom:mainfrom
medarob:patch-1
Open

Update tcpdf.php for undefined array key#860
medarob wants to merge 1 commit intotecnickcom:mainfrom
medarob:patch-1

Conversation

@medarob
Copy link
Copy Markdown

@medarob medarob commented Mar 25, 2026

Core: Error handler (FE): PHP Warning: Undefined array key 0 in /var/www/typo3/vendor/tecnickcom/tcpdf/tcpdf.php line 20833
Core: Error handler (FE): PHP Warning: Undefined array key -1 in /var/www/typo3/vendor/tecnickcom/tcpdf/tcpdf.php line 20689
Core: Error handler (FE): PHP Warning: Undefined array key 0 in /var/www/typo3/vendor/tecnickcom/tcpdf/tcpdf.php line 19318

Core: Error handler (FE): PHP Warning: Undefined array key 0 in /var/www/typo3/vendor/tecnickcom/tcpdf/tcpdf.php line 20833
Core: Error handler (FE): PHP Warning: Undefined array key -1 in /var/www/typo3/vendor/tecnickcom/tcpdf/tcpdf.php line 20689
Core: Error handler (FE): PHP Warning: Undefined array key 0 in /var/www/typo3/vendor/tecnickcom/tcpdf/tcpdf.php line 19318
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@medarob medarob mentioned this pull request Mar 25, 2026
@nicolaasuni
Copy link
Copy Markdown
Member

Thank you very much, @medarob, for the undefined array key fixes - suppressing those PHP warnings keeps projects running cleanly on strict error reporting settings, and it is genuinely appreciated.

TCPDF 6 is now deprecated and no further changes will be merged into this repository. Please see the pinned issue Thank You - and the Future of TCPDF for the full explanation and the path forward.

The array access patterns in tc-lib-pdf are validated by PHPStan at a strict level, so undefined-key warnings of this kind are caught at static-analysis time. If you encounter a related issue there, feel free to open a PR or issue.

I am leaving this PR open in case you or anyone else wishes to continue the discussion, but it is unlikely that this PR will ever be merged.

Thank you again for your patience and understanding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants