Bug 1016 - Operation Log needs to be updated when admin modifies the password for device admin
Review Request #433 — Created Aug. 28, 2024 and submitted — Latest diff uploaded
| Information | |
|---|---|
| apoorva.sn | |
| AMP | |
| amp_3_7 | |
| 1016 | |
| Reviewers | |
| pmurugaiyan, prajesh, shuinvy | |
Problem
When admin modifies password for Device admin, even though the password is updated operation log prints it as failed and also it prints unnecessary logs which are not relevant to device admin
Solution
Some of the fields do not have custom edit function or there is no generic update fuction for those fields.
The update is happening as part of UserAuthorization->update() through the DB query when _insert_or_update is called below.
So have created non_editable_fields list and excluded those fields as they dont have seperate update functions
Tested locally by updating mail and phone number and there are no unnecessary logs created.
Have attached screenshot for the sameThese is duplicate log shown, but there is a seperate bug to track that: bug 993
