[wp-trac] [WordPress Trac] #26869: Hook Docs completed files fixes

WordPress Trac noreply at wordpress.org
Fri Apr 25 11:35:05 UTC 2014


#26869: Hook Docs completed files fixes
----------------------------+-----------------------------
 Reporter:  DrewAPicture    |       Owner:
     Type:  task (blessed)  |      Status:  new
 Priority:  normal          |   Milestone:  Future Release
Component:  General         |     Version:
 Severity:  normal          |  Resolution:
 Keywords:                  |     Focuses:  docs
----------------------------+-----------------------------

Comment (by GaryJ):

 Replying to [comment:23 DrewAPicture]:

 > * [attachment:admin_post_actions.diff] is a proposed attempt to handle
 the dynamic hook currently designated as `$action`

 You've changed behaviour there I think. The old code allowed hooks of:

 * `action_post` (logged in, request action empty)
 * `action_post_nopriv` (not logged in, request action empty)
 * `action_post_{$action}` (logged in, request action populated)
 * `action_post_nopriv_{$action}` (not logged in, request action populated)

 The new code appears to allow (re-ordered to match above):

 * `admin_post_` (logged in, request action empty - '''different''')
 * `action_post_nopriv_` (not logged in, request action empty -
 '''different''')
 * `action_post_{$action}` (logged in, request action populated)
 * `action_post_nopriv_{$action}` (not logged in, request action populated)

 Now I've written it out, it looks like it's just the underscore before an
 empty ${action} that needs to be handled better.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/26869#comment:24>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list