[ http://jira.magnolia.info/browse/MAGNOLIA-921?page=comments#action_12056 
] 

James Reynolds commented on MAGNOLIA-921:
-----------------------------------------

Sorry, I should have been more verbose.  I think this patch fixes the links.  
That is, when a user created link includes http:// (or any protocol), it isn't 
removed.  But with this patch, it is.

> samples/paragraphs/link.jsp index of :// is incorrect
> -----------------------------------------------------
>
>          Key: MAGNOLIA-921
>          URL: http://jira.magnolia.info/browse/MAGNOLIA-921
>      Project: magnolia
>         Type: Bug

>   Components: samples
>     Versions: 2.1.5
>     Reporter: James Reynolds
>     Assignee: Boris Kraft
>     Priority: Minor

>
> Original Estimate: 2 minutes
>         Remaining: 2 minutes
>
> This:
>                       if (html.indexOf("://")==-1) 
> html.append("http://"+link+"\"; target=\"_blank\">");
> Should be:
>                       if (link.indexOf("://")==-1) 
> html.append("http://"+link+"\"; target=\"_blank\">");
>                       else html.append(link+"\" target=\"_blank\">");

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.magnolia.info/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


----------------------------------------------------------------
for list details see
http://www.magnolia.info/en/magnolia/developer.html
----------------------------------------------------------------

Reply via email to