Customize: Fetch presence of published pages more optimally (refactor)

[nagpai]

The function has_published_pages on line 5753 seems to fetch all the data of pages, whereas what is needed is only whether published pages exist or not.

While this may not matter much in sites that have fewer pages, on website that has too many pages, it may create an out of memory error. I will test and submit a patch to fetch only the count of published pages that will help improve efficiency.

[dlh]

Welcome back to Trac, @nagpai! Thanks for the report.

The current logic was added for #38013, where 'number' => 1 was mentioned specifically in the description but not the final patch, so it seems like it was an oversight.

Care to submit a patch or a Pull Request? Once we have a patch, I think this ticket can be added to the 6.0 milestone.

[nagpai]

Hey @dlh , Thank you so much for reviewing this ticket and adding the very useful context! I was traveling last week, and will submit a patch in the next couple of days. Thanks :D

[nagpai]

Fix for ticket #55425

[nagpai]

I just uploaded a patch. I confirm having tested this on a test site with a few pages published, and none published. It seems to work fine. Would be great if someone can test it too.

This is my first patch submission to WordPress and I hope to do more :) . I have tried to follow the instructions in the contributors' handbook. Advance apologies for any mistakes. Thanks for your kind guidance @dlh !

Also props to my colleague and guide @tyxla who guided me with the next steps too.

https://core.trac.wordpress.org/ticket/55425

[peterwilsoncc]

I've moved this on the the 6.0 milestone as it appears to be a quick performance win.

To make sure the unit tests still pass, I've set up a ​linked pull request.

@nagpai I made some ​minor white space changes to your patch to observe ​the WordPress PHP coding standards.

[peterwilsoncc]

In 53057:

Customizer: Improve performance checking for published pages.

Improve the performance of WP_Customize_Manager::has_published_pages() by limiting the database query to a single published page, down from querying all published pages.

Props nagpai, dlh, tyxla.
Fixes #55425.

Merged in https://core.trac.wordpress.org/changeset/53057