Why is this test for None needed? Isn't sites always a list (empty for no sites)?