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.