Hello Dolly : Uncaught exception 'Error' with message 'Call to undefined function add_action()

[deepakrohilla]

error.message LIKE 'Uncaught exception 'Error' with message 'Call to undefined function add_action()' in /var/www/wp-content/plugins/hello.php:69%' AND error.class = 'Error' AND transactionUiName = '/wp-content/plugins/hello.php'

'request_uri': '/wp-content/plugins/hello.php'

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

[SergeyBiryukov]

Hi there, thanks for the ticket!

A similar issue was addressed in [49619] / #35835 for wp-includes/rss-functions.php.

Unless I'm missing something, calling directly pretty much any other file in wp-includes or wp-admin/includes would lead to the same issue, so if we want to add an early exit, it might be a good idea to do that in a consistent way.

Related: #47154, #51806, #55944, #61277, #61286.

[deepakrohilla]

@SergeyBiryukov I have cover maximum number of files of wp-includes & wp-admin directory and raised in https://core.trac.wordpress.org/ticket/61314#ticket with patch

could you confirm that I'm not missing something with this change.

I core I see simple die()-s without -1.

I core I see simple die()-s without -1.

Hi @szepeviktor Thanks, yes this can be simply die without -1. but in core like
\wp-admin\edit-link-form.php and wp-admin\edit-form-comment.php and many other's file of core , we are already using

Don't load directly.
if ( ! defined( 'ABSPATH' ) ) {

die( '-1' );

}

Hi @szepeviktor , Could you please suggest what we should use? In many places in the core files, we are using die('-1');, while in some files, I have seen a simple die();.

@deepakrohillas Exit status is for CLI only.
​https://www.php.net/manual/en/function.exit.php
So die()