Skip to content

Commit a3f0cba

Browse files
committed
Removed placeholder comment for import/export functionality
For six years, we've been carrying around this commented out placeholder for a `Load()` method that would handle importing topics from an `XmlDocument` in order to support import capabilities. We shouldn't maintain commented out placeholder code like this to begin with. But, in addition, it's no longer relevant as we've implemented a import capabilities via the new **OnTopic-Data-Transfer** library—which is also a much more elegant and flexible approach. Good riddance!
1 parent 3693ea0 commit a3f0cba

1 file changed

Lines changed: 0 additions & 15 deletions

File tree

OnTopic/Repositories/ITopicRepository.cs

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -74,21 +74,6 @@ public interface ITopicRepository {
7474
/// <returns>A topic object.</returns>
7575
Topic? Load(int topicId, DateTime version);
7676

77-
/*==========================================================================================================================
78-
| ###TODO JJC080314: An overload to Load() should be created to accept an XmlDocument or XmlNode based on the proposed
79-
| Import/Export schema.
80-
>---------------------------------------------------------------------------------------------------------------------------
81-
| ###NOTE JJC080313: If the topic already exists, return the existing node, by calling its Merge() function. Otherwise,
82-
| construct a new node using its XmlNode constructor.
83-
>---------------------------------------------------------------------------------------------------------------------------
84-
public static Topic Load(XmlNode node, ImportStrategy importStrategy = ImportStrategy.Merge) {
85-
//Process XML
86-
//Construct children objects
87-
//###NOTE JJC080314: May need to cross-reference with Load() and/or TopicRepository to validate against whatever objects
88-
//are already created and available.
89-
}
90-
\-------------------------------------------------------------------------------------------------------------------------*/
91-
9277
/*==========================================================================================================================
9378
| METHOD: ROLLBACK
9479
\-------------------------------------------------------------------------------------------------------------------------*/

0 commit comments

Comments
 (0)