Throw exception in case of non-existant bean method (not in model either) #406

Closed
gabordemooij opened this Issue Dec 14, 2014 · 2 comments

Projects

None yet

1 participant

@gabordemooij
Owner

Rivets:

RB doesn't seem to throw an error if you call an undefined method on a bean. This seems like a very dangerous omission. It's not like fields which get checked with the database. There should at least be an option to make the check happen.

@gabordemooij
Owner

Looked into this however the architecture is geared to this principle right now. FUSE events rely on the fact that these methods may not exist. It could be solved by adding a mechanism to identify the FUSE methods, but that would restrict the whole architecture too much.

I don't think this issue is severe enough to revise the architecture.

@gabordemooij gabordemooij reopened this Jan 3, 2015
@gabordemooij
Owner

Changed my mind about this.
Gonna treat this more like a 'feature'.
Allowing devs to set error handlers on beans...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment