Make Markdown use l() for links

alienbrain - February 13, 2007 - 10:31
Project:Markdown with SmartyPants
Version:4.7.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:alienbrain
Status:won't fix
Description

Hello,
Markdown currently generates links manually without considering Drupal's base_path() which might lead to incorrect links, or headaches if you are moving to/from a subdirectory or a domain. This patch modifies Markdown to use l() which makes sure links will work all the time despite of the hosting environment.

Cheers,

AttachmentSize
markdown_correct_links.patch1.29 KB

#1

moshe weitzman - July 26, 2007 - 16:47
Status:needs review» won't fix

makes sense, but i really don't want to ship with a forked markdown.php. some use regular markdown, others use extra ... if people want this, they should apply this patch.

 
 

Drupal is a registered trademark of Dries Buytaert.