Hook Docs for lib/endpoints/class-wp-rest-users-controller.php#1556
Conversation
* Add some missing filter docs * Spacing, punctuation cleanup as per standard * Change second action from `rest_insert_user` to `rest_update_user` as its in the update function * added some todo notes, I don’t understand what the final false variable is for, maybe its needed?
There was a problem hiding this comment.
@adamsilverstein The third param here should be true on "Create" (it should be false on update).
There was a problem hiding this comment.
Thanks for clarifying, I will add that info to the docblock
|
Note: I believe this also corrects an issue with the first call of the Please confirm this is correct before merging :) |
|
Working on lib/endpoints/class-wp-rest-terms-controller.php now and wondering why I don't see similar do_action( 'rest_insert_term' ...) hooks firing. Is this intentional, or an oversight? If the latter I can go ahead and add them as I go :) |
@adamsilverstein Just an oversight. |
.gitignore
Outdated
There was a problem hiding this comment.
@adamsilverstein can you undo this change to the .gitignore?
There was a problem hiding this comment.
Ugh, should be in my local exclude sorry. kind of switching back to sublime anyway :)
|
@rachelbaker This should be good to go unless I missed some feedback, I'm addressing adding missing hooks in #1571 |
…t-users-controller-php Hook Docs for lib/endpoints/class-wp-rest-users-controller.php
|
✨ |
3 participants
See #1549.
rest_insert_usertorest_update_userasits in the update function
variable is for, maybe its needed?