Fixed to compare after conversion to localtime in alert_localtime logic
[platform/core/pim/calendar-service.git] / server / cal_server_alarm.c
index dfb90f7..3ee36be 100644 (file)
@@ -138,6 +138,18 @@ static int _cal_server_alarm_update_alarm_id(int alarm_id, int event_id, int tic
        return CALENDAR_ERROR_NONE;
 }
 
+static time_t _make_time(struct tm *time)
+{
+       time_t utc = mktime(time);
+
+       /* If an error occurs when isdst is 1, retry it after chaning isdst is 0 */
+       if (utc < 0 && time->tm_isdst != 0 ) {
+               time->tm_isdst = 0;
+               utc = mktime(time);
+       }
+       return utc;
+}
+
 static long long int _get_event_alert_utime(const char *field, int event_id, time_t current)
 {
        int ret = 0;
@@ -169,12 +181,18 @@ static int _get_event_alert_localtime(const char *field, int event_id, time_t cu
        int ret = 0;
        char query[CAL_DB_SQL_MAX_LEN] = {0};
        struct tm st = {0};
-       tzset();
-       localtime_r(&current, &st);
-       time_t mod_current = timegm(&st);
+       struct tm now_s = {0};
+       char buf[256] = {0};
+
+       localtime_r(&current, &now_s);
+       snprintf(buf, sizeof(buf), "%04d-%02d-%02dT%02d:%02d:%02d", (int)(now_s.tm_year + 1900),
+                               (int)(now_s.tm_mon + 1), (int)now_s.tm_mday, (int)now_s.tm_hour,
+                               (int)now_s.tm_min, (int)now_s.tm_sec);
+       DBG("Current time : (%s)", buf);
+
        snprintf(query, sizeof(query), "SELECT %s FROM %s "
-                       "WHERE event_id=%d AND strftime('%%s', %s)>%ld ORDER BY %s LIMIT 1",
-                       field, CAL_TABLE_LOCALTIME_INSTANCE, event_id, field, mod_current, field);
+                       "WHERE event_id=%d AND (strftime('%%s', %s) - strftime('%%s', '%s') > 0) ORDER BY %s LIMIT 1",
+                       field, CAL_TABLE_LOCALTIME_INSTANCE, event_id, field, buf, field);
 
        sqlite3_stmt *stmt = NULL;
        ret = cal_db_util_query_prepare(query, &stmt);
@@ -190,6 +208,7 @@ static int _get_event_alert_localtime(const char *field, int event_id, time_t cu
        if (CAL_SQLITE_ROW == cal_db_util_stmt_step(stmt))
                datetime = (const char *)sqlite3_column_text(stmt, 0);
 
+
        if (NULL == datetime || '\0' == *datetime) {
                /* LCOV_EXCL_START */
                ERR("Invalid datetime [%s]", datetime);
@@ -209,8 +228,9 @@ static int _get_event_alert_localtime(const char *field, int event_id, time_t cu
        st.tm_hour = h;
        st.tm_min = n;
        st.tm_sec = s;
+       st.tm_isdst = cal_time_is_dst_savings();
 
-       return (long long int)mktime(&st);
+       return (long long int)_make_time(&st);
 }
 
 static int64_t _get_todo_alert_utime(const char *field, int id, time_t now_t)
@@ -236,10 +256,10 @@ static int64_t _get_todo_alert_utime(const char *field, int id, time_t now_t)
        case CAL_SQLITE_ROW:
                utime = (int64_t)sqlite3_column_int64(stmt, 0);
                break;
-               /* LCOV_EXCL_START */
+               /* LCOV_EXCL_START
        case SQLITE_DONE:
                ERR("No data");
-               break;
+               break;*/ //Svace:371986
        default:
                ERR("Invalid return(%d)", ret);
                break;
@@ -255,12 +275,18 @@ static int _get_todo_alert_localtime(const char *field, int event_id, time_t cur
        int ret = 0;
        char query[CAL_DB_SQL_MAX_LEN] = {0};
        struct tm st = {0};
-       tzset();
-       localtime_r(&current, &st);
-       time_t mod_current = timegm(&st);
+       struct tm now_s = {0};
+       char buf[256] = {0};
+
+       localtime_r(&current, &now_s);
+       snprintf(buf, sizeof(buf), "%04d-%02d-%02dT%02d:%02d:%02d", (int)(now_s.tm_year + 1900),
+                               (int)(now_s.tm_mon + 1), (int)now_s.tm_mday, (int)now_s.tm_hour,
+                               (int)now_s.tm_min, (int)now_s.tm_sec);
+       DBG("Current time : (%s)", buf);
+
        snprintf(query, sizeof(query), "SELECT %s FROM %s "
-                       "WHERE id=%d AND strftime('%%s', %s)>%ld ORDER BY %s LIMIT 1",
-                       field, CAL_TABLE_SCHEDULE, event_id, field, mod_current, field);
+                       "WHERE id=%d AND (strftime('%%s', %s) - strftime('%%s', '%s') > 0) ORDER BY %s LIMIT 1",
+                       field, CAL_TABLE_SCHEDULE, event_id, field, buf, field);
 
        sqlite3_stmt *stmt = NULL;
        ret = cal_db_util_query_prepare(query, &stmt);
@@ -295,8 +321,9 @@ static int _get_todo_alert_localtime(const char *field, int event_id, time_t cur
        st.tm_hour = h;
        st.tm_min = n;
        st.tm_sec = s;
+       st.tm_isdst = cal_time_is_dst_savings();
 
-       return (long long int)mktime(&st);
+       return (long long int)_make_time(&st);
 }
 /*
  * time(NULL) is not appropriate as parameter.
@@ -347,14 +374,6 @@ static int cal_server_alarm_get_alert_time(int alarm_id, time_t *tt_alert)
                dtend_type = sqlite3_column_int(stmt, 8);
        }
 
-       if (NULL == tt_alert) {
-               /* LCOV_EXCL_START */
-               ERR("Invalid parameter: tt_alert is NULL");
-               sqlite3_finalize(stmt);
-               return CALENDAR_ERROR_INVALID_PARAMETER;
-               /* LCOV_EXCL_STOP */
-       }
-
        if (CALENDAR_ALARM_TIME_UNIT_SPECIFIC == unit) {
                if (CALENDAR_TIME_UTIME == type) {
                        *tt_alert = utime;
@@ -370,8 +389,9 @@ static int cal_server_alarm_get_alert_time(int alarm_id, time_t *tt_alert)
                                st.tm_hour = h;
                                st.tm_min = n;
                                st.tm_sec = s;
-                               *tt_alert = mktime(&st);
-                               DBG("datetime[%s] to %02d:%02d:%02d (%d)", datetime, h, n, s, *tt_alert);
+                               st.tm_isdst = cal_time_is_dst_savings();
+                               *tt_alert = _make_time(&st);
+                               DBG("datetime[%s] to %02d:%02d:%02d (%ld)", datetime, h, n, s, *tt_alert);
                        }
                }
                sqlite3_finalize(stmt);
@@ -380,6 +400,7 @@ static int cal_server_alarm_get_alert_time(int alarm_id, time_t *tt_alert)
        sqlite3_finalize(stmt);
 
        time_t current = time(NULL);
+
        current += (tick * unit);
        current -= 2; /* in case time passed */
 
@@ -408,9 +429,10 @@ static int cal_server_alarm_get_alert_time(int alarm_id, time_t *tt_alert)
                }
                break;
        }
