Introduced isNew variable

The local `isNew` variable is more semantically meaningful and less repetitive than checking `topic.Id != -1` multiple times.

Author: JeremyCaney

OnTopic.Data.Sql/SqlTopicRepository.cs

@@ -330,26 +330,23 @@ public override int Save([NotNull]Topic topic, bool isRecursive = false, bool is
       var connection            = new SqlConnection(_connectionString);
       var procedureName         = topic.Id > 0? "CreateTopic" : "UpdateTopic";
       var command               = new SqlCommand(procedureName, connection) {
-        CommandType           = CommandType.StoredProcedure
+        CommandType             = CommandType.StoredProcedure
       };
-
-      /*------------------------------------------------------------------------------------------------------------------------
-      | Set version
-      \-----------------------------------------------------------------------------------------------------------------------*/
       var version               = DateTime.Now;
+      var isNew                 = topic.Id == -1;
 
       /*------------------------------------------------------------------------------------------------------------------------
       | Establish query parameters
       \-----------------------------------------------------------------------------------------------------------------------*/
-      if (topic.Id != -1) {
+      if (!isNew) {
         command.AddParameter("TopicID", topic.Id);
       }
       else if (topic.Parent != null) {
         command.AddParameter("ParentID", topic.Parent.Id);
       }
       command.AddParameter("Version", version);
       command.Parameters.AddWithValue("@Attributes", attributes);
-      if (topic.Id != -1) {
+      if (!isNew) {
         command.AddParameter("DeleteRelationships", true);
       }
       command.AddParameter("ExtendedAttributes", extendedAttributes);