[PATCH 0/3] mmsd: new tasks definition

[PATCH 0/3] mmsd: new tasks definition

[Ronald Tessier]

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

These patches concern mmsd and update TODO and documentation to describe new
functionalities to add to mmsd.

Ronald Tessier (3):
  TODO: Add new tasks
  doc: Describe delivered group in storage doc
  doc: Add new D-Bus methods to service interface

 TODO                |   67 +++++++++++++++++++++++++++++++++++++++++++++++++++
 doc/service-api.txt |   20 +++++++++++++++
 doc/storage.txt     |   30 ++++++++++++++++++++++
 3 files changed, 117 insertions(+), 0 deletions(-)

--
1.7.4.1

[PATCH 1/3] TODO: Add new tasks

[Ronald Tessier]

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

---
 TODO |   67 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 67 insertions(+), 0 deletions(-)

diff --git a/TODO b/TODO
index a4330b3..6e54f6f 100644
--- a/TODO
+++ b/TODO
@@ -16,6 +16,73 @@ Background
   once the task is better understood.


+MMS Delivery Report
+===================
+
+- Add delivery report request when sending message.
+  Modify SendMessage D-Bus API to add delivery report request flag.
+  Update message structure to manage this flag.
+  Modify MMS M-Send.Req PDU encoder to encode delivery report request.
+  Modify test script related to message sending to support new delivery flag.
+
+  Priority: Medium
+  Complexity: C2
+  Owner: Ronald Tessier <ronald.tessier@linux.intel.com>
+
+- Update message status stored in its associated Meta file.
+  Add a group [delivery_status] in the message status to handle delivery
+  reports, this group contains an entry for each message recipients.
+  Each recipient entry will be updated when the corresponding report will be
+  received and then remove the temporary stored delivery report.
+  Upon M-Send.Conf reception, store the received Message_ID to match delivery
+  reports (needed only if delivery report was requested).
+
+  Priority: Medium
+  Complexity: C2
+  Depends:
+  Owner: Sébastien Bianti <sebastien.bianti@linux.intel.com>
+
+- Add MMS M-Delivery.ind PDU decoding support.
+  Modify mms_service_push_notify() to handle MMS Delivery Report WAP Push
+  notification and temporary store the M-Delivery.ind PDU.
+  Decode M-Delivery.ind PDU.
+  Add MMS M-Delivery.ind PDU sample PDU to unit test.
+
+  Priority: Medium
+  Complexity: C2
+  Depends:
+  Owner: Ronald Tessier <ronald.tessier@linux.intel.com>
+
+- Update message Meta file.
+  Upon MMS M-Delivery.ind PDU reception, update delivery_status recipient entry
+  with the received status (this is described in doc/storage.txt).
+
+  Priority: Medium
+  Complexity: C2
+  Depends:
+  Owner: Sébastien Bianti <sebastien.bianti@linux.intel.com>
+
+
+D-Bus API
+=========
+
+- Add the "org.ofono.mms.Service" interface "DeleteMessages" method
+  implementation.
+  The D-Bus Message API documentation is available under /doc/service-api.txt.
+
+  Priority: Medium
+  Complexity: C2
+  Owner: Sébastien Bianti <sebastien.bianti@linux.intel.com>
+
+- Add the "org.ofono.mms.Service" interface "DeleteConversation" method
+  implementation.
+  The D-Bus Message API documentation is available under /doc/service-api.txt.
+
+  Priority: Medium
+  Complexity: C2
+  Owner: Ronald Tessier <ronald.tessier@linux.intel.com>
+
+
 Miscellaneous
 =============

--
1.7.4.1

[PATCH 2/3] doc: Describe delivered group in storage doc

[Ronald Tessier]

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

---
 doc/storage.txt |   30 ++++++++++++++++++++++++++++++
 1 files changed, 30 insertions(+), 0 deletions(-)

diff --git a/doc/storage.txt b/doc/storage.txt
index 8e76382..0aa6d9d 100644
--- a/doc/storage.txt
+++ b/doc/storage.txt
@@ -18,6 +18,7 @@ Meta file Example
 [info]
 read=false
 state=notification
+message_id=0123456789ABCDEF


 Meta file Keys/Values details
@@ -31,3 +32,32 @@ state: The message local state, possible values can be:
     - "received": m-Retrieve.Conf PDU downloaded and successfully acknowledged.
     - "draft": m-Send.Req PDU ready for sending.
     - "sent": m-Send.Req PDU successfully sent.
+
+message_id: this is the value provided in the M-Send.conf PDU (assigned by MMSC
+in response to a M-Send.req message), this entry will only be created upon
+M-Send.conf message reception if the delivery report was requested.
+
+For sent messages, a group [delivery_status] could take place in addition to
+[info] if delivery report is requested.
+In this group, every recipient has a MMS Delivery status value which can be one
+of the following:
+    - "none": no report has been received yet.
+    - "expired": recipient did not retrieve the MMS before expiration.
+    - "retrieved": MMS successfully retrieved by the recipient.
+    - "rejected": recipient rejected the MMS.
+    - "deferred": recipient decided to retrieve the MMS at a later time.
+    - "indeterminate": cannot determine if the MMS reached its destination.
+    - "forwarded": recipient forwarded the MMS without retrieving it first.
+    - "unreachable": recipient is not reachable.
+
+
+Example of a sent_message meta file with delivery report requested
+==================================================================
+
+[info]
+state=sent
+message_id=0123456789ABCDEF
+
+[delivery_status]
+21345=retrieved
+09876=none
--
1.7.4.1

[PATCH 3/3] doc: Add new D-Bus methods to service interface

[Ronald Tessier]

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

---
 doc/service-api.txt |   20 ++++++++++++++++++++
 1 files changed, 20 insertions(+), 0 deletions(-)

diff --git a/doc/service-api.txt b/doc/service-api.txt
index 36e1161..5c18cad 100644
--- a/doc/service-api.txt
+++ b/doc/service-api.txt
@@ -63,6 +63,26 @@ Methods		array{object,dict} GetMessages()
 				[service].Error.PermanentContentNotAccepted
 				[service].Error.PermanentLackOfPrepaid

+		[TODO] void DeleteMessages(array{object})
+
+			Delete an array of message objects that represents
+			the currently stored messages to delete.
+
+			Possible Errors: [service].Error.InvalidArguments
+
+		[TODO] void DeleteConversation(string number)
+
+			Delete the messages that belongs to the conversation
+			defined by the given number.
+
+			The number parameter contains digits to look for
+			(i.e.: n last digits of the phone number), only messages
+			with a recipient which ends with the given number will
+			be member of the conversation and thus will be deleted.
+
+			Possible Errors: [service].Error.InvalidArguments
+
+
 Signals		MessageAdded(object path, dict properties)

 			Signal that is sent when a new message is added. It
--
1.7.4.1

[PATCH 0/3] mmsd: new tasks definition

[Ronald Tessier]

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

These patches concern mmsd and update TODO and documentation to describe new
functionalities to add to mmsd.

Ronald Tessier (3):
  TODO: Add new tasks
  doc: Describe delivered group in storage doc
  doc: Add new D-Bus methods to service interface

 TODO                |   67 +++++++++++++++++++++++++++++++++++++++++++++++++++
 doc/service-api.txt |   20 +++++++++++++++
 doc/storage.txt     |   30 ++++++++++++++++++++++
 3 files changed, 117 insertions(+), 0 deletions(-)

--
1.7.4.1