Skip to content
This repository has been archived by the owner on Sep 2, 2021. It is now read-only.

Removing all terms via Advanced Translator is ignored #154

Closed
kraftner opened this issue Aug 24, 2015 · 2 comments
Closed

Removing all terms via Advanced Translator is ignored #154

kraftner opened this issue Aug 24, 2015 · 2 comments
Assignees
@tfrommen
Labels
bug post translation
Milestone
2.2.0

Comments

@kraftner
Copy link
Contributor

If you remove all terms from a post using the Advanced Translator this change is ignored and everything just stays the way it was on the linked posts.

I think this is because here you check for an empty tax key and abort tax syncing if empty. But when you remove all terms this being empty is actually a change to process.
@tfrommen tfrommen added this to the 2.2.0 milestone Aug 24, 2015
@tfrommen tfrommen self-assigned this Aug 24, 2015
tfrommen added a commit that referenced this issue Aug 24, 2015
@tfrommen
Allow to remove all terms of a remote post. #154
203609d
@tfrommen
Copy link
Member

Once again, good catch! This seems to have been broken forever.

@tfrommen
Copy link
Member

Closed with 203609d.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@tfrommen tfrommen

Labels
bug post translation
Projects
None yet
Milestone
2.2.0
Development

No branches or pull requests
2 participants
@kraftner @tfrommen