Don't cleanup response content for invalid json strings. #166 WIP

ufna · May 30, 2018 · e24ca58 · e24ca58
1 parent 4c19727
commit e24ca58
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/Source/VaRestPlugin/Private/VaRestRequestJSON.cpp b/Source/VaRestPlugin/Private/VaRestRequestJSON.cpp
@@ -546,8 +546,15 @@ bool UVaRestRequestJSON::HasTag(FName Tag) const
 
 FString UVaRestRequestJSON::GetResponseContentAsString(bool bCacheResponseContent)
 {
-	// Check we have valide response
-	if (!bIsValidJsonResponse || !ResponseJsonObj || !ResponseJsonObj->IsValidLowLevel())
+	// Check we have valid json response
+	if (!bIsValidJsonResponse)
+	{
+		// We've cached response content in OnProcessRequestComplete()
+		return ResponseContent;
+	}
+
+	// Check we have valid response object
+	if (!ResponseJsonObj || !ResponseJsonObj->IsValidLowLevel())
 	{
 		// Discard previous cached string if we had one
 		ResponseContent = DeprecatedResponseString;