Commit f8597b2f authored by Chris T's avatar Chris T

Fix Various Bugs

This fixes a few bugs:
- Makes sure "sent" has an accurate time
- Allows "Draft" to Go to "Sent" on the dbus
- Allows the time to be preserved if mmsd is closed and opened
parent b10efed8
......@@ -120,12 +120,14 @@ struct mms_retrieve_conf {
char *priority;
char *msgid;
time_t date;
char *datestamp;
};
struct mms_send_req {
enum mms_message_status status;
char *to;
time_t date;
char *datestamp;
char *content_type;
gboolean dr;
};
......
......@@ -18,6 +18,7 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*
*/
#define _XOPEN_SOURCE 700
#ifdef HAVE_CONFIG_H
#include <config.h>
......@@ -35,6 +36,8 @@
#include <glib.h>
#include <glib/gstdio.h>
#include <gdbus.h>
#include <time.h>
#include <stdio.h>
#include <gweb/gweb.h>
......@@ -129,6 +132,11 @@ static DBusConnection *connection;
static guint32 transaction_id_start = 0;
static const char *time_to_str(const time_t *t);
void debug_print(const char* s, void* data);
static void mms_load_settings(struct mms_service *service)
{
GError *error;
......@@ -220,6 +228,7 @@ static void emit_msg_status_changed(const char *path, const char *new_status)
signal = dbus_message_new_signal(path, MMS_MESSAGE_INTERFACE,
"PropertyChanged");
if (signal == NULL)
return;
......@@ -269,6 +278,7 @@ static DBusMessage *msg_mark_read(DBusConnection *conn,
g_free(state);
g_key_file_set_boolean(meta, "info", "read", TRUE);
mms->rc.status = MMS_MESSAGE_STATUS_READ;
mms_store_meta_close(service->identity, mms->uuid, meta, TRUE);
......@@ -722,6 +732,7 @@ static gboolean result_request_send_conf(struct mms_request *request)
g_key_file_set_string(meta, "info", "state", "sent");
g_key_file_set_string(meta, "info", "id", msg->sc.msgid);
request->msg->sr.status = MMS_MESSAGE_STATUS_SENT;
mms_message_free(msg);
......@@ -1048,6 +1059,7 @@ static DBusMessage *send_message(DBusConnection *conn,
struct mms_service *service = data;
struct mms_request *request;
GKeyFile *meta;
const char *datestr;
msg = g_new0(struct mms_message, 1);
if (msg == NULL)
......@@ -1060,7 +1072,13 @@ static DBusMessage *send_message(DBusConnection *conn,
msg->sr.dr = service->use_delivery_reports;
if (send_message_get_args(dbus_msg, msg) == FALSE) {
time(&msg->sr.date);
datestr = time_to_str(&msg->sr.date);
msg->sr.datestamp = g_strdup(datestr);
if (send_message_get_args(dbus_msg, msg, service) == FALSE) {
mms_debug("Invalid arguments");
release_attachement_data(msg->attachments);
......@@ -1100,6 +1118,7 @@ static DBusMessage *send_message(DBusConnection *conn,
if (meta == NULL)
goto release_request;
g_key_file_set_string(meta, "info", "date", msg->sr.datestamp);
g_key_file_set_string(meta, "info", "state", "draft");
if (service->use_delivery_reports) {
......@@ -1340,10 +1359,12 @@ static gboolean load_message_from_store(const char *service_id,
char *state = NULL;
gboolean read_status;
char *data_path = NULL;
char *datestr = NULL;
gboolean success = FALSE;
gboolean tainted = FALSE;
void *pdu;
size_t len;
struct tm tm;
meta = mms_store_meta_open(service_id, uuid);
if (meta == NULL)
......@@ -1355,6 +1376,13 @@ static gboolean load_message_from_store(const char *service_id,
read_status = g_key_file_get_boolean(meta, "info", "read", NULL);
datestr = g_key_file_get_string(meta, "info", "date", NULL);
if (datestr != NULL) {
strptime(datestr, "%Y-%m-%dT%H:%M:%S%z", &tm);
} else {
mms_error("src/service.c:load_message_from_store() There is no date stamp!");
}
data_path = mms_store_get_path(service_id, uuid);
if (data_path == NULL)
goto out;
......@@ -1373,25 +1401,28 @@ static gboolean load_message_from_store(const char *service_id,
msg->uuid = g_strdup(uuid);
if (strcmp(state, "received") == 0
&& msg->type == MMS_MESSAGE_TYPE_RETRIEVE_CONF) {
if (strcmp(state, "received") == 0 && msg->type == MMS_MESSAGE_TYPE_RETRIEVE_CONF) {
msg->rc.datestamp = g_strdup(datestr);
msg->rc.date = mktime(&tm);
if (read_status == TRUE)
msg->rc.status = MMS_MESSAGE_STATUS_READ;
else
msg->rc.status = MMS_MESSAGE_STATUS_RECEIVED;
} else if (strcmp(state, "downloaded") == 0
&& msg->type == MMS_MESSAGE_TYPE_RETRIEVE_CONF) {
} else if (strcmp(state, "downloaded") == 0 && msg->type == MMS_MESSAGE_TYPE_RETRIEVE_CONF) {
msg->rc.status = MMS_MESSAGE_STATUS_DOWNLOADED;
if (msg->transaction_id == NULL)
msg->transaction_id = "";
} else if (strcmp(state, "sent") == 0
&& msg->type == MMS_MESSAGE_TYPE_SEND_REQ)
} else if (strcmp(state, "sent") == 0 && msg->type == MMS_MESSAGE_TYPE_SEND_REQ) {
msg->sr.datestamp = g_strdup(datestr);
msg->sr.date = mktime(&tm);
msg->sr.status = MMS_MESSAGE_STATUS_SENT;
else if (strcmp(state, "draft") == 0
&& msg->type == MMS_MESSAGE_TYPE_SEND_REQ)
}
else if (strcmp(state, "draft") == 0 && msg->type == MMS_MESSAGE_TYPE_SEND_REQ) {
msg->sr.datestamp = g_strdup(datestr);
msg->sr.date = mktime(&tm);
msg->sr.status = MMS_MESSAGE_STATUS_DRAFT;
else if (msg->type != MMS_MESSAGE_TYPE_NOTIFICATION_IND &&
msg->type != MMS_MESSAGE_TYPE_DELIVERY_IND)
}
else if (msg->type != MMS_MESSAGE_TYPE_NOTIFICATION_IND && msg->type != MMS_MESSAGE_TYPE_DELIVERY_IND)
goto out;
success = TRUE;
......@@ -1880,14 +1911,13 @@ static void append_rc_msg_properties(DBusMessageIter *dict,
static void append_sr_msg_properties(DBusMessageIter *dict,
struct mms_message *msg)
{
const char *date = time_to_str(&msg->rc.date);
const char *status = mms_message_status_get_string(msg->sr.status);
mms_dbus_dict_append_basic(dict, "Status",
DBUS_TYPE_STRING, &status);
mms_dbus_dict_append_basic(dict, "Date",
DBUS_TYPE_STRING, &date);
DBUS_TYPE_STRING, &msg->sr.datestamp);
if (msg->sr.to != NULL)
append_msg_recipients(dict, msg);
......@@ -2121,6 +2151,9 @@ static gboolean result_request_notify_resp(struct mms_request *request)
if (meta == NULL)
return FALSE;
const char *datestr = time_to_str(&msg->rc.date);
msg->rc.datestamp = g_strdup(datestr);
g_key_file_set_string(meta, "info", "date", msg->rc.datestamp);
g_key_file_set_string(meta, "info", "state", "received");
mms_store_meta_close(request->service->identity,
......@@ -2537,7 +2570,6 @@ void mms_service_bearer_notify(struct mms_service *service, mms_bool_t active,
if (active == FALSE)
goto interface_down;
DBG("interface %s proxy %s", interface, proxy);
if (service->web != NULL) {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment