#42474 closed defect (bug) (fixed)
bulk_edit_posts should honor the show_in_quick_edit property of taxonomies
Reported by: | Chouby | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 6.1 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Quick/Bulk Edit | Keywords: | commit add-to-field-guide |
Focuses: | performance | Cc: |
Description
When bulk editing posts, several database queries are related to taxonomies for which show_in_quick_edit
is set to false. Since the terms are not modified, these queries uselessly harm the performance.
This is because bulk_edit_posts
does not check this property when building arguments to pass to wp_update_post
.
Attachments (3)
Change History (13)
#2
@
4 years ago
- Keywords needs-testing added
- Version 4.2 deleted
@Chouby Your approach looks good. I refreshed the patch to work with the current file.
This ticket was mentioned in Slack in #core by abhanonstopnews. View the logs.
3 years ago
#4
@
3 years ago
Adding summary from @nalininonstopnewsuk as she is unable to add changes to the ticket.
- This has a patch and appears just to need testing. Request to promote it at devchat on 16 March 2022.
- Any additional updates requested to @sabernhardt to try and get this into a future release, and to @chouby if there was any update for the ticket
Nalini:speech_balloon: 20:52
If this is still live, could it be added to open floor for dev chat this week to get some testing of the patch to move it forward @marybaum? (edited
#5
@
3 years ago
Pending testing from @meher. Slack was down for a lot of India last week, wo presumably she'll catch up in due course.
This ticket was mentioned in Slack in #core by abhanonstopnews. View the logs.
3 years ago
#7
@
3 years ago
- Keywords commit added; has-patch needs-testing removed
In a scrub of the component, @webcommsat, @arslan and I tested the latest .diff from @costdev, and it works. Recommend commit!
simple refresh