Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-28443 Action logging improvements META
  3. MDL-20733

When creating a new user, the log doesn't include the id of the newly created user

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Duplicate
    • Affects Version/s: 1.9.5
    • Fix Version/s: None
    • Component/s: Other
    • Labels:
      None
    • Environment:
      independent
    • Database:
      MySQL
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_19_STABLE

      Description

      When creating a new user, the log registered doesn't include the id of the newly created user,
      I suggest to use the Info field to store this information.
      In attach I provide a new version of the 'user/editadvanced.php' file that could offer this minor enhancement: it is based on the movement of the line

      add_to_log($course->id, 'user', 'update', "view.php?id=$user->id&course=$course->id", '');

      from line 80 to line 104 (with the use of the variable $usernew->id for memorizing the new user's id)

      add_to_log($course->id, 'user', 'update', "view.php?id=$user->id&course=$course->id", $usernew->id);

        Gliffy Diagrams

          Issue Links

            Activity

            mazzolal Luca Mazzola created issue -
            dougiamas Martin Dougiamas made changes -
            Field Original Value New Value
            Workflow jira [ 33939 ] MDL Workflow [ 45559 ]
            dougiamas Martin Dougiamas made changes -
            Workflow MDL Workflow [ 45559 ] MDL Full Workflow [ 73899 ]
            skodak Petr Skoda made changes -
            Parent MDL-28443 [ 46008 ]
            Issue Type Improvement [ 4 ] Sub-task [ 5 ]
            dougiamas Martin Dougiamas made changes -
            Assignee Martin Dougiamas [ dougiamas ] moodle.com [ moodle.com ]
            ankit_frenz Ankit Agarwal made changes -
            Fix Version/s STABLE Sprint 17 [ 11550 ]
            Assignee moodle.com [ moodle.com ] Ankit Agarwal [ ankit_frenz ]
            ankit_frenz Ankit Agarwal made changes -
            Status Open [ 1 ] Closed [ 6 ]
            Resolution Duplicate [ 3 ]
            ankit_frenz Ankit Agarwal made changes -
            Link This issue duplicates MDL-28386 [ MDL-28386 ]
            stronk7 Eloy Lafuente (stronk7) made changes -
            Fix Version/s STABLE Sprint 17 [ 11550 ]

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: