opendev/gerrit
Code Issues Proposed changes

Add message to change when it is created to preserve upload timestamp

Browse Source 
On ChangeScreen2 the upload timestamp is not shown after a change has
been updated. Sometimes it is interesting to see the age of a change,
which is why it is useful to have a ChangeMessage with the upload
timestamp. When a change is created we now add 'Uploaded patch set 1'
as ChangeMessage.

Bug: issue 2076
Change-Id: Ie26477b9d168eeb1384c40ee3ce6a2f5f675164f
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
This commit is contained in:
Edwin Kempin
2013-09-13 16:15:43 +02:00
parent 2bd1cb12d1
commit ba5c93d9ea
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gerrit-server/src/main/java/com/google/gerrit/server/git/ReceiveCommits.java
View File

@@ -1454,8 +1454,14 @@ public class ReceiveCommits {
recipients.add(getRecipientsFromFooters(accountResolver, ps, footerLines)); recipients.add(getRecipientsFromFooters(accountResolver, ps, footerLines));
recipients.remove(me); recipients.remove(me);
ChangeMessage msg =
new ChangeMessage(new ChangeMessage.Key(change.getId(),
ChangeUtil.messageUUID(db)), me, ps.getCreatedOn(), ps.getId());
msg.setMessage("Uploaded patch set " + ps.getPatchSetId() + ".");
ins ins
.setReviewers(recipients.getReviewers()) .setReviewers(recipients.getReviewers())
.setMessage(msg)
.setSendMail(false) .setSendMail(false)
.insert(); .insert();
created = true; created = true;