Replace include_once with require_once in wp-includes/comment.php

[wpfy]

The WordPress PHP coding standards recommend using require[_once] for unconditional includes rather than include[_once]. This is because if the file being included is not found, include[_once] will throw a warning but continue execution, potentially leading to other errors or security issues if the file is needed for the application.

However, in the WordPress core file wp-includes/comment.php, on lines 3046 and 3047, the function pingback() includes two files using include_once. To ensure consistency with the WordPress coding standards and improve the security and stability of the code, I propose replacing include_once with require_once on these lines.

I will create a pull request to the WordPress core repository with this change.

The WordPress PHP coding standards recommend using require[_once] for unconditional includes rather than include[_once]. This is because if the file being included is not found, include[_once] will throw a warning but continue execution, potentially leading to other errors or security issues if the file is needed for the application.

However, in the WordPress core file wp-includes/comment.php, on lines 3046 and 3047, the function pingback() includes two files using include_once. To ensure consistency with the WordPress coding standards and improve the security and stability of the code, I propose replacing include_once with require_once on these lines.

I will create a pull request to the WordPress core repository with this change.

Trac ticket: https://core.trac.wordpress.org/ticket/58046

[SergeyBiryukov]

Hi there, thanks for the ticket!

It looks like this was resolved in [55641] / #57839 as part of a few other similar changes.

I have updated the props for that commit in the Core Props tool on make/core to include the props from this ticket too, so that the contribution is properly counted for you as well :)

Thanks for the PR! Merged in r55641 as part of a few other similar changes.