-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
'refresh' method doesn't reuse parameters provided as "query", it only takes "url". #7318
Comments
Please provide an Online Example to show your problem. Thanks! |
We will close this issue as we got no response form you. If you still need help with that please provide us an example as @wenzhixin said. |
@UtechtDustin -- We will close, or already closing? |
@wenzhixin here you go @UtechtDustin please don't close the issue as “Completed” immediately after reminder. This is extremely discouraging. |
Just in case, here is the source code of PHP script which generating content for my example on https://live.bootstrap-table.com/code/utilmind/17717
|
As I mentioned in my top comment, I'm not contributing the fix for this issue because a workaround with ?parameters= in the |
Sorry, my bad. I don't know that the close button marks it as completed or that there is a close option without out the completed part. |
The Working demo: https://live.bootstrap-table.com/code/UtechtDustin/17720 |
@UtechtDustin Okay, yes, my bad I specified the query incorrectly in my example, but it was correct in my real app. Here is another example: https://live.bootstrap-table.com/code/utilmind/17722
|
You're right! That should be an easy fix, as soon we get a query object we have to add/merge it to our base url. |
Should be fixed by #7353. |
Bootstraptable version(s) affected
1.22.5 (latest)
Description
Unfortunately "refresh" button on toolbar is useless, if I originally provide the request parameters as "query" property, instead of straightforward GET-query, as part of "url" property.
So if I originally provide data as follows, this will not work on manual "refresh" (click on refresh button on toolbar), it will just ignore "query" on repeating request:
However, the following will work:
I think it should not ignore data provided in "query" on repeating "refreshes".
The workaround provided above is works for me, so I don't submitting the pool request, but would switch to "query" if it will be fixed in further Bootstrap Table versions.
Example(s)
No response
Possible Solutions
Fix initServer() method, reuse saved "query" parameter from previous "refresh" request.
Additional Context
No response
The text was updated successfully, but these errors were encountered: