Fixes maartenba#230, default HTTP method set incorrectly for DynamicN…

…odes. Also added HttpMethod property to DynamicNode so HttpMethod can be set.
NightOwl888 · Sep 20, 2013 · 57cb085 · 57cb085
1 parent d5e6a4e
commit 57cb085
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/src/MvcSiteMapProvider/MvcSiteMapProvider/DynamicNode.cs b/src/MvcSiteMapProvider/MvcSiteMapProvider/DynamicNode.cs
@@ -159,6 +159,14 @@ public UpdatePriority UpdatePriority
         /// <value>The sort order.</value>
         public int? Order { get; set; }
 
+        /// <summary>
+        /// Gets or sets the HTTP method (such as GET, POST, or HEAD).
+        /// </summary>
+        /// <value>
+        /// The HTTP method.
+        /// </value>
+        public string HttpMethod { get; set; }
+
         /// <summary>
         /// Copies the values for matching properties on an <see cref="T:MvcSiteMapNodeProvider.ISiteMapNode"/> instance, but
         /// doesn't overwrite any values that are not set in this <see cref="T:MvcSiteMapNodeProvider.DynamicNode"/> instance.
@@ -236,6 +244,8 @@ public void SafeCopyTo(ISiteMapNode node)
             }
             if (this.Order != null)
                 node.Order = (int)this.Order;
+            if (!string.IsNullOrEmpty(this.HttpMethod))
+                node.HttpMethod = this.HttpMethod;
         }
 
 

diff --git a/src/MvcSiteMapProvider/MvcSiteMapProvider/SiteMapNode.cs b/src/MvcSiteMapProvider/MvcSiteMapProvider/SiteMapNode.cs
@@ -75,6 +75,7 @@ IUrlPath urlPath
         protected readonly ISiteMap siteMap;
         protected readonly string key;
         protected readonly bool isDynamic;
+        protected string httpMethod = "*";
         protected string title = String.Empty;
         protected string description = String.Empty;
         protected string imageUrl = String.Empty;
@@ -114,12 +115,16 @@ public override ISiteMap SiteMap
         }
 
         /// <summary>
-        /// Gets or sets the HTTP method.
+        /// Gets or sets the HTTP method (such as GET, POST, or HEAD).
         /// </summary>
         /// <value>
         /// The HTTP method.
         /// </value>
-        public override string HttpMethod { get; set; }
+        public override string HttpMethod
+        {
+            get { return this.httpMethod; }
+            set { this.httpMethod = value; }
+        }
 
         /// <summary>
         /// Gets the implicit resource key (optional).