No Code Attached Yet bug
avatar rdeutz
rdeutz
7 Apr 2025

In #25103 we deprecated the onUserBeforeDataValidation event and suggested that users should use the onContentValidateData event, but this event is never triggered.

So far this hasn't so much effect, reasons might be:

  • onUserBeforeDataValidation is not used in core
  • Deprecated documentation is missing

This needs to be fixed before 6.0 comes out.

avatar rdeutz rdeutz - open - 7 Apr 2025
avatar joomla-cms-bot joomla-cms-bot - change - 7 Apr 2025
Labels Added: No Code Attached Yet bug
avatar joomla-cms-bot joomla-cms-bot - labeled - 7 Apr 2025
avatar hiteshm0
hiteshm0 - comment - 7 Apr 2025
$data = $dispatcher->dispatch('onContentBeforeValidateData', new Model\BeforeValidateDataEvent('onContentBeforeValidateData', [
            'subject' => $form,
            'data'    => &$data, // @todo: Remove reference in Joomla 6, see AfterRenderModulesEvent::__constructor()
        ]))->getArgument('data', $data);

onContentBeforeValidateData is being trigerred after a warning is shown that onUserBeforeDataValidation is depracated, and we are supposed to use onContentValidateData instead. But onContentValidateData is never trigerred.

Should I add :

$data = $dispatcher->dispatch('onContentValidateData', new Model\BeforeValidateDataEvent('onContentValidateData', [
            'subject' => $form,
            'data'    => &$data,
        ]))->getArgument('data', $data);

after the code snipped mentioned above ? @rdeutz

avatar Fedik
Fedik - comment - 8 Apr 2025

That is the typo in the deprecation message.
The correct event name is onContentBeforeValidateData

avatar Fedik
Fedik - comment - 8 Apr 2025

Please test #45296

avatar richard67 richard67 - change - 8 Apr 2025
Status New Closed
Closed_Date 0000-00-00 00:00:00 2025-04-08 13:09:09
Closed_By richard67
avatar richard67 richard67 - close - 8 Apr 2025
avatar richard67
richard67 - comment - 8 Apr 2025

Closing as having a pull request. Please test #45296 . Thanks in advance.

avatar Fedik
Fedik - comment - 8 Apr 2025

@richard67 I were leave it to auto-close feature 😉

Add a Comment

Login with GitHub to post a comment