[Rt-commit] rt branch, 4.4/dashboard-subscription-user-group-fix, repushed
Dustin Graves
dustin at bestpractical.com
Tue May 17 15:40:01 EDT 2016
The branch 4.4/dashboard-subscription-user-group-fix was deleted and repushed:
was c01791743f9d99b31d40e5d77f938c2824e69f16
now bf6f1a259f8d1d387db092ae476e41669c09b33e
1: c017917 ! 1: b483a1c change dashboard subscription user loading to use RT->SystemUser and output any user load/create failure message
@@ -1,24 +1,20 @@
Author: Dustin Graves <dustin at bestpractical.com>
- add fix for dashboard subscription rights requirement and message in case of any failure
- remove setting of RealName field on user creation from dashboard subscription
+ change dashboard subscription user loading to use RT->SystemUser and output any user load/create failure message
diff --git a/share/html/Dashboards/Subscription.html b/share/html/Dashboards/Subscription.html
--- a/share/html/Dashboards/Subscription.html
+++ b/share/html/Dashboards/Subscription.html
@@
- my @recipients = @{ $fields{Recipients}->{Users} };
-
for ( RT::EmailParser->ParseEmailAddress( $val ) ) {
-- my ( $email, $name ) = ( $_->address, $_->name );
-+ my $email = $_->address;
+ my ( $email, $name ) = ( $_->address, $_->name );
- my $user = RT::User->new($session{CurrentUser});
- $user->LoadOrCreateByEmail(
+ my $user = RT::User->new(RT->SystemUser);
+ ($ok, $msg) = $user->LoadOrCreateByEmail(
EmailAddress => $email,
-- RealName => $name,
+ RealName => $name,
Comments => 'Autocreated when added as a dashboard subscription recipient',
);
@@ -39,3 +35,4 @@
}
@{ $fields{Recipients}->{Users} } = uniq @recipients;
+
-: ------- > 2: bf6f1a2 don't set RealName on dashboard subscription user creation
More information about the rt-commit
mailing list