Skip to content

Commit

Permalink
[FIXED JENKINS-42389] implement ReconfigurableDescribable to fix data…
Browse files Browse the repository at this point in the history
… lost when folder gets updated
  • Loading branch information
imod committed Mar 15, 2017
1 parent 679be22 commit 3a4282f
Showing 1 changed file with 8 additions and 3 deletions.
Expand Up @@ -6,14 +6,14 @@
import edu.umd.cs.findbugs.annotations.NonNull;
import edu.umd.cs.findbugs.annotations.Nullable;
import hudson.Extension;
import hudson.model.Descriptor;
import hudson.model.Item;
import hudson.model.ItemGroup;
import hudson.model.*;
import net.sf.json.JSONObject;
import org.jenkinsci.lib.configprovider.ConfigProvider;
import org.jenkinsci.lib.configprovider.model.Config;
import org.jenkinsci.plugins.configfiles.ConfigFileStore;
import org.jenkinsci.plugins.configfiles.ConfigProviderComparator;
import org.jenkinsci.plugins.configfiles.buildwrapper.ManagedFile;
import org.kohsuke.stapler.StaplerRequest;

import java.io.IOException;
import java.util.*;
Expand Down Expand Up @@ -113,6 +113,10 @@ private Object readResolve() {
return this;
}

public FolderConfigFileProperty reconfigure(StaplerRequest req, JSONObject form) throws Descriptor.FormException {
return this;
}

@Extension(optional = true)
public static class DescriptorImpl extends AbstractFolderPropertyDescriptor {

Expand All @@ -122,4 +126,5 @@ public String getDisplayName() {
return "";
}
}

}

2 comments on commit 3a4282f

@darxriggs
Copy link
Contributor

@darxriggs darxriggs commented on 3a4282f Jan 13, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@imod I suggest that you delete this commit's orphaned branch and the other ones too - see https://github.com/jenkinsci/config-file-provider-plugin/branches.

@imod
Copy link
Member Author

@imod imod commented on 3a4282f Jan 14, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@darxriggs thanks for the hint - done...

Please sign in to comment.