diff options
author | Dave Lawrence <dlawrence@mozilla.com> | 2013-01-30 19:45:35 +0100 |
---|---|---|
committer | Dave Lawrence <dlawrence@mozilla.com> | 2013-01-30 19:45:35 +0100 |
commit | a512b3b55e3c6222bdd80b5b94302dd356247d2d (patch) | |
tree | 3c3bc4660ef43203a0ca28dc76a8e2bb15a59e18 /Bugzilla | |
parent | 32add3f7b6cdfa932ce07b99c13c140df10bf537 (diff) | |
download | bugzilla-a512b3b55e3c6222bdd80b5b94302dd356247d2d.tar.gz bugzilla-a512b3b55e3c6222bdd80b5b94302dd356247d2d.tar.xz |
Bug 102048: Display the "Blocks" and "Depends On" columns in buglists
r/a=LpSolit
Diffstat (limited to 'Bugzilla')
-rw-r--r-- | Bugzilla/Field.pm | 4 | ||||
-rw-r--r-- | Bugzilla/Search.pm | 13 |
2 files changed, 15 insertions, 2 deletions
diff --git a/Bugzilla/Field.pm b/Bugzilla/Field.pm index f237182c2..3d482b3c9 100644 --- a/Bugzilla/Field.pm +++ b/Bugzilla/Field.pm @@ -209,9 +209,9 @@ use constant DEFAULT_FIELDS => ( buglist => 1}, {name => 'cc', desc => 'CC', in_new_bugmail => 1}, {name => 'dependson', desc => 'Depends on', in_new_bugmail => 1, - is_numeric => 1}, + is_numeric => 1, buglist => 1}, {name => 'blocked', desc => 'Blocks', in_new_bugmail => 1, - is_numeric => 1}, + is_numeric => 1, buglist => 1}, {name => 'attachments.description', desc => 'Attachment description'}, {name => 'attachments.filename', desc => 'Attachment filename'}, diff --git a/Bugzilla/Search.pm b/Bugzilla/Search.pm index 3c574274d..e89e86bc5 100644 --- a/Bugzilla/Search.pm +++ b/Bugzilla/Search.pm @@ -484,6 +484,14 @@ use constant COLUMN_JOINS => { to => 'id', }, }, + blocked => { + table => 'dependencies', + to => 'dependson', + }, + dependson => { + table => 'dependencies', + to => 'blocked', + }, 'longdescs.count' => { table => 'longdescs', join => 'INNER', @@ -551,6 +559,9 @@ sub COLUMNS { . $dbh->sql_string_concat('map_flagtypes.name', 'map_flags.status')), 'keywords' => $dbh->sql_group_concat('DISTINCT map_keyworddefs.name'), + + blocked => $dbh->sql_group_concat('DISTINCT map_blocked.blocked'), + dependson => $dbh->sql_group_concat('DISTINCT map_dependson.dependson'), 'longdescs.count' => 'COUNT(DISTINCT map_longdescs_count.comment_id)', ); @@ -646,7 +657,9 @@ sub REPORT_COLUMNS { # is here because it *always* goes into the GROUP BY as the first item, # so it should be skipped when determining extra GROUP BY columns. use constant GROUP_BY_SKIP => qw( + blocked bug_id + dependson flagtypes.name keywords longdescs.count |