Small performance gain for check_for_subscription()

Since we aren't /actually/ returning the subscription type, it makes no
sense to parse the XML and then search the dict.

On the rpi, parsing and searching takes 0.1s. On a fresh login, we call
this twice. Since this new check is basically instantaneous, rpi users
get a 0.2s speedup.

In the future, if/when we support alternate subscription types
(team only, etc), then we may need to go back to parsing the XML.
Until then, let's take this performance win.
Invert logic flow for sections vs controlIDs

This way each section is grouped together, making it easier to follow
the navigation of a user. For example, all RedZone related actions are
grouped together now, making it less likely that the (recently fixed)
omission of RedZone - Live will happen again.

This does introduce some redundant code (namely week_list, and game_list
resets). There's some additional cleanup to be done and potential to
split some actions into functions.