Address the code review feedback.

joostdenijs · Oct 16, 2012 · 00bb3a6 · 00bb3a6
1 parent 6de7de5
commit 00bb3a6
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 8 deletions.
diff --git a/...rc/main/java/com/microsoft/windowsazure/services/media/implementation/RedirectFilter.java b/...rc/main/java/com/microsoft/windowsazure/services/media/implementation/RedirectFilter.java
@@ -37,7 +37,6 @@ public ClientResponse handle(ClientRequest request) throws ClientHandlerExceptio
         }
 
         URI originalURI = request.getURI();
-        String queryString = originalURI.getQuery();
         request.setURI(locationManager.getRedirectedURI(originalURI));
 
         ClientResponse response = getNext().handle(request);

diff --git a/...ava/com/microsoft/windowsazure/services/media/implementation/ResourceLocationManager.java b/...ava/com/microsoft/windowsazure/services/media/implementation/ResourceLocationManager.java
@@ -17,7 +17,6 @@
 
 import java.net.URI;
 import java.net.URISyntaxException;
-import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.Map;
 
@@ -42,10 +41,7 @@ public URI getRedirectedURI(URI originalURI) {
         String queryString = originalURI.getQuery();
 
         if (queryString != null && !queryString.isEmpty()) {
-            Map<String, String> queryParams = parseQueryString(queryString);
-            for (String key : new ArrayList<String>(queryParams.keySet())) {
-                uriBuilder.queryParam(key, queryParams.get(key));
-            }
+            uriBuilder.replaceQuery(queryString);
         }
         return uriBuilder.build();
     }

diff --git a/...c/main/java/com/microsoft/windowsazure/services/media/models/ListAccessPolicyOptions.java b/...c/main/java/com/microsoft/windowsazure/services/media/models/ListAccessPolicyOptions.java
@@ -19,6 +19,6 @@
  * Options class for listing access policies
  * No options available at this time.
  */
-public class ListAccessPolicyOptions {
+public class ListAccessPolicyOptions extends ListOptions {
 
 }
diff --git a/...i/src/main/java/com/microsoft/windowsazure/services/media/models/ListLocatorsOptions.java b/...i/src/main/java/com/microsoft/windowsazure/services/media/models/ListLocatorsOptions.java
@@ -17,6 +17,6 @@
 /**
  * The Class ListLocatorsOptions.
  */
-public class ListLocatorsOptions {
+public class ListLocatorsOptions extends ListOptions {
 
 }