Skip to content

Comments

Fix GraphQL element date resolver type error#1045

Open
umizoomi wants to merge 2 commits intopimcore:2.xfrom
umizoomi:fix/graphql-element-date-resolver
Open

Fix GraphQL element date resolver type error#1045
umizoomi wants to merge 2 commits intopimcore:2.xfrom
umizoomi:fix/graphql-element-date-resolver

Conversation

@umizoomi
Copy link

@umizoomi umizoomi commented Feb 19, 2026

Root Cause

The resolveModificationDate() and resolveCreationDate() methods were added in DataHub v2.x with strict array type hints, but they don't account for ElementDescriptor objects that are returned when resolving relation fields.

Affected Queries

getObject {
  relation {
    creationDate
    modificationDate
  }
}

Related Files

src/GraphQL/RelationHelper.php
└── processRelation() returns ElementDescriptor

src/GraphQL/ElementDescriptor.php
└── extends ArrayObject (array-accessible but not type-compatible)

src/GraphQL/BaseDescriptor.php
└── extends ArrayObject

@github-actions
Copy link

github-actions bot commented Feb 19, 2026

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@sonarqubecloud
Copy link

@umizoomi
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@umizoomi
Copy link
Author

recheck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant