Fixes for setTags() and empty tags

FIX: setTags() did not call loadTags(). Furtheron, neededSave() would
re-save even though no tags had changed, or not save when empty
setTags(array()) was called.

FIX: in for loop while saving in afterSave(), when empty tag was found,
continue instead of aborting.

CHG: remove empty tags before handling

ETaggableBehavior.php

@@ -143,6 +143,8 @@ private function getModelTableFkName() {
 	 * @return void
 	 */
 	public function setTags($tags) {
+		$this->loadTags();
+
 		$tags = $this->toTagsArray($tags);
 		$this->tags = array_unique($tags);
 
@@ -283,6 +285,7 @@ protected function toTagsArray($tags) {
 		}
 
 		array_walk($tags, array($this, 'trim'));
+		$tags = array_filter($tags, 'strlen');
 		return $tags;
 	}
 	/**
@@ -355,7 +358,7 @@ public function afterSave($event) {
 			// add new tag bindings and tags if there are any
 			if(!empty($this->tags)){
 				foreach($this->tags as $tag){
-					if(empty($tag)) return;
+					if(empty($tag)) continue;
 
 					// try to get existing tag
 					$findCriteria = new CDbCriteria(array(
@@ -699,4 +702,4 @@ protected function updateCount($count) {
 			)->execute();
 		}
 	}
-}
+}