Show owner and responsible full name instead of login.

Fixes bug#9353 - http://bugs.otrs.org/show_bug.cgi?id=9353 Customer Ticket Zoom shows owner login name instead of full name The Customer Ticket Overview already showed the full name of the owner. The Owner Login, which was previously displayed, can often be cryptic (like 'U00123' or 'mb').
OTRS · Apr 22, 2013 · 0c394c6 · 0c394c6
1 parent 8ee82b3
commit 0c394c6
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 7 deletions.
diff --git a/CHANGES b/CHANGES
@@ -1,4 +1,5 @@
 3.2.7 2013-??.??
+ - 2013-04-22 Fixed bug#9353 - Customer Ticket Zoom shows owner login name instead of full name.
  - 2013-04-18 Fixed bug#8599 - Problem with "[]" characters in name of attachment file.
 
 3.2.6 2013-04-23

diff --git a/Kernel/Modules/CustomerTicketOverview.pm b/Kernel/Modules/CustomerTicketOverview.pm
@@ -656,11 +656,11 @@ sub ShowTicketStatus {
     );
 
     if ( $Self->{Owner} ) {
-        my %Owner = $Self->{AgentUserObject}->GetUserData( UserID => $Ticket{OwnerID} );
+        my $OwnerName = $Self->{AgentUserObject}->UserName( UserID => $Ticket{OwnerID} );
         $Self->{LayoutObject}->Block(
             Name => 'RecordOwner',
             Data => {
-                %Owner,
+                OwnerName => $OwnerName,
             },
         );
     }

diff --git a/Kernel/Modules/CustomerTicketZoom.pm b/Kernel/Modules/CustomerTicketZoom.pm
@@ -901,9 +901,12 @@ sub _Mask {
 
     # ticket owner
     if ( $Self->{Config}->{AttributesView}->{Owner} ) {
+        my $OwnerName = $Self->{AgentUserObject}->UserName(
+            UserID => $Param{OwnerID},
+        );
         $Self->{LayoutObject}->Block(
             Name => 'Owner',
-            Data => \%Param,
+            Data => { OwnerName => $OwnerName },
         );
     }
 
@@ -914,9 +917,12 @@ sub _Mask {
         $Self->{Config}->{AttributesView}->{Responsible}
         )
     {
+        my $ResponsibleName = $Self->{AgentUserObject}->UserName(
+            UserID => $Param{ResponsibleID},
+        );
         $Self->{LayoutObject}->Block(
             Name => 'Responsible',
-            Data => \%Param,
+            Data => { ResponsibleName => $ResponsibleName },
         );
     }
 

diff --git a/Kernel/Output/HTML/Standard/CustomerTicketOverview.dtl b/Kernel/Output/HTML/Standard/CustomerTicketOverview.dtl
@@ -109,7 +109,7 @@
                     </td>
                     <td class="Status">$Text{"$Data{"State"}"}</td>
 <!-- dtl:block:RecordOwner -->
-                    <td class="Owner">$QData{"UserFirstname","15"} $QData{"UserLastname","15"}</td>
+                    <td class="Owner">$QData{"OwnerName"}</td>
 <!-- dtl:block:RecordOwner -->
 <!-- dtl:block:RecordQueue -->
                     <td class="Queue" title="$Data{"Queue"}">$QData{"Queue","20"}</td>

diff --git a/Kernel/Output/HTML/Standard/CustomerTicketZoom.dtl b/Kernel/Output/HTML/Standard/CustomerTicketZoom.dtl
@@ -177,10 +177,10 @@
                     <li><span class="Key">$Text{"Queue"}:</span> <span title="$QData{"Queue"}">$QData{"Queue","18"}</span></li>
 <!-- dtl:block:Queue -->
 <!-- dtl:block:Owner -->
-                    <li><span class="Key">$Text{"Owner"}:</span> <span title="$QData{"Owner"}">$QData{"Owner"}</span></li>
+                    <li><span class="Key">$Text{"Owner"}:</span> $QData{"OwnerName"}</li>
 <!-- dtl:block:Owner -->
 <!-- dtl:block:Responsible -->
-                    <li><span class="Key">$Text{"Responsible"}:</span> <span title="$QData{"Responsible"}">$QData{"Responsible"}</span></li>
+                    <li><span class="Key">$Text{"Responsible"}:</span> $QData{"ResponsibleName"}/li>
 <!-- dtl:block:Responsible -->
 <!-- dtl:block:ProcessData -->
                     <li><span class="Key">$Text{"Process"}:</span> <span title="$QData{"Process"}">$QData{"Process"}</span></li>