Skip to content

Conversation

@simonfunk
Copy link

This PR upgrades the project to PHP 8.3, ensuring compatibility with the latest PHP features, performance improvements, and security patches. Below are the major changes and considerations included in this upgrade:

Updated the syntax and refactored parts of the codebase to leverage new features in PHP 8.3 (e.g., null-safe operators, readonly properties).
Replaced deprecated functions and features with their modern alternatives.

Simon Funk added 5 commits October 24, 2024 16:42
Updated the README and composer.json to require PHP 8.3 and later. Changed project author and name details in composer.json to reflect new ownership.
Introduced a new trait, DynamicPropertyTrait, to manage dynamic properties in entities. Updated DataMapper to utilize this trait for handling cases where entities have undeclared properties, and added logging for dynamic property interactions.
Removed unnecessary logging and methods related to dynamic property setting to streamline the code. Introduced a new method `instantiateClass` to handle class instantiation with necessary arguments for subclasses of `ResponseModel`.
@vadmou
Copy link
Collaborator

vadmou commented Nov 28, 2024

Thank you for your contribution! While we appreciate your effort, the proposed changes don’t fully align with our requirements. Unfortunately, your solution does not address the issue of using dynamic properties in Entity.

Additionally, we noticed a modification to the library’s authorship, which we believe might not be appropriate. We kindly ask you to avoid such changes in the future. The solution for handling dynamic properties has already been implemented in release v1.2.7.

We value your contributions and encourage you to review the release and provide any further suggestions!

@vadmou vadmou closed this Nov 28, 2024
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.

2 participants