Issue #3289927: Automated Drupal 10 compatibility fixes
4 open threads
Merge request reports
Activity
35 35 */ 36 36 protected function getDirectoryPath() { 37 37 $library_discovery = new LibraryDiscovery($this->getDrupalRoot()); 38 /** @var $files \Drupal\Core\Extension\Extension[] */ 38 /** @var \Drupal\Core\Extension\Extension $files */ - Comment on lines -38 to +38
I don't think this is needed at all because
\Drupal\Core\Extension\ExtensionDiscovery::scan()
documents a return type. Normally I'd not worry about it, but that method says it returnsExtension[]
so it seems like we're actually overriding it with a wrong type. Later code accesses by array too. changed this line in version 2 of the diff
- composer.json 0 → 100644
1 { 2 "name": "drupal/system_stream_wrapper", 3 "description": "Provides stream wrappers to access files in module, theme, profile, and library files and directories.", 4 "type": "drupal-module", 5 "license": "GPL-2.0-or-later", 6 "authors": [ 7 { 8 "name": "Drupal Community" 9 } 10 ], 11 "require": { 12 "drupal/core": "^8 || ^9 || ^10" changed this line in version 2 of the diff
1 1 name: System stream wrapper 2 2 type: module 3 3 description: Provides stream wrappers to access files in module, theme, profile, and library files and directories. 4 core: 8.x 5 core_version_requirement: ^8 || ^9 4 core_version_requirement: ^8 || ^9 || ^10 changed this line in version 2 of the diff
Please register or sign in to reply