Integrate new Topic.IsNew property

JeremyCaney · JeremyCaney · commit 94f1efc544e6 · 2020-05-30T11:10:19.000-07:00
This is just a shorthand for `Topic.Id &lt; 0`, but it is more semantically meaningful and intuitive to read.
diff --git a/OnTopic.Data.Transfer/Interchange/TopicExtensions.cs b/OnTopic.Data.Transfer/Interchange/TopicExtensions.cs
@@ -451,7 +451,7 @@ bool useCustomMergeRules(AttributeData attribute) =>
     private static string? GetTopicId(Topic topic, string? uniqueKey) {
       if (uniqueKey!.StartsWith("Root", StringComparison.InvariantCultureIgnoreCase)) {
         var target = topic.GetByUniqueKey(uniqueKey);
-        if (target != null && target.Id >= 0) {
+        if (target != null && !target.IsNew) {
           return target.Id.ToString(CultureInfo.CurrentCulture);
         }
         else {

Original file line number	Diff line number	Diff line change
`@@ -451,7 +451,7 @@ bool useCustomMergeRules(AttributeData attribute) =>`
`451`	`451`	`private static string? GetTopicId(Topic topic, string? uniqueKey) {`
`452`	`452`	`if (uniqueKey!.StartsWith("Root", StringComparison.InvariantCultureIgnoreCase)) {`
`453`	`453`	`var target = topic.GetByUniqueKey(uniqueKey);`
`454`		`- if (target != null && target.Id >= 0) {`
	`454`	`+ if (target != null && !target.IsNew) {`
`455`	`455`	`return target.Id.ToString(CultureInfo.CurrentCulture);`
`456`	`456`	`}`
`457`	`457`	`else {`