You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@esme.apache.org by vd...@apache.org on 2010/09/02 21:53:47 UTC
svn commit: r992087 -
/incubator/esme/trunk/server/src/main/scala/org/apache/esme/model/Message.scala
Author: vdichev
Date: Thu Sep 2 19:53:47 2010
New Revision: 992087
URL: http://svn.apache.org/viewvc?rev=992087&view=rev
Log:
ESME-267 Links in messages in pools don't redirect through the shortened URL to disable popularity stats
Modified:
incubator/esme/trunk/server/src/main/scala/org/apache/esme/model/Message.scala
Modified: incubator/esme/trunk/server/src/main/scala/org/apache/esme/model/Message.scala
URL: http://svn.apache.org/viewvc/incubator/esme/trunk/server/src/main/scala/org/apache/esme/model/Message.scala?rev=992087&r1=992086&r2=992087&view=diff
==============================================================================
--- incubator/esme/trunk/server/src/main/scala/org/apache/esme/model/Message.scala (original)
+++ incubator/esme/trunk/server/src/main/scala/org/apache/esme/model/Message.scala Thu Sep 2 19:53:47 2010
@@ -392,9 +392,16 @@ class Message extends LongKeyedMapper[Me
case e: Elem if e.label == "url" =>
e.attribute("url").flatMap(url =>
- e.attribute("uniqueId").map(id =>
- <xml:group> <a class="tiplelement" href={"/u/"+id} target="_blank" title={url}>{url.toString.substring(0,20)}...</a> </xml:group>)).
- getOrElse(Text("") )
+ e.attribute("uniqueId").map { id =>
+ val href =
+ if (pool.defined_?)
+ // disable shortener to avoid popularity statistics
+ url.toString
+ else
+ "/u/" + id
+ <xml:group> <a class="tiplelement" href={href} target="_blank" title={url}>{url.toString.substring(0,20)}...</a> </xml:group>
+ }
+ ).getOrElse(Text("") )
case e: Elem if e.label == "em" =>
e.attribute("text").map(text =>