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

[email protected]
Newsgroups gmane.comp.cms.lenya.cvs
Message-ID <[email protected]>
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;
     }
 
 }
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.