Skip to content

add HttpRequestMessage to DeleteRecord handler #107

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
csantero merged 1 commit into from
Apr 12, 2016
Merged

add HttpRequestMessage to DeleteRecord handler #107

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ...anceTests.EntityFrameworkTestWebApp/DocumentMaterializers/StarshipDocumentMaterializer.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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();
}
Expand Down
2 changes: 1 addition & 1 deletion JSONAPI.EntityFramework/Http/EntityFrameworkDocumentMaterializer.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down
2 changes: 1 addition & 1 deletion JSONAPI/Http/IDocumentMaterializer.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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);
}
}
2 changes: 1 addition & 1 deletion JSONAPI/Http/JsonApiController.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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);
}
}
Expand Down
4 changes: 2 additions & 2 deletions JSONAPI/Http/MappedDocumentMaterializer.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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>
Expand Down