Polishing (including removal of javadoc imports that show as package cycles in IntelliJ)

This commit is contained in:
Juergen Hoeller 2013-12-02 23:57:00 +01:00
parent 41332728f9
commit 2a52decbbc
7 changed files with 25 additions and 26 deletions

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2012 the original author or authors. * Copyright 2002-2013 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -16,7 +16,6 @@
package org.springframework.scheduling.config; package org.springframework.scheduling.config;
import org.springframework.scheduling.annotation.Scheduled;
import org.springframework.scheduling.support.CronTrigger; import org.springframework.scheduling.support.CronTrigger;
/** /**
@ -26,13 +25,13 @@ import org.springframework.scheduling.support.CronTrigger;
* *
* @author Chris Beams * @author Chris Beams
* @since 3.2 * @since 3.2
* @see Scheduled#cron() * @see org.springframework.scheduling.annotation.Scheduled#cron()
* @see ScheduledTaskRegistrar#setCronTasksList(java.util.List) * @see ScheduledTaskRegistrar#setCronTasksList(java.util.List)
* @see org.springframework.scheduling.TaskScheduler * @see org.springframework.scheduling.TaskScheduler
*/ */
public class CronTask extends TriggerTask { public class CronTask extends TriggerTask {
private String expression; private final String expression;
/** /**
@ -54,7 +53,9 @@ public class CronTask extends TriggerTask {
this.expression = cronTrigger.getExpression(); this.expression = cronTrigger.getExpression();
} }
public String getExpression() { public String getExpression() {
return expression; return this.expression;
} }
} }

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2012 the original author or authors. * Copyright 2002-2013 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -16,11 +16,8 @@
package org.springframework.test.util; package org.springframework.test.util;
import static org.springframework.test.util.MatcherAssertionErrors.assertThat;
import java.io.StringReader; import java.io.StringReader;
import java.util.Map; import java.util.Map;
import javax.xml.parsers.DocumentBuilder; import javax.xml.parsers.DocumentBuilder;
import javax.xml.parsers.DocumentBuilderFactory; import javax.xml.parsers.DocumentBuilderFactory;
import javax.xml.transform.Source; import javax.xml.transform.Source;
@ -29,11 +26,12 @@ import javax.xml.transform.dom.DOMSource;
import org.custommonkey.xmlunit.Diff; import org.custommonkey.xmlunit.Diff;
import org.custommonkey.xmlunit.XMLUnit; import org.custommonkey.xmlunit.XMLUnit;
import org.hamcrest.Matcher; import org.hamcrest.Matcher;
import org.springframework.test.web.servlet.result.MockMvcResultMatchers;
import org.w3c.dom.Document; import org.w3c.dom.Document;
import org.w3c.dom.Node; import org.w3c.dom.Node;
import org.xml.sax.InputSource; import org.xml.sax.InputSource;
import static org.springframework.test.util.MatcherAssertionErrors.*;
/** /**
* A helper class for assertions on XML content. * A helper class for assertions on XML content.
* *
@ -73,15 +71,12 @@ public class XmlExpectationsHelper {
* Parse the expected and actual content strings as XML and assert that the * Parse the expected and actual content strings as XML and assert that the
* two are "similar" -- i.e. they contain the same elements and attributes * two are "similar" -- i.e. they contain the same elements and attributes
* regardless of order. * regardless of order.
*
* <p>Use of this method assumes the * <p>Use of this method assumes the
* <a href="http://xmlunit.sourceforge.net/">XMLUnit<a/> library is available. * <a href="http://xmlunit.sourceforge.net/">XMLUnit<a/> library is available.
*
* @param expected the expected XML content * @param expected the expected XML content
* @param actual the actual XML content * @param actual the actual XML content
* * @see org.springframework.test.web.servlet.result.MockMvcResultMatchers#xpath(String, Object...)
* @see MockMvcResultMatchers#xpath(String, Object...) * @see org.springframework.test.web.servlet.result.MockMvcResultMatchers#xpath(String, Map, Object...)
* @see MockMvcResultMatchers#xpath(String, Map, Object...)
*/ */
public void assertXmlEqual(String expected, String actual) throws Exception { public void assertXmlEqual(String expected, String actual) throws Exception {

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2011 the original author or authors. * Copyright 2002-2013 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -16,6 +16,10 @@
package org.springframework.transaction.config; package org.springframework.transaction.config;
/**
* @author Chris Beams
* @since 3.1
*/
public abstract class TransactionManagementConfigUtils { public abstract class TransactionManagementConfigUtils {
/** /**

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2012 the original author or authors. * Copyright 2002-2013 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -32,6 +32,9 @@ import org.springframework.util.ClassUtils;
*/ */
public class AllEncompassingFormHttpMessageConverter extends FormHttpMessageConverter { public class AllEncompassingFormHttpMessageConverter extends FormHttpMessageConverter {
private static final boolean jaxb2Present =
ClassUtils.isPresent("javax.xml.bind.Binder", AllEncompassingFormHttpMessageConverter.class.getClassLoader());
private static final boolean jackson2Present = private static final boolean jackson2Present =
ClassUtils.isPresent("com.fasterxml.jackson.databind.ObjectMapper", AllEncompassingFormHttpMessageConverter.class.getClassLoader()) && ClassUtils.isPresent("com.fasterxml.jackson.databind.ObjectMapper", AllEncompassingFormHttpMessageConverter.class.getClassLoader()) &&
ClassUtils.isPresent("com.fasterxml.jackson.core.JsonGenerator", AllEncompassingFormHttpMessageConverter.class.getClassLoader()); ClassUtils.isPresent("com.fasterxml.jackson.core.JsonGenerator", AllEncompassingFormHttpMessageConverter.class.getClassLoader());
@ -40,9 +43,6 @@ public class AllEncompassingFormHttpMessageConverter extends FormHttpMessageConv
ClassUtils.isPresent("org.codehaus.jackson.map.ObjectMapper", AllEncompassingFormHttpMessageConverter.class.getClassLoader()) && ClassUtils.isPresent("org.codehaus.jackson.map.ObjectMapper", AllEncompassingFormHttpMessageConverter.class.getClassLoader()) &&
ClassUtils.isPresent("org.codehaus.jackson.JsonGenerator", AllEncompassingFormHttpMessageConverter.class.getClassLoader()); ClassUtils.isPresent("org.codehaus.jackson.JsonGenerator", AllEncompassingFormHttpMessageConverter.class.getClassLoader());
private static final boolean jaxb2Present =
ClassUtils.isPresent("javax.xml.bind.Binder", AllEncompassingFormHttpMessageConverter.class.getClassLoader());
@SuppressWarnings("deprecation") @SuppressWarnings("deprecation")
public AllEncompassingFormHttpMessageConverter() { public AllEncompassingFormHttpMessageConverter() {

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2012 the original author or authors. * Copyright 2002-2013 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -19,7 +19,6 @@ package org.springframework.http.converter.xml;
import javax.xml.transform.Source; import javax.xml.transform.Source;
import org.springframework.http.converter.FormHttpMessageConverter; import org.springframework.http.converter.FormHttpMessageConverter;
import org.springframework.http.converter.support.AllEncompassingFormHttpMessageConverter;
/** /**
* Extension of {@link org.springframework.http.converter.FormHttpMessageConverter}, * Extension of {@link org.springframework.http.converter.FormHttpMessageConverter},
@ -27,7 +26,8 @@ import org.springframework.http.converter.support.AllEncompassingFormHttpMessage
* *
* @author Juergen Hoeller * @author Juergen Hoeller
* @since 3.0.3 * @since 3.0.3
* @deprecated in favor of {@link AllEncompassingFormHttpMessageConverter} * @deprecated in favor of
* {@link org.springframework.http.converter.support.AllEncompassingFormHttpMessageConverter}
*/ */
@Deprecated @Deprecated
public class XmlAwareFormHttpMessageConverter extends FormHttpMessageConverter { public class XmlAwareFormHttpMessageConverter extends FormHttpMessageConverter {

View File

@ -28,7 +28,6 @@ import org.springframework.web.HttpRequestMethodNotSupportedException;
import org.springframework.web.HttpSessionRequiredException; import org.springframework.web.HttpSessionRequiredException;
import org.springframework.web.context.request.WebRequest; import org.springframework.web.context.request.WebRequest;
import org.springframework.web.context.support.WebApplicationObjectSupport; import org.springframework.web.context.support.WebApplicationObjectSupport;
import org.springframework.web.servlet.mvc.LastModified;
/** /**
* Convenient superclass for any kind of web content generator, * Convenient superclass for any kind of web content generator,
@ -204,7 +203,7 @@ public abstract class WebContentGenerator extends WebApplicationObjectSupport {
* programmatically do a lastModified calculation as described in * programmatically do a lastModified calculation as described in
* {@link WebRequest#checkNotModified(long)}. Default is "false", * {@link WebRequest#checkNotModified(long)}. Default is "false",
* effectively relying on whether the handler implements * effectively relying on whether the handler implements
* {@link LastModified} or not. * {@link org.springframework.web.servlet.mvc.LastModified} or not.
*/ */
public void setAlwaysMustRevalidate(boolean mustRevalidate) { public void setAlwaysMustRevalidate(boolean mustRevalidate) {
this.alwaysMustRevalidate = mustRevalidate; this.alwaysMustRevalidate = mustRevalidate;