Skip to content

php 8+ Unknown: fseek(): Passing null to parameter #2 ($offset) of ty… #146

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BuslikDrev
Copy link

@BuslikDrev BuslikDrev commented Dec 19, 2024

…pe int is deprecated in FontLib\BinaryStream.php on line 123

Unknown: fseek(): Passing null to parameter 2 ($offset) of type int is deprecated in FontLib\BinaryStream.php on line 123

… of type int is deprecated in FontLib\BinaryStream.php on line 123

Unknown: fseek(): Passing null to parameter dompdf#2 ($offset) of type int is deprecated in FontLib\BinaryStream.php on line 123
@bsweeney
Copy link
Member

Can you outline the scenario where this error occurs? FontLib shouldn't attempt to read the font file with a null offset so this could be indicative of a parsing error.

@BuslikDrev
Copy link
Author

Can you outline the scenario where this error occurs? FontLib shouldn't attempt to read the font file with a null offset so this could be indicative of a parsing error.

Позже посмотрю в какой момент это происходит и пришлю результат.

@bsweeney bsweeney added this to the 1.0.2 milestone Dec 21, 2024
@bsweeney
Copy link
Member

Any luck identifying the cause of the issue? Or, if you can indicate the font and the operation I should be able to figure out what happened.

@BuslikDrev
Copy link
Author

Any luck identifying the cause of the issue? Or, if you can indicate the font and the operation I should be able to figure out what happened.

Со шрифтами всё хорошо работает при использовании стандартного кода для их редактирования.
Если для других целей использовать вызов, например, $glyph = new FontLib\Glyph\Outline($this); $glyph->parse($font); то в $offset пойдёт значение null

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants