Skip to content

Conversation

@StefH
Copy link
Collaborator

@StefH StefH commented Jul 25, 2025

No description provided.

@CDNConsultant
Copy link

Hello StefH - I am somewhat of a newbie at GitHub - is this saying that you have uploaded the fix already? If so - I will get it and test it on my projects.

@StefH
Copy link
Collaborator Author

StefH commented Jul 28, 2025

@CDNConsultant
Not yet. I'll try tonight.

@CDNConsultant
Copy link

Ok - No pressure - just not sure of what some of the icons meant :) Have a great day!

@StefH
Copy link
Collaborator Author

StefH commented Jul 28, 2025

You can always check the changelog, which includes the bugs + PullRequests :
https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/master/CHANGELOG.md

@StefH StefH merged commit dd7267c into master Jul 28, 2025
6 checks passed
@StefH StefH deleted the TryConvertTypes branch July 28, 2025 14:24
@CDNConsultant
Copy link

So - I see the change log about the Config object. Is that all I have to do - or do I need any new code changes?

@StefH
Copy link
Collaborator Author

StefH commented Jul 28, 2025

New version is released on NuGet. So you need that one and that config setting

@CDNConsultant
Copy link

Hello StefH

Ok - just did a re-install from NuGet - Version 1.67 (uploaded today) - and changed my code to use the config object (see below) and it still returns Zero selected items...

Am I missing something?

        private void button_RunLINQ_Click(object sender, EventArgs e)
        {
            LoadFromFile();

            var config = new ParsingConfig();
            config.AllowEqualsAndToStringMethodsOnObject = true;    

            CopyDT = SampleDS.Tables["a"];

            var dynamicQuery = CopyDT.AsEnumerable().AsQueryable();
            var dynamicQuery2 = dynamicQuery.Where(config, "City == \"Seattle\"").ToList();
            MessageBox.Show("Dynamic LINQ Total List Recs: " + dynamicQuery2.Count.ToString());


        }

@StefH
Copy link
Collaborator Author

StefH commented Jul 28, 2025

@CDNConsultant
Copy link

That seems to have resolved the issue. Thank you very much - I will test it out further over the next few days!!

I really am grateful for your help on this! I would like to offer you a free copy of the tool that this will be used in - called InSights Into Data (www.insightsintodata.com). You can check out the program - and if it is something you want to use - just send me an email to brad_AT_insightsintodata_DOT_com and I will get you a free license key!

@CDNConsultant
Copy link

StefH - in all my current testing - this now seems to be functioning well.

Thank you for the very quick change and update!!

@StefH
Copy link
Collaborator Author

StefH commented Jul 30, 2025

StefH - in all my current testing - this now seems to be functioning well.

Thank you for the very quick change and update!!

Good to hear.

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

Labels

Development

Successfully merging this pull request may close these issues.

3 participants