From 9c18fe1cc7e22077763baa545144cc01d5d94eab Mon Sep 17 00:00:00 2001
From: Christian Weiske <cweiske@cweiske.de>
Date: Fri, 5 Aug 2011 06:47:37 +0200
Subject: Fix bug #3386178: "system:unfiled" secret tag does not work

---
 doc/ChangeLog                       | 1 +
 src/SemanticScuttle/Service/Tag.php | 1 +
 tests/Bookmark2TagTest.php          | 9 +++++++++
 3 files changed, 11 insertions(+)

diff --git a/doc/ChangeLog b/doc/ChangeLog
index b512d3c..d71a3d0 100644
--- a/doc/ChangeLog
+++ b/doc/ChangeLog
@@ -6,6 +6,7 @@ ChangeLog for SemantiScuttle
 0.98.2 - 2011-08-XX
 -------------------
 - Fix bug #3385724: Rename tag ends with XML Parsing Error
+- Fix bug #3386178: "system:unfiled" secret tag does not work
 
 
 0.98.1 - 2011-08-01
diff --git a/src/SemanticScuttle/Service/Tag.php b/src/SemanticScuttle/Service/Tag.php
index 8325285..2476608 100644
--- a/src/SemanticScuttle/Service/Tag.php
+++ b/src/SemanticScuttle/Service/Tag.php
@@ -143,6 +143,7 @@ class SemanticScuttle_Service_Tag extends SemanticScuttle_DbService
         if(!is_array($tags)) {
             $tags = utf8_strtolower(trim($tags));
         } else {
+            $tags = array_filter($tags);//remove empty values
             for($i=0; $i<count($tags); $i++) {
                 $tags[$i] = utf8_strtolower(trim($tags[$i]));
             }
diff --git a/tests/Bookmark2TagTest.php b/tests/Bookmark2TagTest.php
index 1367a0e..f466a7d 100644
--- a/tests/Bookmark2TagTest.php
+++ b/tests/Bookmark2TagTest.php
@@ -80,6 +80,15 @@ class Bookmark2TagTest extends TestBase
         );
     }
 
+    public function testAttachTagsWithArrayWithEmptyStringAddsSystemUnfiled()
+    {
+        $bid = $this->addBookmark(null, null, 0, array(''));
+        $this->assertEquals(
+            array('system:unfiled'),
+            $this->b2ts->getTagsForBookmark($bid, true)
+        );
+    }
+
 
 
     /**
-- 
cgit v1.2.3