I'm moving a project from project.json to the new-style csproj format, and it includes a class derived from DbParameterCollection. In my real project I'm using multi-targeting, but for the purposes of this question we only need to care about net45.
The compiler is telling me that I have to override three properties that I didn't have to before:
If you follow those documentation links (which are for .NET 4.5) you'll see that all the properties are virtual - not abstract. If I build the code just by calling csc, all is well... it's only when using the .NET Core SDK that I run into the issue.
Here's sample code to reproduce the problem:
Project file:
<Project Sdk="Microsoft.NET.Sdk">
<PropertyGroup>
<TargetFramework>net45</TargetFramework>
</PropertyGroup>
</Project>
C# code:
using System;
using System.Collections;
using System.Data.Common;
public class DummyParameterCollection : DbParameterCollection
{
public override int Count => 0;
public override object SyncRoot => null;
public override void Remove(object value) {}
public override void RemoveAt(int index) {}
public override void RemoveAt(string parameterName) {}
public override int Add(object value) => 0;
public override void Insert(int index, object value) {}
public override void AddRange(Array values) {}
public override void Clear() {}
public override bool Contains(object value) => false;
public override bool Contains(string value) => false;
public override void CopyTo(Array array, int index) {}
public override int IndexOf(object value) => -1;
public override int IndexOf(string parameterName) => -1;
protected override DbParameter GetParameter(int index) => null;
protected override DbParameter GetParameter(string parameterName) => null;
protected override void SetParameter(int index, DbParameter value) {}
protected override void SetParameter(string parameterName, DbParameter value) {}
public override IEnumerator GetEnumerator() => null;
}
Errors:
DummyParameterCollection.cs(5,14): error CS0534: 'DummyParameterCollection' does not implement inherited abstract member 'DbParameterCollection.IsSynchronized.get' [c:\Users\skeet\Test\ParameterCollection\ParameterCollection.csproj]
DummyParameterCollection.cs(5,14): error CS0534: 'DummyParameterCollection' does not implement inherited abstract member 'DbParameterCollection.IsFixedSize.get' [c:\Users\skeet\Test\ParameterCollection\ParameterCollection.csproj]
DummyParameterCollection.cs(5,14): error CS0534: 'DummyParameterCollection' does not implement inherited abstract member 'DbParameterCollection.IsReadOnly.get' [c:\Users\skeet\Test\ParameterCollection\ParameterCollection.csproj]
I believe I know the immediate cause of the problem, but not the reasons why it's like this, or the best workaround.
It looks like the .NET Core SDK (and VS2017 when it's loaded this project) uses the reference assemblies. If I open C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.5\System.Data.dll in Reflector, that shows the properties as being abstract as well. Whereas if I open c:\Windows\Microsoft.NET\Framework\v4.0.30319\System.Data.dll, that shows the properties as being virtual.
I can work around this by overriding the properties and just returning false from all of them - but is that the best way of handling this situation? Beyond that, is there any good reason why the reference assemblies don't match the real assemblies (and documentation) in this case? I would expect the reference assemblies to be autogenerated, so it's odd for some things to be incorrect like this...
