That is how I would expect it to be wired. It is called the loop in method, there is nothing wrong with it, and as you found switching off the switch does not "kill" anything. I wouldn't want to see you get it all wrong, and as it is not what you expected, I can only suggest you get a comp...