The method attachTags was raising a Notice error, fix it and adds a test for this...

diff --git a/src/SemanticScuttle/Service/Bookmark2Tag.php b/src/SemanticScuttle/Service/Bookmark2Tag.php
index 6e8e28c7c1bbb1c1d79317e47c2dbebd09c660d4..ed5e3285da6e9e31b4514c5cbe11e139f662464e 100644 (file)
--- a/src/SemanticScuttle/Service/Bookmark2Tag.php
+++ b/src/SemanticScuttle/Service/Bookmark2Tag.php
@@ -97,12 +97,13 @@ class SemanticScuttle_Service_Bookmark2Tag extends SemanticScuttle_DbService
 
 
         $tags_count = is_array($tags)?count($tags):0;
-
-        foreach($tags as $i => $tag) {
-            $tags[$i] = trim(utf8_strtolower($tags[$i]));
-            if ($fromApi) {
-                include_once 'SemanticScuttle/functions.php';
-                $tags[$i] = convertTag($tags[$i], 'in');
+        if (is_array($tags)) {
+            foreach($tags as $i => $tag) {
+                $tags[$i] = trim(utf8_strtolower($tags[$i]));
+                if ($fromApi) {
+                    include_once 'SemanticScuttle/functions.php';
+                    $tags[$i] = convertTag($tags[$i], 'in');
+                }
             }
         }
 

diff --git a/tests/Bookmark2TagTest.php b/tests/Bookmark2TagTest.php
index b0eccd195c8fd6afdf269d881237e222e0b4ece1..fdfc42ec9570a8d96f3b3f9c8ada6abe36f71c71 100644 (file)
--- a/tests/Bookmark2TagTest.php
+++ b/tests/Bookmark2TagTest.php
@@ -88,6 +88,21 @@ class Bookmark2TagTest extends TestBase
             $this->b2ts->getTagsForBookmark($bid, true)
         );
     }
+    
+    public function testAttachTagsWithEmptyStringAddsSystemUnfiled()
+    {
+        $originalDisplayErros = ini_get('display_errors');
+        $originalErrorReporting = ini_get('error_reporting');
+        ini_set('display_errors', 1);
+        error_reporting(E_ALL);
+        $bid = $this->addBookmark(null, null, 0, '');
+        $this->assertEquals(
+            array('system:unfiled'),
+            $this->b2ts->getTagsForBookmark($bid, true)
+        );
+        ini_set('display_errors', $originalDisplayErros);
+        error_reporting($originalErrorReporting);
+    }
 
     public function testAttachTagsWithSomeEmptyTags()
     {