SQLAlchemy 0.9.X对None条件处理的差异

原创
2014/01/07 21:40
阅读数 395

SQLAlchemy 最新的版本已经是0.9.1了,不过有人报告说Uliweb在使用这个版本时有问题,get()不到数据。后来查了一下,发现生成的查询条件多了一个NULL,比如:

Blog.get(1)
SELECT todo.title, todo.post_date, todo.finished, todo.id FROM todo 
WHERE todo.id = 1 AND NULL LIMIT 1 OFFSET 0;

在条件中多了一个NULL,很奇怪,換成0.8.X就好了。然后我仔细查了一下,在Uliweb中,get()会调用filter,而filter本身可以传入多个条件,因此需要将条件以与的方式合并,因此代码是:

cond = None
for c in condition:
    if c is not None:
        cond = c & cond

因此,最后条件会和None相与。

然后就发现是 and_(&) 的处理和以前不同的。以前的代码是在sqlalchemy.sql.expression中的ClauseList的类中,以0.8.3版本为例是在3325行左右,代码为:

if self.group_contents:
    self.clauses = [
        _literal_as_text(clause).self_group(against=self.operator)
        for clause in clauses if clause is not None]
else:
    self.clauses = [
        _literal_as_text(clause)
        for clause in clauses if clause is not None]

可以看到,在for中已经装饰None的条件过滤了,所以对于条件是None的会自动过滤掉。

但是到了0.9.1中,代码发生了很大的变化,这块代码都移到了elements.py中了,而且真正的and_的处理从ClauseList中移到了BooleanClauseList中,并且代码处理也有所不同:

convert_clauses = []

clauses = util.coerce_generator_arg(clauses)
for clause in clauses:
    clause = _literal_as_text(clause)

    if isinstance(clause, continue_on):
        continue
    elif isinstance(clause, skip_on):
        return clause.self_group(against=operators._asbool)

    convert_clauses.append(clause)

没有了对None的过滤。所以造成了None在0.9.1中与0.8.X中不兼容的情况。不知道这算不算一个Bug。不过我已经把uliweb中的cond = None 改为了 cond = '' 。这样结果在0.8.X中还是在0.9.X中都是对的。

所以如果你是在0.9.X中使用None条件,建议改为 true()。true 可以从 sqlalchemy.sql 中导出。

update

如果你希望0.9.x和0.8.x对None的处理一样,我找到一个打补丁的方法:

from sqlalchemy.sql.compiler import SQLCompiler
def visit_null(self, expr, **kw):
    return ''
setattr(SQLCompiler, 'visit_null', visit_null)

这样 NULL 就不会出现。你也可以考虑在这个地方加一个抛异常,强制用户修改,这也是一个办法。

展开阅读全文
打赏
0
5 收藏
分享
加载中
已经更新Uliweb,找时间再试下
2014/01/08 12:52
回复
举报
更多评论
打赏
1 评论
5 收藏
0
分享
返回顶部
顶部