I don't think using the transaction name to determine which action to take is the right model. Your application already knows whether the user intends to insert, update or delete...

In your database, create a user-defined table type that matches the columns in "myorder". Your stored procedure can then specify a parameter that uses this table type instead of one...

Not an explanation of your problem, but in our warehouse we tend to drop indexes then truncate/load then recreate the indexes. It is generally a lot quicker on low-spec servers...

Regarding your 2nd question "should I worry about it", I'd say it's definitely worth giving them sensible names if you have copies of the database for development, QA, etc. and...

When you insert the copied rows...

Select DeveloperName, HitCount
from AppDevelopers
cross apply ufn_BigStick(AppDeveloperID)

GO 20

Why do you want to exclude the IDENTITY columns in the first place? If you use SET IDENTITY_INSERT <your table> ON you can copy the identity values across as well.

You can sum a nullable column. Rows where the column is null will not contribute to the total, so leaving the marks null when the student was absent (for whatever...

I'm assuming you can't use suser_name() or an equivalent because the application connects using a SQL Server login rather than windows authentication. You could prevent deletes, etc. from occurring outside...

I think the UserID variable in your code will be receiving the RETURN value from executing the stored procedure. This is not the same as the results of the SELECT...

Running multiple SQL statements passed in as a string.

This method looks highly vulnerable to SQL injection attacks.

"hiring to support growth" - what is that?

Are you able to change the parameter to a table-parameter? If so, you can add your array of values to it and use it in an inner join.

Just as an aside; I have got into the habit of using the "Script as create" instead of "Script as alter" or "Modify" options when looking at stored procedures, functions,...