TagBehavior的afterSave，没有考虑更新时完全删除tag的情况

[dakcom]

RT。图省事直接加了
if(isset($data[static::$formName]) && empty($data[static::$formName])) {
if(!$this->owner->isNewRecord) {
$this->beforeDelete();
}
}
如果规范，是否加个if嵌套？

[yidashi]

恩。

public function afterSave()
    {
        if (\Yii::$app->request->isConsoleRequest ) {
            return;
        }
        $data = \Yii::$app->request->post($this->owner->formName());
        if(isset($data[static::$formName])) {
            if(!$this->owner->isNewRecord) {
                $this->beforeDelete();
            }
            if (!empty($data[static::$formName])) {
                $tags = $data[static::$formName];
                foreach ($tags as $tag) {
                    $tagModel = Tag::findOne(['name' => $tag]);
                    if (empty($tagModel)) {
                        $tagModel = new Tag();
                        $tagModel->name = $tag;
                        $tagModel->save();
                    }
                    $articleTag = new ArticleTag();
                    $articleTag->article_id = $this->owner->id;
                    $articleTag->tag_id = $tagModel->id;
                    $articleTag->save();
                }
                Tag::updateAllCounters(['article' => 1], ['name' => $tags]);
            }
        }
    }