I must be missing something. Doesn't dr["Id"] return an object? There are only 10 types of people in the world, those who understand binary and those who don't.

The next gem is to be expected in the Untested Sql Procedure (usp_) ErrorLogInsert: in case of an exception happening there, it will check the location parameter, and if it starts with "insertErrorLog" not throw an exception but set errorID to null (that's why it's nullable). And the guy never checks the value of errorID after execution!