Issue 3204: When merging multiple changes with "submit whole topic", changes do not have the correct submitter
Status:  Released
Owner: ----
Closed:  Mar 2015
Cc:
Project Member Reported by david.pu...@sonymobile.com, Feb 23, 2015
*****************************************************************
*****                                                       *****
***** !!!! THIS BUG TRACKER IS FOR GERRIT CODE REVIEW !!!!  *****
*****                                                       *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, CYANOGENMOD,  *****
***** INTERNAL ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.*****
*****                                                       *****
*****   THOSE ISSUES BELONG IN DIFFERENT ISSUE TRACKERS     *****
*****                                                       *****
*****************************************************************

Affected Version: master

What steps will reproduce the problem?
1. Several changes in a topic are approved and verified, and the "Submit whole topic" button is active
2. Press "Submit whole topic" on one of the changes
3.

What is the expected output? What do you see instead?

For the change on which "Submit whole topic" was pressed, the submitter is correctly mentioned in the review message, i.e. "Change has been successfully merged into the git repository by User Name"

For the other changes in the topic, the message is only "Change has been successfully merged into the git repository".  Also, for all the other changes in the topic an email notification "Anonymous Coward has submitted this change and it was merged" is sent.

Please provide any additional information below.

See the secure-store-lib topic on stable-2.11 [1].  From those changes, [2] is the one that I pressed the "submit whole topic" button on and that is the only one that shows my name as the submitter.  The others don't, and for those I received the email notification.


[1] https://gerrit-review.googlesource.com/#/q/status:merged+project:gerrit+branch:stable-2.11+topic:secure-store-lib

[2] https://gerrit-review.googlesource.com/#/c/65102/

Feb 24, 2015
Project Member #1 david.pu...@sonymobile.com
https://gerrit-review.googlesource.com/65260
Status: ChangeUnderReview
Mar 11, 2015
Project Member #2 david.pu...@sonymobile.com
(No comment was entered for this change.)
Status: Submitted
Labels: FixedIn-2.12
Dec 21, 2015
Project Member #3 david.pu...@sonymobile.com
(No comment was entered for this change.)
Status: Released