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

I had to remove that line cause, apparently, it was trying to delete my entity before the children. #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Leftyx
Copy link

@Leftyx Leftyx commented Apr 7, 2011

No description provided.

@Leftyx Leftyx closed this Apr 7, 2011
@Leftyx Leftyx reopened this Apr 7, 2011
@nrstott
Copy link

nrstott commented Apr 7, 2011

Can you provide a unit test demonstrating the problem that this solves? Thanks!

@Leftyx
Copy link
Author

Leftyx commented Apr 11, 2011

I've prepared it but I don't know how to update the repository :-s I am new to github.

@nrstott
Copy link

nrstott commented Apr 11, 2011

Try doing a 'git commit -am "added unit test"' then 'git push origin master'

vandalo added 2 commits April 12, 2011 09:57
…d when working with MS Sql Server (2008) and the DeleteEntityEventListener.
…d when working with MS Sql Server (2008) and the DeleteEntityEventListener.

Signed-off-by: vandalo <[email protected]>

Signed-off-by: vandalo <[email protected]>
@Leftyx
Copy link
Author

Leftyx commented Apr 12, 2011

nrstott: I think I've done it. I've added a unit test and I had to create a new database fixture cause the existing one works with SqlLite and everything works fine there. My problem (apparently) is with Sql Server.

@ols
Copy link

ols commented Oct 11, 2011

We looked at this row and the reason it's failing for us is because of the auto-flush in NHibernate before potentially dirty (out of sync) reads, which is the case for the listing below the line in question in this pull request. See our pull request. /Olle, Henrik

joramwangi pushed a commit to joramwangi/rhino-security that referenced this pull request Dec 3, 2018
Add simple implementation of OnPreDeleteAsync()
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

Successfully merging this pull request may close these issues.

3 participants