Comment 3 for bug 392216

«text_name = filter.get("text_name", None)
if text_name is not None:»

That's overkill, just «if "text_name" in filter:» and the «filter["text_name"]» is more than enough.

Looks good otherwise. Thanks for working on this.