opendev/gerrit
Code Issues Proposed changes

Fix putTopic hook firing

Browse Source 
The oldTopicName and newTopicName variables were only being set in a
local scope. Fix so that the instance variables (that the hook uses) are
set instead.

Change-Id: I0fcf4b07861f4e0c7df2df88784fe2f1f611a5ff
This commit is contained in:
Nasser Grainawi
2015-11-11 09:55:04 -08:00
parent e6b3015bcc
commit 5e2585f6a1
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-server/src/main/java/com/google/gerrit/server/change/PutTopic.java
View File

@@ -101,8 +101,8 @@ public class PutTopic implements RestModifyView<ChangeResource, Input>,
@Override @Override
public void updateChange(ChangeContext ctx) throws OrmException { public void updateChange(ChangeContext ctx) throws OrmException {
change = ctx.getChange(); change = ctx.getChange();
String newTopicName = Strings.nullToEmpty(input.topic); newTopicName = Strings.nullToEmpty(input.topic);
String oldTopicName = Strings.nullToEmpty(change.getTopic()); oldTopicName = Strings.nullToEmpty(change.getTopic());
if (oldTopicName.equals(newTopicName)) { if (oldTopicName.equals(newTopicName)) {
return; return;
} }