Two newlines before the URL breaks linking even in the default template where this value is used as the url parameter.

scottwalters · scottwalters · commit e2f95778ce48 · 2011-09-08T14:33:59.000-04:00
The newlines get changed to &lt;br /&gt;'s and that stuffed into the param of an href.  A URL really should just be a URL.
Or a URI.  Or one of those things.
diff --git a/lib/WebGUI/Workflow/Activity/NotifyAboutVersionTag.pm b/lib/WebGUI/Workflow/Activity/NotifyAboutVersionTag.pm
@@ -101,7 +101,7 @@ sub execute {
 	if ($versionTag->getAssetCount) {
 		# if there's only one asset in the tag, we might as well give them a direct link to it
 		my $asset = $versionTag->getAssets->[0];	
-		$urlOfSingleAsset = "\n\n".$self->session->url->getSiteURL().$asset->getUrl("func=view;revision=".$asset->get("revisionDate"));
+		$urlOfSingleAsset = $self->session->url->getSiteURL().$asset->getUrl("func=view;revision=".$asset->get("revisionDate"));
 	}
     my $var = {
         message  => $self->get('message'),

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ sub execute {`
`101`	`101`	`if ($versionTag->getAssetCount) {`
`102`	`102`	`# if there's only one asset in the tag, we might as well give them a direct link to it`
`103`	`103`	`my $asset = $versionTag->getAssets->[0];`
`104`		`- $urlOfSingleAsset = "\n\n".$self->session->url->getSiteURL().$asset->getUrl("func=view;revision=".$asset->get("revisionDate"));`
	`104`	`+ $urlOfSingleAsset = $self->session->url->getSiteURL().$asset->getUrl("func=view;revision=".$asset->get("revisionDate"));`
`105`	`105`	`}`
`106`	`106`	`my $var = {`
`107`	`107`	`message => $self->get('message'),`