[Jet Engine + Jet Popup + Jet Smart Filters] When paginate tables, the template collum with popup button not instance if query has "per page" limit #7351
Describe the bug
Using all the plugins described in the title, when I created a table with Table Builder, I placed action buttons similar to a CRUD and used jet popup to display the rest of the data in that table and then I placed smart filters for pagination , other pages did not populate the values of dynamic fields placed in the popup, using xdebug and pausing on the hook 'jet-engine/compatibility/popup-package/post-object' I noticed that the "per page" of the Query Builder, within the " case 'repeater':" does not search the current page, thus losing $items[$item_index] and going empty, as well as getting the popup template completely to prevent errors
Sorry if anything is incoherent, I'm using Google translate '-'
To Reproduce
Steps to reproduce the behavior:
1: Install the 3 plugins described in the title
2: Make a Repeater Field wherever you want
3: Create a table to display this data, using the Query Builder with a limit by pages (limit by 1 and create 2 will be enough)
4: Place a template in the table, coming from the jet engine listings, as well as in the documentation
5: Create a button that activates the Popup within the template
6:Add it to the table
7: Create either pagination or any filter, with Jet SmartFilters, for this table
8:Place some metafield of this repeater created with dynamic fields in the popup
9: See that when you open the popup of the second item (if limited by 1 per page), the dynamic field of the popup will not appear, whereas the first one will
Expected behavior
I already corrected it with the Hook that I mentioned before, just calculating the current page using the index that is sent to me via AJAX, but what was expected was, even if filtered and having its limit per page in the Query Builder, getting the index similar to "9- 895-15"(respectively Query_ID-Post_ID-Item_ID) it would bring all the data to the dynamic fields, something that is not happening, when it gets the $items, it is always on the first page, the part where I saw this will follow in the print:
Screenshots
Desktop (please complete the following information):
OS: Linux Debian based distribution
Browser Chrome and Firefox
Version Engine: 3.3.5, Popup: 2.0.3.1, SmartFilters: 3.3.2, tables builder: 1.0.9
Smartphone (please complete the following information):
Device: Android 9 - Moto Z3 << buggin too
OS: Android 9
Browser: Chrome
Version [e.g. 22]
Additional context
I solved the problem as I mentioned before, searching for the correct page either using the index or some way of bringing up the current page from Smart Filters
A screenshot of the add_filter I made and solved the problem, I know it's not the best way!
awsj007
commented
7 months ago
Describe the bug Using all the plugins described in the title, when I created a table with Table Builder, I placed action buttons similar to a CRUD and used jet popup to display the rest of the data in that table and then I placed smart filters for pagination , other pages did not populate the values of dynamic fields placed in the popup, using xdebug and pausing on the hook 'jet-engine/compatibility/popup-package/post-object' I noticed that the "per page" of the Query Builder, within the " case 'repeater':" does not search the current page, thus losing $items[$item_index] and going empty, as well as getting the popup template completely to prevent errors Sorry if anything is incoherent, I'm using Google translate '-'
To Reproduce Steps to reproduce the behavior: 1: Install the 3 plugins described in the title 2: Make a Repeater Field wherever you want 3: Create a table to display this data, using the Query Builder with a limit by pages (limit by 1 and create 2 will be enough) 4: Place a template in the table, coming from the jet engine listings, as well as in the documentation 5: Create a button that activates the Popup within the template 6:Add it to the table 7: Create either pagination or any filter, with Jet SmartFilters, for this table 8:Place some metafield of this repeater created with dynamic fields in the popup 9: See that when you open the popup of the second item (if limited by 1 per page), the dynamic field of the popup will not appear, whereas the first one will
Expected behavior I already corrected it with the Hook that I mentioned before, just calculating the current page using the index that is sent to me via AJAX, but what was expected was, even if filtered and having its limit per page in the Query Builder, getting the index similar to "9- 895-15"(respectively Query_ID-Post_ID-Item_ID) it would bring all the data to the dynamic fields, something that is not happening, when it gets the $items, it is always on the first page, the part where I saw this will follow in the print:
Screenshots
Desktop (please complete the following information):
Smartphone (please complete the following information):
Additional context I solved the problem as I mentioned before, searching for the correct page either using the index or some way of bringing up the current page from Smart Filters A screenshot of the add_filter I made and solved the problem, I know it's not the best way!