Merge pull request #129 from michaelborn/patch/handle-response-wrong-…

…argument-type

🐛 FIX: Correct wrong type error when deleteByID() tries to delegate error handling to onResponseFailure

box.json

@@ -2,7 +2,7 @@
     "name":"Elasticsearch for the Coldbox Framework",
     "author":"Ortus Solutions <[email protected]",
     "location":"https://downloads.ortussolutions.com/ortussolutions/coldbox-modules/cbelasticsearch/@build.version@/[email protected]@[email protected]@.zip",
-    "version":"3.1.4",
+    "version":"3.2.0",
     "slug":"cbelasticsearch",
     "type":"modules",
     "homepage":"https://cbelasticsearch.ortusbooks.com",

changelog.md

@@ -8,7 +8,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ----
 ## [Unreleased]
 
-## [3.1.3] - 06-08-2023
+## [3.2.1] - 06-20-2023
+### Fixed
+* Corrected a typing error when performing error handling in `deleteById()`
+
+## [3.2.0] - 06-08-2023
 ### Added
 * [Added `getTermVectors` to SearchBuilder and Client](https://cbelasticsearch.ortusbooks.com/searching/search#term-vectors) to allow for fetching term vectors on document field(s)
 

models/io/HyperClient.cfc

@@ -1031,10 +1031,11 @@ component accessors="true" threadSafe singleton {
 			deleteRequest.setQueryParam( param.name, param.value );
 		} );
 
-		var deleteResult = deleteRequest.send().json();
+		var response = deleteRequest.send();
+		var deleteResult = response.json();
 
 		if ( arguments.throwOnError && structKeyExists( deleteResult, "error" ) ) {
-			onResponseFailure( deleteResult );
+			onResponseFailure( response );
 		}
 
 		return deleteResult.keyExists( "error" ) ? false : deleteResult.result == "deleted";