Closed mrabbani closed 2 weeks ago
The changes involve a modification to the __get
magic method in the WeDevs_Dokan
class found in dokan-class.php
. The updated logic alters the order of property retrieval by first checking the legacy_container
array for the requested property before checking the main container. This adjustment may affect how properties are accessed and returned within the class.
File | Change Summary |
---|---|
dokan-class.php | Modified the logic in the __get method to first check legacy_container for properties before the main container. |
WeDevs_Dokan
class in dokan-class.php
, which may interact with the __get
magic method, particularly in how properties are accessed and returned.In the code where rabbits hop,
Properties now take a new stop.
From legacy to the main,
They dance in a logical chain.
With a flick of a tail, they say,
"Access made bright, hip-hip-hooray!" 🐇✨
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?
All Submissions:
Changes proposed in this Pull Request:
Related Pull Request(s)
Closes
How to test the changes in this Pull Request:
Changelog entry
Prioritize Legacy Container
If someone override the service through
dokan_get_class_container
filter then the legacy container service will be used instead of Container Service.Before Changes
Describe the issue before changes with screenshots(s).
After Changes
Describe the issue after changes with screenshot(s).
Feature Video (optional)
Link of detailed video if this PR is for a feature.
PR Self Review Checklist:
FOR PR REVIEWER ONLY:
Summary by CodeRabbit
These changes may lead to more reliable access to properties within the application, ensuring a smoother user experience.