[Rt-commit] rt branch, 4.2/user-transactions-privileged-groups, created. rt-4.2.12-127-g068faac

[Dustin Graves]

The branch, 4.2/user-transactions-privileged-groups has been created
        at  068faac12273437b9ad83c38f5424b8605a1c18e (commit)

- Log -----------------------------------------------------------------
commit 068faac12273437b9ad83c38f5424b8605a1c18e
Author: Dustin Graves <dustin at bestpractical.com>
Date:   Fri Jun 3 21:54:35 2016 +0000

    log a transaction when adding a User to a Group or removing them from a Group

diff --git a/lib/RT/Transaction.pm b/lib/RT/Transaction.pm
index d73bf38..08be124 100644
--- a/lib/RT/Transaction.pm
+++ b/lib/RT/Transaction.pm
@@ -1311,6 +1311,18 @@ sub _FormatUser {
         } else {
             return ("Reminder completed"); #loc()
         }
+    },
+    AddMembership => sub {
+        my $self = shift;
+        my $group = RT::Group->new($self->CurrentUser);
+        $group->Load($self->Field);
+        return ("Added to group '[_1]'", $group->Name); #loc()
+    },
+    DeleteMembership => sub {
+        my $self = shift;
+        my $group = RT::Group->new($self->CurrentUser);
+        $group->Load($self->Field);
+        return ("Removed from group '[_1]'", $group->Name); #loc()
     }
 );
 
diff --git a/share/html/Admin/Elements/MembershipsPage b/share/html/Admin/Elements/MembershipsPage
index 75afd49..97fcf50 100644
--- a/share/html/Admin/Elements/MembershipsPage
+++ b/share/html/Admin/Elements/MembershipsPage
@@ -101,6 +101,10 @@ if ( $Update ) {
             next;
         }
         my ($status, $msg) = $group->AddMember( $id );
+        $object->_NewTransaction(
+            Type => 'AddMembership',
+            Field     => $group->id,
+        );
         push @results, $msg;
     }
     foreach my $gid( @Delete ) {
@@ -111,6 +115,10 @@ if ( $Update ) {
             next;
         }
         my ($status, $msg) = $group->DeleteMember( $id );
+        $object->_NewTransaction(
+            Type => 'DeleteMembership',
+            Field     => $group->id,
+        );
         push @results, $msg;
     }
     MaybeRedirectForResults(

-----------------------------------------------------------------------