Skip to content

Commit d076682

Browse files
SreejithpinKenitoInc
authored andcommitted
updates
1 parent 8c74cc8 commit d076682

File tree

4 files changed

+12
-15
lines changed

4 files changed

+12
-15
lines changed

src/Microsoft.AspNet.OData.Shared/EdmChangedObjectCollection.cs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ internal EdmChangedObjectCollection Patch(EdmODataAPIHandler apiHandler, ODataEd
101101
internal EdmChangedObjectCollection CopyChangedValues(EdmODataAPIHandler apiHandler, ODataEdmAPIHandlerFactory apiHandlerFactory = null)
102102
{
103103
EdmChangedObjectCollection changedObjectCollection = new EdmChangedObjectCollection(_entityType);
104-
IEdmStructuralProperty[] keys = _entityType.Key().ToArray();
104+
string[] keys = _entityType.Key().Select(x=>x.Name).ToArray();
105105

106106
foreach (IEdmChangedObject changedObj in Items)
107107
{
@@ -199,18 +199,18 @@ internal EdmChangedObjectCollection CopyChangedValues(EdmODataAPIHandler apiHand
199199
return changedObjectCollection;
200200
}
201201

202-
private static IDictionary<string, object> GetKeyValues(IEdmStructuralProperty[] keys, IEdmChangedObject changedObj)
202+
private static IDictionary<string, object> GetKeyValues(string[] keys, IEdmChangedObject changedObj)
203203
{
204204
IDictionary<string, object> keyValues = new Dictionary<string, object>();
205205

206-
foreach (IEdmStructuralProperty key in keys)
206+
foreach (string key in keys)
207207
{
208208
object value;
209-
changedObj.TryGetPropertyValue(key.Name, out value);
209+
changedObj.TryGetPropertyValue(key, out value);
210210

211211
if (value != null)
212212
{
213-
keyValues.Add(key.Name, value);
213+
keyValues.Add(key, value);
214214
}
215215
}
216216

@@ -323,7 +323,7 @@ private void ApplyProperties(EdmStructuredObject changedObj, EdmStructuredObject
323323
}
324324

325325
private IEdmChangedObject HandleFailedOperation(EdmEntityObject changedObj, DataModificationOperationKind operation, IEdmStructuredObject originalObj,
326-
IEdmStructuralProperty[] keys, string errorMessage, EdmODataAPIHandler apiHandler)
326+
string[] keys, string errorMessage, EdmODataAPIHandler apiHandler)
327327
{
328328
IEdmChangedObject edmChangedObject = null;
329329
DataModificationExceptionType dataModificationExceptionType = new DataModificationExceptionType(operation);
@@ -368,13 +368,13 @@ private IEdmChangedObject HandleFailedOperation(EdmEntityObject changedObj, Data
368368
}
369369

370370
//This is for ODL to work to set id as empty, because if there are missing keys, id wouldnt be set and we need to set it as empty.
371-
private static void ValidateForDeletedEntityId(IEdmStructuralProperty[] keys, EdmDeltaDeletedEntityObject edmDeletedObject)
371+
private static void ValidateForDeletedEntityId(string[] keys, EdmDeltaDeletedEntityObject edmDeletedObject)
372372
{
373373
bool hasNullKeys = false;
374374
for (int i = 0; i < keys.Length; i++)
375375
{
376376
object value;
377-
if (edmDeletedObject.TryGetPropertyValue(keys[i].Name, out value))
377+
if (edmDeletedObject.TryGetPropertyValue(keys[i], out value))
378378
{
379379
hasNullKeys = true;
380380
break;

src/Microsoft.AspNet.OData.Shared/EdmDeltaDeletedEntityObject.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ public class EdmDeltaDeletedEntityObject : EdmEntityObject, IEdmDeltaDeletedEnti
2121
{
2222
private string _id;
2323
private DeltaDeletedEntryReason _reason;
24-
private EdmDeltaType _edmType;
2524
private IEdmNavigationSource _navigationSource;
2625

2726
/// <summary>
@@ -50,7 +49,7 @@ public EdmDeltaDeletedEntityObject(IEdmEntityTypeReference entityTypeReference)
5049
public EdmDeltaDeletedEntityObject(IEdmEntityType entityType, bool isNullable)
5150
: base(entityType, isNullable)
5251
{
53-
_edmType = new EdmDeltaType(entityType, EdmDeltaEntityKind.DeletedEntry);
52+
5453
}
5554

5655
/// <inheritdoc />

src/Microsoft.AspNet.OData.Shared/EdmDeltaDeletedLink.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ public class EdmDeltaDeletedLink : EdmEntityObject, IEdmDeltaDeletedLink
2121
private Uri _source;
2222
private Uri _target;
2323
private string _relationship;
24-
private EdmDeltaType _edmType;
2524

2625
/// <summary>
2726
/// Initializes a new instance of the <see cref="EdmDeltaDeletedLink"/> class.
@@ -49,7 +48,7 @@ public EdmDeltaDeletedLink(IEdmEntityTypeReference entityTypeReference)
4948
public EdmDeltaDeletedLink(IEdmEntityType entityType, bool isNullable)
5049
: base(entityType, isNullable)
5150
{
52-
_edmType = new EdmDeltaType(entityType, EdmDeltaEntityKind.DeletedLinkEntry);
51+
5352
}
5453

5554
/// <inheritdoc />

src/Microsoft.AspNet.OData.Shared/EdmDeltaEntityObject.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,7 @@ namespace Microsoft.AspNet.OData
1717
/// </summary>
1818
[NonValidatingParameterBinding]
1919
public class EdmDeltaEntityObject : EdmEntityObject, IEdmChangedObject
20-
{
21-
private EdmDeltaType _edmType;
20+
{
2221
private IEdmNavigationSource _navigationSource;
2322

2423
/// <summary>
@@ -47,7 +46,7 @@ public EdmDeltaEntityObject(IEdmEntityTypeReference entityTypeReference)
4746
public EdmDeltaEntityObject(IEdmEntityType entityType, bool isNullable)
4847
: base(entityType, isNullable)
4948
{
50-
_edmType = new EdmDeltaType(entityType, EdmDeltaEntityKind.Entry);
49+
5150
}
5251

5352
/// <inheritdoc />

0 commit comments

Comments
 (0)