- Issue created by @cedricl
- Assigned to abhishek_virasat
- Issue was unassigned.
- Status changed to Needs review
7 months ago 10:19am 16 April 2024 - ๐ฎ๐ณIndia abhishek_virasat
@CedricL, I have fixed the issue and created MR. please review it once
- ๐ฆ๐บAustralia almunnings Melbourne, ๐ฆ๐บ
Not sure thats it, `getDefinitions` should always return an array. I feel we have a bigger issue with merging non graphql_compose schemas into the final result.
- ๐ฆ๐บAustralia almunnings Melbourne, ๐ฆ๐บ
Ok, it probably boils down to...
$this->getConfiguration()['extensions']
I'll take a look deeper there tonight after everyone goes to bed :)
There was some schema changes upstream in GraphQL which I had to try work with.
Can you try going to the GraphQL schema in the GraphQL admin interface, and just clicking "save" ?
Eg URL:
/admin/config/graphql/servers/manage/YOUR-CORE-ID
- ๐ฆ๐บAustralia almunnings Melbourne, ๐ฆ๐บ
If I put in a sneaky little
dump($this->getConfiguration());
inweb/modules/contrib/graphql/src/Plugin/GraphQL/Schema/ComposableSchema.php
in the methodgetExtensions
I get the following:
/** * {@inheritdoc} */ protected function getExtensions() { dump($this->getConfiguration()); return array_map(function ($id) { return $this->extensionManager->createInstance($id); }, array_filter($this->getConfiguration()['extensions'])); }
array:2 [โผ "extensions" => array:16 [โผ 0 => "graphql_compose_entity_schema_extension" 1 => "graphql_compose_language_schema_extension" 2 => "graphql_compose_information_schema_extension" 3 => "graphql_compose_blocks_schema_extension" 4 => "graphql_compose_comments_schema_extension" 5 => "graphql_compose_edges_schema_extension" 6 => "graphql_compose_image_style_schema_extension" 7 => "graphql_compose_layout_builder_schema_extension" 8 => "graphql_compose_layout_paragraphs_schema_extension" 9 => "graphql_compose_layout_schema_extension" 10 => "graphql_compose_menus_schema_extension" 11 => "graphql_compose_metatags_schema_extension" 12 => "graphql_compose_preview" 13 => "graphql_compose_route_schema_extension" 14 => "graphql_compose_views_schema_extension" 15 => "example_extension" ] "enabled" => true ]
What do you get? (Plugin IDs might vary between dev and beta4)
- ๐ฆ๐บAustralia almunnings Melbourne, ๐ฆ๐บ
Looks like its the schema changes.
schema_configuration: {}
Should be
schema_configuration: graphql_compose: enabled: true
I'll create an update function to check for graphql compose schemas and update that.
-
almunnings โ
committed 3377d629 on 2.1.x
Issue #3441242: Undefined array key "extensions"
-
almunnings โ
committed 3377d629 on 2.1.x
- ๐ฆ๐บAustralia almunnings Melbourne, ๐ฆ๐บ
almunnings โ changed the visibility of the branch 3441242-undefined-array-key to hidden.
- ๐ฆ๐บAustralia almunnings Melbourne, ๐ฆ๐บ
@CedricL Please pull dev at hash, or apply patch
https://git.drupalcode.org/project/graphql_compose/-/commit/3377d6294360...
Let me know if that is fixed after running
drush updatedb
or the database updates via the Drupal UI.Thanks!
- Status changed to RTBC
7 months ago 11:57am 17 April 2024 - ๐ง๐ชBelgium cedricl
The patch file fixed the issue. Thanks for the fast responses.
- ๐ฆ๐บAustralia almunnings Melbourne, ๐ฆ๐บ
Sweet. The update will rerun next release due to a number change, but glad thatโs the issue found.
Will push up b5 now.
- Status changed to Fixed
7 months ago 12:06pm 17 April 2024 - ๐ฆ๐บAustralia almunnings Melbourne, ๐ฆ๐บ
Tagged and released
Thanks all
- Status changed to Fixed
7 months ago 8:16pm 18 April 2024