Managers abstract methods should probably raise NotImplementedErrors

vertical-knowledge / ripozo

A tool for quickly creating REST/HATEOAS/Hypermedia APIs in python

http://ripozo.readthedocs.org/

GNU General Public License v2.0

201 stars 22 forks source link

Managers abstract methods should probably raise NotImplementedErrors #56

Open timmartin19 opened 8 years ago

timmartin19 commented 8 years ago

This would prevent dumb errors that I tend to make. Like calling the wrong super. E.g.

class MyManager(AlchemyManager):
     model = MyModel

     def create(self, values, *args, **kwargs):
          super(AlchemyManager, self).create(values, *args, **kwargs)
          # Do something else