Pagination Error

Post

Posted
Rating:
#5492 (In Topic #1273)
Avatar
Standard member
SoccerDad is in the usergroup ‘Community saint’
Hey Composr's!

Had this one around for a wee while but hadn't gotten around to asking. Anyone experienced this bad boy:

Screenshot from 2019-01-28 13-33-48.png

100% recreatable on every page pagination is used. Sorry for the delay in reporting; it's been around for the last 5+ releases or so. I get it on a fresh (read: not yet mangled by me ;) ) installation as well. Unfortunately my Javascript sux worse than my PHP so I'm not much help in troubleshooting.

Thanx!

"no violence, no hate, no pain, no enemies
just peace, unity, tolerance and love" - The Beloved
Online now: No Back to the top

Post

Posted
Rating:
#5496
Avatar
Standard member
SoccerDad is in the usergroup ‘Community saint’
A little further: using the chainsaw approach, removing all instances of WRAPPER_ID from AJAX_PAGINATION.tpl stops the error from happening on comcode pages without seeming to affect the ability to toggle through entries. This action however fires a syntax error in the member profile activities screen. Interesting part: restoring AJAX_PAGINATION.tpl starts the error again on comcode pages but there is no error at all from member profile activities screen even with the pagination thingy loading at the bottom of the page.

"no violence, no hate, no pain, no enemies
just peace, unity, tolerance and love" - The Beloved
Online now: No Back to the top

Post

Posted
Rating:
#5506
Avatar
Site director
Chris Graham is in the usergroup ‘Administrators’
Hey,

Do you have an overridden template that is including AJAX_PAGINATION?
I think there was a bug with this a while back, but it was fixed.

Anything including the template should be passing a WRAPPER_ID, e.g.

Code

         {+START,INCLUDE,AJAX_PAGINATION}
            ALLOW_INFINITE_SCROLL=1
            WRAPPER_ID={$GET,ajax_block_main_activities_wrapper}
         {+END}


Become a fan of Composr on Facebook or add me as a friend. Add me on on Mastodon. Follow me on Minds (where I am most active). Support me on Patreon

Was I helpful?
  • If not, please let us know how we can do better (please try and propose any bigger ideas in such a way that they are fundable and scalable).
  • If so, please let others know about Composr whenever you see the opportunity or support me on Patreon.
  • If my reply is too Vulcan or expressed too much in business-strategy terms, and not particularly personal, I apologise. As a company & project maintainer, time is very limited to me, so usually when I write a reply I try and make it generic advice to all readers. I'm also naturally a joined-up thinker, so I always express my thoughts in combined business and technical terms. I recognise not everyone likes that, don't let my Vulcan-thinking stop you enjoying Composr on fun personal projects.
  • If my response can inspire a community tutorial, that's a great way of giving back to the project as a user.
Online now: No Back to the top

Post

Posted
Rating:
#5508
Avatar
Standard member
SoccerDad is in the usergroup ‘Community saint’
Hey Chris! As far as I know and can tell from date stamps the only one ever used was the one included with the default template. Interesting new happenstance: playing around with something different last night saw me starting with a 10.0.18 package and voila, the error was gone. Upon applying the upgrade to 10.0.22 it returned. I copied the AJAX file from the .18 package default theme overwriting .22 default theme version and the error disappeared again. The biggest difference I can see in the files is the reference to wrapper_id in that the .18 package calls

Code

{$GET%,wrapper_id}
whereas the .22 package is

Code

{WRAPPER_ID%}

"no violence, no hate, no pain, no enemies
just peace, unity, tolerance and love" - The Beloved
Online now: No Back to the top

Post

Posted
Rating:
Item has a rating of 5 (Liked by SoccerDad)
#5559
Avatar
Site director
Chris Graham is in the usergroup ‘Administrators’
Hi,

Sorry for the delay in replying, this got stuck too far down my todo list.

I'm not convinced there's an active bug here, I think I didn't communicate what I meant clearly enough.

Maybe this commit will clarify what has happened:
https://github.com/ocproducts/composr/commit/71c30fb108897aecb20eb310cbd3bfb516e5c14f

Before Jul 31, 2018, there was an issue whereby nested paginations would mess up, because we were setting IDs based on shared variables (creating a conflict). So I instead switched it to so each template would assign a template-specific ID.

Any of the changed templates overridden from an earlier patch would have lost compatibility at this point. I suspect you have one or more of them overridden.

The Tempcode that loads up AJAX_PAGINATION passes through WRAPPER_ID now, as used by the new version of that template.


Become a fan of Composr on Facebook or add me as a friend. Add me on on Mastodon. Follow me on Minds (where I am most active). Support me on Patreon

Was I helpful?
  • If not, please let us know how we can do better (please try and propose any bigger ideas in such a way that they are fundable and scalable).
  • If so, please let others know about Composr whenever you see the opportunity or support me on Patreon.
  • If my reply is too Vulcan or expressed too much in business-strategy terms, and not particularly personal, I apologise. As a company & project maintainer, time is very limited to me, so usually when I write a reply I try and make it generic advice to all readers. I'm also naturally a joined-up thinker, so I always express my thoughts in combined business and technical terms. I recognise not everyone likes that, don't let my Vulcan-thinking stop you enjoying Composr on fun personal projects.
  • If my response can inspire a community tutorial, that's a great way of giving back to the project as a user.
Online now: No Back to the top

Post

Posted
Rating:
#5573
Avatar
Standard member
Paul Flavel is in the usergroup ‘Honoured member’
I know this is something that is affecting the themes i made, so will go through them all and update with the above. I will also include the instructions on how to manually fix them for those who have further customised their theme.
Online now: No Back to the top

Post

Posted
Rating:
#5574
Avatar
Site director
Chris Graham is in the usergroup ‘Administrators’

Paul Flavel said

I know this is something that is affecting the themes i made, so will go through them all and update with the above. I will also include the instructions on how to manually fix them for those who have further customised their theme.

Sorry for the inconvenience, thanks for doing that.


Become a fan of Composr on Facebook or add me as a friend. Add me on on Mastodon. Follow me on Minds (where I am most active). Support me on Patreon

Was I helpful?
  • If not, please let us know how we can do better (please try and propose any bigger ideas in such a way that they are fundable and scalable).
  • If so, please let others know about Composr whenever you see the opportunity or support me on Patreon.
  • If my reply is too Vulcan or expressed too much in business-strategy terms, and not particularly personal, I apologise. As a company & project maintainer, time is very limited to me, so usually when I write a reply I try and make it generic advice to all readers. I'm also naturally a joined-up thinker, so I always express my thoughts in combined business and technical terms. I recognise not everyone likes that, don't let my Vulcan-thinking stop you enjoying Composr on fun personal projects.
  • If my response can inspire a community tutorial, that's a great way of giving back to the project as a user.
Online now: No Back to the top

Post

Posted
Rating:
#5575
Avatar
Standard member
Paul Flavel is in the usergroup ‘Honoured member’
Not an issue at all. Very easy fix.
Online now: No Back to the top
1 guest and 0 members have just viewed this.

Statistics

Forum statistics:
  • 2,052 topics, 7,195 posts, 10,827 members
  • Our newest member is LowfareMart
Birthdays:
Back to Top