coderabbitai[bot]
commented
4 months ago Walkthrough
The changes involve refactoring several classes within the project to remove the AllowDynamicProperties attribute and improve instance management through the use of centralized container properties. This enhances code organization and maintainability by encapsulating instances within a single property, making the structure more modular and easier to extend.
Changes
| Files | Change Summary |
|---|---|
Lib/invisible_recaptcha.php |
Removed #[AllowDynamicProperties] attribute and adjusted class formatting. |
admin/class-admin-settings.php |
Removed AllowDynamicProperties import and attribute from WPUF_Admin_Settings class. |
includes/Admin.php |
Refactored Admin class to use a container for instance management, replacing individual properties. |
includes/Admin/Forms/... |
Removed AllowDynamicProperties and updated form builder management in Admin_Form and Form classes. |
includes/Admin/Forms/...Templates/ |
Added checks for existence of variables and removed AllowDynamicProperties in Post_Form_Template_WooCommerce and Pro_Form_Preview_EDD. |
includes/Ajax/Frontend_Form_Ajax.php |
Removed fully-qualified AllowDynamicProperties usage. |
includes/Fields/Form_Field_Post_Taxonomy.php |
Removed #[AllowDynamicProperties] attribute and import statement. |
includes/Frontend.php |
Refactored to use a container for instance management instead of individual properties. |
includes/Pro_Upgrades.php |
Updated register_pro_fields method to use the container array for managing pro_fields. |
includes/Setup_Wizard.php |
Added custom_admin_bar_styles method and removed emoji styles from setup_wizard. |
languages/wp-user-frontend.pot |
Updated various strings, labels, and messages in the translation file. |
package.json |
Incremented version number from "4.0.8" to "4.0.9". |
wpuf.php |
Introduced $container property in WP_User_Frontend to hold class instances, refactoring the instantiation process. |
Sequence Diagram(s)
sequenceDiagram
participant User
participant WP_User_Frontend
participant Admin
participant Ajax
participant Component
User->>WP_User_Frontend: Initialize
WP_User_Frontend->>Admin: Instantiate if is_admin()
Admin-->>WP_User_Frontend: Instance stored in container
WP_User_Frontend->>Ajax: Instantiate if DOING_AJAX
Ajax-->>WP_User_Frontend: Instance stored in container
WP_User_Frontend->>Component: Instantiate various components
Component-->>WP_User_Frontend: Instances stored in container
User->>WP_User_Frontend: Access instance from container
WP_User_Frontend->>User: Return instancePoem
In the code where instances used to roam, We found a better, organized home. No more dynamic properties to see, Just neat containers, filled with glee. Version bumped, to four point nine, Our plugin shines, in organized design. 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Some of our plugin pages were showing PHP AllowDynamicProperty warning for PHP 8.* users
Summary by CodeRabbit
New Features
WP_User_Frontendclass, improving code organization and extensibility.Bug Fixes
Enhancements
Maintenance
AllowDynamicPropertiesattribute from multiple classes for better code compliance.