View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0013718 | mantisbt | customization | public | 2012-01-04 04:19 | 2014-09-23 18:05 |
| Reporter | JanHegewald | Assigned To | dregad | ||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 1.2.1 | ||||
| Target Version | 1.2.9 | Fixed in Version | 1.2.9 | ||
| Summary | 0013718: Custom severity not displayed cross-project | ||||
| Description | Assuming issue "1" in project "A" has a custom severity "N/A" defined as described under Additional Information below, the severity string is displayed as @X@ in the view issues page :
Same behavior in view issue details page, looking at another issue "2" defined in project "B", relationship to issue 1 This is related to the other 'custom resolution/status not displayed cross project' issues (0013707, 0011323). | ||||
| Additional Information | Sample definition for custom resolution: severity_enum_string = '10:feature,20:trivial,30:text,40:tweak,50:minor,60:major,70:crash,80:block,100:N/A'; | ||||
| Tags | No tags attached. | ||||
| Attached Files | 0001-Fix-cross-project-display-of-custom-severity-and-pri.patch (3,948 bytes)
From eb011e1696afe41212121cd892e44a756c8c1fa7 Mon Sep 17 00:00:00 2001
From: Damien Regad <damien.regad@merckgroup.com>
Date: Fri, 6 Jan 2012 00:44:09 +0100
Subject: [PATCH] Fix cross-project display of custom severity and priority
When a custom severity is defined in a different project, its code
was displayed as @X@ instead of the corresponding string in the
severity column on the View Issues page.
The same problem applied to custom priorities
Fixes #13718
---
core/columns_api.php | 4 ++--
core/print_api.php | 16 ++++++++--------
my_view_inc.php | 2 +-
3 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/core/columns_api.php b/core/columns_api.php
index dbba861..3acb4ac 100644
--- a/core/columns_api.php
+++ b/core/columns_api.php
@@ -931,7 +931,7 @@ function print_column_edit( $p_bug, $p_columns_target = COLUMNS_TARGET_VIEW_PAGE
function print_column_priority( $p_bug, $p_columns_target = COLUMNS_TARGET_VIEW_PAGE ) {
echo '<td>';
if( ON == config_get( 'show_priority_text' ) ) {
- print_formatted_priority_string( $p_bug->status, $p_bug->priority );
+ print_formatted_priority_string( $p_bug );
} else {
print_status_icon( $p_bug->priority );
}
@@ -1070,7 +1070,7 @@ function print_column_category_id( $p_bug, $p_columns_target = COLUMNS_TARGET_VI
*/
function print_column_severity( $p_bug, $p_columns_target = COLUMNS_TARGET_VIEW_PAGE ) {
echo '<td class="center">';
- print_formatted_severity_string( $p_bug->status, $p_bug->severity );
+ print_formatted_severity_string( $p_bug );
echo '</td>';
}
diff --git a/core/print_api.php b/core/print_api.php
index 764d5be..9e6304b 100644
--- a/core/print_api.php
+++ b/core/print_api.php
@@ -1143,13 +1143,13 @@ function print_bug_link( $p_bug_id, $p_detail_info = true ) {
# formats the priority given the status
# shows the priority in BOLD if the bug is NOT closed and is of significant priority
-function print_formatted_priority_string( $p_status, $p_priority ) {
- $t_pri_str = get_enum_element( 'priority', $p_priority );
+function print_formatted_priority_string( $p_bug ) {
+ $t_pri_str = get_enum_element( 'priority', $p_bug->priority );
$t_priority_threshold = config_get( 'priority_significant_threshold' );
if( $t_priority_threshold >= 0 &&
- $p_priority >= $t_priority_threshold &&
- $p_status < config_get( 'bug_closed_status_threshold' ) ) {
+ $p_bug->priority >= $t_priority_threshold &&
+ $p_bug->status < config_get( 'bug_closed_status_threshold' ) ) {
echo "<span class=\"bold\">$t_pri_str</span>";
} else {
echo $t_pri_str;
@@ -1158,13 +1158,13 @@ function print_formatted_priority_string( $p_status, $p_priority ) {
# formats the severity given the status
# shows the severity in BOLD if the bug is NOT closed and is of significant severity
-function print_formatted_severity_string( $p_status, $p_severity ) {
- $t_sev_str = get_enum_element( 'severity', $p_severity );
+function print_formatted_severity_string( $p_bug ) {
+ $t_sev_str = get_enum_element( 'severity', $p_bug->severity, auth_get_current_user_id(), $p_bug->project_id );
$t_severity_threshold = config_get( 'severity_significant_threshold' );
if( $t_severity_threshold >= 0 &&
- $p_severity >= $t_severity_threshold &&
- $p_status < config_get( 'bug_closed_status_threshold' ) ) {
+ $p_bug->severity >= $t_severity_threshold &&
+ $p_bug->status < config_get( 'bug_closed_status_threshold' ) ) {
echo "<span class=\"bold\">$t_sev_str</span>";
} else {
echo $t_sev_str;
diff --git a/my_view_inc.php b/my_view_inc.php
index cedc67f..c40b04b 100644
--- a/my_view_inc.php
+++ b/my_view_inc.php
@@ -461,7 +461,7 @@ echo "($v_start - $v_end / $t_bug_count)";
}
if( ON == config_get( 'show_priority_text' ) ) {
- print_formatted_priority_string( $t_bug->status, $t_bug->priority );
+ print_formatted_priority_string( $t_bug );
} else {
print_status_icon( $t_bug->priority );
}
--
1.7.0.4
| ||||
|
In my opinion this can be fixed by changing to things: In print_api.php: function print_formatted_severity_string( $p_status, $p_severity, $p_bug=null ) { to function print_formatted_severity_string( $p_status, $p_severity, $p_bug=null ) { In columns_api.php: function print_column_severity( $p_bug, $p_columns_target = COLUMNS_TARGET_VIEW_PAGE ) { to function print_column_severity( $p_bug, $p_columns_target = COLUMNS_TARGET_VIEW_PAGE ) { Can anyone verify that these changes are working? |
|
|
I'll have a look. Thanks for the bug report. |
|
|
Please test the attached patch |
|
|
Jan, did you get a chance to test ? |
|
|
Sorry for not answering. I'm going to test the patch on wednesday or thursday. |
|
|
I tested your patch and it seems to work for severities on 1.2.1. Priorities aren't customized here, so I could not test this point. I also could not test on 1.2.8, because I haven't customized severities and priorities there. |
|
|
Marking as 'acknowledged' not resolved/closed to track that change gets ported to master-2.0.x branch |
|
|
MantisBT: master c077ca60 2012-01-05 10:44 Details Diff |
Fix cross-project display of custom severity and priority When a custom severity is defined in a different project, its code was displayed as @X@ instead of the corresponding string in the severity column on the View Issues page. The same problem applied to custom priorities Fixes 0013718 |
Affected Issues 0013718 |
|
| mod - core/columns_api.php | Diff File | ||
| mod - core/print_api.php | Diff File | ||
| mod - my_view_inc.php | Diff File | ||
|
MantisBT: master-1.2.x 8538a5d1 2012-01-05 10:44 Details Diff |
Fix cross-project display of custom severity and priority When a custom severity is defined in a different project, its code was displayed as @X@ instead of the corresponding string in the severity column on the View Issues page. The same problem applied to custom priorities Fixes 0013718 |
Affected Issues 0013718 |
|
| mod - core/columns_api.php | Diff File | ||
| mod - core/print_api.php | Diff File | ||
| mod - my_view_inc.php | Diff File | ||