[NFC] Fixed TODO sections.
authorTomasz Marciniak <t.marciniak@samsung.com>
Wed, 7 Jan 2015 14:32:13 +0000 (15:32 +0100)
committerTomasz Marciniak <t.marciniak@samsung.com>
Wed, 7 Jan 2015 14:43:09 +0000 (15:43 +0100)
[Verification] Code compiles without errors.

Change-Id: Ie3b6baef0d8ae9216f829216b76946c44872ef71
Signed-off-by: Tomasz Marciniak <t.marciniak@samsung.com>
src/nfc/nfc_adapter.cc
src/nfc/nfc_api.js

index 4257af79e6414d875d969047776d03342eabae8c..87478f26affb22daeeacb0714438cabfc58299ac 100644 (file)
@@ -951,10 +951,7 @@ void NFCAdapter::TagWriteNDEF(int tag_id, const picojson::value& args) {
     if(message){
         int result = nfc_tag_write_ndef(m_last_tag_handle, message, NULL, NULL);
 
-        // Message is no more needed - can be removed
-        // TODO: uncomment line below after merging commit
-        // that adds this function
-        //NFCMessageUtils::removeMessageHandle(message)
+        NFCMessageUtils::RemoveMessageHandle(message);
         if (NFC_ERROR_NONE != result){
 
             // for permission related error throw exception
index 599416ca7b97fd6bd50414e84ac2f79036621319..7503d5f2e811662e6a9ca6f13813d0e7a3183e2e 100644 (file)
@@ -273,13 +273,21 @@ NFCAdapter.prototype.setTagListener  = function() {
         },
         {
             name : 'tagType',
-            type : types_.STRING,
+            type : types_.ARRAY,
+            values : types_.STRING,
             optional : true,
             nullable : true
         }
     ]);
 
-    // TODO: NFCTag type value validation needed here
+    if(!types_.isNullOrUndefined(args.tagType)) {
+        for (var i = 0; i < args.tagType.length; i++) {
+            if (NFCTagType[args.tagType[i]] === undefined) {
+                throw new tizen.WebAPIException(
+                        tizen.WebAPIException.TYPE_MISMATCH_ERR, 'Invalid tag type.');
+            }
+        }
+    }
 
     // Listener object creation
     var listenerCallback = function(message) {
@@ -288,11 +296,10 @@ NFCAdapter.prototype.setTagListener  = function() {
         if('onattach' === message.action) {
             tagObject = new NFCTag(message.id);
 
-            if(!types_.isNullOrUndefined(args.tagType)) {
-                // If filter set for listener then check tag type
-                if(tagObject.type !== args.tagType) {
-                    return;
-                }
+            // If filter is set for listener but tag type is not searched one
+            if(!types_.isNullOrUndefined(args.tagType) &&
+                    args.tagType.indexOf(tagObject.type) < 0) {
+                return;
             }
         }
         args.listener[message.action](tagObject);