Skip to content

Commit fa113c3

Browse files
committed
Removed outdated override of ClearItems()
With the underlying `ClearItems()` now calling `Remove()` and, thus, `RemoveItem()` (abfe213), it's no longer necessary to override `ClearItems()` to remove the `IncomingRelationships`, as this is already being done by `RemoveItem()`.
1 parent abfe213 commit fa113c3

1 file changed

Lines changed: 0 additions & 20 deletions

File tree

OnTopic/Associations/TopicReferenceCollection.cs

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -135,25 +135,5 @@ protected override sealed void RemoveItem(int index) {
135135

136136
}
137137

138-
/*==========================================================================================================================
139-
| OVERRIDE: CLEAR ITEMS
140-
\-------------------------------------------------------------------------------------------------------------------------*/
141-
/// <inheritdoc/>
142-
protected override sealed void ClearItems() {
143-
144-
/*------------------------------------------------------------------------------------------------------------------------
145-
| Handle recipricol references
146-
\-----------------------------------------------------------------------------------------------------------------------*/
147-
foreach (var item in Items) {
148-
item.Value?.IncomingRelationships.Remove(item.Key, AssociatedTopic, true);
149-
}
150-
151-
/*------------------------------------------------------------------------------------------------------------------------
152-
| Provide base logic
153-
\-----------------------------------------------------------------------------------------------------------------------*/
154-
base.ClearItems();
155-
156-
}
157-
158138
} //Class
159139
} //Namespace

0 commit comments

Comments
 (0)