Re: [PATCH 3/4] service: Add DBus method to set delivery-report

[Denis Kenzior <denkenz@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2983 bytes --]

Hi Ronald,

On 08/06/2012 10:30 AM, Ronald Tessier wrote:
> ---
>   doc/service-api.txt |    6 ++++++
>   src/service.c       |   46 ++++++++++++++++++++++++++++++++++++++++++++++
>   2 files changed, 52 insertions(+)

Please separate this into two patches, one for api changes and one for 
code changes.

>
> diff --git a/doc/service-api.txt b/doc/service-api.txt
> index 02396ff..f247d3a 100644
> --- a/doc/service-api.txt
> +++ b/doc/service-api.txt
> @@ -82,6 +82,12 @@ Methods		array{object,dict} GetMessages()
>
>   			Possible Errors: [service].Error.InvalidArguments
>
> +		void SetProperty(string name, variant value)
> +
> +			Changes the value of the specified property.
> +
> +			Possible Errors: [service].Error.InvalidArguments
> +
>
>   Signals		MessageAdded(object path, dict properties)
>

You're missing the Property section additions and the description of the 
UseDeliveryReports property.

> diff --git a/src/service.c b/src/service.c
> index 7243efd..c1ca4ab 100644
> --- a/src/service.c
> +++ b/src/service.c
> @@ -887,6 +887,48 @@ out:
>   	return reply;
>   }
>
> +static DBusMessage *set_property(DBusConnection *conn, DBusMessage *dbus_msg,
> +								void *data)
> +{
> +	struct mms_service *service = data;
> +	DBusMessageIter iter;
> +	DBusMessageIter var;
> +	const char *property;
> +
> +	if (!dbus_message_iter_init(dbus_msg,&iter))
> +		return __mms_error_invalid_args(dbus_msg);
> +
> +	if (dbus_message_iter_get_arg_type(&iter) != DBUS_TYPE_STRING)
> +		return __mms_error_invalid_args(dbus_msg);
> +
> +	dbus_message_iter_get_basic(&iter,&property);
> +	dbus_message_iter_next(&iter);
> +
> +	if (dbus_message_iter_get_arg_type(&iter) != DBUS_TYPE_VARIANT)
> +		return __mms_error_invalid_args(dbus_msg);
> +
> +	dbus_message_iter_recurse(&iter,&var);
> +
> +	if (!strcmp(property, "UseDeliveryReports")) {
> +		dbus_bool_t value;
> +
> +		if (dbus_message_iter_get_arg_type(&var) != DBUS_TYPE_BOOLEAN)
> +			return __mms_error_invalid_args(dbus_msg);
> +
> +		dbus_message_iter_get_basic(&var,&value);
> +
> +		if (service->use_delivery_reports != (gboolean) value) {
> +			DBG("use_delivery_reports = %d", value);
> +
> +			service->use_delivery_reports = value;
> +		}
> +
> +		return g_dbus_create_reply(dbus_msg, DBUS_TYPE_INVALID);
> +	}
> +
> +	return __mms_error_invalid_args(dbus_msg);
> +}
> +
>   static gboolean mms_attachment_is_smil(const struct mms_attachment *part)
>   {
>   	if (g_str_has_prefix(part->content_type, "application/smil"))
> @@ -1023,6 +1065,10 @@ static const GDBusMethodTable service_methods[] = {
>   			GDBUS_ARGS({ "number", "s" }, { "count", "s" }),
>   			GDBUS_ARGS({ "messages_with_properties", "a(oa{sv}" }),
>   			get_conversation) },
> +	{ GDBUS_METHOD("SetProperty",
> +			GDBUS_ARGS({ "property", "s" }, { "value", "v" }),
> +		       NULL,
> +			set_property) },
>   	{ }
>   };
>

Regards,
-Denis