only include users which have bugs

[debbugs.git] / Debbugs / Bug / Tag.pm

diff --git a/Debbugs/Bug/Tag.pm b/Debbugs/Bug/Tag.pm
index 8ed9246ab5b666516f2c7714ce685f6a3dd4c23d..06dfb3f55132254b16cbc0a4d2a9b532bbd5720a 100644 (file)
--- a/Debbugs/Bug/Tag.pm
+++ b/Debbugs/Bug/Tag.pm
@@ -25,40 +25,110 @@ use strictures 2;
 use namespace::clean;
 use v5.10; # for state
 
+use Debbugs::User;
+use List::AllUtils qw(uniq);
 use Debbugs::Config qw(:config);
+use Carp qw(croak);
 
 state $valid_tags =
     {map {($_,1)} @{$config{tags}}};
 
+state $short_tags =
+   {%{$config{tags_single_letter}}};
+
 extends 'Debbugs::OOBase';
 
 around BUILDARGS => sub {
     my $orig = shift;
     my $class = shift;
     if (@_ == 1 && !ref $_[0]) {
-       my @tags = split / /,$_[0];
-       my %tags;
-       @tags{@tags} = (1) x @tags;
-       return $class->$orig(tags => \%tags);
+       return $class->$orig(keywords => $_[0]);
     } else {
        return $class->$orig(@_);
     }
 };
 
-has tags => (is => 'ro', isa => 'HashRef[Str]',
-            default => sub {{}},
+sub BUILD {
+    my $self = shift;
+    my $args = shift;
+    if (exists $args->{keywords}) {
+        my @tags;
+        if (ref($args->{keywords})) {
+            @tags = @{$args->{keywords}}
+        } else {
+            @tags = split /[, ]/,$args->{keywords};
+        }
+        return unless @tags;
+        $self->_set_tag(map {($_,1)} @tags);
+        delete $args->{keywords};
+    }
+}
+
+has tags => (is => 'ro',
+            isa => 'HashRef[Str]',
+            traits => ['Hash'],
+            lazy => 1,
+            reader => '_tags',
+            builder => '_build_tags',
+            handles => {has_tags => 'count',
+                         _set_tag => 'set',
+                         unset_tag => 'delete',
+                        },
            );
-has usertags => (is => 'ro',isa => 'HashRef[Str]',
-                default => sub {{}},
+has usertags => (is => 'ro',
+                isa => 'HashRef[Str]',
+                lazy => 1,
+                 traits => ['Hash'],
+                 handles => {unset_usertag => 'delete',
+                             has_usertags => 'count',
+                            },
+                reader => '_usertags',
+                builder => '_build_usertags',
                );
 
-sub tag_is_set {
-    return exists $_[0]->tags->{$_[1]} ? 1 : 0;
+sub has_any_tags {
+    my $self = shift;
+    return ($self->has_tags || $self->has_usertags);
+}
+
+has bug => (is => 'ro',
+            isa => 'Debbugs::Bug',
+            required => 1,
+           );
+
+has users => (is => 'ro',
+              isa => 'ArrayRef[Debbugs::User]',
+              default => sub {[]},
+             );
+
+sub _build_tags {
+    return {};
 }
 
-sub unset_tag {
+sub _build_usertags {
     my $self = shift;
-    delete $self->tags->{$_} foreach @_;
+    local $_;
+    my $t = {};
+    my $id = $self->bug->id;
+    for my $user (@{$self->users}) {
+        for my $tag ($user->tags_on_bug($id)) {
+            $t->{$tag} = $user->email;
+        }
+    }
+    return $t;
+}
+
+sub is_set {
+    return ($_[0]->tag_is_set($_[1]) or
+        $_[0]->usertag_is_set($_[1]));
+}
+
+sub tag_is_set {
+    return exists $_[0]->_tags->{$_[1]} ? 1 : 0;
+}
+
+sub usertag_is_set {
+    return exists $_[0]->_usertags->{$_[1]} ? 1 : 0;
 }
 
 sub set_tag {
@@ -67,7 +137,7 @@ sub set_tag {
        if (not $self->valid_tag($tag)) {
            confess("Invalid tag $tag");
        }
-       $self->tags->{$tag} = 1;
+       $self->_tags->{$tag} = 1;
     }
     return $self;
 }
@@ -77,7 +147,58 @@ sub valid_tag {
 }
 
 sub as_string {
-    return join(' ',sort keys %{$_[0]->tags})
+    my $self = shift;
+    return $self->join_all(' ');
+}
+
+sub join_all {
+    my $self = shift;
+    my $joiner = shift;
+    $joiner //= ', ';
+    return join($joiner,$self->all_tags);
+}
+
+sub join_usertags {
+    my $self = shift;
+    my $joiner = shift;
+    $joiner //= ', ';
+    return join($joiner,$self->usertags);
+}
+
+sub join_tags {
+    my $self = shift;
+    my $joiner = shift;
+    $joiner //= ', ';
+    return join($joiner,$self->tags);
+}
+
+sub all_tags {
+    return uniq sort $_[0]->tags,$_[0]->usertags;
+}
+
+sub tags {
+    return sort keys %{$_[0]->_tags}
+}
+
+sub short_tags {
+    my $self = shift;
+    my @r;
+    for my $tag ($self->tags) {
+       next unless exists $short_tags->{$tag};
+       push @r,
+          {long => $tag,
+           short => $short_tags->{$tag},
+          };
+    }
+    if (wantarray) {
+       return @r;
+    } else {
+       return [@r];
+    }
+}
+
+sub usertags {
+    return sort keys %{$_[0]->_usertags}
 }
 
 no Mouse;