You've implemented this differently than the previous controllers - is there a reason? All the other fetch the parent object first, this does it inline.  I don't really care which way it's done, though I think this way will result in less database hits, but we should be consistent.