Shouldn't all the checking be in a model? I see they are all here in the my_subscription controller which are actions that render nothing and redirect no where but return true or false