But this would:

create procedure procname

as BEGIN

statement 1

statement 2

...

statement n

END

grant execute to procname to role1

grant...

Which,...

As for preferences, I always use the Begin/End in IF/ELSE constructs but also for stored procs and, yes, I indent the entire block of code. This is personal preference but...

The index is the best solution, except that as you are querying for the MAX date, then I would override the default and define the index as descending.

create index IX_Name...

It's not so much that you're not returning a value from a function. You could hard code a return value if you want and it still won't compile. However, the...

I think that one of the methods you could use to make this code a lot easier to work with is to write out just exactly what it is doing...

You have a many-to-many relationship between stations and categories but have not modeled the intersection table correctly. Instead of

Table2 : StationCategoryLink

SampleData : StationCode ...

You can truncate the time part and simplify the calculation in one swell foop:

select DateAdd( m, DateDiff( m, 0, @NextMonth ) + 1, 0 );

Leave out the "+ 1" and...

You may want to version the table (see the link in my sig to get a document explaining the details). But that allows you to query the table to see...

You don't have begin-end delimiters after the else. So only the first print statement (print @pallet) is getting executed when the ifcondition is false. The statements from the second print...

You have got to get off the track you are on or it will be very difficult...

None of which are Money. All those rates will be Decimal, Numeric,...

By any chance, would there be an Instead Of trigger defined on the table?

Not exactly. The problem is not quite...

Obviously, what you are trying to do can be done and it makes for a good exercise in solving problems using SQL. However, my advice is don't do it at...