sensord: remove sensor_permission_t from sensor_info 52/123852/2
authorkibak.yoon <kibak.yoon@samsung.com>
Fri, 7 Apr 2017 08:18:00 +0000 (17:18 +0900)
committerkibak.yoon <seseki17@gmail.com>
Tue, 11 Apr 2017 17:50:34 +0000 (02:50 +0900)
- it just only needs privilege string.

Change-Id: I82b25ae10075167781ed21baf62fe868a7b48450
Signed-off-by: kibak.yoon <kibak.yoon@samsung.com>
src/shared/sensor_info.cpp
src/shared/sensor_info.h

index 808c07f..7ff31f4 100644 (file)
@@ -40,7 +40,7 @@ sensor_info::sensor_info()
 , m_min_interval(0)
 , m_max_batch_count(0)
 , m_wakeup_supported(false)
-, m_permission(SENSOR_PERMISSION_STANDARD) /* TODO: change it to string */
+, m_privilege("")
 {
 }
 
@@ -56,7 +56,7 @@ sensor_info::sensor_info(const sensor_info &info)
 , m_min_interval(info.m_min_interval)
 , m_max_batch_count(info.m_max_batch_count)
 , m_wakeup_supported(info.m_wakeup_supported)
-, m_permission(SENSOR_PERMISSION_STANDARD)
+, m_privilege(info.m_privilege)
 {
 }
 
@@ -78,7 +78,8 @@ sensor_info::sensor_info(const sensor_info_t &info)
        set_min_interval(info.min_interval);
        set_max_batch_count(info.max_batch_count);
        set_wakeup_supported(info.wakeup_supported);
-       set_permission(SENSOR_PERMISSION_STANDARD);
+       /* TODO: sensor_info_t should have privilege string */
+       set_privilege("");
 }
 
 sensor_info::sensor_info(const sensor_info2_t &info)
@@ -97,13 +98,7 @@ sensor_info::sensor_info(const sensor_info2_t &info)
        set_min_interval(info.min_interval);
        set_max_batch_count(info.max_batch_count);
        set_wakeup_supported(info.wakeup_supported);
-
-       /* TODO : store string just itself */
-       std::string privilege = info.privilege;
-       if (privilege == "http://tizen.org/privilege/healthinfo")
-               set_permission(SENSOR_PERMISSION_HEALTH_INFO);
-       else
-               set_permission(SENSOR_PERMISSION_STANDARD);
+       set_privilege(info.privilege);
 }
 
 sensor_type_t sensor_info::get_type(void)
@@ -161,9 +156,9 @@ bool sensor_info::is_wakeup_supported(void)
        return m_wakeup_supported;
 }
 
-sensor_permission_t sensor_info::get_permission(void)
+std::string &sensor_info::get_privilege(void)
 {
-       return m_permission;
+       return m_privilege;
 }
 
 void sensor_info::set_type(sensor_type_t type)
@@ -221,9 +216,9 @@ void sensor_info::set_wakeup_supported(bool supported)
        m_wakeup_supported = supported;
 }
 
-void sensor_info::set_permission(sensor_permission_t permission)
+void sensor_info::set_privilege(const char *privilege)
 {
-       m_permission = permission;
+       m_privilege = privilege;
 }
 
 void sensor_info::serialize(raw_data_t &data)
@@ -239,12 +234,11 @@ void sensor_info::serialize(raw_data_t &data)
        put(data, m_min_interval);
        put(data, m_max_batch_count);
        put(data, m_wakeup_supported);
-       put(data, (int)m_permission);
+       put(data, m_privilege);
 }
 
 void sensor_info::deserialize(const char *data, int data_len)
 {
-       int permission;
        int type;
 
        raw_data_t raw_data(&data[0], &data[data_len]);
@@ -262,9 +256,7 @@ void sensor_info::deserialize(const char *data, int data_len)
        it = get(it, m_min_interval);
        it = get(it, m_max_batch_count);
        it = get(it, m_wakeup_supported);
-
-       it = get(it, permission);
-       m_permission = (sensor_permission_t)permission;
+       it = get(it, m_privilege);
 }
 
 void sensor_info::show(void)
@@ -279,7 +271,7 @@ void sensor_info::show(void)
        _I("Min_interval = %d", m_min_interval);
        _I("Max_batch_count = %d", m_max_batch_count);
        _I("Wakeup_supported = %d", m_wakeup_supported);
-       _I("Privilege = %d", (int)m_permission);
+       _I("Privilege = %s", m_privilege.c_str());
 }
 
 void sensor_info::clear(void)
@@ -295,7 +287,7 @@ void sensor_info::clear(void)
        m_min_interval = 0;
        m_max_batch_count = 0;
        m_wakeup_supported = false;
-       m_permission = SENSOR_PERMISSION_STANDARD;
+       m_privilege.clear();
 }
 
 void sensor_info::put(raw_data_t &data, int value)
index 93ae3aa..2611948 100644 (file)
@@ -44,14 +44,14 @@ public:
        std::string &get_type_uri(void);
        std::string &get_uri(void);
        std::string &get_model(void);
-       std::string & get_vendor(void);
+       std::string &get_vendor(void);
        float get_min_range(void);
        float get_max_range(void);
        float get_resolution(void);
        int get_min_interval(void);
        int get_max_batch_count(void);
        bool is_wakeup_supported(void);
-       sensor_permission_t get_permission(void);
+       std::string &get_privilege(void);
 
        /* TODO: it would be better to get_type() returns type(URI) */
        void set_type(sensor_type_t type);
@@ -65,7 +65,7 @@ public:
        void set_min_interval(int min_interval);
        void set_max_batch_count(int max_batch_count);
        void set_wakeup_supported(bool supported);
-       void set_permission(sensor_permission_t permission);
+       void set_privilege(const char *privilege);
 
        void clear(void);
 
@@ -85,7 +85,7 @@ private:
        int m_min_interval;
        int m_max_batch_count;
        bool m_wakeup_supported;
-       sensor_permission_t m_permission;
+       std::string m_privilege;
 
        /* TODO: use template */
        void put(raw_data_t &data, int value);