Skip to content

Use TryConvertTypes also for strings#938

Merged
StefH merged 1 commit into
masterfrom
TryConvertTypes
Jul 28, 2025
Merged

Use TryConvertTypes also for strings#938
StefH merged 1 commit into
masterfrom
TryConvertTypes

Conversation

@StefH

@StefH StefH commented Jul 25, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@CDNConsultant

Copy link
Copy Markdown

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

StefH commented Jul 28, 2025

Copy link
Copy Markdown
Collaborator Author

@CDNConsultant
Not yet. I'll try tonight.

@CDNConsultant

Copy link
Copy Markdown

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

@StefH

StefH commented Jul 28, 2025

Copy link
Copy Markdown
Collaborator Author

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
Copy Markdown

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

StefH commented Jul 28, 2025

Copy link
Copy Markdown
Collaborator Author

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

@CDNConsultant

Copy link
Copy Markdown

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

StefH commented Jul 28, 2025

Copy link
Copy Markdown
Collaborator Author

@CDNConsultant

Copy link
Copy Markdown

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
Copy Markdown

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

Thank you for the very quick change and update!!

@StefH

StefH commented Jul 30, 2025

Copy link
Copy Markdown
Collaborator Author

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.

2 participants