You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lenya.apache.org by an...@apache.org on 2008/03/07 00:04:44 UTC

svn commit: r634447 - in /lenya/sandbox/modules/externallinks: config/sitemap/transformers.xmap java/src/org/apache/lenya/modules/externallinks/ExternalLinksRewriter.java

Author: andreas
Date: Thu Mar  6 15:04:43 2008
New Revision: 634447

URL: http://svn.apache.org/viewvc?rev=634447&view=rev
Log:
Mark non-existing ext: links.

Modified:
    lenya/sandbox/modules/externallinks/config/sitemap/transformers.xmap
    lenya/sandbox/modules/externallinks/java/src/org/apache/lenya/modules/externallinks/ExternalLinksRewriter.java

Modified: lenya/sandbox/modules/externallinks/config/sitemap/transformers.xmap
URL: http://svn.apache.org/viewvc/lenya/sandbox/modules/externallinks/config/sitemap/transformers.xmap?rev=634447&r1=634446&r2=634447&view=diff
==============================================================================
--- lenya/sandbox/modules/externallinks/config/sitemap/transformers.xmap (original)
+++ lenya/sandbox/modules/externallinks/config/sitemap/transformers.xmap Thu Mar  6 15:04:43 2008
@@ -28,6 +28,7 @@
     <transform namespace="http://www.w3.org/1999/xhtml" element="link" attribute="href"/>
     <transform namespace="http://www.w3.org/1999/xhtml" element="form" attribute="action"/>
     <transform namespace="http://www.w3.org/1999/xhtml" element="script" attribute="src"/>
+    <markBrokenLinks attribute="class" value="brokenlink"/>
   </map:transformer>
 
 </xmap>

Modified: lenya/sandbox/modules/externallinks/java/src/org/apache/lenya/modules/externallinks/ExternalLinksRewriter.java
URL: http://svn.apache.org/viewvc/lenya/sandbox/modules/externallinks/java/src/org/apache/lenya/modules/externallinks/ExternalLinksRewriter.java?rev=634447&r1=634446&r2=634447&view=diff
==============================================================================
--- lenya/sandbox/modules/externallinks/java/src/org/apache/lenya/modules/externallinks/ExternalLinksRewriter.java (original)
+++ lenya/sandbox/modules/externallinks/java/src/org/apache/lenya/modules/externallinks/ExternalLinksRewriter.java Thu Mar  6 15:04:43 2008
@@ -36,7 +36,7 @@
     protected static final String PROTOCOL = "ext:";
 
     public boolean matches(String url) {
-        return url.startsWith(PROTOCOL) && this.links.contains(getLinkName(url));
+        return url.startsWith(PROTOCOL);
     }
 
     protected String getLinkName(String url) {
@@ -44,7 +44,8 @@
     }
 
     public String rewrite(String url) {
-        return this.links.getUri(getLinkName(url));
+        String linkName = getLinkName(url);
+        return this.links.contains(linkName) ? this.links.getUri(linkName) : null;
     }
 
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@lenya.apache.org
For additional commands, e-mail: commits-help@lenya.apache.org