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

Fix for table.ajax.reload() with data source "ssd" and ajax object in configuration #135

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

gregorschatz
Copy link

In Microsoft IE 11.0.40 and Apple Safari 12.0.3 I get additional
DataTable initialisations with data source detected as 'dom' and settings Object containing ajax set to null.
It nevertheless tries to build an ajax request and thus and breaks when
ajax from settions object comes into use.

var table = $("#table").DataTable({
serverSide: true,
ajax: {
url:"...",
method:"...",
data: function (){
// ...
}
}
});

$("#table").DataTable().ajax.reload();

…x object in configuration:

In Microsoft IE 11.0.40 and Apple Safari 12.0.3 I get additional
DataTable initialisations with data source detected as 'dom' and settings Object containing ajax set to null.
It nevertheless tries to build an ajax request and thus and breaks when
ajax from settions object comes into use.

var table = $("#table").DataTable({
	serverSide: true,
	ajax: {
		url:"...",
		method:"...",
		data: function (){
			// ...
		}
	}
});

$("#table").DataTable().ajax.reload();
@DataTables
Copy link
Collaborator

Hi

Thanks for the PR. Are you able to give me a link to a test case showing the issue please? If you are also able to add a unit test for the issue it is addressing, that would be great and let me understand the problem more fully.

@gregorschatz
Copy link
Author

It will take some time but I try to construct a live example for the issue.

@DataTables
Copy link
Collaborator

That would be excellent. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants