Changeset 9841 in josm for trunk/src/org

Timestamp:

2016-02-20T01:34:07+01:00 (8 years ago)

Author:

Don-vip

Message:

add unit test

Location:

trunk/src/org/openstreetmap/josm/gui/io

Files:

• UploadStrategySelectionPanel.java (modified) (2 diffs)
• UploadStrategySpecification.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/io/UploadStrategySelectionPanel.java

@@ -240 +240 @@
 
     public void setUploadStrategySpecification(UploadStrategySpecification strategy) {
-        if (strategy == null) return;
+        if (strategy == null)
+            return;
         rbStrategy.get(strategy.getStrategy()).setSelected(true);
         tfChunkSize.setEnabled(strategy.getStrategy() == UploadStrategy.CHUNKED_DATASET_STRATEGY);
@@ -256 +257 @@
         int chunkSize = getChunkSize();
         UploadStrategySpecification spec = new UploadStrategySpecification();
-        switch(strategy) {
-        case INDIVIDUAL_OBJECTS_STRATEGY:
-        case SINGLE_REQUEST_STRATEGY:
-            spec.setStrategy(strategy);
-            break;
-        case CHUNKED_DATASET_STRATEGY:
-            spec.setStrategy(strategy).setChunkSize(chunkSize);
-            break;
+        if (strategy != null) {
+            switch(strategy) {
+            case INDIVIDUAL_OBJECTS_STRATEGY:
+            case SINGLE_REQUEST_STRATEGY:
+                spec.setStrategy(strategy);
+                break;
+            case CHUNKED_DATASET_STRATEGY:
+                spec.setStrategy(strategy).setChunkSize(chunkSize);
+                break;
+            }
         }
         if (pnlMultiChangesetPolicyPanel.isVisible()) {

trunk/src/org/openstreetmap/josm/gui/io/UploadStrategySpecification.java

@@ -35 +35 @@
 
     /**
-     * Clones another upload strategy. If other is null,assumes default
-     * values.
+     * Clones another upload strategy. If other is null, assumes default values.
      *
      * @param other the other upload strategy
      */
     public UploadStrategySpecification(UploadStrategySpecification other) {
-        if (other == null) return;
-        this.strategy = other.strategy;
-        this.chunkSize = other.chunkSize;
-        this.policy = other.policy;
-        this.closeChangesetAfterUpload = other.closeChangesetAfterUpload;
+        if (other != null) {
+            this.strategy = other.strategy;
+            this.chunkSize = other.chunkSize;
+            this.policy = other.policy;
+            this.closeChangesetAfterUpload = other.closeChangesetAfterUpload;
+        }
     }
 
@@ -93 +93 @@
 
     public int getNumRequests(int numObjects) {
-        if (numObjects <= 0) return 0;
+        if (numObjects <= 0)
+            return 0;
         switch(strategy) {
         case INDIVIDUAL_OBJECTS_STRATEGY: return numObjects;
@@ -114 +115 @@
     @Override
     public boolean equals(Object obj) {
-        if (this == obj) return true;
-        if (obj == null || getClass() != obj.getClass()) return false;
+        if (this == obj)
+            return true;
+        if (obj == null || getClass() != obj.getClass())
+            return false;
         UploadStrategySpecification that = (UploadStrategySpecification) obj;
         return chunkSize == that.chunkSize &&