Fixed incorrect logic in SiteMapPathHelper.

maartenba · Aug 18, 2013 · 15692d0 · 15692d0
1 parent 99f48b9
commit 15692d0
Showing 1 changed file with 11 additions and 4 deletions.
diff --git a/src/MvcSiteMapProvider/MvcSiteMapProvider/Web/Html/SiteMapPathHelper.cs b/src/MvcSiteMapProvider/MvcSiteMapProvider/Web/Html/SiteMapPathHelper.cs
@@ -189,13 +189,20 @@ private static SiteMapPathHelperModel BuildModel(MvcSiteMapHtmlHelper helper, So
             // Build model
             var model = new SiteMapPathHelperModel();
             var node = startingNode;
-            // Check visibility and ACL
-            if (node != null && node.IsVisible(sourceMetadata) && node.IsAccessibleToUser())
+            while (node != null)
             {
-                // Add node
-                model.Nodes.AddRange((new SiteMapNodeModel(node, sourceMetadata)).Ancestors);
+                bool nodeVisible = node.IsVisible(sourceMetadata);
+                if (nodeVisible && node.IsAccessibleToUser())
+                {
+                    var nodeToAdd = new SiteMapNodeModel(node, sourceMetadata);
+                    model.Nodes.Add(nodeToAdd);
+                }
+                node = node.ParentNode;
             }
+            model.Nodes.Reverse();
+
             return model;
+
         }
 
         /// <summary>