[wp-trac] [WordPress Trac] #38700: REST API: Cannot send an empty or no-op comment update
WordPress Trac
noreply at wordpress.org
Tue Jun 30 14:11:16 UTC 2020
#38700: REST API: Cannot send an empty or no-op comment update
-------------------------------------------------+-------------------------
Reporter: jnylen0 | Owner: jnylen0
Type: defect (bug) | Status: closed
Priority: normal | Milestone: 4.7.1
Component: REST API | Version: 4.7
Severity: normal | Resolution: fixed
Keywords: has-patch has-unit-tests commit | Focuses:
fixed-major |
-------------------------------------------------+-------------------------
Comment (by SergeyBiryukov):
In [changeset:"48235" 48235]:
{{{
#!CommitTicketReference repository="" revision="48235"
Comments: Make `wp_update_comment()` return `false` instead of 0 for an
invalid comment or post ID.
This addresses an inconsistency where 0 could mean one of the three
scenarios:
* Invalid comment ID.
* Invalid comment post ID.
* No DB rows updated. This is not an error and should not be treated as
one.
With this change, `wp_update_comment()` always returns either `false` or a
`WP_Error` object on failure, depending on the value of the `$wp_error`
parameter.
Follow-up to [48154], [48215], [48216], [48218], [48230].
Props dd32, jnylen0, enrico.sorcinelli.
Fixes #39732. See #38700, #39735.
}}}
--
Ticket URL: <https://core.trac.wordpress.org/ticket/38700#comment:28>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list