Nullable columns with a default set should not be "required" by registerSchema

[fubu]

A column of the form is_public = Column(Boolean, default=False, nullable=False) produces a schema with required=True, although the intention of the default argument is explicitly to not require a user to provide a value. The culprit is the registerSchema decorator.

I propose changing that line to something like:

                schema['required'] = not (col.nullable or col.default is None) \
                    if not col.primary_key else False

[jeroenes]

I second fubu's proposal.

Is there a workaround?

[dkellner]

I can confirm this bug. The right way to tackle this would be:

• Create a test case (similar to the one I created here: https://github.com/dkellner/eve-sqlalchemy/tree/fix-sqlalchemy-default)
• Incorporate @fubu's code and see if all tests still pass