Fields starting with int throw error if not renamed.

Open on Drupal.org →

Created on 16 December 2023, 6 months ago

Updated 6 January 2024, 6 months ago

Create a field: field_88_bananas
Enable field in GraphQL Compose, which defaults to: 88Bananas

GraphQL\Error\SyntaxError: Syntax Error: Expected Name, found Int "88" in GraphQL\Language\Parser->expect() (line 366 of /app/vendor/webonyx/graphql-php/src/Language/Parser.php).

This is currently easy enough to mitigate by changing the field name in the GraphQL Compose config.

Would be nice to make the rename required on enabling the field, if the field name starts with an int.

✨ Feature request

Status

Fixed

Version

2.1

Component

Code

Created by

🇦🇺Australia AlMunnings Melbourne

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!57Fields starting with int throw error if not renamed.
Merged
🇦🇺Australia AlMunnings
updated 6 months ago

Comments & Activities

Issue created by @AlMunnings
Merge request !57Issue #3409260 by almunnings: Fields starting with int throw error if not renamed → (Merged) created by AlMunnings
Status changed to Needs review 6 months ago12:48am 17 December 2023
Comment 6 months ago →
🇦🇺Australia AlMunnings Melbourne
Comment 6 months ago →
System Message

almunnings → committed 31778a37 on 2.1.x
Issue #3409260 by almunnings: Fields starting with int throw error if...
Comment 6 months ago →
🇦🇺Australia AlMunnings Melbourne
Status changed to Fixed 6 months ago4:15am 23 December 2023
Comment 6 months ago →
🇦🇺Australia AlMunnings Melbourne
Comment 6 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024