Changes between Version 4 and Version 5 of Ticket #34560, comment 30
- Timestamp:
- 11/10/2015 05:42:23 PM (8 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #34560, comment 30
v4 v5 17 17 It would not surprise me if there were other problematical uses of "wp_get_post_revisions", but [https://github.com/WordPress/WordPress/search?q=wp_get_post_revisions code review for that] could perhaps become a new issue. 18 18 19 One remaining concern is the one in comment 17 referencing issue #25233 by @jkudish (the second concern under "Possible bottlenecks") about WordPress needing to load all revisions to upgrade them. Does that concern apply here? I do not see such code in the [https://github.com/WordPress/WordPress/blob/4.3-branch/wp-includes/revision.php 4.3 version of that function]. There is a separate "_wp_upgrade_revisions_of_post" function in that file. It seems to [https://github.com/WordPress/WordPress/search?q=_wp_upgrade_revisions_of_post be called] separately when the edit process is started via edit_post (code below). Based on inspecting the code for WordPress 4.3 on GitHub, I do not think that concern does notapply here far as the patch itself goes -- but someone else should review that just in case to ensure there is no risk of data corruption.19 One remaining concern is the one in comment 17 referencing issue #25233 by @jkudish (the second concern under "Possible bottlenecks") about WordPress needing to load all revisions to upgrade them. Does that concern apply here? I do not see such code in the [https://github.com/WordPress/WordPress/blob/4.3-branch/wp-includes/revision.php 4.3 version of that function]. There is a separate "_wp_upgrade_revisions_of_post" function in that file. It seems to [https://github.com/WordPress/WordPress/search?q=_wp_upgrade_revisions_of_post be called] separately when the edit process is started via edit_post (code below). Based on inspecting the code for WordPress 4.3 on GitHub, I think that concern '''does *not*''' apply here far as the patch itself goes -- but someone else should review that just in case to ensure there is no risk of data corruption. 20 20 21 21 See this code in [https://github.com/WordPress/WordPress/blob/4.3-branch/wp-admin/includes/post.php includes/post.php] to understand what the concern may be: