pyeve / eve-sqlalchemy

SQLAlchemy data layer for Eve-powered RESTful APIs
http://eve-sqlalchemy.readthedocs.io
Other
232 stars 70 forks source link

ValidatorSQL class doesn't handle/pass allow_unknown #70

Closed lingfish closed 9 years ago

lingfish commented 9 years ago

Hi, just discovered that the abovementioned class doesn't notice when allow_unknown is set to True.

The key lines missing/needed from Eve are here (in __init__):

        if resource:
            self.allow_unknown = config.DOMAIN[resource]['allow_unknown']

I've worked around this (and it works fine) by subclassing and adding those two lines.

amleczko commented 9 years ago

Can you make a PR instead with a tiny test?