Skip to content

Merge 0-4-0 into master #91

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 16 commits into from
May 19, 2015
Merged

Merge 0-4-0 into master #91

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
033df32
Merge pull request #92 from csantero/filter
csantero Mar 15, 2015
1be6db3
allow specifying model manager to fluent config
Mar 15, 2015
0a26101
add model manager method to determine whether type is registered
Mar 16, 2015
44fa79b
extract key-name determination facility
Mar 19, 2015
0f5f9cd
make DbContextExtensions public
Mar 20, 2015
d60024e
rethrow HttpResponseException
Mar 20, 2015
6f2e16b
return more useful exception message for pagination errors
Apr 13, 2015
775330c
use related key instead of href
Apr 14, 2015
740c0ce
Properly serialize null values
Apr 14, 2015
a9f8c53
support deserialization of linkage objects
Apr 16, 2015
9fbe305
search parent classes for object sets
Apr 16, 2015
7b01416
ensure materialized updates create empty collection property values
Apr 16, 2015
94608cd
add some tests for EF materializer
Apr 29, 2015
30929b5
make Merge protected
May 6, 2015
0038034
make metadata manager protected
May 6, 2015
b73859e
Simplify IMaterializer
May 6, 2015
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
Prev Previous commit
Next Next commit
make metadata manager protected
  • Loading branch information
Chris Santero committed May 6, 2015
commit 003803405a86f81bf46656da737018fbee14ff7b
6 changes: 3 additions & 3 deletions JSONAPI.EntityFramework/EntityFrameworkMaterializer.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ namespace JSONAPI.EntityFramework
/// </summary>
public partial class EntityFrameworkMaterializer : IMaterializer
{
private readonly IMetadataManager _metadataManager;
protected readonly IMetadataManager MetadataManager;

/// <summary>
/// The DbContext instance used to perform materializer operations
Expand All @@ -31,7 +31,7 @@ public partial class EntityFrameworkMaterializer : IMaterializer
/// <param name="context">The DbContext instance used to perform materializer operations</param>
public EntityFrameworkMaterializer(DbContext context, IMetadataManager metadataManager)
{
_metadataManager = metadataManager;
MetadataManager = metadataManager;
DbContext = context;
}

Expand Down Expand Up @@ -325,7 +325,7 @@ protected async Task Merge (Type type, object ephemeral, object material)
foreach (PropertyInfo prop in props)
{
// Comply with the spec, if a key was not set, it should not be updated!
if (!_metadataManager.PropertyWasPresent(ephemeral, prop)) continue;
if (!MetadataManager.PropertyWasPresent(ephemeral, prop)) continue;

if (IsMany(prop.PropertyType))
{
Expand Down