add HttpRequestMessage to DeleteRecord handler (#107)

csantero · csantero · commit 9e305ecff354 · 2016-04-12T19:16:35.000-04:00
diff --git a/JSONAPI.AcceptanceTests.EntityFrameworkTestWebApp/DocumentMaterializers/StarshipDocumentMaterializer.cs b/JSONAPI.AcceptanceTests.EntityFrameworkTestWebApp/DocumentMaterializers/StarshipDocumentMaterializer.cs
@@ -62,7 +62,7 @@ public override Task<ISingleResourceDocument> UpdateRecord(string id, ISingleRes
             throw new NotImplementedException();
         }
 
-        public override Task<IJsonApiDocument> DeleteRecord(string id, CancellationToken cancellationToken)
+        public override Task<IJsonApiDocument> DeleteRecord(string id, HttpRequestMessage request, CancellationToken cancellationToken)
         {
             throw new NotImplementedException();
         }
diff --git a/JSONAPI.EntityFramework/Http/EntityFrameworkDocumentMaterializer.cs b/JSONAPI.EntityFramework/Http/EntityFrameworkDocumentMaterializer.cs
@@ -86,7 +86,7 @@ public virtual async Task<ISingleResourceDocument> UpdateRecord(string id, ISing
             return returnDocument;
         }
 
-        public virtual async Task<IJsonApiDocument> DeleteRecord(string id, CancellationToken cancellationToken)
+        public virtual async Task<IJsonApiDocument> DeleteRecord(string id, HttpRequestMessage request, CancellationToken cancellationToken)
         {
             var singleResource = await _dbContext.Set<T>().FindAsync(cancellationToken, id);
             _dbContext.Set<T>().Remove(singleResource);
diff --git a/JSONAPI/Http/IDocumentMaterializer.cs b/JSONAPI/Http/IDocumentMaterializer.cs
@@ -39,6 +39,6 @@ Task<ISingleResourceDocument> UpdateRecord(string id, ISingleResourceDocument re
         /// <summary>
         /// Deletes the record corresponding to the given id.
         /// </summary>
-        Task<IJsonApiDocument> DeleteRecord(string id, CancellationToken cancellationToken);
+        Task<IJsonApiDocument> DeleteRecord(string id, HttpRequestMessage request, CancellationToken cancellationToken);
     }
 }
diff --git a/JSONAPI/Http/JsonApiController.cs b/JSONAPI/Http/JsonApiController.cs
@@ -78,7 +78,7 @@ public virtual async Task<IHttpActionResult> Patch(string resourceType, string i
         public virtual async Task<IHttpActionResult> Delete(string resourceType, string id, CancellationToken cancellationToken)
         {
             var materializer = _documentMaterializerLocator.GetMaterializerByResourceTypeName(resourceType);
-            var document = await materializer.DeleteRecord(id, cancellationToken);
+            var document = await materializer.DeleteRecord(id, Request, cancellationToken);
             return Ok(document);
         }
     }
diff --git a/JSONAPI/Http/MappedDocumentMaterializer.cs b/JSONAPI/Http/MappedDocumentMaterializer.cs
@@ -102,8 +102,8 @@ public abstract Task<ISingleResourceDocument> UpdateRecord(string id, ISingleRes
             HttpRequestMessage request,
             CancellationToken cancellationToken);
 
-        public abstract Task<IJsonApiDocument> DeleteRecord(string id, CancellationToken cancellationToken);
-
+        public abstract Task<IJsonApiDocument> DeleteRecord(string id, HttpRequestMessage request, CancellationToken cancellationToken);
+        
         /// <summary>
         /// Returns a list of property paths to be included when constructing a query for this resource type
         /// </summary>

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ public override Task<ISingleResourceDocument> UpdateRecord(string id, ISingleRes`
`62`	`62`	`throw new NotImplementedException();`
`63`	`63`	`}`
`64`	`64`
`65`		`- public override Task<IJsonApiDocument> DeleteRecord(string id, CancellationToken cancellationToken)`
	`65`	`+ public override Task<IJsonApiDocument> DeleteRecord(string id, HttpRequestMessage request, CancellationToken cancellationToken)`
`66`	`66`	`{`
`67`	`67`	`throw new NotImplementedException();`
`68`	`68`	`}`
Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ public virtual async Task<ISingleResourceDocument> UpdateRecord(string id, ISing`
`86`	`86`	`return returnDocument;`
`87`	`87`	`}`
`88`	`88`
`89`		`- public virtual async Task<IJsonApiDocument> DeleteRecord(string id, CancellationToken cancellationToken)`
	`89`	`+ public virtual async Task<IJsonApiDocument> DeleteRecord(string id, HttpRequestMessage request, CancellationToken cancellationToken)`
`90`	`90`	`{`
`91`	`91`	`var singleResource = await _dbContext.Set<T>().FindAsync(cancellationToken, id);`
`92`	`92`	`_dbContext.Set<T>().Remove(singleResource);`
Original file line number	Diff line number	Diff line change
`@@ -39,6 +39,6 @@ Task<ISingleResourceDocument> UpdateRecord(string id, ISingleResourceDocument re`
`39`	`39`	`/// <summary>`
`40`	`40`	`/// Deletes the record corresponding to the given id.`
`41`	`41`	`/// </summary>`
`42`		`- Task<IJsonApiDocument> DeleteRecord(string id, CancellationToken cancellationToken);`
	`42`	`+ Task<IJsonApiDocument> DeleteRecord(string id, HttpRequestMessage request, CancellationToken cancellationToken);`
`43`	`43`	`}`
`44`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ public virtual async Task<IHttpActionResult> Patch(string resourceType, string i`
`78`	`78`	`public virtual async Task<IHttpActionResult> Delete(string resourceType, string id, CancellationToken cancellationToken)`
`79`	`79`	`{`
`80`	`80`	`var materializer = _documentMaterializerLocator.GetMaterializerByResourceTypeName(resourceType);`
`81`		`- var document = await materializer.DeleteRecord(id, cancellationToken);`
	`81`	`+ var document = await materializer.DeleteRecord(id, Request, cancellationToken);`
`82`	`82`	`return Ok(document);`
`83`	`83`	`}`
`84`	`84`	`}`