Implemented When the class is not found, say which

Rigel Kentaurus

Well-known member
Small enhancement. This code:
$this->error(new XenForo_Phrase('please_enter_valid_callback_method'), 'cron_method');

To this please :(
$this->error(new XenForo_Phrase('please_enter_valid_callback_method', array('class' => $class, 'method' => $method)), 'cron_method');

It appears on DataWriter/AddOn.php, CodeEventListener, CronEntry, Page and UserField

And well, of course the phrase needs to be updated, that would reduce the hunting time down on misbehaving add-ons

Today, whenever I try to install an add-on that has a missing class, it says "Please enter a valid callback method" and leaves me hunting down whether it meant the installer, an add-on, a cron job, etc.

And this usually happens whenever it works perfectly on non-case-sensitive OS and then the add-on is installed on a case-sensitive OS and suddenly it cannot find one file.
Upvote 6
This suggestion has been implemented. Votes are no longer accepted.
Top Bottom