From: Pawel Andruszkiewicz <p.andruszkie@samsung.com>
Date: Mon, 26 Jan 2015 08:00:49 +0000 (+0100)
Subject: [Messaging] Fix attachment updates (JS).
X-Git-Tag: submit/tizen_tv/20150603.064601~1^2~566^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ba24a042b6d8b16768e0b76facdd386840115fe9;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[Messaging] Fix attachment updates (JS).

Change-Id: Ib02769a60abb4ca8d410de71cc8d6e0782acda14
Signed-off-by: Pawel Andruszkiewicz <p.andruszkie@samsung.com>
---

diff --git a/src/messaging/messaging_api.js b/src/messaging/messaging_api.js
index 8e7a6739..d0614826 100644
--- a/src/messaging/messaging_api.js
+++ b/src/messaging/messaging_api.js
@@ -124,7 +124,7 @@ function InternalValues_(data) {
 function updateInternal_(internal, data) {
     var values = new InternalValues_(data);
     for(var key in data) {
-        if (data.hasOwnProperty(key) && internal.hasOwnProperty(key)) {
+        if (values.hasOwnProperty(key) && internal.hasOwnProperty(key)) {
             internal[key] = values;
         }
     }
@@ -429,10 +429,18 @@ function Message(type, data) {
         {
             get: function () {return _internal.attachments;},
             set: function(value) {
-                if (value instanceof InternalValues_) value = value.attachments;
-                for (var k = 0; k < value.length; ++k) {
-                    if (!(value[k] instanceof tizen.MessageAttachment)) {
-                        return;
+                if (value instanceof InternalValues_) {
+                    value = value.attachments;
+                    for (var k = 0; k < value.length; ++k) {
+                        if (!(value[k] instanceof tizen.MessageAttachment)) {
+                            value[k] = new MessageAttachment_(value[k]);
+                        }
+                    }
+                } else {
+                    for (var k = 0; k < value.length; ++k) {
+                        if (!(value[k] instanceof tizen.MessageAttachment)) {
+                            return;
+                        }
                     }
                 }
                 _internal.attachments = value;