From 2bf23c0ef4f6582383c943729f961ac9b3c31adc Mon Sep 17 00:00:00 2001 From: Kohsuke Kawaguchi Date: Tue, 18 Mar 2014 15:53:51 -0700 Subject: [PATCH] Revert "The intent was for this code to move into the doReload() method" This reverts commit 71f3acfc3c1ccdd4e761c5a47e4738bcaf8df3b6. Test regressions --- core/src/main/java/hudson/model/AbstractItem.java | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/core/src/main/java/hudson/model/AbstractItem.java b/core/src/main/java/hudson/model/AbstractItem.java index b5045ad49f..92f369b7f9 100644 --- a/core/src/main/java/hudson/model/AbstractItem.java +++ b/core/src/main/java/hudson/model/AbstractItem.java @@ -605,9 +605,18 @@ public abstract class AbstractItem extends Actionable implements Item, HttpDelet // try to reflect the changes by reloading doReload(); + new XmlFile(Items.XSTREAM, out.getTemporaryFile()).unmarshal(this); + Items.whileUpdatingByXml(new Callable() { + @Override public Void call() throws IOException { + onLoad(getParent(), getRootDir().getName()); + return null; + } + }); + Jenkins.getInstance().rebuildDependencyGraphAsync(); // if everything went well, commit this new version out.commit(); + SaveableListener.fireOnChange(this, getConfigFile()); } finally { out.abort(); // don't leave anything behind } -- GitLab