NETProvider icon indicating copy to clipboard operation
NETProvider copied to clipboard

HasRows should be hidden or throw NotSupportedException

Open Conman-123 opened this issue 3 years ago • 1 comments

According to https://github.com/FirebirdSQL/NETProvider/issues/317, HasRows always returns true as it is not supported by Firebird. However, this is not documented anywhere and is causing "There are no data to read" exceptions because it always returns true.

It would be very helpful to hide this property, or throw NotSupportedException, or change it to always return false with a docstring explaining that it is not supported by Firebird.

Obviously this would be a big breaking change.

Conman-123 avatar Feb 14 '23 06:02 Conman-123

It's unfortunate situation. Although the change to NotSupportedException is trivial, I'd like to hear more opinions whether such breaking change is worth it. I personally don't have strong feeling about one or the other.

cincuranet avatar Feb 14 '23 09:02 cincuranet