PHP 8.2 Deprecated function: Creation of dynamic property

Hi @dharmendart,
I have fixed this issue "PHP 8.2 Deprecated function: Creation of dynamic property" and also attached patch,
please review and verify,

You have to add AllowedDynamicProperties to final class to skip this error.
I have added this patch

geoanders → closed merge request !10

#6 seems like the correct fix here.

More information on the matter: https://www.php.net/manual/en/class.allowdynamicproperties.php

New PR, because dosen't work on D10 latest.

Ad patch 1

A New patch fix all above

Tested on D 10.1.8 PHP8.2 and the error no longer appears.

Why are we muting a deprecation notice instead of using the newer API? Seems like we're treating a symptom here instead of the root cause. Or am I missing something?

ckhalilo → changed the visibility of the branch 3360409-16 to hidden.

ckhalilo → changed the visibility of the branch 3360409-16 to active.

ckhalilo → changed the visibility of the branch 8.x-2.x to hidden.

I have already pushed a new changes

I looked into the deprecated function and found the solution that @ckhalio has in https://git.drupalcode.org/project/az_blob_fs/-/merge_requests/19 is sooo close. The simplest and most elegant solution is, in fact, to add $context as a property to the class. The AzBlobFsStream's is a version of the PHP streamWrapper, and it would appear that the `$context` property has been around since at least PHP 5 or so. Therefore, adding it would be backwardly compatible as well.

This would be a quick win and be one less thing in the issue queue, were it to be fixed.

In that merge request, all that is required is to convert the property from private to public, as documented in the PHP docs. If you look at core, there are a couple of classes in core that demonstrate using public $context:

• core/lib/Drupal/Core/StreamWrapper/ReadOnlyStream.php
• core/lib/Drupal/Core/StreamWrapper/LocalStream.php