Skip to content

Commit 8135653

Browse files
committed
fix(graphile-postgis): remove isPgConnectionFilterMany scope to avoid some/every/none collision
Setting isPgConnectionFilterMany: true on our per-relation spatial filter type caused ConnectionFilterBackwardRelationsPlugin to auto-register some/every/none fields with FK-join semantics — colliding with (and semantically different from) the ones we register. Our plugin fully owns those fields for spatial-relation filter types, so we leave that flag off.
1 parent 16ad9f9 commit 8135653

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

graphile/graphile-postgis/src/plugins/spatial-relations.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -572,7 +572,11 @@ export const PostgisSpatialRelationsPlugin: GraphileConfig.Plugin = {
572572
build.registerInputObjectType(
573573
typeName,
574574
{
575-
isPgConnectionFilterMany: true,
575+
// NOTE: intentionally NOT setting `isPgConnectionFilterMany`.
576+
// That flag triggers ConnectionFilterBackwardRelationsPlugin
577+
// (and friends) to auto-register `some`/`every`/`none` fields
578+
// with FK-join semantics, which would collide with — and
579+
// semantically differ from — ours. We own those fields here.
576580
foreignTable: rel.targetResource,
577581
isPgGISSpatialFilter: true,
578582
pgGISSpatialRelation: rel,

0 commit comments

Comments
 (0)