Yes all of this is already stated above. This commit is “the easy way”, mostly because categories loaded come from category_list_serializer
which doesn’t include allowed_tags
and allowed_tag_groups
.
I don’t disagree with what you say here as I asked about it sooner. What I disagree, is the frequency on which it would be an issue compared to changes involved.
-> tags are enabled
-> you need to use tags on this topic
-> change your category after setting tags
-> to a category with not tags restriction
-> and I would even say, have more than one tag, otherwise this is really not that much annoying
So I would want to first try to live with this and see if it’s an actual issue. If it’s an actual issue we can add this to category_list_serializer
and check on allowed_tags before deciding to remove tags or not.