Opened 9 years ago
Closed 7 years ago
#38784 closed defect (bug) (wontfix)
Set no_found_rows inside _WP_Editors::wp_link_query
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Severity: | normal | Version: | |
| Component: | Editor | Keywords: | |
| Focuses: | performance | Cc: |
Description
Since the WP_Query inside the _WP_Editors::wp_link_query is not taking advantage of found_posts nor of max_num_pages, not setting no_found_rows and thus triggering a SQL query with SQL_CALC_FOUND_ROWS does not feel right.
The extra queried information is not being used and forces the SQL query to scan much more rows than it needs to.
Attachments (1)
Change History (2)
#1
@
7 years ago
- Milestone Awaiting Review deleted
- Resolution set to wontfix
- Status changed from new to closed
- Version 4.6.1 deleted
While Core is not relying on found_rows or max_num_pages, it's possible a plugin or custom code could be relying on it being present through the wp_link_query filter.
I am going to close this one out since this only affects the Classic Editor, which will mostly remain the same moving forward with the exception of bug fixes.
If someone feels strongly that this should be updated and can investigate further, feel free to reopen.
Setting
no_found_rowsinside the_WP_Editors::wp_link_query