Deprecated error when passing null to non-nullable argument for PHP 8.1

[jawira]

Hi, this line is causing deprecated error when executed in PHP 8.1 (with GitHub Actions):

vfsStream/src/vfsFile.php

Line 63 in 7299174

$this->content = new StringBasedFileContent(null);

The problem is the null value passed to substr function later. Example:

vfsStream/src/content/StringBasedFileContent.php

Line 70 in 7299174

return (string) substr($this->content, $offset, $count);

Passing null to non-nullable arguments is deprecated since PHP 8.1.

Souce: https://wiki.php.net/rfc/deprecate_null_to_scalar_internal_arg

[allejo]

This has been fixed in our master branch already.

vfsStream/src/vfsStreamFile.php

Lines 65 to 70 in f2b8d8c

	public function __construct(string $name, ?int $permissions = null)
	{
	$this->content = new StringBasedFileContent('');
	$this->type = vfsStreamContent::TYPE_FILE;
	parent::__construct($name, $permissions);
	}

I've back ported the fix to the v1.x branch in 1483c50. We'll tag a new v1.6.x or 1.7.x release with this fix before or around PHP 8.1's release.

[alexpott]

@allejo @jawira thanks for fixing the PHP 8.1 deprecation. Is there any chance of a 1.6.x or 1.7.x release sooner than PHP 8.1.x release with this fix in? This gives projects that are dependent on vfsStream a chance to work through all their PHP 8.1. deprecations prior to PHP 8.1's release. Thanks!

[allejo]

This gives projects that are dependent on vfsStream a chance to work through all their PHP 8.1. deprecations prior to PHP 8.1's release.

I hadn't thought about this! I'll tag a new release this week, just want to do some cleanup/organization first.

[allejo]

I've tagged v1.6.9 which includes this fix.

[alexpott]

@allejo thanks for release! Nice one :)