Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changing taxonomy term id looses existing content values #127

Open
laulaz opened this issue Mar 15, 2022 · 1 comment
Open

Changing taxonomy term id looses existing content values #127

laulaz opened this issue Mar 15, 2022 · 1 comment

Comments

@laulaz
Copy link
Member

laulaz commented Mar 15, 2022

With the new ability to change terms identifiers (see #72 & #113), we can now change the identifier of a term, even if it is already used on a content (value selected on the behavior).
As there is no migration code in the pull request, the existing values are then "lost".

IMHO, we should :

  • migrate the existing values used (if their identifier changed)
  • prevent user to change terms identifiers that are already used
  • or at least warn the user in the control panel

@Petchesi-Iulian @ale-rt @alecghica @avoinea what do you think of this ?

@alecghica
Copy link
Member

@laulaz make perfect sense to me!

We can take care of this but it will take some time to be implemented as it's low prio for us at least.
Do you plan to implement it? If so we won't put it on our radar. Just let me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants