Merge pull request #447 from mharen/all-caps-url-payload-generator

url payload generator should detect all-caps protocol

Author: codebude

QRCoder/PayloadGenerator.cs

@@ -312,7 +312,7 @@ public class Url : Payload
             private readonly string url;
 
             /// <summary>
-            /// Generates a link. If not given, http/https protocol will be added.
+            /// Generates a link. If the protocol is not specified, the http protocol will be added.
             /// </summary>
             /// <param name="url">Link url target</param>
             public Url(string url)
@@ -322,7 +322,7 @@ public Url(string url)
 
             public override string ToString()
             {
-                return (!this.url.StartsWith("http") ? "http://" + this.url : this.url);
+                return (!this.url.StartsWith("http", StringComparison.OrdinalIgnoreCase) ? "http://" + this.url : this.url);
             }
         }
 

QRCoderTests/PayloadGeneratorTests.cs

@@ -757,6 +757,18 @@ public void url_should_build_https()
         }
 
 
+        [Fact]
+        [Category("PayloadGenerator/Url")]
+        public void url_should_build_https_all_caps()
+        {
+            var url = "HTTPS://CODE-BUDE.NET";
+
+            var generator = new PayloadGenerator.Url(url);
+
+            generator.ToString().ShouldBe("HTTPS://CODE-BUDE.NET");
+        }
+
+
         [Fact]
         [Category("PayloadGenerator/Url")]
         public void url_should_add_http()