[#1239] kavitha|yekkanti Refactored code

arjunk · Feb 11, 2013 · 84e8d29 · 84e8d29
1 parent 40d4f94
commit 84e8d29
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/RapidFTR-Android/src/main/java/com/rapidftr/service/ChildService.java b/RapidFTR-Android/src/main/java/com/rapidftr/service/ChildService.java
@@ -180,7 +180,14 @@ public Child apply(Map content) {
     public void setPhoto(Child child) throws IOException, JSONException {
         PhotoCaptureHelper photoCaptureHelper = new PhotoCaptureHelper(context);
 
-        JSONArray photoKeys = new JSONArray(child.optString("photo_keys"));
+        JSONArray photoKeys = child.optJSONArray("photo_keys");
+        if(photoKeys != null){
+            getPhotoFromServerIfNeeded(child, photoCaptureHelper, photoKeys);
+        }
+
+    }
+
+    private void getPhotoFromServerIfNeeded(Child child, PhotoCaptureHelper photoCaptureHelper, JSONArray photoKeys) throws JSONException, IOException {
         for(int i = 0; i < photoKeys.length(); i++){
             String photoKey = photoKeys.get(i).toString();
             try {
@@ -192,7 +199,6 @@ public void setPhoto(Child child) throws IOException, JSONException {
                 getPhotoFromServer(child, photoCaptureHelper, photoKey);
             }
         }
-
     }
 
     public void getPhotoFromServer(Child child, PhotoCaptureHelper photoCaptureHelper, String fileName) throws IOException {