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

Property conditions broken when changing the tag alias #1411

Open
4 tasks done
dmolineus opened this issue Jun 24, 2020 · 2 comments
Open
4 tasks done

Property conditions broken when changing the tag alias #1411

dmolineus opened this issue Jun 24, 2020 · 2 comments
Assignees
Labels
enhancement This issue is about an enhancement (aka new feature)
Milestone

Comments

@dmolineus
Copy link
Contributor

dmolineus commented Jun 24, 2020

Checklist before I submit this issue report

I confirm that:

My environment is:

(Please fill in the actual values from your environment - check Contao Manager or use composer show)

Key Value Comments
PHP version: 7.2.10
Contao version: 4.9.3
MetaModels version: 2.2@dev
Installed MetaModels packages: core,attribute_tags,... Some more but not relevant
DCG version: 2.2@dev

Issue description

(Describe the problem you are having)

Steps to reproduce

  1. I have a tags attribute which uses the alias as alias field
  2. The tags field is also referenced as condition for tl_metamodel_dcasetting_condition of type conditionpropertycontainanyof
  3. I change the tag alias to id the property condition

Describe the behaviour of the application

Because of duplicate aliases I want to migrate to the id as alias field. If I change the tag alias to id the property conditions are broken because the serialized information of the property condition of type conditionpropertycontainanyof are not adjusted

Describe the expected behaviour of the application

  1. I have a tags attribute which uses the alias as alias field
  2. And I use the tags field as referenced field for property conditions of type conditionpropertycontainanyof.
  3. I expect that the property conditions still work / get autoadjusted when changing the property condition tag alias.
@zonky2 zonky2 added the bug A bug! A bug! Fast, squish it! label Jun 24, 2020
@zonky2 zonky2 added this to the 2.2.0 milestone Jun 24, 2020
@zonky2 zonky2 added the Up for discussion This ticket will be up for discussion in one of our next calls label Dec 1, 2020
@zonky2 zonky2 modified the milestones: 2.2.0, Future Dec 1, 2020
@zonky2 zonky2 added enhancement This issue is about an enhancement (aka new feature) and removed Up for discussion This ticket will be up for discussion in one of our next calls bug A bug! A bug! Fast, squish it! labels Dec 1, 2020
@zonky2
Copy link
Contributor

zonky2 commented Dec 1, 2020

Have we discussed in MM call ... Aliases should be unique anyway :D

@zonky2
Copy link
Contributor

zonky2 commented May 25, 2021

see #904

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This issue is about an enhancement (aka new feature)
Projects
None yet
Development

No branches or pull requests

3 participants