RefactoringEssentials icon indicating copy to clipboard operation
RefactoringEssentials copied to clipboard

Inconsistent RECS0082

Open sm-g opened this issue 8 years ago • 1 comments

public class QbCacheWriterTests : QbCacheTestsBase
{
	private StoreWithQbItems store;
	
	private QbCacheWriter CreateQbCacheWriter(DataStub store) // warns Parameter 'store' hides field 'store'
	{
		...
	}

	protected override IReadOnlyCollection<object> MakeDbSetMocks(DataStub store, dynamic settings = null) // no warning
	{
		...
	}
}

sm-g avatar Apr 27 '17 09:04 sm-g

Current implementation ignore public, protected, abstract and override methods as well as constructors and expicit interface implementations. As for me it doesn't have a sense, maybe except abstract modifier, but i could probably miss something due to my low experience in C#.

So before fixing i would like someone more experienced to let me know which of these restrictions are really necessary and which can be removed.

AndrewBoklashko avatar May 13 '17 11:05 AndrewBoklashko