Skip to content

Commit e054209

Browse files
committedSep 22, 2012
Fix #14677: Summary: developer names should be links
1 parent 6c6c3d7 commit e054209

File tree

1 file changed

+20
-1
lines changed

1 file changed

+20
-1
lines changed
 

‎core/summary_api.php

+20-1
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,25 @@ function summary_helper_print_row( $p_label, $p_open, $p_resolved, $p_closed, $p
6262
print( '</tr>' );
6363
}
6464

65+
/**
66+
* Returns a string representation of the user, together with a link to the issues
67+
* acted on by the user ( reported, handled or commented on )
68+
*
69+
* @param int $p_user_id
70+
* @return string
71+
*/
72+
function summary_helper_get_developer_label ( $p_user_id ) {
73+
74+
$t_user = string_display_line( user_get_name( $p_user_id ) );
75+
76+
return "<a class='subtle' href='view_all_set.php?type=1&amp;temporary=y
77+
&amp;".FILTER_PROPERTY_REPORTER_ID."=$p_user_id
78+
&amp;".FILTER_PROPERTY_HANDLER_ID."=$p_user_id
79+
&amp;".FILTER_PROPERTY_NOTE_USER_ID."=$p_user_id
80+
&amp;".FILTER_PROPERTY_MATCH_TYPE."=".FILTER_MATCH_ANY."'>$t_user</a>";
81+
82+
}
83+
6584
# Used in summary reports
6685
# this function prints out the summary for the given enum setting
6786
# The enum field name is passed in through $p_enum
@@ -837,7 +856,7 @@ function summary_print_developer_resolution( $p_resolution_enum_string ) {
837856
echo '<tr ' . helper_alternate_class( $t_row_count ) . '>';
838857
$t_row_count++;
839858
echo '<td>';
840-
echo string_display_line( user_get_name( $t_handler_id ) );
859+
echo summary_helper_get_developer_label( $t_handler_id );
841860
echo '</td>';
842861

843862
# We need to track the percentage of bugs that are considered fixed, as well as

0 commit comments

Comments
 (0)
Please sign in to comment.