[wp-trac] Re: [WordPress Trac] #890: support relative paths for
enclosures
WordPress Trac
wp-trac at lists.automattic.com
Fri Sep 29 13:41:42 GMT 2006
#890: support relative paths for enclosures
-----------------------------------+----------------------------------------
Reporter: anonymousbugger | Owner: markjaquith
Type: enhancement | Status: assigned
Priority: normal | Milestone: 2.1
Component: Administration | Version: 1.5
Severity: normal | Resolution:
Keywords: has-patch 2nd-opinion |
-----------------------------------+----------------------------------------
Comment (by zamoose):
The patch looks good except for one thing: both relative and absolute
server paths are caught by it and (incorrectly) routed to
$SERVER['HTTP_HOST'], n'est ce pas? The ticket submitter requested that
both relative (''audio/audio.mp3'') '''and''' absolute
(''/audio/audio.mp3'') work. Would not the current solution direct both
links to http://$SERVER_URL/audio/audio.mp3, which is fine for
''/audio/audio.mp3'' but which would be incorrect for anyone with a blog
URL in any place other than the root for ''audio/audio.mp3''. If there's
no leading slash, oughtn't we just use ''bloginfo('url')'' or similar?
--
Ticket URL: <http://trac.wordpress.org/ticket/890>
WordPress Trac <http://wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list