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

Fix #4049 Nested tags not showing under some circumstances #25218 #25254

Closed
wants to merge 1 commit into from

Conversation

gellweiler
Copy link
Contributor

@gellweiler gellweiler commented Jan 1, 2023

I accidentally closed my old PR: #25218. So here is a new PR with a test class.

This commit fixes #4049.

Bug description: When nested tags were not created in the same order as they are nested, they are not shown in the add tag to contact dialog.

See: https://lab.civicrm.org/dev/core/-/issues/4049

@civibot
Copy link

civibot bot commented Jan 1, 2023

(Standard links)

@civibot civibot bot added the master label Jan 1, 2023
@gellweiler gellweiler changed the title Fix #4049 Fix #4049 Nested tags not showing under some circumstances #25218 Jan 1, 2023
This commit fixes civicrm#4049: Nested tags are not showing under some circumstances.
@eileenmcnaughton
Copy link
Contributor

@gellweiler thank you for the test & the clear instructions - I was able to replicate the bug & see that this fixes it.

I realised working through the issue that this is a regression - and hence our practice is to target the rc with the patch .

Since the rc is being cut tomorrow to the new 5.57.0 release I figured I should create a new PR porting this rather than ask you to rebase against the 5.57 branch - #25272

Can you please do another PR against 5.57 branch adding yourself to the contributor-key.yml file https://github.com/civicrm/civicrm-core/blob/master/contributor-key.yml so we can include you in the release notes

@gellweiler
Copy link
Contributor Author

@eileenmcnaughton will do

@gellweiler
Copy link
Contributor Author

@eileenmcnaughton #25273

@eileenmcnaughton
Copy link
Contributor

@gellweiler great!

@eileenmcnaughton
Copy link
Contributor

@gellweiler I'm closing this now as I've up-merged from 5.57 here #25274

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

Successfully merging this pull request may close these issues.

2 participants