Missed variables and unused transaction

[volodymyrdovhan]

Hello,

I discovered an unused $USER variable and unnecessary transaction usage. Transactions are only necessary when processing insert/update operations, not for data reads. In any case, transactions should be closed with $transaction->allow_commit();

https://moodledev.io/docs/4.4/apis/core/dml/delegated-transactions

Thank you