[wp-trac] [WordPress Trac] #23643: Redundant use of esc_url() in wp-admin/includes/export.php

[WordPress Trac]

#23643: Redundant use of esc_url() in wp-admin/includes/export.php
------------------------------+------------------
 Reporter:  pauldewouters     |       Owner:
     Type:  defect (bug)      |      Status:  new
 Priority:  normal            |   Milestone:  3.6
Component:  Export            |     Version:  3.0
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |
------------------------------+------------------
Changes (by SergeyBiryukov):

 * keywords:  dev-feedback has-patch => has-patch commit
 * version:   => 3.0
 * milestone:  Awaiting Review => 3.6


Old description:

> In ticket [https://core.trac.wordpress.org/ticket/13555 13555], esc_url()
> was moved into the_guid() function. See:
>
> [https://core.trac.wordpress.org/changeset/14949]
>
> but it wasn't removed from this line
> [https://core.trac.wordpress.org/browser/trunk/wp-
> admin/includes/export.php#L374]
>
> I'd like to submit a fix to remove esc_url() as it is already called
> inside the_guid()

New description:

 In ticket #13555, esc_url() was moved into the_guid() function. See:
 [14949]

 but it wasn't removed from this line
 [http://core.trac.wordpress.org/browser/tags/3.5.1/wp-
 admin/includes/export.php#L374]

 I'd like to submit a fix to remove esc_url() as it is already called
 inside the_guid()

--

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/23643#comment:2>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software