fix: use `Post::$databaseId` to prevent fatals - Githubissues

AxeWP / wp-graphql-rank-math

Add GraphQL support for RankMath SEO

GNU General Public License v3.0

50 stars 7 forks source link

fix: use `Post::$databaseId` to prevent fatals #109

Closed justlevine closed 3 weeks ago

justlevine commented 3 weeks ago

What

This PR fixes an issue in NodeWithSeo so the model is resolved using Post::$databaseId instead of Post::$ID

Why

On private posts, the Post::$ID is protected, leading to potential fatal errors with strict types.

Fixes #108

How

Testing Instructions

Additional Info

Checklist:

[x] My code is tested to the best of my abilities.
[x] My code follows the WordPress Coding Standards.
[x] My code has proper inline documentation.
[x] I have added unit tests to verify the code works as intended.
[x] The changes in this PR have been noted in CHANGELOG.md