Comment 1 for bug 493722

Good catch! This is a case where the user has made a mistake by using findAll('tag-name', text=...) or findAll(True, text=...). According to the API, text=... should override the tag searching.

I put a fix in a private branch. I also combined some of the checks in the two elif tests.

There was one possible issue with using 'not text': it matches for an empty string (""). That is fixed by using 'text is None' instead.