Fixed: Respect long defaults in converters

dcodeIO · dcodeIO · commit 62eef58aa3b0 · 2017-01-04T12:34:41.000+01:00
diff --git a/src/converter.js b/src/converter.js
@@ -21,7 +21,7 @@ function genConvert(field, fieldIndex, prop) {
         case "fixed64":
         case "sfixed64":
             // longs
-            return sprintf("f.longs(s%s,%d,%d,%j,o)", prop, 0, 0, field.type.charAt(0) === "u");
+            return sprintf("f.longs(s%s,%d,%d,%j,o)", prop, field.typeDefault.low, field.typeDefault.high, field.type.charAt(0) === "u");
         case "bytes":
             // bytes
             return sprintf("f.bytes(s%s,%j,o)", prop, Array.prototype.slice.call(field.typeDefault));
diff --git a/tests/data/ambiguous-names.js b/tests/data/ambiguous-names.js
@@ -134,7 +134,7 @@ $root.A = (function() {
         if (!options) {
             options = {};
         }
-        var dst = impl.create(src, this, options) || null;
+        var dst = impl.create(src, this, options);
         if (dst) {
             if (dst.whatever === undefined && options.defaults) {
                 dst.whatever = "";
@@ -294,9 +294,9 @@ $root.B = (function() {
         if (!options) {
             options = {};
         }
-        var dst = impl.create(src, this, options) || null;
+        var dst = impl.create(src, this, options);
         if (dst) {
-            if (!(src.A === undefined || src.A === null) || options.defaults) {
+            if (src.A !== undefined && src.A !== null || options.defaults) {
                 dst.A = types[0].convert(src.A, impl, options);
             }
         }
diff --git a/tests/data/mapbox/vector_tile.js b/tests/data/mapbox/vector_tile.js
@@ -157,7 +157,7 @@ $root.vector_tile = (function() {
             if (!options) {
                 options = {};
             }
-            var dst = impl.create(src, this, options) || null;
+            var dst = impl.create(src, this, options);
             if (dst) {
                 if (src.layers && src.layers.length) {
                     dst.layers = [];
@@ -445,7 +445,7 @@ $root.vector_tile = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.stringValue === undefined && options.defaults) {
                         dst.stringValue = "";
@@ -456,13 +456,13 @@ $root.vector_tile = (function() {
                     if (dst.doubleValue === undefined && options.defaults) {
                         dst.doubleValue = 0;
                     }
-                    if (!(src.intValue === undefined || src.intValue === null) || options.defaults) {
+                    if (src.intValue !== undefined && src.intValue !== null || options.defaults) {
                         dst.intValue = impl.longs(src.intValue, 0, 0, false, options);
                     }
-                    if (!(src.uintValue === undefined || src.uintValue === null) || options.defaults) {
+                    if (src.uintValue !== undefined && src.uintValue !== null || options.defaults) {
                         dst.uintValue = impl.longs(src.uintValue, 0, 0, true, options);
                     }
-                    if (!(src.sintValue === undefined || src.sintValue === null) || options.defaults) {
+                    if (src.sintValue !== undefined && src.sintValue !== null || options.defaults) {
                         dst.sintValue = impl.longs(src.sintValue, 0, 0, false, options);
                     }
                     if (dst.boolValue === undefined && options.defaults) {
@@ -721,9 +721,9 @@ $root.vector_tile = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
-                    if (!(src.id === undefined || src.id === null) || options.defaults) {
+                    if (src.id !== undefined && src.id !== null || options.defaults) {
                         dst.id = impl.longs(src.id, 0, 0, true, options);
                     }
                     if (src.tags && src.tags.length) {
@@ -736,7 +736,7 @@ $root.vector_tile = (function() {
                             dst.tags = [];
                         }
                     }
-                    if (!(src.type === undefined || src.type === null) || options.defaults) {
+                    if (src.type !== undefined || options.defaults) {
                         dst.type = impl.enums(src.type, undefined, types[2], options);
                     }
                     if (src.geometry && src.geometry.length) {
@@ -1017,7 +1017,7 @@ $root.vector_tile = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.version === undefined && options.defaults) {
                         dst.version = 1;
diff --git a/tests/data/package.js b/tests/data/package.js
@@ -503,7 +503,7 @@ $root.Package = (function() {
         if (!options) {
             options = {};
         }
-        var dst = impl.create(src, this, options) || null;
+        var dst = impl.create(src, this, options);
         if (dst) {
             if (dst.name === undefined && options.defaults) {
                 dst.name = "";
@@ -520,7 +520,7 @@ $root.Package = (function() {
             if (dst.license === undefined && options.defaults) {
                 dst.license = "";
             }
-            if (!(src.repository === undefined || src.repository === null) || options.defaults) {
+            if (src.repository !== undefined && src.repository !== null || options.defaults) {
                 dst.repository = types[5].convert(src.repository, impl, options);
             }
             if (dst.bugs === undefined && options.defaults) {
@@ -726,7 +726,7 @@ $root.Package = (function() {
             if (!options) {
                 options = {};
             }
-            var dst = impl.create(src, this, options) || null;
+            var dst = impl.create(src, this, options);
             if (dst) {
                 if (dst.type === undefined && options.defaults) {
                     dst.type = "";
diff --git a/tests/data/rpc.js b/tests/data/rpc.js
@@ -226,7 +226,7 @@ $root.MyRequest = (function() {
         if (!options) {
             options = {};
         }
-        var dst = impl.create(src, this, options) || null;
+        var dst = impl.create(src, this, options);
         if (dst) {
             if (dst.path === undefined && options.defaults) {
                 dst.path = "";
@@ -382,7 +382,7 @@ $root.MyResponse = (function() {
         if (!options) {
             options = {};
         }
-        var dst = impl.create(src, this, options) || null;
+        var dst = impl.create(src, this, options);
         if (dst) {
             if (dst.status === undefined && options.defaults) {
                 dst.status = 0;
diff --git a/tests/data/test.js b/tests/data/test.js

Original file line number	Diff line number	Diff line change
`@@ -134,7 +134,7 @@ $root.A = (function() {`
`134`	`134`	`if (!options) {`
`135`	`135`	`options = {};`
`136`	`136`	`}`
`137`		`- var dst = impl.create(src, this, options) \|\| null;`
	`137`	`+ var dst = impl.create(src, this, options);`
`138`	`138`	`if (dst) {`
`139`	`139`	`if (dst.whatever === undefined && options.defaults) {`
`140`	`140`	`dst.whatever = "";`
`@@ -294,9 +294,9 @@ $root.B = (function() {`
`294`	`294`	`if (!options) {`
`295`	`295`	`options = {};`
`296`	`296`	`}`
`297`		`- var dst = impl.create(src, this, options) \|\| null;`
	`297`	`+ var dst = impl.create(src, this, options);`
`298`	`298`	`if (dst) {`
`299`		`- if (!(src.A === undefined \|\| src.A === null) \|\| options.defaults) {`
	`299`	`+ if (src.A !== undefined && src.A !== null \|\| options.defaults) {`
`300`	`300`	`dst.A = types[0].convert(src.A, impl, options);`
`301`	`301`	`}`
`302`	`302`	`}`
Original file line number	Diff line number	Diff line change
`@@ -157,7 +157,7 @@ $root.vector_tile = (function() {`
`157`	`157`	`if (!options) {`
`158`	`158`	`options = {};`
`159`	`159`	`}`
`160`		`- var dst = impl.create(src, this, options) \|\| null;`
	`160`	`+ var dst = impl.create(src, this, options);`
`161`	`161`	`if (dst) {`
`162`	`162`	`if (src.layers && src.layers.length) {`
`163`	`163`	`dst.layers = [];`
`@@ -445,7 +445,7 @@ $root.vector_tile = (function() {`
`445`	`445`	`if (!options) {`
`446`	`446`	`options = {};`
`447`	`447`	`}`
`448`		`- var dst = impl.create(src, this, options) \|\| null;`
	`448`	`+ var dst = impl.create(src, this, options);`
`449`	`449`	`if (dst) {`
`450`	`450`	`if (dst.stringValue === undefined && options.defaults) {`
`451`	`451`	`dst.stringValue = "";`
`@@ -456,13 +456,13 @@ $root.vector_tile = (function() {`
`456`	`456`	`if (dst.doubleValue === undefined && options.defaults) {`
`457`	`457`	`dst.doubleValue = 0;`
`458`	`458`	`}`
`459`		`- if (!(src.intValue === undefined \|\| src.intValue === null) \|\| options.defaults) {`
	`459`	`+ if (src.intValue !== undefined && src.intValue !== null \|\| options.defaults) {`
`460`	`460`	`dst.intValue = impl.longs(src.intValue, 0, 0, false, options);`
`461`	`461`	`}`
`462`		`- if (!(src.uintValue === undefined \|\| src.uintValue === null) \|\| options.defaults) {`
	`462`	`+ if (src.uintValue !== undefined && src.uintValue !== null \|\| options.defaults) {`
`463`	`463`	`dst.uintValue = impl.longs(src.uintValue, 0, 0, true, options);`
`464`	`464`	`}`
`465`		`- if (!(src.sintValue === undefined \|\| src.sintValue === null) \|\| options.defaults) {`
	`465`	`+ if (src.sintValue !== undefined && src.sintValue !== null \|\| options.defaults) {`
`466`	`466`	`dst.sintValue = impl.longs(src.sintValue, 0, 0, false, options);`
`467`	`467`	`}`
`468`	`468`	`if (dst.boolValue === undefined && options.defaults) {`
`@@ -721,9 +721,9 @@ $root.vector_tile = (function() {`
`721`	`721`	`if (!options) {`
`722`	`722`	`options = {};`
`723`	`723`	`}`
`724`		`- var dst = impl.create(src, this, options) \|\| null;`
	`724`	`+ var dst = impl.create(src, this, options);`
`725`	`725`	`if (dst) {`
`726`		`- if (!(src.id === undefined \|\| src.id === null) \|\| options.defaults) {`
	`726`	`+ if (src.id !== undefined && src.id !== null \|\| options.defaults) {`
`727`	`727`	`dst.id = impl.longs(src.id, 0, 0, true, options);`
`728`	`728`	`}`
`729`	`729`	`if (src.tags && src.tags.length) {`
`@@ -736,7 +736,7 @@ $root.vector_tile = (function() {`
`736`	`736`	`dst.tags = [];`
`737`	`737`	`}`
`738`	`738`	`}`
`739`		`- if (!(src.type === undefined \|\| src.type === null) \|\| options.defaults) {`
	`739`	`+ if (src.type !== undefined \|\| options.defaults) {`
`740`	`740`	`dst.type = impl.enums(src.type, undefined, types[2], options);`
`741`	`741`	`}`
`742`	`742`	`if (src.geometry && src.geometry.length) {`
`@@ -1017,7 +1017,7 @@ $root.vector_tile = (function() {`
`1017`	`1017`	`if (!options) {`
`1018`	`1018`	`options = {};`
`1019`	`1019`	`}`
`1020`		`- var dst = impl.create(src, this, options) \|\| null;`
	`1020`	`+ var dst = impl.create(src, this, options);`
`1021`	`1021`	`if (dst) {`
`1022`	`1022`	`if (dst.version === undefined && options.defaults) {`
`1023`	`1023`	`dst.version = 1;`