yv01p/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

remove a ref to requestHandler, fix DefaultRoutesCache management

Browse Source
This commit is contained in:
Stephan
2012-09-28 09:04:49 -02:00
parent 3ed1b4c77d
commit d82cbfccc5
2 changed files with 24 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
src/Umbraco.Web/Routing/DefaultRoutesCache.cs
View File

@@ -28,17 +28,31 @@ namespace Umbraco.Web.Routing
if (bindToEvents)
{
//FIXME:
//
// here we must register handlers to clear the cache when content changes
// this was done by presentation.library, which cleared everything when content changed
// but really, we should do some partial refreshes
// these are the two events that were used by presentation.library
// are they enough?
// content - whenever the entire XML cache is rebuilt (from disk cache or from database)
// we must clear the cache entirely
global::umbraco.content.AfterRefreshContent += (sender, e) => Clear();
// document - whenever a document is updated in, or removed from, the XML cache
// we must clear the cache - at the moment, we clear the entire cache
// TODO could we do partial updates instead of clearing the whole cache?
global::umbraco.content.AfterUpdateDocumentCache += (sender, e) => Clear();
global::umbraco.content.AfterClearDocumentCache += (sender, e) => Clear();
// domains - whenever a domain change we must clear the cache
// because routes contain the id of root nodes of domains
// TODO could we do partial updates instead of clearing the whole cache?
global::umbraco.cms.businesslogic.web.Domain.AfterDelete += (sender, e) => Clear();
global::umbraco.cms.businesslogic.web.Domain.AfterSave += (sender, e) => Clear();
global::umbraco.cms.businesslogic.web.Domain.New += (sender, e) => Clear();
// FIXME
// the content class needs to be refactored - at the moment
// content.XmlContentInternal setter does not trigger any event
// content.UpdateDocumentCache(List<Document> Documents) does not trigger any event
// content.RefreshContentFromDatabaseAsync triggers AfterRefresh _while_ refreshing
// etc...
// in addition some events do not make sense... we trigger Publish when moving
// a node, which we should not (the node is moved, not published...) etc.
}
}

2
src/Umbraco.Web/umbraco.presentation/content.cs
View File

@@ -155,7 +155,7 @@ namespace umbraco
{
Cache.ClearCacheObjectTypes("MS.Internal.Xml.XPath.XPathSelectionIterator");
}
requestHandler.ClearProcessedRequests();
_xmlContent = value;
if (!UmbracoSettings.isXmlContentCacheDisabled && UmbracoSettings.continouslyUpdateXmlDiskCache)