From c25b318fc741b8b65a769446df6179a45b58b04e Mon Sep 17 00:00:00 2001
From: Mark Côté <mcote@mozilla.com>
Date: Thu, 5 Mar 2015 23:33:39 -0500
Subject: Bug 1140049 - Advertise MozReview on Create New Attachment page.
 r=glob

---
 extensions/MozReview/Extension.pm                  | 30 ++++++++++++++--------
 .../hook/attachment/create-before_form.html.tmpl   | 17 ++++++++++++
 .../en/default/hook/global/header-start.html.tmpl  | 11 ++++++++
 extensions/MozReview/web/style/attachment.css      | 13 ++++++++++
 4 files changed, 60 insertions(+), 11 deletions(-)
 create mode 100644 extensions/MozReview/template/en/default/hook/attachment/create-before_form.html.tmpl
 create mode 100644 extensions/MozReview/template/en/default/hook/global/header-start.html.tmpl
 create mode 100644 extensions/MozReview/web/style/attachment.css

(limited to 'extensions')

diff --git a/extensions/MozReview/Extension.pm b/extensions/MozReview/Extension.pm
index 2287a2136..ab7afb18a 100644
--- a/extensions/MozReview/Extension.pm
+++ b/extensions/MozReview/Extension.pm
@@ -22,25 +22,33 @@ sub template_before_process {
     my $vars = $args->{'vars'};
 
     return unless (($file eq 'bug/show-header.html.tmpl' ||
-                    $file eq 'bug/edit.html.tmpl') &&
+                    $file eq 'bug/edit.html.tmpl' ||
+                    $file eq 'attachment/create.html.tmpl') &&
                    Bugzilla->params->{mozreview_base_url});
 
     my $bug = exists $vars->{'bugs'} ? $vars->{'bugs'}[0] : $vars->{'bug'};
 
     if ($bug) {
-        my @rrids;
-        my $attachments = Bugzilla::Attachment->get_attachments_by_bug($bug);
+        if ($file eq 'attachment/create.html.tmpl') {
+            if ($bug->product eq 'Core' || $bug->product eq 'Firefox' ||
+                $bug->product eq 'Firefox for Android') {
+                $vars->{'mozreview_enabled'} = 1;
+            }
+        } else {
+            my @rrids;
+            my $attachments = Bugzilla::Attachment->get_attachments_by_bug($bug);
 
-        foreach my $attachment (@$attachments) {
-            if ($attachment->contenttype eq 'text/x-review-board-request' &&
-                !$attachment->isobsolete) {
-                push @rrids, ($attachment->data =~ m#/r/(\d+)/?$#);
+            foreach my $attachment (@$attachments) {
+                if ($attachment->contenttype eq 'text/x-review-board-request'
+                    && !$attachment->isobsolete) {
+                    push @rrids, ($attachment->data =~ m#/r/(\d+)/?$#);
+                }
             }
-        }
 
-        if (scalar @rrids) {
-            $vars->{'mozreview'} = 1;
-            $vars->{'review_request_ids'} = \@rrids;
+            if (scalar @rrids) {
+                $vars->{'mozreview'} = 1;
+                $vars->{'review_request_ids'} = \@rrids;
+            }
         }
     }
 }
diff --git a/extensions/MozReview/template/en/default/hook/attachment/create-before_form.html.tmpl b/extensions/MozReview/template/en/default/hook/attachment/create-before_form.html.tmpl
new file mode 100644
index 000000000..0bf6fe543
--- /dev/null
+++ b/extensions/MozReview/template/en/default/hook/attachment/create-before_form.html.tmpl
@@ -0,0 +1,17 @@
+[%# This Source Code Form is subject to the terms of the Mozilla Public
+  # License, v. 2.0. If a copy of the MPL was not distributed with this
+  # file, You can obtain one at http://mozilla.org/MPL/2.0/.
+  #
+  # This Source Code Form is "Incompatible With Secondary Licenses", as
+  # defined by the Mozilla Public License, v. 2.0.
+  #%]
+
+[% RETURN UNLESS mozreview_enabled %]
+
+<div class="mozreview-ad">
+  Are you attaching a patch? Consider trying out
+  <a href="https://reviewboard.mozilla.org/">MozReview</a>, Mozilla's
+  new repository-based code-review tool.
+  <a href="http://mozilla-version-control-tools.readthedocs.org/en/latest/mozreview.html">Read
+    the docs</a> to get started.
+</div>
diff --git a/extensions/MozReview/template/en/default/hook/global/header-start.html.tmpl b/extensions/MozReview/template/en/default/hook/global/header-start.html.tmpl
new file mode 100644
index 000000000..6ad026de6
--- /dev/null
+++ b/extensions/MozReview/template/en/default/hook/global/header-start.html.tmpl
@@ -0,0 +1,11 @@
+[%# This Source Code Form is subject to the terms of the Mozilla Public
+  # License, v. 2.0. If a copy of the MPL was not distributed with this
+  # file, You can obtain one at http://mozilla.org/MPL/2.0/.
+  #
+  # This Source Code Form is "Incompatible With Secondary Licenses", as
+  # defined by the Mozilla Public License, v. 2.0.
+  #%]
+
+[% IF template.name == 'attachment/create.html.tmpl' %]
+  [% style_urls.push('extensions/MozReview/web/style/attachment.css') %]
+[% END %]
diff --git a/extensions/MozReview/web/style/attachment.css b/extensions/MozReview/web/style/attachment.css
new file mode 100644
index 000000000..474bf466e
--- /dev/null
+++ b/extensions/MozReview/web/style/attachment.css
@@ -0,0 +1,13 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this file,
+ * You can obtain one at http://mozilla.org/MPL/2.0/.
+ *
+ * This Source Code Form is "Incompatible With Secondary Licenses", as
+ * defined by the Mozilla Public License, v. 2.0. */
+
+.mozreview-ad {
+    background-color: #fff9db;
+    color: #666458;
+    padding: 5px;
+    margin-bottom: 10px;
+}
-- 
cgit v1.2.3-24-g4f1b