#2030 Added a setting ('commonsettings.bbcodeeditoropenlinksinnewwindow') indicating whether BBCode Editor should generate URL with "target=_blank" (new window).

Author: AndreiMaz

src/Libraries/Nop.Core/Domain/Common/CommonSettings.cs

@@ -11,9 +11,19 @@ public CommonSettings()
             IgnoreLogWordlist = new List<string>();
         }
 
+
+        /// <summary>
+        /// Gets or sets a value indicating whether the contacts form should have "Subject"
+        /// </summary>
         public bool SubjectFieldOnContactUsForm { get; set; }
+        /// <summary>
+        /// Gets or sets a value indicating whether the contacts form should use system email
+        /// </summary>
         public bool UseSystemEmailForContactUsForm { get; set; }
 
+        /// <summary>
+        /// Gets or sets a value indicating whether stored procedures are enabled (should be used if possible)
+        /// </summary>
         public bool UseStoredProceduresIfSupported { get; set; }
 
         /// <summary>
@@ -76,5 +86,10 @@ public CommonSettings()
         /// Gets or sets ignore words (phrases) to be ignored when logging errors/messages
         /// </summary>
         public List<string> IgnoreLogWordlist { get; set; }
+
+        /// <summary>
+        /// Gets or sets a value indicating whether links generated by BBCode Editor should be opened in a new window
+        /// </summary>
+        public bool BbcodeEditorOpenLinksInNewWindow { get; set; }
     }
 }

src/Libraries/Nop.Core/Html/BBCodeHelper.cs

@@ -1,6 +1,8 @@
 using System;
 using System.Text.RegularExpressions;
+using Nop.Core.Domain.Common;
 using Nop.Core.Html.CodeFormatter;
+using Nop.Core.Infrastructure;
 
 namespace Nop.Core.Html
 {
@@ -62,13 +64,14 @@ public static string FormatText(string text, bool replaceBold, bool replaceItali
 
             if (replaceUrl)
             {
+                var newWindow = EngineContext.Current.Resolve<CommonSettings>().BbcodeEditorOpenLinksInNewWindow;
                 // format the url tags: [url=http://www.nopCommerce.com]my site[/url]
                 // becomes: <a href="http://www.nopCommerce.com">my site</a>
-                text = regexUrl1.Replace(text, "<a href=\"$1\" rel=\"nofollow\">$2</a>");
+                text = regexUrl1.Replace(text, string.Format("<a href=\"$1\" rel=\"nofollow\"{0}>$2</a>", newWindow ? " target=_blank" : ""));
 
                 // format the url tags: [url]http://www.nopCommerce.com[/url]
                 // becomes: <a href="http://www.nopCommerce.com">http://www.nopCommerce.com</a>
-                text = regexUrl2.Replace(text, "<a href=\"$1\" rel=\"nofollow\">$1</a>");
+                text = regexUrl2.Replace(text, string.Format("<a href=\"$1\" rel=\"nofollow\"{0}>$1</a>", newWindow ? " target=_blank" : ""));
             }
 
             if (replaceQuote)

src/Libraries/Nop.Services/Installation/CodeFirstInstallationService.cs

@@ -5766,7 +5766,8 @@ protected virtual void InstallSettings()
                 Log404Errors = true,
                 BreadcrumbDelimiter = "/",
                 RenderXuaCompatible = false,
-                XuaCompatibleValue = "IE=edge"
+                XuaCompatibleValue = "IE=edge",
+                BbcodeEditorOpenLinksInNewWindow = false
             });
 
             settingService.SaveSetting(new SeoSettings

upgradescripts/3.80-the next version/upgrade.sql

@@ -3710,4 +3710,12 @@ BEGIN
     INSERT [Setting] ([Name], [Value], [StoreId])
     VALUES (N'shippingsettings.considerassociatedproductsdimensions', N'True', 0)
 END
+GO
+
+ --new setting
+IF NOT EXISTS (SELECT 1 FROM [Setting] WHERE [name] = N'commonsettings.bbcodeeditoropenlinksinnewwindow')
+BEGIN
+    INSERT [Setting] ([Name], [Value], [StoreId])
+    VALUES (N'commonsettings.bbcodeeditoropenlinksinnewwindow', N'false', 0)
+END
 GO