In views_ajax the pager_element is in the request, but is not properly set on the view display because the display has already been initialized. As a result the pager_element is set but the actual pager id is not.

CommentFileSizeAuthor
#10 views-n2562745-10.patch701 bytesdamienmckenna
#8 ajax-pager-id-missing-2562745-8.patch889 bytesguaneagler
#5 views-views-ajax-pager-id-2562745-5-7.x.patch733 bytesmalcolm_p
#2 views-views-ajax-pager-id-2562745-2-7.x.patch732 bytesmalcolm_p

Comments

malcolm_p created an issue. See original summary.

malcolm_p’s picture

malcolm_p commented
StatusFileSize
new views-views-ajax-pager-id-2562745-2-7.x.patch732 bytes
happysnowmantech’s picture

happysnowmantech commented

I ran into this issue and patch #2 resolved it. Thanks malcolm_p!

darrenwh’s picture

darrenwh commented
Status: Needs review » Needs work 
+++ b/includes/ajax.inc
@@ -64,6 +64,9 @@ function views_ajax() {
+        // setting pager_element doesn't update the pager id

Hi, this is a bit trivial, but could you add an ending period(.) to this line.

malcolm_p’s picture

malcolm_p commented
StatusFileSize
new views-views-ajax-pager-id-2562745-5-7.x.patch733 bytes

Updating comment on this patch.

malcolm_p’s picture

malcolm_p commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 5: views-views-ajax-pager-id-2562745-5-7.x.patch, failed testing. View results

guaneagler’s picture

guaneagler commented
StatusFileSize
new ajax-pager-id-missing-2562745-8.patch889 bytes

I do the patch like this, it seems to work. Need to check.

I get the issue because I use the views-field-view module, and it manually set the pager-id. When I use ajax with the pager, and the pager not work as the pager-id in ajax is not set.

darrenwh’s picture

darrenwh commented
Status: Needs work » Needs review
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new views-n2562745-10.patch701 bytes

Rerolled with the correct file path.