Skip to content

Commit

Permalink
Fix #14677: Summary: developer names should be links
Browse files Browse the repository at this point in the history
  • Loading branch information
rombert committed Sep 22, 2012
1 parent 6c6c3d7 commit e054209
Showing 1 changed file with 20 additions and 1 deletion.
21 changes: 20 additions & 1 deletion core/summary_api.php
Expand Up @@ -62,6 +62,25 @@ function summary_helper_print_row( $p_label, $p_open, $p_resolved, $p_closed, $p
print( '</tr>' );
}

/**
* Returns a string representation of the user, together with a link to the issues
* acted on by the user ( reported, handled or commented on )
*
* @param int $p_user_id
* @return string
*/
function summary_helper_get_developer_label ( $p_user_id ) {

$t_user = string_display_line( user_get_name( $p_user_id ) );

return "<a class='subtle' href='view_all_set.php?type=1&amp;temporary=y
&amp;".FILTER_PROPERTY_REPORTER_ID."=$p_user_id
&amp;".FILTER_PROPERTY_HANDLER_ID."=$p_user_id
&amp;".FILTER_PROPERTY_NOTE_USER_ID."=$p_user_id
&amp;".FILTER_PROPERTY_MATCH_TYPE."=".FILTER_MATCH_ANY."'>$t_user</a>";

}

# Used in summary reports
# this function prints out the summary for the given enum setting
# The enum field name is passed in through $p_enum
Expand Down Expand Up @@ -837,7 +856,7 @@ function summary_print_developer_resolution( $p_resolution_enum_string ) {
echo '<tr ' . helper_alternate_class( $t_row_count ) . '>';
$t_row_count++;
echo '<td>';
echo string_display_line( user_get_name( $t_handler_id ) );
echo summary_helper_get_developer_label( $t_handler_id );
echo '</td>';

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

0 comments on commit e054209

Please sign in to comment.