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

[Bug]: A portal cannot have a List ProfileProperty identical to another portal #6313

Open
2 tasks done
skamphuis opened this issue Jan 16, 2025 · 0 comments · May be fixed by #6314
Open
2 tasks done

[Bug]: A portal cannot have a List ProfileProperty identical to another portal #6313

skamphuis opened this issue Jan 16, 2025 · 0 comments · May be fixed by #6314

Comments

@skamphuis
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

What happened?

In an installation with multiple portals, different portals can't have a profile property with the same name and values. If you enter a list value, you will get a popup saying it has been added, but it's not:

Image

This is caused by Unique constraints on the Lsts table, not containing the PortalId column.

Steps to reproduce?

  1. Create a profile prop of type list (FavoriteColor)
  2. give it a few values (Red: 1, Green: 2, Blue: 3)
  3. create a new portal in the same installation
  4. Create a profile prop of type list (FavoriteColor) in the new portal
  5. Try to give it a value you already used in the other portal (Red: 1)

Current Behavior

If you enter a list value, you will get a popup saying it has been added, but it's not:

Image

Expected Behavior

The value should be added.

Relevant log output

none

Anything else?

I'm working on a fix.

Affected Versions

9.13.7 (alpha)

What browsers are you seeing the problem on?

Firefox, Chrome, Safari, Microsoft Edge

Code of Conduct

  • I agree to follow this project's Code of Conduct
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant