moodle-tool_mergeusers icon indicating copy to clipboard operation
moodle-tool_mergeusers copied to clipboard

Creation of dynamic property AssignSubmissionTableMerger::$duplicateddatamerger is deprecated

Open leonstr opened this issue 1 year ago • 1 comments

This issue is reported in #262. Contrary to this comment the message appears when debugging is not enabled.

Steps to reproduce issue

  1. Install plugin.
  2. As a site admin go to Site administration > Users > Accounts > Merge user accounts > Merge user accounts.

Expected results

No errors on the page and no errors in the error log.

Actual results

The following message appears at the top of the page (behind the navigation bar) and in the PHP error log:

Deprecated: Creation of dynamic property AssignSubmissionTableMerger::$duplicateddatamerger is deprecated in .../admin/tool/mergeusers/lib/table/assignsubmissiontablemerger.php on line 34

Environment

Moodle 4.4.2+ (Build: 20240823), PHP 8.3.4 and tool_mergeusers 2024060300.

leonstr avatar Aug 26 '24 16:08 leonstr

PR #279 with proposed fix.

leonstr avatar Aug 26 '24 17:08 leonstr

Experiencing the same problem, Moodle 4.5 (Build: 20241007), MySQL (8.0.39-0ubuntu0.20.04.1) and too_mergeusers 2024060300

timbahula-horizon avatar Nov 07 '24 15:11 timbahula-horizon

Thanks @leonstr for the PR. It was merged right now.

Please, all those you reported this problem, could you try the latest master branch version? Put a reminder on the Moodle version you tried, please, just to confirm that now it works properly, without any problem.

With your ok, I will update Moodle Plugins directory, bumping up its the plugin version.

Jordi

jpahullo avatar Nov 08 '24 11:11 jpahullo

applied on M4.1 and 4.5 versions.

to be applied from M4.2 to 4.4

jpahullo avatar Feb 05 '25 14:02 jpahullo