[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