[wp-trac] [WordPress Trac] #39193: Fix two hook name to meet the naming convention for Dynamic-Hooks

WordPress Trac noreply at wordpress.org
Fri Dec 9 00:38:16 UTC 2016


#39193: Fix two hook name to meet the naming convention for Dynamic-Hooks
-------------------------------------+------------------------------
 Reporter:  ramiy                    |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  Awaiting Review
Component:  General                  |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-refresh  |     Focuses:  docs
-------------------------------------+------------------------------
Changes (by jnylen0):

 * keywords:  has-patch 2nd-opinion => has-patch needs-refresh


Comment:

 The proposed patches here will not work as intended:

 {{{
 $ wp shell
 wp> function a() {}
 wp> "some_text_{a()}"
 => string(15) "some_text_{a()}"
 }}}

 String interpolations inside curly braces must start with `{$`.  See
 http://php.net/manual/en/language.types.string.php#language.types.string.parsing

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


More information about the wp-trac mailing list