If you want to be stricter on what should be interpreted as numeric (and are using 2012 or later), you could change the WHERE clause to

where try_cast(TPN...

My two favourites were both from COBOL programs many years ago.

1. At the very top of the program, "This program operates on a finite state machine concept". That was the...

I mostly use PRINT and temp tables. One other thing I do in all multi-step stored procedures is to declare a variable called @ErrorLocation that is incremented before each step....

If GP does all its writes via stored procedures, you can deny write to his login and grant execute to the stored procedures.

The thing that disheartened me the most about the Sarbanes-Oxley act was that the very same companies that caused the crisis (eg. Enron's auditors) were able to make vast sums...

One advantage of casting 1 and 0 to bit is that some UIs will default to displaying these as True/False or ticked/unticked without any special coding.

There's a good reason most TV programmes are around the 50-55 minute mark (even on the BBC which doesn't stop for adverts every few minutes) - that's probably the length...

Thanks all. I have some things to try once I have write access to a development copy of the database.

Is it possible that LGTC_ShipmentsClaims is empty (or at least contains no matching rows) at the time your stored procedure is run?

Here's the next key table.
create table SEQ_SYSTEMKEY
  (LASTKEY int null) on [PRIMARY];

and here's the current stored procedure.
create procedure nextSID
as
 ...

The code in the sproc just does an update of column Last_Key on the one row in that table and returns the new value. That's not a real sequence is...