5

So this is a continuation of post: Best way to get identity of inserted row?

That post proposes, and I agree, to use Inserted feature to safely return inserted id column(s).

While implementing this feature, it seems SqlClient of the .net framework does not support this feature, and fails while trying to execute command, I get the following exception:

System.Data.SqlClient.SqlException: 'Cannot find either column "INSERTED" or the user-defined function or aggregate "INSERTED.Id", or the name is ambiguous.'

I'm just using:

return (T)command.ExecuteScalar();

Where the query is:

INSERT INTO MyTable 
OUTPUT INSERTED.Id 
(Description) 
VALUES (@Description)

And the table just contains

ID (identity int)
Description (varchar(max))

If impossible to do, is there other safe way without using variables in the middle that might affect performance?

Thanks

marc_s
  • 732,580
  • 175
  • 1,330
  • 1,459
Yogurtu
  • 2,656
  • 3
  • 23
  • 23

1 Answers1

9

You are doing everything correctly, but you have misplaced the OUTPUT clause: it goes after the list of columns and before the VALUES, i.e.

INSERT INTO MyTable (Description) 
OUTPUT INSERTED.Id 
VALUES (@Description)
Sergey Kalinichenko
  • 714,442
  • 84
  • 1,110
  • 1,523
  • Ah thanks, example I saw didn't contain target columns, didn't noticed that... clearly I need to sleep. – Yogurtu Apr 12 '18 at 01:59