Files
@ 260a7a01b054
Branch filter:
Location: kallithea/rhodecode/lib/dbmigrate/versions/009_version_1_5_1.py - annotation
260a7a01b054
722 B
text/x-python
follow Python conventions for boolean values
True and False might be singletons and the "default" values for "boolean"
expressions, but "all" values in Python has a boolean value and should be
evaluated as such. Checking with 'is True' and 'is False' is thus confusing,
error prone and unnessarily complex.
If we anywhere rely and nullable boolean fields from the database layer and
don't want the null value to be treated as False then we should check
explicitly for null with 'is None'.
True and False might be singletons and the "default" values for "boolean"
expressions, but "all" values in Python has a boolean value and should be
evaluated as such. Checking with 'is True' and 'is False' is thus confusing,
error prone and unnessarily complex.
If we anywhere rely and nullable boolean fields from the database layer and
don't want the null value to be treated as False then we should check
explicitly for null with 'is None'.
102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 102ef3f1b849 | import logging
import datetime
from sqlalchemy import *
from sqlalchemy.exc import DatabaseError
from sqlalchemy.orm import relation, backref, class_mapper, joinedload
from sqlalchemy.orm.session import Session
from sqlalchemy.ext.declarative import declarative_base
from rhodecode.lib.dbmigrate.migrate import *
from rhodecode.lib.dbmigrate.migrate.changeset import *
from rhodecode.model.meta import Base
from rhodecode.model import meta
log = logging.getLogger(__name__)
def upgrade(migrate_engine):
"""
Upgrade operations go here.
Don't create your own engine; bind migrate_engine to your metadata
"""
pass
def downgrade(migrate_engine):
meta = MetaData()
meta.bind = migrate_engine
|