Please fix post_per_page assertion for recurring queries

[papertower]

In events-calendar-pro/src/Tribe/Recurrence/Event_Query.php on line 66, please check to see whether the posts_per_page value is set (like you’re doing in the standard events query) before imposing the internal setting. I’m doing a request via AJAX and I need to be able to set the number of posts being requested. After some digging I realized that line cuts off any prior value set.

Thanks!

• This topic was modified 10 years, 1 month ago by papertower.

[Geoff B.]

Good evening Brenden and welcome back!

Thanks for bringing this to our attention.

I have logged this recommendation in our system and passed it along to our Dev team for further review.

Unfortunately, I cannot commit to a release date at this point. But stay tuned!

Best regards,

Geoff B.

[papertower]

Awesome! Is it possible to have this ticket updated when a commit made it into a scheduled release?

Right now I’m having to use a workaround via the tribe_event_pre_get_posts filter, and I’d really like to remove it when it’s no longer necessary.

Thanks!

• This reply was modified 10 years, 1 month ago by papertower.

[Geoff B.]

Good afternoon Brenden,

Absolutely. We update threads whenever we complete their attached tickets 🙂

Have a good week-end,

Geoff B.

[Support Droid]

This topic has not been active for quite some time and will now be closed.

If you still need assistance please simply open a new topic (linking to this one if necessary)
and one of the team will be only too happy to help.

[Leah]

Hi there,

I just wanted to pop in here and give you an update. While we weren’t able to get this feature into our upcoming 4.2 release, it is still on our list of ideas to implement in future releases.

We appreciate your patience, and we will be sure to update this thread when we have more news. Thank you for your continued support, and keep an eye out for version 4.2 coming in the next few weeks!

Best,
Leah
and The Events Calendar team

[Author]

Posts