Merge pull request #8 from apiaryio/master

do not add separator after last var

Merge pull request #8 from apiaryio/master
do not add separator after last var
80642db3 · Franz X Antesberger · 901b8520 · 47bb0737 · 80642db3 · 80642db3
Commit 80642db3 authored Jan 19, 2013 by Franz X Antesberger
Show whitespace changes
Inline Side-by-side

Showing with 6 additions and 6 deletions

bin/uritemplate.js bin/uritemplate.js +3 -3

src/VariableExpression.js src/VariableExpression.js +3 -3

No files found.
--- a/bin/uritemplate.js
+++ b/bin/uritemplate.js
@@ -525,9 +525,9 @@ var VariableExpression = (function () {
        for (index = 0; index < this.varspecs.length; index += 1) {
            varspec = this.varspecs[index];
            value = variables[varspec.varname];
-            // if (!isDefined(value)) {
-            //     continue;
-            // }
+            if (!isDefined(value)) {
+                continue;
+            }
            if (isFirstVarspec) {
                result += this.operator.first;
                isFirstVarspec = false;

--- a/src/VariableExpression.js
+++ b/src/VariableExpression.js
@@ -70,9 +70,9 @@ var VariableExpression = (function () {
        for (index = 0; index < this.varspecs.length; index += 1) {
            varspec = this.varspecs[index];
            value = variables[varspec.varname];
-            // if (!isDefined(value)) {
-            //     continue;
-            // }
+            if (!isDefined(value)) {
+                 continue;
+            }
            if (isFirstVarspec) {
                result += this.operator.first;
                isFirstVarspec = false;