[ 
https://issues.apache.org/jira/browse/DOXIASITETOOLS-150?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15136417#comment-15136417
 ] 

Hudson commented on DOXIASITETOOLS-150:
---------------------------------------

SUCCESS: Integrated in doxia-all #241 (See 
[https://builds.apache.org/job/doxia-all/241/])
[DOXIASITETOOLS-150] added isLink(href) method for use in skins: 
$decoration.isLink($href) (hboutemy: 
[http://svn.apache.org/viewvc/?view=rev&rev=1729027])
* 
./doxia-sitetools/doxia-decoration-model/src/main/java/org/apache/maven/doxia/site/decoration/DecorationUtils.java
* ./doxia-sitetools/doxia-decoration-model/src/main/mdo/decoration.mdo
* 
./doxia-sitetools/doxia-decoration-model/src/test/java/org/apache/maven/doxia/site/decoration/DecorationUtilsTest.java


> create a "isLink(href)" function for use in skins
> -------------------------------------------------
>
>                 Key: DOXIASITETOOLS-150
>                 URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-150
>             Project: Maven Doxia Sitetools
>          Issue Type: New Feature
>          Components: Decoration model
>    Affects Versions: 1.6
>            Reporter: Hervé Boutemy
>             Fix For: 1.7
>
>
> many skins copy/paste following code:
> {code} #if ( $href.toLowerCase().startsWith("http:/") || 
> $href.toLowerCase().startsWith("https:/") ||
>     $href.toLowerCase().startsWith("ftp:/") || 
> $href.toLowerCase().startsWith("mailto:";) ||
>     $href.toLowerCase().startsWith("file:/") || 
> ($href.toLowerCase().indexOf("://") != -1) ){code}
> putting this as a function available from skins will improve their maintenance
> one idea is to add this code as a method of decoration model, which is 
> already available in skins as {{$decoration}} variable: this would permit to 
> replace previous snippet with
> {code}#if $decoration.isLink($href){code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to