lsm/Orchard
1
0
Fork 0
mirror of https://github.com/OrchardCMS/Orchard.git synced 2025-10-21 19:34:40 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

#19380: Removing needless content type definition fetching from CommonPartHandler

Browse Source 
Work Item: 19380

--HG--
branch : 1.x
This commit is contained in:
Zoltán Lehóczky
2013-05-15 16:59:31 +02:00
parent 6ca76c68bb
commit 3fef593881
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/Orchard.Web/Core/Common/Handlers/CommonPartHandler.cs
View File

@@ -9,6 +9,7 @@ using Orchard.ContentManagement;
using Orchard.ContentManagement.Handlers;
using Orchard.Security;
using Orchard.Services;
using Orchard.ContentManagement.MetaData.Models;
namespace Orchard.Core.Common.Handlers {
[UsedImplicitly]
@@ -63,14 +64,12 @@ namespace Orchard.Core.Common.Handlers {
public Localizer T { get; set; }
protected override void Activating(ActivatingContentContext context) {
if (ContentTypeWithACommonPart(context.ContentType))
if (ContentTypeWithACommonPart(context.Definition))
context.Builder.Weld<ContentPart<CommonPartVersionRecord>>();
}
protected bool ContentTypeWithACommonPart(string typeName) {
protected bool ContentTypeWithACommonPart(ContentTypeDefinition contentTypeDefinition) {
//Note: What about content type handlers which activate "CommonPart" in code?
var contentTypeDefinition = _contentDefinitionManager.GetTypeDefinition(typeName);
if (contentTypeDefinition != null)
return contentTypeDefinition.Parts.Any(part => part.PartDefinition.Name == "CommonPart");