cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From wid...@apache.org
Subject [2/3] git commit: updated refs/heads/master to 120e1cc
Date Sun, 13 Sep 2015 20:24:29 GMT
CLOUDSTACK-8817: listFirewallRules response JSON startport/endport not an int


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/221624d2
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/221624d2
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/221624d2

Branch: refs/heads/master
Commit: 221624d2fe70289f1a70eb908027167634858012
Parents: 4d626c1
Author: Ashwin Swaroop <ashwinswaroop93@gmail.com>
Authored: Sun Sep 13 14:45:44 2015 -0400
Committer: Ashwin Swaroop <ashwinswaroop93@gmail.com>
Committed: Sun Sep 13 14:45:44 2015 -0400

----------------------------------------------------------------------
 api/src/org/apache/cloudstack/api/response/FirewallResponse.java | 4 ++--
 server/src/com/cloud/api/ApiResponseHelper.java                  | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/221624d2/api/src/org/apache/cloudstack/api/response/FirewallResponse.java
----------------------------------------------------------------------
diff --git a/api/src/org/apache/cloudstack/api/response/FirewallResponse.java b/api/src/org/apache/cloudstack/api/response/FirewallResponse.java
index 368f732..462bd1b 100644
--- a/api/src/org/apache/cloudstack/api/response/FirewallResponse.java
+++ b/api/src/org/apache/cloudstack/api/response/FirewallResponse.java
@@ -87,11 +87,11 @@ public class FirewallResponse extends BaseResponse {
         this.protocol = protocol;
     }
 
-    public void setStartPort(String startPort) {
+    public void setStartPort(Integer startPort) {
         this.startPort = startPort;
     }
 
-    public void setEndPort(String endPort) {
+    public void setEndPort(Integer endPort) {
         this.endPort = endPort;
     }
 

http://git-wip-us.apache.org/repos/asf/cloudstack/blob/221624d2/server/src/com/cloud/api/ApiResponseHelper.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/api/ApiResponseHelper.java b/server/src/com/cloud/api/ApiResponseHelper.java
index 06c68c4..6ee7c26 100644
--- a/server/src/com/cloud/api/ApiResponseHelper.java
+++ b/server/src/com/cloud/api/ApiResponseHelper.java
@@ -2124,11 +2124,11 @@ public class ApiResponseHelper implements ResponseGenerator {
         response.setId(fwRule.getUuid());
         response.setProtocol(fwRule.getProtocol());
         if (fwRule.getSourcePortStart() != null) {
-            response.setStartPort(Integer.toString(fwRule.getSourcePortStart()));
+            response.setStartPort(fwRule.getSourcePortStart());
         }
 
         if (fwRule.getSourcePortEnd() != null) {
-            response.setEndPort(Integer.toString(fwRule.getSourcePortEnd()));
+            response.setEndPort(fwRule.getSourcePortEnd());
         }
 
         List<String> cidrs = ApiDBUtils.findFirewallSourceCidrs(fwRule.getId());


Mime
View raw message