-
Notifications
You must be signed in to change notification settings - Fork 363
Fix #827 #828
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
Fix #827 #828
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR addresses issue #827 by enhancing schema scanning and preventing unintended alterations to sibling ref properties. Key changes include:
- Updating function signatures with strict type hints and return types in both SchemaStorageInterface and SchemaStorage.
- Switching to strict equality (===) for schema id comparisons.
- Adding a recursive subschema registration method in SchemaStorage to scan for nested id properties.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/JsonSchema/SchemaStorageInterface.php | Updated function signatures with type hints and return types. |
| src/JsonSchema/SchemaStorage.php | Enhanced addSchema and expandRefs methods for better reference resolution and introduced subschema registration. |
| phpstan-baseline.neon | Adjusted baseline messages to reflect type validations and return type updates. |
| CHANGELOG.md | Updated changelog with new issue fix information. |
Comments suppressed due to low confidence (1)
src/JsonSchema/SchemaStorage.php:105
- [nitpick] The variable names '$parentId' and '$childId' are used to manage schema reference resolution; adding a brief comment or using more descriptive names may improve clarity for future maintainers.
if (property_exists($schema, 'id') && is_string($schema->id) && $childId !== $schema->id) {
|
Hello DannyvdSluijs, I encountered the error message Thank you! |
This PR will:
idproperties and register them if valid schemasidfor siblingrefpropertiesThis will fix #827