introduced string interpolation where it could simplify the code

Author: ThisWillDoIt

BlogEngine/BlogEngine.Core/API/MetaWeblog/MetaWeblogHandler.cs

@@ -177,7 +177,7 @@ internal bool DeletePage(string blogId, string pageId, string userName, string p
             }
             catch (Exception ex)
             {
-                throw new MetaWeblogException("15", string.Format("DeletePage failed.  Error: {0}", ex.Message));
+                throw new MetaWeblogException("15", $"DeletePage failed.  Error: {ex.Message}");
             }
 
             return true;
@@ -220,7 +220,7 @@ internal bool DeletePost(string appKey, string postId, string userName, string p
             }
             catch (Exception ex)
             {
-                throw new MetaWeblogException("12", string.Format("DeletePost failed.  Error: {0}", ex.Message));
+                throw new MetaWeblogException("12", $"DeletePost failed.  Error: {ex.Message}");
             }
 
             return true;
@@ -712,7 +712,7 @@ internal MWAMediaInfo NewMediaObject(
 
             var mediaInfo = new MWAMediaInfo();
 
-            var rootPath = string.Format("{0}files/", Blog.CurrentInstance.StorageLocation);
+            var rootPath = $"{Blog.CurrentInstance.StorageLocation}files/";
             var serverPath = request.Server.MapPath(rootPath);
             var saveFolder = serverPath;
             string mediaObjectName = mediaObject.name.Replace(" ", "_");
@@ -749,7 +749,7 @@ internal MWAMediaInfo NewMediaObject(
                 // Find unique fileName
                 for (var count = 1; count < 30000; count++)
                 {
-                    var tempFileName = fileName.Insert(fileName.LastIndexOf('.'), string.Format("_{0}", count));
+                    var tempFileName = fileName.Insert(fileName.LastIndexOf('.'), $"_{count}");
                     if (File.Exists(saveFolder + tempFileName))
                     {
                         continue;

BlogEngine/BlogEngine.Core/API/MetaWeblog/XMLRPCRequest.cs

@@ -339,7 +339,7 @@ private void LoadXmlRequest(string xml)
             }
             catch (Exception ex)
             {
-                throw new MetaWeblogException("01", string.Format("Invalid XMLRPC Request. ({0})", ex.Message));
+                throw new MetaWeblogException("01", $"Invalid XMLRPC Request. ({ex.Message})");
             }
 
             // Method name is always first
@@ -449,7 +449,7 @@ private void LoadXmlRequest(string xml)
                     this.PageID = this.inputParams[3].InnerText;
                     break;
                 default:
-                    throw new MetaWeblogException("02", string.Format("Unknown Method. ({0})", this.MethodName));
+                    throw new MetaWeblogException("02", $"Unknown Method. ({MethodName})");
             }
         }
 

BlogEngine/BlogEngine.Core/AuthorProfile.cs

@@ -300,7 +300,7 @@ public string FullName
         {
             get
             {
-                return string.Format("{0} {1} {2}", this.FirstName, this.MiddleName, this.LastName).Replace("  ", " ");
+                return $"{FirstName} {MiddleName} {LastName}".Replace("  ", " ");
             }
         }
 
@@ -439,7 +439,7 @@ public string RelativeLink
         {
             get
             {
-                return string.Format("{0}author/{1}{2}", Utils.RelativeWebRoot, this.Id, BlogConfig.FileExtension);
+                return $"{Utils.RelativeWebRoot}author/{Id}{BlogConfig.FileExtension}";
             }
         }
 

BlogEngine/BlogEngine.Core/Blog.cs

@@ -629,7 +629,7 @@ public string StorageLocation
                     return BlogConfig.StorageLocation;
                 }
 
-                return string.Format("{0}{1}/{2}/", BlogConfig.StorageLocation, BlogConfig.BlogInstancesFolderName, this.StorageContainerName);
+                return $"{BlogConfig.StorageLocation}{BlogConfig.BlogInstancesFolderName}/{StorageContainerName}/";
             }
         }
 
@@ -676,7 +676,7 @@ public Uri AbsoluteWebRoot
         {
             get
             {
-                string contextItemKey = string.Format("{0}-absolutewebroot", this.Id);
+                string contextItemKey = $"{Id}-absolutewebroot";
 
                 var context = HttpContext.Current;
                 if (context == null)
@@ -955,7 +955,7 @@ internal bool CopyExistingBlogFolderToNewBlogFolder(Blog existingBlog)
                 existingBlogStoragePath = HostingEnvironment.MapPath(existingBlog.StorageLocation);
                 if (!Directory.Exists(existingBlogStoragePath))
                 {
-                    throw new Exception(string.Format("Storage folder for existing blog instance to copy from does not exist.  Directory not found is: {0}", existingBlogStoragePath));
+                    throw new Exception($"Storage folder for existing blog instance to copy from does not exist.  Directory not found is: {existingBlogStoragePath}");
                 }
             }
             catch (Exception ex)
@@ -965,7 +965,7 @@ internal bool CopyExistingBlogFolderToNewBlogFolder(Blog existingBlog)
             }
 
             // Ensure "BlogInstancesFolderName" exists.
