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

[BuslikDrev]

…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

[bsweeney]

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]

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]

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]

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

[bsweeney]

I can see how that would result in the issue, but this particular use case doesn't really make much sense even though it's valid logic. The Outline classes require an offset and size to do anything. Otherwise parsing the table returns nothing.

It probably makes sense to require a valid integer instead, though that's for later. In the meantime, this change does address the notice while maintaining existing functionality.