ryanwelcher / advanced-query-loop

WordPress plugin that introduces a Query Loop block variation that can create advanced queries.
87 stars 10 forks source link

Posts Per Page Bug #31

Closed carlbtmn closed 7 months ago

carlbtmn commented 11 months ago

Not sure if is just me, but I cannot change the number of posts from (10) when inherit query is enabled. It works when inherit is disabled.

image

ryanwelcher commented 11 months ago

Confirming this is an issue. I'm looking into it now.

carlbtmn commented 11 months ago

I had to change the amount of posts under settings - reading - "Blog pages show at most" to show less posts, but that is not a great solution.

Thank you.

ryanwelcher commented 11 months ago

This looks to have been introduced with WordPress 6.4. Specifically with this PR. I have done some preliminary testing and I am not sure I can create a work around for it.

carlbtmn commented 11 months ago

I can confirm this was happening before 6.4 and I thought I was doing something wrong.

ryanwelcher commented 11 months ago

Interesting, can you confirm if you had the Gutenberg plugin installed and active? I tested this today and it seemed to work in WP 6.3.1

carlbtmn commented 11 months ago

You mean this plugin: https://wordpress.org/plugins/gutenberg/

I don't have this installed. Is this required when using AQL?

ryanwelcher commented 11 months ago

No, it isn't. I only ask because the changes that caused the issue were in that plugin before WordPress 6.4 was released.

carlbtmn commented 11 months ago

Ok. I've never used it.

I know I had the issue in 6.3.1, because I updated last night to 6.4.

carlbtmn commented 10 months ago

Any updates with this issue?

ryanwelcher commented 10 months ago

Hi and thanks for your patience.

I have confirmed that the bug existed since WP 6.2 and was introduced with this PR.

Unfortunately, I have not been able to find a way to override the changes that were implemented. I will continue to work on this but in the meantime, I am going to remove the control from the interface when the query is being inherited. It's not ideal but it makes sense until there is a definitive solution for bringing that feature back.

carlbtmn commented 10 months ago

I agree... it should be removed until there is a fix. Thank you for the update.