Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Revert "[FIXED JENKINS-18629]"
  • Loading branch information
kohsuke committed Oct 25, 2013
1 parent 900e12d commit b322594
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 120 deletions.
75 changes: 0 additions & 75 deletions core/src/main/java/hudson/model/Descriptor.java
Expand Up @@ -29,7 +29,6 @@
import hudson.XmlFile;
import hudson.BulkChange;
import hudson.Util;
import hudson.init.Initializer;
import hudson.model.listeners.SaveableListener;
import hudson.util.FormApply;
import hudson.util.FormValidation.CheckMethod;
Expand Down Expand Up @@ -63,7 +62,6 @@
import java.util.Locale;
import java.util.Arrays;
import java.util.Collections;
import java.util.Stack;
import java.util.concurrent.ConcurrentHashMap;
import java.util.logging.Level;
import java.util.logging.Logger;
Expand Down Expand Up @@ -987,77 +985,4 @@ public void generateResponse(StaplerRequest req, StaplerResponse rsp, Object nod
public static final class Self {}

protected static Class self() { return Self.class; }

/**
* Register a global {@link BindInterceptor} that uses {@link Descriptor#newInstance(StaplerRequest, JSONObject)}
* for instantiation
*/
@Initializer
public static void initGlobalBindInterceptor() {
boolean newInstance = WebApp.get(Jenkins.getInstance().servletContext).bindInterceptors.add(new BindInterceptor() {
final class Input {
final Class type;
final JSONObject json;

private Input(Class type, JSONObject json) {
this.type = type;
this.json = json;
}

@Override
public boolean equals(Object o) {
if (this == o) return true;
if (o == null || getClass() != o.getClass()) return false;

Input rhs = (Input) o;
return json==rhs.json && type==rhs.type;

}

@Override
public int hashCode() {
return 31*type.hashCode() + json.hashCode();
}
}

private final ThreadLocal<Stack<Input>> inputs = new ThreadLocal<Stack<Input>>() {
protected Stack<Input> initialValue() {
return new Stack<Input>();
}
};

@Override
public Object instantiate(Class actualType, JSONObject json) {
if (Describable.class.isAssignableFrom(actualType)) {
Descriptor d = Jenkins.getInstance().getDescriptor(actualType);
if (d != null) {
try {
// only when Descriptor.newInstance is overridden
Method m = d.getClass().getMethod("newInstance", StaplerRequest.class, JSONObject.class);
if (m.getDeclaringClass() != Descriptor.class) {
Input newFrame = new Input(actualType,json);
if (!inputs.get().contains(newFrame)) {
// prevent infinite recursion in case Descriptor.newInstance calls right back into
// bindJSON
inputs.get().push(newFrame);
try {
StaplerRequest req = Stapler.getCurrentRequest();
if (req != null)
return d.newInstance(req, json);
} finally {
inputs.get().pop();
}
}
}
} catch (NoSuchMethodException e) {
throw new AssertionError(e); // this can't happen because Descriptor defines such a method
} catch (FormException e) {
throw new IllegalArgumentException(e);
}
}
}
return DEFAULT;
}
});
}
}
45 changes: 0 additions & 45 deletions test/src/test/java/hudson/model/DescriptorTest.java
Expand Up @@ -27,17 +27,10 @@
import hudson.model.Descriptor.PropertyType;
import hudson.tasks.Shell;
import static org.junit.Assert.*;

import net.sf.json.JSONObject;
import org.junit.Rule;
import org.junit.Test;
import org.jvnet.hudson.test.Bug;
import org.jvnet.hudson.test.JenkinsRule;
import org.jvnet.hudson.test.TestExtension;
import org.kohsuke.stapler.Stapler;
import org.kohsuke.stapler.StaplerRequest;

import java.util.concurrent.Callable;

public class DescriptorTest {

Expand All @@ -58,42 +51,4 @@ public class DescriptorTest {
}
}

/**
* Make sure Descriptor.newInstance gets invoked.
*/
@Bug(18629) @Test
public void callDescriptor_newInstance() throws Exception {
rule.executeOnServer(new Callable<Object>() {
@Override
public Object call() throws Exception {
DataBoundBean v = Stapler.getCurrentRequest().bindJSON(DataBoundBean.class, new JSONObject());
assertEquals(5,v.x);
return null;
}
});
}

public static class DataBoundBean extends AbstractDescribableImpl<DataBoundBean> {
int x;

// not @DataBoundConstructor
public DataBoundBean(int x) {
this.x = x;
}

@TestExtension
public static class DescriptorImpl extends Descriptor<DataBoundBean> {
@Override
public String getDisplayName() {
return "";
}

@Override
public DataBoundBean newInstance(StaplerRequest req, JSONObject formData) throws FormException {
return new DataBoundBean(5);
}
}

}

}

0 comments on commit b322594

Please sign in to comment.