Skip to content

Commit

Permalink
Merge pull request #11277 from umbraco/v9/bugfix/11081_userRepo_doesn…
Browse files Browse the repository at this point in the history
…t_retrieve_tourdata

Removed obsolete method
  • Loading branch information
nikolajlauridsen authored Oct 5, 2021
2 parents 846b1c4 + 80c7496 commit a013cfb
Showing 1 changed file with 1 addition and 16 deletions.
17 changes: 1 addition & 16 deletions src/Umbraco.Infrastructure/Persistence/UmbracoPocoDataBuilder.cs
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
using System;
using System.Reflection;
using NPoco;
using Umbraco.Cms.Infrastructure.Persistence.Dtos;

namespace Umbraco.Cms.Infrastructure.Persistence
{
Expand All @@ -18,6 +16,7 @@ namespace Umbraco.Cms.Infrastructure.Persistence
/// <para>So far, this is very manual. We don't try to be clever and figure out whether the
/// columns exist already. We just ignore it.</para>
/// <para>Beware, the application MUST restart when this class behavior changes.</para>
/// <para>You can override the GetColmunnInfo method to control which columns this includes</para>
/// </remarks>
internal class UmbracoPocoDataBuilder : PocoDataBuilder
{
Expand All @@ -28,19 +27,5 @@ public UmbracoPocoDataBuilder(Type type, MapperCollection mapper, bool upgrading
{
_upgrading = upgrading;
}

protected override ColumnInfo GetColumnInfo(MemberInfo mi, Type type)
{
var columnInfo = base.GetColumnInfo(mi, type);

// TODO: Is this upgrade flag still relevant? It's a lot of hacking to just set this value
// including the interface method ConfigureForUpgrade for this one circumstance.
if (_upgrading)
{
if (type == typeof(UserDto) && mi.Name == "TourData") columnInfo.IgnoreColumn = true;
}

return columnInfo;
}
}
}

0 comments on commit a013cfb

Please sign in to comment.