Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Destroy(Expression) method in ActiveRecord throws error #42

Open
georgexcollins opened this issue Feb 8, 2010 · 1 comment
Open

Destroy(Expression) method in ActiveRecord throws error #42

georgexcollins opened this issue Feb 8, 2010 · 1 comment

Comments

@georgexcollins
Copy link

See stackoverflow:
http://stackoverflow.com/questions/2141738/subsonic-3-activerecord-delete-destroy-throwing-strange-exception/2187630#2187630

If a table has logical delete isDeleted column, then the templates generate a static Destroy(Expression) method. However this calls the underlying SubSonicRepository Delete(Expression) which doesn't exist.

I think the solution is to change the template to call SubSonicRepository.DeleteMany(Expression) instead which does exist. Should the ActiveRecord static method also be renamed to "DestroyMany"?

@georgexcollins
Copy link
Author

Pull request submitted with a fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant