Merge pull request spring-attic#28 from ramnivas/fix-RestTemple

Removed conflicting overloaded methods from RestTemplate

Author: Arjen Poutsma

src/main/scala/org/springframework/scala/web/client/RestTemplate.scala

@@ -30,6 +30,7 @@ import scala.reflect.ClassTag
  * advantage of functions and Scala types.
  *
  * @author Arjen Poutsma
+ * @author Ramnivas Laddad
  * @since 1.0
  * @constructor Creates a `RestTemplate` that wraps the given Java template, defaulting to the standard `RestTemplate`
  * @param javaTemplate the Java `RestTemplate` to wrap
@@ -75,17 +76,6 @@ class RestTemplate(val javaTemplate: org.springframework.web.client.RestOperatio
     Option(javaTemplate.getForObject(url, typeToClass[T], uriVariables.asJava))
   }
 
-  /**
-   * Retrieve a representation by doing a GET on the URL.
-   * The response (if any) is converted and returned.
-   *
-   * @param url the URL
-   * @return the converted object
-   */
-  def getForAny[T: ClassTag](url: URI): Option[T] = {
-    Option(javaTemplate.getForObject(url, typeToClass[T]))
-  }
-
   /**
    * Retrieve an entity by doing a GET on the specified URL.
    * The response is converted and stored in an `ResponseEntity`.
@@ -112,15 +102,6 @@ class RestTemplate(val javaTemplate: org.springframework.web.client.RestOperatio
     javaTemplate.getForEntity(url, typeToClass[T], uriVariables.asJava)
   }
 
-  /**
-   * Retrieve a representation by doing a GET on the URL .
-   * The response is converted and stored in an [[org.springframework.http.ResponseEntity]]}.
-   * @param url the URL
-   * @return the converted object
-   */
-  def getForEntity[T: ClassTag](url: URI): ResponseEntity[T] = {
-    javaTemplate.getForEntity(url, typeToClass[T])
-  }
 
   // HEAD
   /**
@@ -149,14 +130,6 @@ class RestTemplate(val javaTemplate: org.springframework.web.client.RestOperatio
     javaTemplate.headForHeaders(url, uriVariables.asJava)
   }
 
-  /**
-   * Retrieve all headers of the resource specified by the URL.
-   * @param url the URL
-   * @return all HTTP headers of that resource
-   */
-  def headForHeaders(url: URI): HttpHeaders = {
-    javaTemplate.headForHeaders(url)
-  }
 
   // POST
   /**
@@ -199,22 +172,6 @@ class RestTemplate(val javaTemplate: org.springframework.web.client.RestOperatio
     javaTemplate.postForLocation(url, request.orNull, uriVariables.asJava)
   }
 
-  /**
-   * Create a new resource by POSTing the given object to the URL, and returns the value of the `Location` header.
-   * This header typically indicates where the new resource is stored.
-   *
-   * The `request` parameter can be a [[org.springframework.http.HttpEntity]] in order to add additional HTTP headers
-   * to the request.
-   *
-   * @param url the URL
-   * @param request the Object to be POSTed
-   * @return the value for the `Location` header
-   * @see HttpEntity
-   */
-  def postForLocation(url: URI, request: Option[Any]): URI = {
-    javaTemplate.postForLocation(url, request.orNull)
-  }
-
   /**
    * Create a new resource by POSTing the given object to the URI template, and returns the representation found in
    * the response.
@@ -253,22 +210,6 @@ class RestTemplate(val javaTemplate: org.springframework.web.client.RestOperatio
     Option(javaTemplate.postForObject(url, request.orNull, typeToClass[T], uriVariables.asJava))
   }
 
-  /**
-   * Create a new resource by POSTing the given object to the URL, and returns the representation found in the
-   * response.
-   *
-   * The `request` parameter can be a [[org.springframework.http.HttpEntity]] in order to add additional HTTP headers
-   * to the request.
-   *
-   * @param url the URL
-   * @param request the Object to be POSTed
-   * @return the converted object
-   * @see HttpEntity
-   */
-  def postForObject[T: ClassTag](url: URI, request: Option[Any]): Option[T] = {
-    Option(javaTemplate.postForObject(url, request.orNull, typeToClass[T]))
-  }
-
   /**
    * Create a new resource by POSTing the given object to the URI template, and returns the response as
    * [[org.springframework.http.ResponseEntity]].
@@ -307,23 +248,6 @@ class RestTemplate(val javaTemplate: org.springframework.web.client.RestOperatio
     javaTemplate.postForEntity(url, request.orNull, typeToClass[T], uriVariables.asJava)
   }
 
-  /**
-   * Create a new resource by POSTing the given object to the URL, and returns the response as
-   * [[org.springframework.http.ResponseEntity]].
-   *
-   * The `request` parameter can be a [[org.springframework.http.HttpEntity]] in order to add additional HTTP headers
-   * to the request.
-   *
-   * @param url the URL
-   * @param request the Object to be POSTed, may be `null`
-   * @return the converted object
-   * @see HttpEntity
-   * @since 3.0.2
-   */
-  def postForEntity[T: ClassTag](url: URI, request: Option[Any]): ResponseEntity[T] = {
-    javaTemplate.postForEntity(url, request.orNull, typeToClass[T])
-  }
-
   // PUT
   /**
    * Create or update a resource by PUTting the given object to the URI.
@@ -359,19 +283,6 @@ class RestTemplate(val javaTemplate: org.springframework.web.client.RestOperatio
     javaTemplate.put(url, request.orNull, uriVariables.asJava)
   }
 
-  /**
-   * Creates a new resource by PUTting the given object to URL.
-   *
-   * The `request` parameter can be a [[org.springframework.http.HttpEntity]] in order to add additional HTTP headers
-   * to the request.
-   *
-   * @param url the URL
-   * @param request the Object to be PUT
-   * @see HttpEntity
-   */
-  def put(url: URI, request: Option[Any]) {
-    javaTemplate.put(url, request.orNull)
-  }
 
   // DELETE
   /**
@@ -397,14 +308,6 @@ class RestTemplate(val javaTemplate: org.springframework.web.client.RestOperatio
     javaTemplate.delete(url, uriVariables.asJava)
   }
 
-  /**
-   * Delete the resources at the specified URL.
-   *
-   * @param url the URL
-   */
-  def delete(url: URI) {
-    javaTemplate.delete(url)
-  }
 
   // OPTIONS
   /**
@@ -433,16 +336,6 @@ class RestTemplate(val javaTemplate: org.springframework.web.client.RestOperatio
     javaTemplate.optionsForAllow(url, uriVariables.asJava).asScala
   }
 
-  /**
-   * Return the value of the Allow header for the given URL.
-   *
-   * @param url the URL
-   * @return the value of the allow header
-   */
-  def optionsForAllow(url: URI): Set[HttpMethod] = {
-    javaTemplate.optionsForAllow(url).asScala
-  }
-
   // exchange
   /**
    * Execute the HTTP method to the given URI template, writing the given request entity to the request, and
@@ -477,18 +370,6 @@ class RestTemplate(val javaTemplate: org.springframework.web.client.RestOperatio
     javaTemplate.exchange(url, method, requestEntity.orNull, typeToClass[T], uriVariables.asJava)
   }
 
-  /**
-   * Execute the HTTP method to the given URI template, writing the given request entity to the request, and
-   * returns the response as `ResponseEntity`.
-   *
-   * @param url the URL
-   * @param method the HTTP method (GET, POST, etc)
-   * @param requestEntity the entity (headers and/or body) to write to the request
-   * @return the response as entity
-   */
-  def exchange[T: ClassTag](url: URI, method: HttpMethod, requestEntity: Option[HttpEntity[_]]): ResponseEntity[T] = {
-    javaTemplate.exchange(url, method, requestEntity.orNull, typeToClass[T])
-  }
 
   // general execution
   /**
@@ -533,24 +414,6 @@ class RestTemplate(val javaTemplate: org.springframework.web.client.RestOperatio
       uriVariables.asJava))
   }
 
-  /**
-   * Execute the HTTP method to the given URL, preparing the request with the given function, and reading the response
-   * with a function.
-   *
-   * @param url the URL
-   * @param method the HTTP method (GET, POST, etc)
-   * @param requestFunction function that prepares the request
-   * @param responseFunction function that extracts the return value from the response
-   * @return an arbitrary object, as returned by the response object
-   */
-  def execute[T](url: URI, method: HttpMethod)
-                (requestFunction: ClientHttpRequest => Unit)
-                (responseFunction: ClientHttpResponse => T): Option[T] = {
-    Option(javaTemplate.execute(url, method, functionToRequestCallback(requestFunction),
-      functionToResponseExtractor(responseFunction)))
-
-  }
-
   private def asInstanceOfAnyRef(seq: Seq[Any]) = {
     seq.map(_.asInstanceOf[AnyRef])
   }