Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

warmupCache() soll nur relevante post_type=page durchsuchen #1725

Open
nelarsen opened this issue Jan 9, 2025 · 0 comments · May be fixed by #1727
Open

warmupCache() soll nur relevante post_type=page durchsuchen #1725

nelarsen opened this issue Jan 9, 2025 · 0 comments · May be fixed by #1727
Labels
enhancement New feature or request
Milestone

Comments

@nelarsen
Copy link
Contributor

nelarsen commented Jan 9, 2025

Die SQL-Query am Anfang von warmupCache() sucht nach CB-Shortcodes. Eigentlich sollten nur Shortcodes auf veröffentlichen Posts (vor allem post_type: page aber vielleicht auch post?) gefunden werden, es werden aber in allen Posts (auch post_type: revision, cb_timeframe, cb_booking, attachment, ...) gesucht.
Es können 10000ende Posts geben (viele cb_booking|s ...) und allein die SQL-Abfrage kann etwas dauern. Aber es werden so auch potentiell nicht benötigte Shortcode/Parameter-Kombinationen berechnet, wenn ein Shortcode wie z.B. [cb_items_table itemcat=tippfehler] nur auf einer nicht veröffentlichen oder alten Revision (post_type=revision) vorhanden ist, aber nirgendwo (mehr) veröffentlich ist. Das kann potentiell eine wesentlich zusätzliche Serverbelastung verursachen.

Ich reiche einen einfachen PR nach.

@nelarsen nelarsen added the enhancement New feature or request label Jan 9, 2025
nelarsen added a commit to nelarsen/commonsbooking that referenced this issue Jan 9, 2025
…y on post_type=page|s

Before this fix all kinds of (irrelevant) post_type|s were searched (revision, cb_booking, ...)
@hansmorb hansmorb added this to the 2.10.2 milestone Jan 9, 2025
@hansmorb hansmorb moved this from Backlog to Todo Review DEV in commonsbooking (> v2.0.0) Jan 9, 2025
nelarsen added a commit to nelarsen/commonsbooking that referenced this issue Jan 11, 2025
…y on published post_type=page|s

Before this fix all kinds of (irrelevant) post_type|s were searched (revision, cb_booking, ...)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Todo Review DEV
2 participants