The structure looks okay, the issue I think you may have is in dealing with how you deal with transactions and when/how you commit or rollback based on the errors...

Really sick of JC and his comments.

Basically, you need to explicit list the columns you want returned from the tables instead of using the *.

So, I post code because I get called out and he goes all crickets on us.

Okay, call me out.  This is also a rough example based solely on your procedure.  it is not what I would do in a production environment as it is missing...

Perhaps to encourage the original author to continue the series or find someone else that may be willing to do so.