-
Notifications
You must be signed in to change notification settings - Fork 122
ACF_Link is being registered multiple times #189
Comments
I have exactly the same problem. Thought I'd just put a +1 on this issue |
Same issue:
|
This is a known bug apparently. |
Is there any movement on this? My site does re-use ACF field groups across multiple templates and custom post types and this bug makes this plugin useless. I otherwise love WPGraphQL and really would like to use it on this and other projects. |
First off, many thanks to the creator of this amazing plugin! |
Are we close to resolving this issue? |
Same issue here |
Same here |
We also have the same problem, we cannot update the plugin because the site stops working. @kidunot89 Does it look like a small or big fix for you? |
Same issue here. |
Same issue here :( |
Same issue! |
Hello, Apologies for the delay in responding. The debug message should not be affecting execution. This bug has actually existed for a long time, but a change to WPGraphQL core leading up to the v1.0 release surfaced it in a debug message, where before it would silently happen, just without letting us know. I explain a bit more about this here: #198 (comment) I'm going to close this issue as a duplicate of #198 |
In trying to track down why my Gatsby previews are failing, and in the process I noticed a warning stating:
I found where this message is printed out in
wp-graphql-1.0/src/Registry/TypeRegistry.php
and put anerror_log
statement in place. My error log got flooded out with this same error aboutACF_Link
being registered multiple times.In looking at
wp-graphql-acf-release-v0.3.5/src/class-config.php
in the methodregister_graphql_field
, for the caselink
, it appears to always register the$field_type_name
ofACF_Link
, whereas other cases in here seem to append other variances to the end of the field name before registering them. When puttingerror_log()
statements in this case I find dozens/hundreds of timesACF_Link
gets used for the $field_type_name (even though they each have differing$type_name
values).I'm not sure if this is related to my preview not working, but I wanted to file a ticket as it seems this warning is firing hundreds of times under the covers either way and potentially should be revisited..?
The text was updated successfully, but these errors were encountered: