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

Unexpected Behavior #84

Open
sethrothman opened this issue Sep 5, 2024 · 3 comments
Open

Unexpected Behavior #84

sethrothman opened this issue Sep 5, 2024 · 3 comments

Comments

@sethrothman
Copy link

Hi Joel,

First off, your library has been my "go to" for years so I'll start by expressing my gratitude. And then I'll apologize for this lengthy post...

Here's my situation. I use the DevExpress suite of components when building WinForms apps. In particular, their Grid control exhibits weird behavior when bound to an AsTrackable() collection - but only when using a more current version of Castle.Core (v5+). I opened a support ticket with DevExpress and they helped troubleshoot and narrow the scope of the issue.

Their grid has a property called "ImmediateUpdateRowPosition" which determines whether or not an edited column's sorting/grouping is applied once the field (not row) is committed. I always turn this setting to False because the result is disconcerting otherwise IMHO. In other words, if you have a grid that's grouped, changing the value of the grouped field will automatically move the row to its "new" grouped location which means you can't edit the remaining fields in the row first. I attached a small video showing the undesired behavior.

Anyway, the issue is that the ImmediateUpdateRowPosition property is basically ignored in the scenario I described above. If I revert to v4 of Castle.Core, everything works as expected. Any thoughts on why the behavior changed? I generally try to keep current with 3rd party nuget packages and Castle.Core is the one standout I'm unable to upgrade.

2023-07-12_05-51-48.mp4

FYI, I attached the small sample I was working with DevExpress on in case you wanted to see it in action. It'd require you install a trial of the DevEx components unfortunately.

T1177040.zip

If you have any thoughts, I'd love to hear them. Thx.

@barmendarez
Copy link

We also have had unexpected behavior with this library. There is something breaking with the the latest Castle.Core (EF Core uses the latest version so we had to update)

@joelweiss
Copy link
Owner

This library is not compatible with Castle.Core v5 due to #647, the issue you are experiencing might be related.

Unfortunately I have not had the time (yet) to come up with a fix.

@sethrothman
Copy link
Author

sethrothman commented Oct 8, 2024 via email

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

4 participants
@joelweiss @sethrothman @barmendarez and others