diff --git a/mozilla/webtools/bugzilla/Bugzilla/Bug.pm b/mozilla/webtools/bugzilla/Bugzilla/Bug.pm
index 2a248905d8f..726316808ad 100644
--- a/mozilla/webtools/bugzilla/Bugzilla/Bug.pm
+++ b/mozilla/webtools/bugzilla/Bugzilla/Bug.pm
@@ -3085,8 +3085,7 @@ sub GetBugActivity {
# Only includes attachments the user is allowed to see.
my $suppjoins = "";
my $suppwhere = "";
- if (Bugzilla->params->{"insidergroup"}
- && !Bugzilla->user->in_group(Bugzilla->params->{'insidergroup'}))
+ if (!Bugzilla->user->is_insider)
{
$suppjoins = "LEFT JOIN attachments
ON attachments.attach_id = bugs_activity.attach_id";
diff --git a/mozilla/webtools/bugzilla/importxml.pl b/mozilla/webtools/bugzilla/importxml.pl
index bd4aee186ce..f5385d17833 100755
--- a/mozilla/webtools/bugzilla/importxml.pl
+++ b/mozilla/webtools/bugzilla/importxml.pl
@@ -512,7 +512,7 @@ sub process_bug {
$long_desc{'isprivate'} = $comment->{'att'}->{'isprivate'} || 0;
# if one of the comments is private we need to set this flag
- if ( $long_desc{'isprivate'} && $exporter->in_group($params->{'insidergroup'})) {
+ if ( $long_desc{'isprivate'} && $exporter->is_insider) {
$private = 1;
}
my $data = $comment->field('thetext');
@@ -1198,7 +1198,7 @@ sub process_bug {
$err .= "No attachment ID specified, dropping attachment\n";
next;
}
- if (!$exporter->in_group($params->{'insidergroup'}) && $att->{'isprivate'}){
+ if (!$exporter->is_insider) && $att->{'isprivate'}){
$err .= "Exporter not in insidergroup and attachment marked private.\n";
$err .= " Marking attachment public\n";
$att->{'isprivate'} = 0;
diff --git a/mozilla/webtools/bugzilla/template/en/default/attachment/create.html.tmpl b/mozilla/webtools/bugzilla/template/en/default/attachment/create.html.tmpl
index 72844a36e3f..a6eb038247a 100644
--- a/mozilla/webtools/bugzilla/template/en/default/attachment/create.html.tmpl
+++ b/mozilla/webtools/bugzilla/template/en/default/attachment/create.html.tmpl
@@ -55,8 +55,7 @@
(optional) Check each existing attachment made obsolete by your new attachment.
[% IF attachments.size %]
[% FOREACH attachment = attachments %]
- [% IF ((attachment.isprivate == 0) || (Param("insidergroup")
- && user.in_group(Param("insidergroup")))) %]
+ [% IF ((attachment.isprivate == 0) || user.is_insider) %]
[% attachment.id %]: [% attachment.description FILTER html %]
@@ -108,7 +107,7 @@
%]
- [% IF (Param("insidergroup") && user.in_group(Param("insidergroup"))) %]
+ [% IF user.is_insider %]