From 8d33488923e3062c3b8dccf315bf09a134294502 Mon Sep 17 00:00:00 2001
From: Stephan <sgay@pilotine.com>
Date: Sun, 31 Mar 2013 18:40:55 -0200
Subject: [PATCH] Web.PublishedCache - make parts of XmlPublishedCache virtual

---
 .../XmlPublishedCache/PublishedContentCache.cs     | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedContentCache.cs b/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedContentCache.cs
index b48d680918..7d14797fd1 100644
--- a/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedContentCache.cs
+++ b/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedContentCache.cs
@@ -37,7 +37,7 @@ namespace Umbraco.Web.PublishedCache.XmlPublishedCache
         /// <para>A valid route is either a simple path eg <c>/foo/bar/nil</c> or a root node id and a path, eg <c>123/foo/bar/nil</c>.</para>
         /// <para>If <param name="hideTopLevelNode" /> is <c>null</c> then the settings value is used.</para>
         /// </remarks>
-        public IPublishedContent GetByRoute(UmbracoContext umbracoContext, string route, bool? hideTopLevelNode = null)
+        public virtual IPublishedContent GetByRoute(UmbracoContext umbracoContext, string route, bool? hideTopLevelNode = null)
         {
             if (route == null) throw new ArgumentNullException("route");
 
@@ -79,7 +79,7 @@ namespace Umbraco.Web.PublishedCache.XmlPublishedCache
         /// <param name="umbracoContext">The context.</param>
         /// <param name="contentId">The content unique identifier.</param>
         /// <returns>The route.</returns>
-        public string GetRouteById(UmbracoContext umbracoContext, int contentId)
+        public virtual string GetRouteById(UmbracoContext umbracoContext, int contentId)
         {
             // try to get from cache if not previewing
             var route = umbracoContext.InPreviewMode
@@ -176,6 +176,8 @@ namespace Umbraco.Web.PublishedCache.XmlPublishedCache
             if (node.Parent == null)
             {
                 var rootNode = umbracoContext.ContentCache.GetByRoute("/", true);
+                if (rootNode == null)
+                    throw new Exception("Failed to get node at /.");
                 if (rootNode.Id == node.Id) // remove only if we're the default node
                     pathParts.RemoveAt(pathParts.Count - 1);
             }
@@ -270,12 +272,12 @@ namespace Umbraco.Web.PublishedCache.XmlPublishedCache
     		return ConvertToDocument(GetXml(umbracoContext).GetElementById(nodeId.ToString()));
     	}
 
-		public IEnumerable<IPublishedContent> GetAtRoot(UmbracoContext umbracoContext)
+		public virtual IEnumerable<IPublishedContent> GetAtRoot(UmbracoContext umbracoContext)
 		{
 			return (from XmlNode x in GetXml(umbracoContext).SelectNodes(XPathStrings.RootDocuments) select ConvertToDocument(x)).ToList();
 		}
 
-        public IPublishedContent GetSingleByXPath(UmbracoContext umbracoContext, string xpath, params XPathVariable[] vars)
+        public virtual IPublishedContent GetSingleByXPath(UmbracoContext umbracoContext, string xpath, params XPathVariable[] vars)
         {
             if (xpath == null) throw new ArgumentNullException("xpath");
             if (string.IsNullOrWhiteSpace(xpath)) return null;
@@ -287,7 +289,7 @@ namespace Umbraco.Web.PublishedCache.XmlPublishedCache
             return ConvertToDocument(node);
         }
 
-        public IEnumerable<IPublishedContent> GetByXPath(UmbracoContext umbracoContext, string xpath, params XPathVariable[] vars)
+        public virtual IEnumerable<IPublishedContent> GetByXPath(UmbracoContext umbracoContext, string xpath, params XPathVariable[] vars)
         {
             if (xpath == null) throw new ArgumentNullException("xpath");
             if (string.IsNullOrWhiteSpace(xpath)) return Enumerable.Empty<IPublishedContent>();
@@ -299,7 +301,7 @@ namespace Umbraco.Web.PublishedCache.XmlPublishedCache
             return ConvertToDocuments(nodes);
         }
 
-        public bool HasContent()
+        public virtual bool HasContent()
         {
 	        var xml = GetXml();
 			if (xml == null)