You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@abdera.apache.org by jm...@apache.org on 2011/11/17 23:16:56 UTC
svn commit: r1203393 -
/abdera/abdera2/common/src/main/java/org/apache/abdera2/common/templates/Operation.java
Author: jmsnell
Date: Thu Nov 17 22:16:56 2011
New Revision: 1203393
URL: http://svn.apache.org/viewvc?rev=1203393&view=rev
Log:
bug fix in the expansion.. wasn't handling nulls properly
Modified:
abdera/abdera2/common/src/main/java/org/apache/abdera2/common/templates/Operation.java
Modified: abdera/abdera2/common/src/main/java/org/apache/abdera2/common/templates/Operation.java
URL: http://svn.apache.org/viewvc/abdera/abdera2/common/src/main/java/org/apache/abdera2/common/templates/Operation.java?rev=1203393&r1=1203392&r2=1203393&view=diff
==============================================================================
--- abdera/abdera2/common/src/main/java/org/apache/abdera2/common/templates/Operation.java (original)
+++ abdera/abdera2/common/src/main/java/org/apache/abdera2/common/templates/Operation.java Thu Nov 17 22:16:56 2011
@@ -445,7 +445,7 @@ public abstract class Operation implemen
for (VarSpec varspec : exp) {
String val = eval(varspec, context, false, "&", ""); // Per Draft Seven (http://tools.ietf.org/html/draft-gregorio-uritemplate-07)
if (context.contains(varspec.getName())) {
- if (!first) buf.append('&');
+ if (!first && val != null) buf.append('&');
if ((val != null && !varspec.isExplode()) || varspec.isNoval()) {
buf.append(varspec.getName());
}