-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
Comments
Hi Alex, thanks for your comments, you can do a PR whenever you want, it's open and everyone can collaborate. |
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:
|
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 |
Ok, I'll review the issue and if possible, a new version will be published in a few days. |
Hi @al3xmeister, I've just uploaded a new release 3.3.2, your issue should be solved now. |
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.
The text was updated successfully, but these errors were encountered: