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

Null reference in LoadTable method #6

Open
al3xmeister opened this issue Aug 1, 2020 · 5 comments
Open

Null reference in LoadTable method #6

al3xmeister opened this issue Aug 1, 2020 · 5 comments
Labels
bug Something isn't working

Comments

@al3xmeister
Copy link

Line 43 (dtparameters.Order reference) and 37 (dtParameters.Search reference) leave null references. Happy to do a PR and contribute if you'd like me to give a hand. Well done on setting 3.1 up.

@DavidSuescunPelegay
Copy link
Owner

Hi Alex, thanks for your comments, you can do a PR whenever you want, it's open and everyone can collaborate.

@DavidSuescunPelegay DavidSuescunPelegay added the bug Something isn't working label Aug 12, 2020
@DavidSuescunPelegay
Copy link
Owner

Hi, I've been trying to solve the bug you issued, but I've not been able to reproduce it.

Please, could you try some of the tips below and, if the bug continues, you can send me screenshots or a small video and I'll research the bug more deeply:

  • Delete bin and obj cache folders
  • Clear NuGet cache
  • Clean and Rebuild the project several times
  • Try different build setups (IIS Express or custom)
  • Update your NuGet Packages using this command Update-Package -reinstall
  • Enable NuGet package restore

@al3xmeister
Copy link
Author

Hi David,

Sorry I've been exceptionally busy but been meaning to reply. The error I was getting was the dtParameters being null, this was caused by the version of Jquery I was using, but is quite a common error as seen https://stackoverflow.com/questions/59327018/asp-net-core-3-1-controller-method-always-recieves-null-parameter-from-httppost.

I was suggesting either adding a null check against dtParameters .search or .Order references, or adding some sort of comment for the developers in the same position as I was to be aware dtParameters should never be null and if that is the case the implementation of dataTables failed somewhere.

Best regards

@DavidSuescunPelegay
Copy link
Owner

Ok, I'll review the issue and if possible, a new version will be published in a few days.

@DavidSuescunPelegay
Copy link
Owner

Hi @al3xmeister, I've just uploaded a new release 3.3.2, your issue should be solved now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants