From: Heewon Park <h_w.park@samsung.com>
Date: Mon, 22 Aug 2016 10:29:58 +0000 (+0900)
Subject: Update EnrolleeConf data model
X-Git-Tag: 1.2.0+RC1~73
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b9e04b058dcffd656fab282b5fed725ee555a38b;p=platform%2Fupstream%2Fiotivity.git

Update EnrolleeConf data model

 - chaged the getter/setter logic

Change-Id: I1871bc29aa0bae9d49fff4756e8dae54c5c6467d
Signed-off-by: Heewon Park <h_w.park@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/10733
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Jihun Ha <jihun.ha@samsung.com>
Reviewed-by: Uze Choi <uzchoi@samsung.com>
---

diff --git a/service/easy-setup/mediator/richsdk/android/EasySetupCore/src/main/java/org/iotivity/service/easysetup/mediator/EnrolleeConf.java b/service/easy-setup/mediator/richsdk/android/EasySetupCore/src/main/java/org/iotivity/service/easysetup/mediator/EnrolleeConf.java
index b2ba173..d7a8862 100755
--- a/service/easy-setup/mediator/richsdk/android/EasySetupCore/src/main/java/org/iotivity/service/easysetup/mediator/EnrolleeConf.java
+++ b/service/easy-setup/mediator/richsdk/android/EasySetupCore/src/main/java/org/iotivity/service/easysetup/mediator/EnrolleeConf.java
@@ -39,7 +39,7 @@ import java.util.List;
 public class EnrolleeConf
 {
     private static final String TAG = EnrolleeConf.class.getName();
-    private OcRepresentation mProvRep = null, mWiFiRep = null, mDevConfRep = null, mCloudRep = null;
+    protected OcRepresentation mProvRep = null;
     /**
      * Constructor
      *
@@ -49,25 +49,11 @@ public class EnrolleeConf
     public EnrolleeConf(OcRepresentation rep)
     {
         mProvRep = rep;
+    }
 
-        List<OcRepresentation> children = rep.getChildren();
-
-        for (OcRepresentation child : children) {
-            List<String> rts = child.getResourceTypes();
-
-            if(child.getUri().indexOf(ESConstants.OC_RSRVD_ES_URI_WIFI) != -1)
-            {
-                mWiFiRep = child;
-            }
-            else if(child.getUri().indexOf(ESConstants.OC_RSRVD_ES_URI_DEVCONF) != -1)
-            {
-                mDevConfRep = child;
-            }
-            else if(child.getUri().indexOf(ESConstants.OC_RSRVD_ES_URI_CLOUDSERVER) != -1)
-            {
-                mCloudRep = child;
-            }
-        }
+    public EnrolleeConf(EnrolleeConf enrolleeConf)
+    {
+        mProvRep = enrolleeConf.getProvResRep();
     }
 
     /**
@@ -77,14 +63,22 @@ public class EnrolleeConf
      */
     public String getDeviceName()
     {
-        try
-        {
-            if(mDevConfRep != null && mDevConfRep.hasAttribute(ESConstants.OC_RSRVD_ES_DEVNAME)) {
-                return (String) mDevConfRep.getValue(ESConstants.OC_RSRVD_ES_DEVNAME);
+        List<OcRepresentation> children = mProvRep.getChildren();
+
+        for (OcRepresentation child : children) {
+            if(child.getUri().indexOf(ESConstants.OC_RSRVD_ES_URI_DEVCONF) != -1)
+            {
+                try
+                {
+                    if(child.hasAttribute(ESConstants.OC_RSRVD_ES_DEVNAME)) {
+                        return (String) child.getValue(ESConstants.OC_RSRVD_ES_DEVNAME);
+                    }
+                } catch (OcException e) {
+                    Log.e(TAG, "getWiFiModes is failed.");
+                }
             }
-        } catch (OcException e) {
-                Log.e(TAG, "getWiFiModes is failed.");
         }
+
         return new String("");
     }
 
@@ -95,14 +89,22 @@ public class EnrolleeConf
      */
     public String getModelNumber()
     {
-        try
-        {
-            if(mDevConfRep != null && mDevConfRep.hasAttribute(ESConstants.OC_RSRVD_ES_MODELNUMBER)) {
-                return (String) mDevConfRep.getValue(ESConstants.OC_RSRVD_ES_MODELNUMBER);
+        List<OcRepresentation> children = mProvRep.getChildren();
+
+        for (OcRepresentation child : children) {
+            if(child.getUri().indexOf(ESConstants.OC_RSRVD_ES_URI_DEVCONF) != -1)
+            {
+                try
+                {
+                    if(child.hasAttribute(ESConstants.OC_RSRVD_ES_MODELNUMBER)) {
+                        return (String) child.getValue(ESConstants.OC_RSRVD_ES_MODELNUMBER);
+                    }
+                } catch (OcException e) {
+                    Log.e(TAG, "getModelNumber is failed.");
+                }
             }
-        } catch (OcException e) {
-                Log.e(TAG, "getModelNumber is failed.");
         }
+
         return new String("");
     }
 
@@ -113,17 +115,24 @@ public class EnrolleeConf
      */
     public ArrayList<WIFI_MODE> getWiFiModes()
     {
+        List<OcRepresentation> children = mProvRep.getChildren();
         ArrayList<WIFI_MODE> modes = new ArrayList<WIFI_MODE>();
-        try {
-            if (mWiFiRep != null && mWiFiRep.hasAttribute(ESConstants.OC_RSRVD_ES_SUPPORTEDWIFIMODE)) {
-                int modes_int[] = mWiFiRep.getValue(ESConstants.OC_RSRVD_ES_SUPPORTEDWIFIMODE);
-                for (int i = 0 ; i < modes_int.length ; ++i) {
-                    modes.add(WIFI_MODE.fromInt(modes_int[i]));
+        for (OcRepresentation child : children) {
+            if(child.getUri().indexOf(ESConstants.OC_RSRVD_ES_URI_WIFI) != -1)
+            {
+                try {
+                    if (child.hasAttribute(ESConstants.OC_RSRVD_ES_SUPPORTEDWIFIMODE)) {
+                        int modes_int[] = child.getValue(ESConstants.OC_RSRVD_ES_SUPPORTEDWIFIMODE);
+                        for (int i = 0 ; i < modes_int.length ; ++i) {
+                            modes.add(WIFI_MODE.fromInt(modes_int[i]));
+                        }
+                    }
+                } catch (OcException e) {
+                    Log.e(TAG, "getWiFiModes is failed.");
                 }
             }
-        } catch (OcException e) {
-            Log.e(TAG, "getWiFiModes is failed.");
         }
+
         return modes;
     }
 
@@ -134,12 +143,19 @@ public class EnrolleeConf
      */
     public WIFI_FREQ getWiFiFreq()
     {
-        try{
-            if(mWiFiRep != null && mWiFiRep.hasAttribute(ESConstants.OC_RSRVD_ES_SUPPORTEDWIFIFREQ))
-                return WIFI_FREQ.fromInt(
-                        (int)mWiFiRep.getValue(ESConstants.OC_RSRVD_ES_SUPPORTEDWIFIFREQ));
-        } catch (OcException e) {
-            Log.e(TAG, "getWiFiFreq is failed.");
+        List<OcRepresentation> children = mProvRep.getChildren();
+
+        for (OcRepresentation child : children) {
+            if(child.getUri().indexOf(ESConstants.OC_RSRVD_ES_URI_WIFI) != -1)
+            {
+                try{
+                    if(child.hasAttribute(ESConstants.OC_RSRVD_ES_SUPPORTEDWIFIFREQ))
+                        return WIFI_FREQ.fromInt(
+                                (int)child.getValue(ESConstants.OC_RSRVD_ES_SUPPORTEDWIFIFREQ));
+                } catch (OcException e) {
+                    Log.e(TAG, "getWiFiFreq is failed.");
+                }
+            }
         }
         return WIFI_FREQ.WIFI_FREQ_NONE;
     }
@@ -152,11 +168,21 @@ public class EnrolleeConf
      */
     public boolean isCloudAccessible()
     {
-        if(mCloudRep != null && mCloudRep.getUri().indexOf(ESConstants.OC_RSRVD_ES_URI_CLOUDSERVER) != -1)
-            return true;
+        List<OcRepresentation> children = mProvRep.getChildren();
 
+        for (OcRepresentation child : children) {
+            if(child.getUri().indexOf(ESConstants.OC_RSRVD_ES_URI_CLOUDSERVER) != -1)
+            {
+                return true;
+            }
+        }
         return false;
     }
+
+    public  OcRepresentation getProvResRep()
+    {
+        return mProvRep;
+    }
 }