you probably want to return [] rather than None