stop url components being escaped too early and breaking the breadcrumb links

plugins/function.buildpath.php

@@ -2,7 +2,7 @@
 	function smarty_function_buildpath($params, &$smarty) {
 		if ($params['item']=="") return "/";
 		foreach ($params['list'] as $crumb) {
-			if ($crumb!="") $output.="/".$crumb;
+			if ($crumb!="") $output.="/".rawurlencode($crumb);
 			if ($crumb==$params['item']) return $output;
 		}
 

templates/breadcrumb.tpl

@@ -2,7 +2,7 @@
 {if $pathlist}
 				<ul>
 {foreach name=pathlist from=$pathlist key=key item=item}
-					<li>{if !$smarty.foreach.pathlist.first}{if $item!=""}&gt; {/if}{/if}{if !$smarty.foreach.pathlist.last}<a href="{buildpath list=$pathlist item=$item|escape:'url'}">{/if}{if $item}{$item|escape:'htmlall'}{else}{if $smarty.foreach.pathlist.first}SUCS{/if}{/if}{if !$smarty.foreach.pathlist.last}</a>{/if}</li>
+					<li>{if !$smarty.foreach.pathlist.first}{if $item!=""}&gt; {/if}{/if}{if !$smarty.foreach.pathlist.last}<a href="{buildpath list=$pathlist item=$item}">{/if}{if $item}{$item|escape:'htmlall'}{else}{if $smarty.foreach.pathlist.first}SUCS{/if}{/if}{if !$smarty.foreach.pathlist.last}</a>{/if}</li>
 {/foreach}
 				</ul>
 {/if}