[wp-trac] [WordPress Trac] #57743: Add `$found` parameter to `wp_cache_get_multiple()`
WordPress Trac
noreply at wordpress.org
Sun Mar 19 23:35:03 UTC 2023
#57743: Add `$found` parameter to `wp_cache_get_multiple()`
----------------------------------------+-----------------------------
Reporter: peterwilsoncc | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: Future Release
Component: Cache API | Version:
Severity: normal | Resolution:
Keywords: needs-unit-tests has-patch | Focuses:
----------------------------------------+-----------------------------
Comment (by peterwilsoncc):
Thank you for creating the patch @thakkarhardik
On the line following the existing `@since` tag, a note will need to be
added to indicate the version in which the parameter was added. As this is
currently marked as for a future release, you can hold off on that for
now.
It would be good to add some unit tests for the new parameter, do you have
any experience with writing tests in WordPress core?
When adding tests, it's easiest if you create a pull request on the
[https://github.com/WordPress/wordpress-develop/ WordPress-Develop GitHub
repository] as the tests will run on your pull request.
If you don't have experience writing tests, you can create a pull request
with the patch as is and another contributor will be able to help out with
some suggestions.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/57743#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list