Mvc.JQuery.DataTables.Common's dependency on System.Linq.Dynamic.Core causes run-time error in an ASP.NET app

Mvc.JQuery.DataTables.Common (latest is 1.5.34) depends on System.Linq.Dynamic.Core.1.0.6.13. Problem is that System.Linq.Dynamic.Core now has version 1.0.7.5. Even though these are stepping version changes only, when I upgrade System.Linq.Dynamic.Core to the latest I get a run-time error:

System.IO.FileLoadException: ‘Could not load file or assembly ‘System.Linq.Dynamic.Core, Version=1.0.0.0, Culture=neutral, PublicKeyToken=0f07ec44de6ac832’ or one of its dependencies. The located assembly’s manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)’

When I try to create the return data:

 public DataTablesResult<CheckItemModel> GetCalculationCheckData(DataTablesParam dataTableParam)
    {
        ...
        var calculationChecks = EngineController.getCalculationChecks(id);
        var queryable = calculationChecks.AsQueryable().InterceptWith(new SetComparerExpressionVisitor(StringComparison.CurrentCultureIgnoreCase));
        return DataTablesResult.Create(queryable, dataTableParam, null);  // Here I get the error
    }

[mcintyre321]

I think if this is going to cause lots of people problems, it’s probably worth updating the dependency withing Mvc.Jquery.DataTables - can you send me a PR with the updated dependency? I’ll merge it and get a new version out ASAP.

On 4 July 2017 at 01:44, Csaba Toth [e-mail] wrote:

If I modify the binding redirect (since it is trippin’ about the major version) to

then the app starts. But then I have the runtime error again. Maybe the Mvc.Jquery Datatables package needs a binding redirect internally as well?

[image: screenshot_2017-07-03_17-42-01] https://user-images.githubusercontent.com/744949/27811110-33fabd4a-6017-11e7-82d0-7088a39939a7.png

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mcintyre321/mvc.jquery.datatables/issues/165#issuecomment-312755212, or mute the thread https://github.com/notifications/unsubscribe-auth/AAQ0-o2rghJS6_fCTcPKfgEno0wGo5P4ks5sKYrWgaJpZM4OLvNI .

[mcintyre321]

You’re right, although there is no guarantee that a minor version isn’t breaking - just an intention (and even then only if the developer is intending to use SemVer)…

Anyhow, the new package should be out there now!