Migrated to string interpolation

Replaced string concactenation with string interpolation.

Author: JeremyCaney

Ignia.Topics/Collections/AttributeValueCollection.cs

@@ -558,8 +558,8 @@ private bool EnforceBusinessLogic(AttributeValue originalAttribute, out Attribut
         _setCounter++;
         if (_setCounter > 3) {
           throw new Exception(
-            "An infinite loop has occurred when setting `" + originalAttribute.Key +
-            "`; be sure to call `Topic.SetAttributeValue()` when setting attributes from `Topic` properties."
+            $"An infinite loop has occurred when setting '{originalAttribute.Key}'; be sure that you are referencing " +
+            $"`Topic.SetAttributeValue()` when setting attributes from `Topic` properties."
           );
         }
         _typeCache.SetPropertyValue(_associatedTopic, originalAttribute.Key, originalAttribute.Value);

Ignia.Topics/Mapping/CachedTopicMappingService.cs

@@ -190,7 +190,7 @@ private object CacheViewModel(string contentType, object viewModel, (int, Type,
       if (cacheKey.Item2 != null) {
         cacheKey = (cacheKey.Item1, null, cacheKey.Item3);
       }
-      if (cacheKey.Item1 > 0 && viewModel.GetType().Name.Equals(contentType + "TopicViewModel")) {
+      if (cacheKey.Item1 > 0 && viewModel.GetType().Name.Equals($"{contentType}TopicViewModel")) {
         _cache.TryAdd(cacheKey, viewModel);
       }
       return viewModel;

Ignia.Topics/Mapping/TopicMappingService.cs

@@ -116,7 +116,7 @@ private async Task<object> MapAsync(Topic topic, Relationships relationships, Co
       /*----------------------------------------------------------------------------------------------------------------------
       | Instantiate object
       \---------------------------------------------------------------------------------------------------------------------*/
-      var viewModelType = _typeLookupService.GetType(topic.ContentType + "TopicViewModel");
+      var viewModelType = _typeLookupService.GetType($"{topic.ContentType}TopicViewModel");
       var target = Activator.CreateInstance(viewModelType);
 
       /*----------------------------------------------------------------------------------------------------------------------
@@ -251,7 +251,7 @@ ConcurrentDictionary<int, object> cache
       | Establish per-property variables
       \-----------------------------------------------------------------------------------------------------------------------*/
       var configuration         = new PropertyConfiguration(property);
-      var topicReferenceId      = source.Attributes.GetInteger(property.Name + "Id", 0);
+      var topicReferenceId      = source.Attributes.GetInteger($"{property.Name}Id", 0);
 
       /*------------------------------------------------------------------------------------------------------------------------
       | Assign default value
@@ -315,7 +315,7 @@ protected static void SetScalarValue(Topic source, object target, PropertyConfig
       /*------------------------------------------------------------------------------------------------------------------------
       | Attempt to retrieve value from topic.Get{Property}()
       \-----------------------------------------------------------------------------------------------------------------------*/
-      var attributeValue = _typeCache.GetMethodValue(source, "Get" + configuration.AttributeKey)?.ToString();
+      var attributeValue = _typeCache.GetMethodValue(source, $"Get{configuration.AttributeKey}")?.ToString();
 
       /*------------------------------------------------------------------------------------------------------------------------
       | Attempt to retrieve value from topic.{Property}
@@ -473,7 +473,7 @@ protected IList<Topic> GetSourceCollection(Topic source, Relationships relations
       | Handle Metadata relationship
       \-----------------------------------------------------------------------------------------------------------------------*/
       if (listSource.Count == 0 && !String.IsNullOrWhiteSpace(configuration.MetadataKey)) {
-        var metadataKey = "Root:Configuration:Metadata:" + configuration.MetadataKey + ":LookupList";
+        var metadataKey = $"Root:Configuration:Metadata:{configuration.MetadataKey}:LookupList";
         listSource = _topicRepository.Load(metadataKey)?.Children.ToList();
       }
 

Ignia.Topics/Topic.cs

@@ -97,7 +97,7 @@ public Topic(string key, string contentType, Topic parent, int id = -1) {
     ///   The unique identifier for the <see cref="Topic"/>.
     /// </value>
     /// <exception cref="ArgumentException">
-    ///   The value of this topic has already been set to " + _id + "; it cannot be changed.
+    ///   The value of this topic has already been set to {_id}; it cannot be changed.
     /// </exception>
     /// <requires description="The id is expected to be a positive value." exception="T:System.ArgumentException">
     ///   value &gt; 0
@@ -107,7 +107,7 @@ public int Id {
       set {
         Contract.Requires<ArgumentOutOfRangeException>(value > 0, "The id is expected to be a positive value.");
         if (_id > 0 && !_id.Equals(value)) {
-          throw new ArgumentException("The value of this topic has already been set to " + _id + "; it cannot be changed.");
+          throw new ArgumentException($"The value of this topic has already been set to {_id}; it cannot be changed.");
         }
         _id = value;
       }
@@ -405,8 +405,8 @@ public DateTime LastModified {
     /// <param name="sibling">The <see cref="Topic" /> to move this <see cref="Topic" /> to the right of.</param>
     /// <exception cref="ArgumentOutOfRangeException">parent - A descendant cannot be its own parent.</exception>
     /// <exception cref="InvalidKeyException">
-    ///   Duplicate key when setting Parent property: the topic with the name '" + Key + "' already exists in the '" +
-    ///   parent.Key + "' topic.
+    ///   Duplicate key when setting Parent property: the topic with the name '{Key}' already exists in the '{parent.Key}'
+    ///   topic.
     /// </exception>
     public void SetParent(Topic parent, Topic sibling = null) {
 
@@ -428,9 +428,9 @@ public void SetParent(Topic parent, Topic sibling = null) {
       \-----------------------------------------------------------------------------------------------------------------------*/
       if (parent != _parent && parent.Children.Contains(Key)) {
         throw new InvalidKeyException(
-          "Duplicate key when setting Parent property: the topic with the name '" + Key +
-          "' already exists in the '" + parent.Key + "' topic."
-          );
+          $"Duplicate key when setting Parent property: the topic with the name '{Key}' already exists in the '{parent.Key}' " +
+          $"topic."
+        );
       }
 
       /*------------------------------------------------------------------------------------------------------------------------
@@ -478,7 +478,7 @@ public string GetUniqueKey() {
       var topic = this;
 
       for (var i = 0; i < 100; i++) {
-        if (uniqueKey.Length > 0) uniqueKey = ":" + uniqueKey;
+        if (uniqueKey.Length > 0) uniqueKey = $":{uniqueKey}";
         uniqueKey = topic.Key + uniqueKey;
         topic = topic.Parent;
         if (topic == null) break;
@@ -507,7 +507,7 @@ public string GetWebPath() {
       Contract.Ensures(Contract.Result<string>() != null);
       var uniqueKey = GetUniqueKey().Replace("Root:", "/").Replace(":", "/") + "/";
       if (!uniqueKey.StartsWith("/")) {
-        uniqueKey = "/" + uniqueKey;
+        uniqueKey = $"/{uniqueKey}";
       }
       return uniqueKey;
     }