Date: prev next · Thread: first prev next last
2013 Archives by date, by thread · List index


Quoting Stephan Bergmann <sbergman@redhat.com>:

On 03/28/2013 10:22 AM, Bjoern Michaelsen wrote:
On Wed, Mar 27, 2013 at 01:28:18PM +0100, Stephan Bergmann wrote:
Just noticed that when I add a reviewer to somebody else's change,
gerrit sends a mail taking on my identity ("Stephan Bergmann (via
Code Review) <gerrit@gerrit.libreoffice.org>") claiming "I have
submitted a patch for review."  But I did not submit anything for
review, I merely invited somebody else to also review something.  I
find that confusing at least.

If you find a better wording, submit a patch for:

https://gerrit.libreoffice.org/gitweb?p=gerrit-etc.git;a=blob;f=mail/NewChange.vm;h=34af52f6a6c9a90fe9e9bb758d9c9721eea9bba0;hb=a60a0f9771a6d31c3e6c0c1b3745613264678b3f

But is that text only used when one adds an additional reviewer to an existing change, or is it also used for the initial mail when a new change is uploaded?

let code speak:

https://gerrit.googlesource.com/gerrit/+/master/gerrit-server/src/main/java/com/google/gerrit/server/change/PostReviewers.java

in the method

private void postAdd(Change change, PostResult result)
mail is unconditionally sent using AddReviewerSender factory:

if (!added.isEmpty()) {
      AddReviewerSender cm;
[...]
      cm.send();
    }

looking at it:

public class AddReviewerSender extends NewChangeSender

you can figure out, that it inherits from NewChangeSender:

https://gerrit.googlesource.com/gerrit/+/master/gerrit-server/src/main/java/com/google/gerrit/server/mail/NewChangeSender.java

using that template:

@Override
  protected void formatChange() throws EmailException {
    appendText(velocifyFile("NewChange.vm"));
  }

and looking at RTFM it is even documented:

http://gerrit-documentation.googlecode.com/svn/Documentation/2.6/config-mail.html#_newchange_vm

If that is not what you expect, file a gerrit bug here:
https://code.google.com/p/gerrit/issues/list

David

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.