I went through the code and was planning on proposing a recommendation
- I can create the patch for the code if you think it's worthwhile.
The behavior I would expect is that if None is provided in the
contains() statement, it would just return no results (because GAE
won't allow None in lists). I
session.company_id is None in your case and None cannot be contained
in a list of strings.
It has to fail in some place. We could catch the error somewhere else
and have a more meaningful error but you really want something like:
if session.company_id:
rows =
db(db.Site_Main.companyID.contains(
2 matches
Mail list logo