dotnet optimize fails because of private JsonReaderWriters

[arizowa]

Hi, I upgraded my project to .NET 8.0
And when I run optimize dbcontext, I received error:

'NpgsqlTimestampTzTypeMapping.NpgsqlJsonTimestampTzDateTimeOffsetReaderWriter' is inaccessible due to its protection level

Please see attact image

The command is:

ef dbcontext optimize -o OptimizedModels -n ProjectName.Infrastructure.OptimizedModels -c PrimaryApplicationDbContext

What can I do to fix this issue? Any workaround for this?

[JustArchi]

@roji got the same problem compiling my own project, says "Cannot access private class NpgsqlJsonTimestampTzDateTimeOffsetReaderWriter here". Seems like oversight and just making that class public or whatever would be sufficient, but of course you're expert here 👍

If you need reproduction, you can try:

[PrimaryKey(nameof(WalletType), nameof(AssetType), nameof(RecordedAt))]
[Table]
public sealed class AssetEntry {
	[Column]
	[Key]
	public int AssetType { get; set; }

	[Column]
	[Key]
	public DateTimeOffset RecordedAt { get; set; }

	[Column]
	[Key]
	public int WalletType { get; set; }
}

[roji]

@JustArchi thanks for reporting, I'll take a look at this for 8.0.1.

[arizowa]

How can we workaround to resolve this issue while waiting for the next release?

[eero-dev]

Getting a similar issue, but with NpgsqlJsonTimestampTzDateTimeReaderWriter, so it's probably all related?

error CS0122: 'NpgsqlTimestampTzTypeMapping.NpgsqlJsonTimestampTzDateTimeReaderWriter' is inaccessible due to its prot
ection level

Optimized code:

           var endDate = runtimeEntityType.AddProperty(
               "EndDate",
               typeof(DateTime),
               propertyInfo: typeof(DateModel).GetProperty("EndDate", BindingFlags.Public | BindingFlags.Instance | BindingFlags.DeclaredOnly),
               fieldInfo: typeof(DateModel).GetField("<EndDate>k__BackingField", BindingFlags.NonPublic | BindingFlags.Instance | BindingFlags.DeclaredOnly),
               valueConverter: new UtcDateTimeConverter());
           endDate.TypeMapping = NpgsqlTimestampTzTypeMapping.Default.Clone(
               comparer: new ValueComparer<DateTime>(
                   (DateTime v1, DateTime v2) => v1.Equals(v2),
                   (DateTime v) => v.GetHashCode(),
                   (DateTime v) => v),
               keyComparer: new ValueComparer<DateTime>(
                   (DateTime v1, DateTime v2) => v1.Equals(v2),
                   (DateTime v) => v.GetHashCode(),
                   (DateTime v) => v),
               providerValueComparer: new ValueComparer<DateTime>(
                   (DateTime v1, DateTime v2) => v1.Equals(v2),
                   (DateTime v) => v.GetHashCode(),
                   (DateTime v) => v),
               converter: new ValueConverter<DateTime, DateTime>(
                   (DateTime v) => v,
                   (DateTime v) => DateTime.SpecifyKind(v, DateTimeKind.Utc)),
               jsonValueReaderWriter: new JsonConvertedValueReaderWriter<DateTime, DateTime>(
                   new NpgsqlTimestampTzTypeMapping.NpgsqlJsonTimestampTzDateTimeReaderWriter(),
                   new ValueConverter<DateTime, DateTime>(
                       (DateTime v) => v,
                       (DateTime v) => DateTime.SpecifyKind(v, DateTimeKind.Utc))));
           endDate.SetSentinelFromProviderValue(new DateTime(1, 1, 1, 0, 0, 0, 0, DateTimeKind.Unspecified));
           endDate.AddAnnotation("Npgsql:ValueGenerationStrategy", NpgsqlValueGenerationStrategy.None);

😅

[roji]

Everyone, I'm currently away and so unable to investigate this; I'm sure there's an actual issue here, but it'll take me a few days before I can investigate and fix it. You can probably expect a 8.0.1 to be released soon with the fix.

[springy76]

@MINHMIT my workaround is to change the generated files so they look like this:

jsonValueReaderWriter: new NpgsqlTimestampTzTypeMapping(typeof(DateTimeOffset)).JsonValueReaderWriter);

which finally should return NpgsqlJsonTimestampTzDateTimeOffsetReaderWriter.Instance

[2Xpro-pop]

@MINHMIT my workaround is to change the generated files so they look like this:

jsonValueReaderWriter: new NpgsqlTimestampTzTypeMapping(typeof(DateTimeOffset)).JsonValueReaderWriter);

which finally should return NpgsqlJsonTimestampTzDateTimeOffsetReaderWriter.Instance

Thank you very much.You helped me solve this problem, I wouldn't have done it without you.

[arizowa]

@MINHMIT my workaround is to change the generated files so they look like this:

jsonValueReaderWriter: new NpgsqlTimestampTzTypeMapping(typeof(DateTimeOffset)).JsonValueReaderWriter);

which finally should return NpgsqlJsonTimestampTzDateTimeOffsetReaderWriter.Instance

Thank you so much.