[wp-trac] [WordPress Trac] #30799: Scrutinizer in 4.2

WordPress Trac noreply at wordpress.org
Sat Jan 10 21:20:03 UTC 2015


#30799: Scrutinizer in 4.2
----------------------------+--------------------
 Reporter:  wonderboymusic  |       Owner:
     Type:  defect (bug)    |      Status:  new
 Priority:  normal          |   Milestone:  4.2
Component:  General         |     Version:  trunk
 Severity:  normal          |  Resolution:
 Keywords:                  |     Focuses:
----------------------------+--------------------

Comment (by wonderboymusic):

 In [changeset:"31131"]:
 {{{
 #!CommitTicketReference repository="" revision="31131"
 In `edit-form-advanced.php`:

 * `get_permalink( $post_ID )` can return `false`, set it to a variable and
 check it
 * Using the variable allows us to replace 11 separate calls to
 `get_permalink( $post_ID )` in the file
 * These notices were triggered by the potential for `false` to be passed
 to `esc_url()`

 See #30799.
 }}}

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


More information about the wp-trac mailing list