Skip to content

[BUGFIX] Handle "dirty" objects.inv.json data #853

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Feb 19, 2024
Merged

Conversation

linawolf
Copy link
Contributor

Handle null values as not set, ignore inventory entries that have non-scalar values or null in the URI.

Until now those were causing type errors

Copy link
Contributor

@brotkrueml brotkrueml left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, currently I don't have enough information for a review: Which content holds $linkArray?

@linawolf
Copy link
Contributor Author

linawolf commented Feb 19, 2024

$linkArraycomes from the imported JSON data, see the fixtures in the Unit Tests https://github.com/phpDocumentor/guides/pull/853/files#diff-615ca620b21221a05c303d924a986d8334be1c95db29e038e95c7a23d60f21b1

Handle null values as not set, ignore inventory entries that have non-scalar values or null in the URI.

Until now those were causing type errors
@linawolf linawolf merged commit 74b3ed0 into main Feb 19, 2024
@linawolf linawolf deleted the bugfix/interlinks branch February 19, 2024 17:13
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.

3 participants