-       DBG("alert_time(%d) = utime(%lld) - (tick(%d) * unit(%d))", *tt_alert, utime, tick, unit);
-
        *tt_alert = utime - (tick * unit);
+
+       DBG("alert_time(%ld) = utime(%lld) - (tick(%d) * unit(%d))", *tt_alert, utime, tick, unit);
+
        return CALENDAR_ERROR_NONE;
 }
 
@@ -425,10 +447,12 @@ static void _cal_server_alarm_get_upcoming_specific_utime(time_t utime, bool get
 {
        int ret = 0;
        char query[CAL_DB_SQL_MAX_LEN] = {0};
-       snprintf(query, sizeof(query), "SELECT event_id,remind_tick_unit,remind_tick,"
-                       "alarm_type,alarm_utime,alarm_datetime "
-                       "FROM %s WHERE remind_tick_unit =%d AND alarm_type =%d AND alarm_utime %s %ld %s",
-                       CAL_TABLE_ALARM, CALENDAR_ALARM_TIME_UNIT_SPECIFIC, CALENDAR_TIME_UTIME,
+       snprintf(query, sizeof(query), "SELECT A.event_id, A.remind_tick_unit, A.remind_tick,"
+                       "A.alarm_type, A.alarm_utime, A.alarm_datetime "
+                       "FROM "CAL_TABLE_ALARM" as A, "CAL_TABLE_SCHEDULE" as S ON A.event_id = S.id "
+                       "WHERE S.link_base_id = 0 AND A.remind_tick_unit = %d AND A.alarm_type = %d "
+                       "AND A.alarm_utime %s %ld %s",
+                       CALENDAR_ALARM_TIME_UNIT_SPECIFIC, CALENDAR_TIME_UTIME,
                        true == get_all ? "=" : ">", utime,
                        true == get_all ? "" : "ORDER BY alarm_utime ASC LIMIT 1");
 
@@ -471,10 +495,12 @@ static void _cal_server_alarm_get_upcoming_specific_localtime(const char *dateti
 {
        int ret = 0;
        char query[CAL_DB_SQL_MAX_LEN] = {0};
-       snprintf(query, sizeof(query), "SELECT event_id,remind_tick_unit,remind_tick,"
-                       "alarm_type,alarm_utime,alarm_datetime "
-                       "FROM %s WHERE remind_tick_unit=%d AND alarm_type=%d AND alarm_datetime %s '%s' %s",
-                       CAL_TABLE_ALARM, CALENDAR_ALARM_TIME_UNIT_SPECIFIC, CALENDAR_TIME_LOCALTIME,
+       snprintf(query, sizeof(query), "SELECT A.event_id, A.remind_tick_unit, A.remind_tick,"
+                       "A.alarm_type, A.alarm_utime, A.alarm_datetime "
+                       "FROM "CAL_TABLE_ALARM" as A, "CAL_TABLE_SCHEDULE" as S ON A.event_id = S.id "
+                       "WHERE S.link_base_id = 0 AND A.remind_tick_unit = %d AND A.alarm_type = %d "
+                       "AND A.alarm_datetime %s '%s' %s",
+                       CALENDAR_ALARM_TIME_UNIT_SPECIFIC, CALENDAR_TIME_LOCALTIME,
                        true == get_all ? "=" : ">", datetime,
                        true == get_all ? "" : "ORDER BY alarm_datetime ASC LIMIT 1");
 
@@ -519,7 +545,8 @@ static void _cal_server_alarm_get_upcoming_specific_localtime(const char *dateti
                st.tm_hour = h;
                st.tm_min = n;
                st.tm_sec = s;
-               ad->alert_utime = (long long int)mktime(&st);
+               st.tm_isdst = cal_time_is_dst_savings();
+               ad->alert_utime = (long long int)_make_time(&st);
                if (false == get_all) break;
        }
        sqlite3_finalize(stmt);
@@ -535,9 +562,11 @@ static void _cal_server_alarm_get_upcoming_nonspecific_event_utime(time_t utime,
        char query[CAL_DB_SQL_MAX_LEN] = {0};
        snprintf(query, sizeof(query), "SELECT A.event_id,A.remind_tick_unit,A.remind_tick, "
                        "A.alarm_type,B.dtstart_utime,A.alarm_datetime "
-                       "FROM %s as A, %s as B ON A.event_id = B.event_id "
-                       "WHERE A.remind_tick_unit >%d AND (B.dtstart_utime - (A.remind_tick_unit * A.remind_tick)) %s %ld %s",
-                       CAL_TABLE_ALARM, CAL_TABLE_UTIME_INSTANCE, CALENDAR_ALARM_TIME_UNIT_SPECIFIC,
+                       "FROM "CAL_TABLE_ALARM" as A, "CAL_TABLE_UTIME_INSTANCE" as B, "CAL_TABLE_SCHEDULE" as S "
+                       "ON A.event_id = B.event_id AND B.event_id = S.id "
+                       "WHERE S.link_base_id = 0 AND A.remind_tick_unit > %d "
+                       "AND (B.dtstart_utime - (A.remind_tick_unit * A.remind_tick)) %s %ld %s",
+                       CALENDAR_ALARM_TIME_UNIT_SPECIFIC,
                        true == get_all ? "=" : ">", utime,
                        true == get_all ? "" : "ORDER BY (B.dtstart_utime - (A.remind_tick_unit * A.remind_tick)) LIMIT 1");
 
@@ -585,12 +614,13 @@ static void _cal_server_alarm_get_upcoming_nonspecific_event_localtime(const cha
         * B:localtime
         */
        char query[CAL_DB_SQL_MAX_LEN] = {0};
-       snprintf(query, sizeof(query), "SELECT A.event_id,A.remind_tick_unit,A.remind_tick, "
-                       "A.alarm_type,A.alarm_utime,B.dtstart_datetime "
-                       "FROM %s as A, %s as B ON A.event_id = B.event_id "
-                       "WHERE A.remind_tick_unit >%d AND "
+       snprintf(query, sizeof(query), "SELECT A.event_id, A.remind_tick_unit, A.remind_tick, "
+                       "A.alarm_type, A.alarm_utime, B.dtstart_datetime "
+                       "FROM "CAL_TABLE_ALARM" as A, "CAL_TABLE_LOCALTIME_INSTANCE" as B, "CAL_TABLE_SCHEDULE" as S "
+                       "ON A.event_id = B.event_id AND B.event_id = S.id "
+                       "WHERE S.link_base_id = 0 AND A.remind_tick_unit >%d AND "
                        "(strftime('%%s', B.dtstart_datetime) - (A.remind_tick_unit * A.remind_tick) - strftime('%%s', '%s') %s 0) %s",
-                       CAL_TABLE_ALARM, CAL_TABLE_LOCALTIME_INSTANCE, CALENDAR_ALARM_TIME_UNIT_SPECIFIC,
+                       CALENDAR_ALARM_TIME_UNIT_SPECIFIC,
                        datetime, true == get_all ? "=" : ">",
                        true == get_all ? "" : "ORDER BY (strftime('%s', B.dtstart_datetime) - (A.remind_tick_unit * A.remind_tick)) LIMIT 1 ");
        sqlite3_stmt *stmt = NULL;
@@ -634,7 +664,8 @@ static void _cal_server_alarm_get_upcoming_nonspecific_event_localtime(const cha
                st.tm_hour = h;
                st.tm_min = n;
                st.tm_sec = s;
-               ad->alert_utime = (long long int)mktime(&st) - (ad->tick * ad->unit);
+               st.tm_isdst = cal_time_is_dst_savings();
+               ad->alert_utime = (long long int)_make_time(&st) - (ad->tick * ad->unit);
                if (false == get_all) break;
        }
        sqlite3_finalize(stmt);
@@ -645,18 +676,17 @@ static void _cal_server_alarm_get_upcoming_nonspecific_todo_utime(time_t utime,
        int ret = 0;
        /*
         * A:alarm
-        * B:todo(utime)
+        * S:todo(utime)
         */
        char query[CAL_DB_SQL_MAX_LEN] = {0};
-       snprintf(query, sizeof(query), "SELECT A.event_id,A.remind_tick_unit,A.remind_tick,"
-                       "A.alarm_type,B.dtend_utime,A.alarm_datetime "
-                       "FROM %s as A, %s as B ON A.event_id = B.id "
-                       "WHERE A.remind_tick_unit >%d AND B.type =%d "
-                       "AND (B.dtend_utime - (A.remind_tick_unit * A.remind_tick)) %s %ld %s",
-                       CAL_TABLE_ALARM, CAL_TABLE_SCHEDULE,
+       snprintf(query, sizeof(query), "SELECT A.event_id, A.remind_tick_unit, A.remind_tick,"
+                       "A.alarm_type, S.dtend_utime, A.alarm_datetime "
+                       "FROM "CAL_TABLE_ALARM" as A, "CAL_TABLE_SCHEDULE" as S ON A.event_id = S.id "
+                       "WHERE A.remind_tick_unit > %d AND S.type = %d "
+                       "AND (S.dtend_utime - (A.remind_tick_unit * A.remind_tick)) %s %ld %s",
                        CALENDAR_ALARM_TIME_UNIT_SPECIFIC, CALENDAR_BOOK_TYPE_TODO,
                        true == get_all ? "=" : ">", utime,
-                       true == get_all ? "" : "ORDER BY (B.dtend_utime - (A.remind_tick_unit * A.remind_tick)) LIMIT 1 ");
+                       true == get_all ? "" : "ORDER BY (S.dtend_utime - (A.remind_tick_unit * A.remind_tick)) LIMIT 1 ");
 
        sqlite3_stmt *stmt = NULL;
        ret = cal_db_util_query_prepare(query, &stmt);
@@ -704,10 +734,9 @@ static void _cal_server_alarm_get_upcoming_nonspecific_todo_localtime(const char
        char query[CAL_DB_SQL_MAX_LEN] = {0};
        snprintf(query, sizeof(query), "SELECT A.event_id,A.remind_tick_unit,A.remind_tick,"
                        "A.alarm_type,A.alarm_utime,B.dtend_datetime "
-                       "FROM %s as A, %s as B ON A.event_id = B.id "
+                       "FROM "CAL_TABLE_ALARM" as A, "CAL_TABLE_SCHEDULE" as B ON A.event_id = B.id "
                        "WHERE A.remind_tick_unit >%d AND B.type =%d "
                        "AND (strftime('%%s', B.dtend_datetime) - (A.remind_tick_unit * A.remind_tick) - strftime('%%s', '%s') %s 0) %s",
-                       CAL_TABLE_ALARM, CAL_TABLE_SCHEDULE,
                        CALENDAR_ALARM_TIME_UNIT_SPECIFIC, CALENDAR_BOOK_TYPE_TODO,
                        datetime, true == get_all ? "=" : ">",
                        true == get_all ? "" : "ORDER BY (strftime('%s', B.dtend_datetime) - (A.remind_tick_unit * A.remind_tick)) LIMIT 1 ");
@@ -753,7 +782,8 @@ static void _cal_server_alarm_get_upcoming_nonspecific_todo_localtime(const char
                st.tm_hour = h;
                st.tm_min = n;
                st.tm_sec = s;
-               ad->alert_utime = (long long int)mktime(&st) - (ad->tick * ad->unit);
+               st.tm_isdst = cal_time_is_dst_savings();
+               ad->alert_utime = (long long int)_make_time(&st) - (ad->tick * ad->unit);
                if (false == get_all) break;
        }
        sqlite3_finalize(stmt);
@@ -761,6 +791,7 @@ static void _cal_server_alarm_get_upcoming_nonspecific_todo_localtime(const char
 
 static void _cal_server_alarm_get_latest(time_t utime, bool get_all, GList **out_l)
 {
+       int ret = 0;
        CAL_FN_CALL();
        RET_IF(NULL == out_l);
 
@@ -769,9 +800,14 @@ static void _cal_server_alarm_get_latest(time_t utime, bool get_all, GList **out
        localtime_r(&utime, &st_local);
 
        char datetime[CAL_STR_SHORT_LEN32] = {0};
-       snprintf(datetime, sizeof(datetime), CAL_FORMAT_LOCAL_DATETIME,
+       ret = snprintf(datetime, sizeof(datetime), CAL_FORMAT_LOCAL_DATETIME,
                        st_local.tm_year +1900, st_local.tm_mon + 1, st_local.tm_mday,
                        st_local.tm_hour, st_local.tm_min, st_local.tm_sec);
+
+       if(ret < 0){
+               WARN("datetime is truncated (%s)",datetime);
+       }
+
        DBG("get alert to register with given time (%ld) datetime[%s]", utime, datetime);
 
        GList *l = NULL;
@@ -799,8 +835,8 @@ static gint _cal_server_alarm_sort_cb(gconstpointer a, gconstpointer b)
 static GFunc _cal_server_alarm_print_cb(gpointer data, gpointer user_data)
 {
        struct _alarm_data_s *ad = (struct _alarm_data_s *)data;
-       DBG("id(%d) unit(%d) tick(%d) type(%d) time(%lld) datetime[%s]",
-                       ad->event_id, ad->unit, ad->tick, ad->type, ad->time, ad->datetime);
+       DBG("id(%d) unit(%d) tick(%d) type(%d) time(%lld) datetime[%s] alert_utime[%lld]",
+                       ad->event_id, ad->unit, ad->tick, ad->type, ad->time, ad->datetime, ad->alert_utime);
        return 0;
 }
 
@@ -811,6 +847,8 @@ static int _cal_server_alarm_register(GList *alarm_list)
 
        int ret = CALENDAR_ERROR_NONE;
        GList *l = g_list_first(alarm_list);
+       RETV_IF(NULL == l, CALENDAR_ERROR_INVALID_PARAMETER);
+
        struct _alarm_data_s *ad = (struct _alarm_data_s *)l->data;
        RETVM_IF(NULL == ad, CALENDAR_ERROR_DB_FAILED, "No data");
 
@@ -945,6 +983,10 @@ static bool __app_matched_cb(app_control_h app_control, const char *package, voi
        }
        GList *cursor = g_list_first(alarm_list);
        for (i = 0; i < len; i++) {
+               if (NULL == cursor) {
+                       ERR("cursor is NULL");
+                       break;
+               }
                struct _alarm_data_s *ad = (struct _alarm_data_s *)cursor->data;
                if (NULL == ad) {
                        WARN("No data");
@@ -979,7 +1021,6 @@ static bool __app_matched_cb(app_control_h app_control, const char *package, voi
        app_control_send_launch_request(ac, NULL, NULL);
        app_control_destroy(ac);
 
-       g_list_free_full(alarm_list, free);
        for (i = 0; i < len; i++) {
                free(ids[i]);
                ids[i] = NULL;
@@ -1096,13 +1137,13 @@ static void cal_server_alarm_alert(time_t tt_alert)
 
        _cal_server_alarm_noti_with_callback(l);
        _cal_server_alarm_noti_with_control(l);
-       /* DO NOT FREE LIST, list is freed in callback */
+       g_list_free_full(l, free);
 }
 
 static int _alert_cb(alarm_id_t alarm_id, void *data)
 {
        CAL_FN_CALL();
-       DBG("alarm_id (%ld)", alarm_id);
+       DBG("alarm_id (%d)", (int)alarm_id);
 
        time_t tt_alert = 0;
        cal_server_alarm_get_alert_time(alarm_id, &tt_alert);
@@ -1160,18 +1201,18 @@ static int cal_server_alarm_init(void)
        _cal_server_alarm_set_timechange();
        _cal_server_alarm_set_inotify(_changed_cb);
 
-       ret = alarmmgr_init("calendar-service");
+       ret = alarmmgr_set_cb(_alert_cb, NULL);
        if (ret < 0) {
                /* LCOV_EXCL_START */
-               ERR("alarmmgr_init() Fail(%d)", ret);
+               ERR("alarmmgr_set_cb() Fail(%d)", ret);
                return CALENDAR_ERROR_SYSTEM;
                /* LCOV_EXCL_STOP */
        }
 
-       ret = alarmmgr_set_cb(_alert_cb, NULL);
+       ret = alarmmgr_init("calendar-service");
        if (ret < 0) {
                /* LCOV_EXCL_START */
-               ERR("alarmmgr_set_cb() Fail(%d)", ret);
+               ERR("alarmmgr_init() Fail(%d)", ret);
                return CALENDAR_ERROR_SYSTEM;
                /* LCOV_EXCL_STOP */
        }