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

Member attribute order in sets not honored in profile #6155

Closed
MrKarlDilkington opened this issue Nov 21, 2017 · 2 comments
Closed

Member attribute order in sets not honored in profile #6155

MrKarlDilkington opened this issue Nov 21, 2017 · 2 comments
Labels
Status:Available Reviewed issue, it’s real, we’d review a pull request. Type:Enhancement A need for something new.

Comments

@MrKarlDilkington
Copy link
Contributor

https://www.concrete5.org/community/forums/customizing_c5/member-attribute-order-not-kept/

Steps to reproduce:

  1. create 3 attributes in the following order (each with Editable in Profile, Displayed in Public Profile, and Displayed on Member List checked)
  • 3 (test_3)
  • 2 (test_2)
  • 1 (test_1)

image

  1. create an Attribute Set (Test)

image

  1. add the 3 attributes to the Test attribute set

image

  1. change the order of the attributes from 3, 2, 1 to 1, 2, 3

image

  1. edit the profile through a public profile or My Account > Edit Profile

image

The attribute order will be 3, 2, 1.

@aembler aembler added Status:Available Reviewed issue, it’s real, we’d review a pull request. priority:like to have Type:Enhancement A need for something new. labels Nov 21, 2017
@qodo
Copy link

qodo commented Apr 24, 2018

Just found that this is also an issue for the registration page. We have a series of custom attributes within custom sets and the sort order for this is not honoured.

Having looked through the Doctrine set-up for this I can see that there is no relationship between the AttributeSetKeys and the UserAttributeKeys so I am not able to change the sort order for this. Not being that familiar with Doctrine I'm not sure what needs editing here so that we can use the asDisplayOrder Column.

Thanks,

@tylerryan
Copy link
Contributor

I can confirm this is no longer an issue as of 8.5.1 and likely before.

@aembler aembler closed this as completed Jul 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status:Available Reviewed issue, it’s real, we’d review a pull request. Type:Enhancement A need for something new.
Projects
None yet
Development

No branches or pull requests

4 participants