remove unused ul var
move isEmpty check into Hours.prototype.serialize

We're checking for it everywhere serialize is used. Subtle behavior change is
assigning null instead of not assigning at all, but I think this is clearer at least.
cleanup edit_view.js

* remove some console.log statements
* move variable declarations to where they're used
* break up addService behavior