lsm/Orchard
1
0
Fork 0
mirror of https://github.com/OrchardCMS/Orchard.git synced 2025-11-24 08:46:48 +08:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Save MemberBindingRecords in memory (#8371)

Browse Source 
Prevent fetching the same table from the database 5+ times per request by
loading it and saving it in a private property for a request.
This commit is contained in:
Matteo Piovanelli
2020-05-07 19:29:58 +02:00
committed by GitHub
parent a75203b883
commit 60464035f7
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/Orchard.Web/Modules/Orchard.Projections/Services/CustomMemberBindingProvider.cs
View File

@@ -1,4 +1,5 @@
using System;
using System.Collections.Generic;
using System.Linq;
using System.Reflection;
using Orchard.Data;
@@ -16,11 +17,18 @@ namespace Orchard.Projections.Services {
_sessionFactoryHolder = sessionFactoryHolder;
}
private List<MemberBindingRecord> memberBindings;
public void GetMemberBindings(BindingBuilder builder) {
var recordBluePrints = _sessionFactoryHolder.GetSessionFactoryParameters().RecordDescriptors;
foreach(var member in _repository.Table.ToList()) {
// save this in memory once per request, to avoid hitting the database 5+
// times per projection per request.
if (memberBindings == null) {
memberBindings = _repository.Table.ToList();
}
foreach (var member in memberBindings) {
var record = recordBluePrints.FirstOrDefault(r => String.Equals(r.Type.FullName, member.Type, StringComparison.OrdinalIgnoreCase));
if (record == null) {
@@ -28,7 +36,7 @@ namespace Orchard.Projections.Services {
}
var property = record.Type.GetProperty(member.Member, BindingFlags.Instance | BindingFlags.Public);
if(property == null) {
if (property == null) {
continue;
}