| Issue 3127: | change-merged event is not emitted when note-db is enabled | |
| 1 person starred this issue and may be notified of changes. | Back to list |
*****************************************************************
***** *****
***** !!!! 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. Run the site with note-db enabled
2. Merge a change
3.
What is the expected output? What do you see instead?
Expect to received a ChangeMergedEvent. No event is received.
Please provide any additional information below.
Running in the debugger shows that the event is not fired because submitter is null.
MergeOp.java @ line 976:
if (submitter != null) {
try {
hooks.doChangeMergedHook(c,
accountCache.get(submitter.getAccountId()).getAccount(),
merged, db, mergeResultRev);
} catch (OrmException ex) {
logError("Cannot run hook for submitted patch set " + c.getId(), ex);
}
}
Nov 9, 2015
Project Member
#1
david.pu...@sonymobile.com
Status:
ChangeUnderReview
|
|
| ► Sign in to add a comment |