-            string blogInstancesFolder = HostingEnvironment.MapPath(string.Format("{0}{1}", BlogConfig.StorageLocation, BlogConfig.BlogInstancesFolderName));
+            string blogInstancesFolder = HostingEnvironment.MapPath($"{BlogConfig.StorageLocation}{BlogConfig.BlogInstancesFolderName}");
             if (!Utils.CreateDirectoryIfNotExists(blogInstancesFolder))
                 return false;
 
@@ -976,7 +976,7 @@ internal bool CopyExistingBlogFolderToNewBlogFolder(Blog existingBlog)
             {
                 if (Directory.Exists(newBlogStoragePath))
                 {
-                    throw new Exception(string.Format("Blog destination folder already exists. {0}", newBlogStoragePath));
+                    throw new Exception($"Blog destination folder already exists. {newBlogStoragePath}");
                 }
             }
             catch (Exception ex)

BlogEngine/BlogEngine.Core/BlogSettings.cs

@@ -149,7 +149,7 @@ public bool IsRazorTheme
         /// </summary>
         public static bool IsThemeRazor(string themeName)
         {
-            string path = HostingEnvironment.MapPath(string.Format("~/Custom/Themes/{0}/site.cshtml", themeName));
+            string path = HostingEnvironment.MapPath($"~/Custom/Themes/{themeName}/site.cshtml");
             return File.Exists(path);
         }
 
@@ -1312,7 +1312,7 @@ private void Load(Blog blog)
                     }
                     catch (Exception e)
                     {
-                        Utils.Log(string.Format("Error loading blog settings: {0}", e.Message));
+                        Utils.Log($"Error loading blog settings: {e.Message}");
                     }
                 }
 

BlogEngine/BlogEngine.Core/Category.cs

@@ -366,7 +366,7 @@ public string CompleteTitle()
 
             var cat = GetCategory((Guid)parent, Blog.CurrentInstance.IsSiteAggregation);
 
-            return cat == null ? title : string.Format("{0} - {1}", cat.CompleteTitle(), title);
+            return cat == null ? title : $"{cat.CompleteTitle()} - {title}";
         }
 
 

BlogEngine/BlogEngine.Core/Comment.cs

@@ -69,7 +69,7 @@ public Uri AbsoluteLink
         {
             get
             {
-                return new Uri(string.Format("{0}#id_{1}", Parent.AbsoluteLink, Id));
+                return new Uri($"{Parent.AbsoluteLink}#id_{Id}");
             }
         }
 
@@ -240,7 +240,7 @@ public string RelativeLink
         {
             get
             {
-                return string.Format("{0}#id_{1}", Parent.RelativeLink, Id);
+                return $"{Parent.RelativeLink}#id_{Id}";
             }
         }
 
@@ -268,7 +268,7 @@ public string Teaser
             get
             {
                 var ret = Utils.StripHtml(Content).Trim();
-                return ret.Length > 120 ? string.Format("{0} ...", ret.Substring(0, 116)) : ret;
+                return ret.Length > 120 ? $"{ret.Substring(0, 116)} ..." : ret;
             }
         }
 

BlogEngine/BlogEngine.Core/Data/CategoryRepository.cs

@@ -115,7 +115,7 @@ public CategoryItem Add(CategoryItem item)
             }
             catch (Exception ex)
             {
-                Utils.Log(string.Format("CategoryRepository.Add: {0}", ex.Message));
+                Utils.Log($"CategoryRepository.Add: {ex.Message}");
                 return null;
             }
         }
@@ -172,7 +172,7 @@ public bool Remove(Guid id)
             }
             catch (Exception ex)
             {
-                Utils.Log(string.Format("CategoryRepository.Remove: {0}", ex.Message));
+                Utils.Log($"CategoryRepository.Remove: {ex.Message}");
                 return false;
             }
         }

BlogEngine/BlogEngine.Core/Data/CustomFilterRepository.cs

@@ -81,11 +81,11 @@ public JsonResponse ResetCounters(string filterName)
                     }
                     ExtensionManager.SaveSettings("MetaExtension", CustomFilters);
                 }
-                return new JsonResponse() { Success = true, Message = string.Format("Counters for {0} reset", filterName) };
+                return new JsonResponse() { Success = true, Message = $"Counters for {filterName} reset"};
             }
             catch (Exception ex)
             {
-                Utils.Log(string.Format("CustomFilterRepository.ResetCounters: {0}", ex.Message));
+                Utils.Log($"CustomFilterRepository.ResetCounters: {ex.Message}");
                 return new JsonResponse() { Message = "Error resetting counters" };
             }
         }

BlogEngine/BlogEngine.Core/Data/LookupsRepository.cs

@@ -72,7 +72,7 @@ void LoadCultures()
             }
             else
             {
-                var path = HostingEnvironment.MapPath(string.Format("{0}App_GlobalResources/", Utils.ApplicationRelativeWebRoot));
+                var path = HostingEnvironment.MapPath($"{Utils.ApplicationRelativeWebRoot}App_GlobalResources/");
                 foreach (var file in Directory.GetFiles(path, "labels.*.resx"))
                 {
                     var index = file.LastIndexOf(Path.DirectorySeparatorChar) + 1;