Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge User Accounts error #261

Closed
dragos5436 opened this issue Feb 21, 2024 · 3 comments
Closed

Merge User Accounts error #261

dragos5436 opened this issue Feb 21, 2024 · 3 comments

Comments

@dragos5436
Copy link

dragos5436 commented Feb 21, 2024

Hi there,

We are trying to merge two user accounts, but are getting the following error:
image

Any tips on what the problem might be and what we could do about it? We suspect there is something related to this particular account, as we are able to merge different sets of accounts, so nothing wrong with the merge process itself.

Thanks

Moodle version 4.3.3
PHP version 8.0

@jpahullo
Copy link
Owner

Hi @dragos5436 ,

It seems that you have some kind of inconsistence in your database.

It tries to regrade a certain activity of type "attendance", which nowadays it does not exist anymore. Maybe it is something related to the plugin type or maybe of your Moodle instance.

We have the plugin "attendance" in production, and we never have seen this kind of error.

If it was me, I would investigate on the database to find where this instance appears and try to do some cleanup or fixing of the database records.

It is something outside the scope of this plugin. This plugin cannot afford about database inconsistences, since for certain tasks, it involves the use of Moodle core APIs, as in this case for regrading, and cannot do anything else.

Maybe in the future, it could be handled this exception to pass to the next item to regrade, but it feels to me to provide false sensation of sucess.

Above all, it is not related to this plugin, but to your Moodle database consistence of data.

Thanks for contacting.

Jordi

@dragos5436
Copy link
Author

Thank you for the help Jordi, we have now managed to complete the merge. We believe some module cleanup task might have got stuck and failed to delete the problematic modules, which was preventing the merge process from running correctly.

Kind regards,

Dragos

@jpahullo
Copy link
Owner

jpahullo commented May 6, 2024

Good to know you finally merged those users.

Thank you for your feedback. It is much apreciated.

Thanks,

Jordi

Repository owner locked and limited conversation to collaborators May 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants