Merge pull request #55 from yonaichin/master

AleksandrRogov · web-flow · commit 1ab3bd495fdb · 2019-08-08T07:53:13.000-04:00
Avoid undefined error in batch operation
diff --git a/lib/requests/helpers/parseResponse.js b/lib/requests/helpers/parseResponse.js
@@ -148,7 +148,7 @@ function parseBatchResponse(response, parseParams, requestNumber) {
                     result.push(isNaN(plainContent) ? plainContent : parseInt(plainContent));
                 }
                 else
-                    if (parseParams.length && parseParams[requestNumber].hasOwnProperty('valueIfEmpty')) {
+                    if (parseParams.length && parseParams[requestNumber] && parseParams[requestNumber].hasOwnProperty('valueIfEmpty')) {
                         result.push(parseParams[requestNumber].valueIfEmpty);
                     }
                     else {
@@ -213,4 +213,4 @@ module.exports = function parseResponse(response, responseHeaders, parseParams)
     }
 
     return parseResult;
-};
+};

Original file line number	Diff line number	Diff line change
`@@ -148,7 +148,7 @@ function parseBatchResponse(response, parseParams, requestNumber) {`
`148`	`148`	`result.push(isNaN(plainContent) ? plainContent : parseInt(plainContent));`
`149`	`149`	`}`
`150`	`150`	`else`
`151`		`- if (parseParams.length && parseParams[requestNumber].hasOwnProperty('valueIfEmpty')) {`
	`151`	`+ if (parseParams.length && parseParams[requestNumber] && parseParams[requestNumber].hasOwnProperty('valueIfEmpty')) {`
`152`	`152`	`result.push(parseParams[requestNumber].valueIfEmpty);`
`153`	`153`	`}`
`154`	`154`	`else {`
`@@ -213,4 +213,4 @@ module.exports = function parseResponse(response, responseHeaders, parseParams)`
`213`	`213`	`}`
`214`	`214`
`215`	`215`	`return parseResult;`
`216`		`-};`
	`216`	`+};`