Skip to content

Commit

Permalink
UHF-X: Add $context property to flysystem stream wrapper
Browse files Browse the repository at this point in the history
PHP requires that stream wrapper implementations have public $context
property. This property is missing from drupal/flysystem stream
wrappers. Since PHP 8.2, assigning dynamic properties is deprecated.

See:
 - https://www.php.net/manual/en/class.streamwrapper.php#streamwrapper.props.context
 - https://www.drupal.org/project/flysystem/issues/3387094
 - twistor/flysystem-stream-wrapper#27
  • Loading branch information
hyrsky committed Nov 7, 2023
1 parent ddb2226 commit ba64a9c
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,5 +11,12 @@
"require-dev": {
"dealerdirect/phpcodesniffer-composer-installer": "^0.7.0",
"drupal/coder": "^8.3"
},
"extra": {
"patches": {
"twistor/flysystem-stream-wrapper": {
"PHP 8.2 support (https://www.drupal.org/project/flysystem/issues/3387094)": "https://raw.githubusercontent.com/City-of-Helsinki/drupal-module-helfi-azure-fs/ddb222622b92d1c2b7db975a84167a00579a1ad0/patches/3387094-add-context-property-to-stream-wrapper.patch"
}
}
}
}

0 comments on commit ba64a9c

Please sign in to comment.