Enhancement for rest api comment controller create_item method to check for WP error

[dspilka]

In the WP_REST_Posts_Controller class, the create_item method calls the prepare_item_for_database method and checks its response for a WP error object.

<?php
    $prepared_post = $this->prepare_item_for_database( $request );

     if ( is_wp_error( $prepared_post ) ) {
          return $prepared_post;
     }

This allows for further filtering and custom validation via the rest_pre_insert_{$this->post_type} filter and for returning a WP Error if needed to stop the request and return messaging.

I'd like to propose similar in the WP_REST_Comments_Controller class. Right now when the create_item method applys the rest_pre_insert_comment filter, it doesn't check for a WP Error response.

<?php
    $prepared_comment = apply_filters( 'rest_pre_insert_comment', $prepared_comment, $request );
    $comment_id = wp_insert_comment( $prepared_comment );

I'd like to propose checking for a WP Error.

<?php
        $prepared_comment = apply_filters( 'rest_pre_insert_comment', $prepared_comment, $request );

        if ( is_wp_error( $prepared_comment ) ) {
                return $prepared_comment;
        }

        $comment_id = wp_insert_comment( $prepared_comment );

[jnylen0]

Good idea, thanks. Want to write a patch? ​https://make.wordpress.org/core/handbook/contribute/#your-first-patch

[dspilka]

Replying to jnylen0:

Good idea, thanks. Want to write a patch? ​https://make.wordpress.org/core/handbook/contribute/#your-first-patch

Yes definitely, I will get a patch together!

[dspilka]

Patch to check for WP Error.

[rmccue]

@dspilka Thanks for the patch :) Just to note: your indentation is off in the patch, but seeing how it's only 3 lines, not a huge issue here. Something to keep in mind for future patches though.

[rmccue]

Add phpDoc

[rmccue]

In 39922:

REST API: Allow shortcircuiting rest_pre_insert_comment

rest_pre_insert_{post_type} allows returning a WP_Error from the filter to shortcircuit actually creating the object, so it makes sense to do so for comments too.

Props dspilka.
Fixes #39578.

[rmccue]

I pushed this forward to 4.8 instead, as I don't see a real reason to backport to a minor release. Thanks for the patch! :)

[dspilka]

Replying to rmccue:

@dspilka Thanks for the patch :) Just to note: your indentation is off in the patch, but seeing how it's only 3 lines, not a huge issue here. Something to keep in mind for future patches though.

*EDIT* Nevermind. I clearly see it now when opening/viewing the patches online! Didn't look that way in my IDE for some reason. But I got it now. Thanks.

Thanks @rmccue. Very happy to have my first WordPress patch! Would you mind pointing out how the indentation should be or how I can check it so I can get it right for my next patch? It seems to look like the others in my IDE, so I just want to make sure I understand it and know how to do it right. Thanks, Dan.

[rmccue]

It'll look fine in your editor since your tab size is set to 4 (i.e. 1 tab character is 4 spaces wide), whereas in Trac, the tab size is 8, so the four spaces don't line up with the 8-space-width of the tab :)