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

Extended User Fields Read Access / Write Access not working #1799

Closed
martyshep opened this issue Aug 10, 2016 · 5 comments
Closed

Extended User Fields Read Access / Write Access not working #1799

martyshep opened this issue Aug 10, 2016 · 5 comments
Labels
type: bug A problem that should not be happening
Milestone

Comments

@martyshep
Copy link
Contributor

Hi devs, I experience an issue where I have Extended User Fields assigned to capture members phone number and address. I want the member to be able to update these fields and view their own but not able to see others unless they are Admin users.

The way it seems to operate is a normal member user can view an Admin users telephone number and address rather than the other way around.

I have the Extended User Field set to Applicable-MEMBERS Read Access-Admin Write Access-Member:
image

@Moc Moc added type: bug A problem that should not be happening status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. labels Aug 17, 2016
@Moc Moc added this to the e107 v2.1.2 milestone Aug 23, 2016
@CaMer0n CaMer0n modified the milestones: e107 v2.1.2, e107 v2.1.3 Oct 27, 2016
@CaMer0n CaMer0n modified the milestones: e107 v2.1.3, e107 v2.1.4 Dec 23, 2016
@CaMer0n CaMer0n modified the milestones: e107 v2.1.4, e107 v2.1.5 Jan 17, 2017
@CaMer0n CaMer0n modified the milestones: e107 v2.1.5, e107 v2.1.6 Mar 21, 2017
@CaMer0n CaMer0n modified the milestones: e107 v2.1.6, e107 v2.1.7 Aug 10, 2017
@CaMer0n CaMer0n modified the milestones: e107 v2.1.7, e107 2.1.8 Dec 24, 2017
@Moc
Copy link
Member

Moc commented Jun 25, 2018

What happens when you change 'applicable to' to admin?

@martyshep
Copy link
Contributor Author

Hi @Moc when I change the 'applicable' to admin, regular members cannot access the field to fill it in, it is only presented to admin.

The same behaviour on my site is still present. The 'applicable' field dictates who would be presented the field to fill in, in their settings. Where it gets messy is that the Read field works opposite to how I would imagine it should. I use the field for members phone number. I do not want other regular members to view other regular members phone numbers, but 'admin' people should be able.

At present my 'admins' they have to be website Admins and go into Admin mode to list users, but they then see all the details. It would be useful if my admins could see the field in the normal operation of the website, as I don't want to give them 'proper' admin access.

So to get this working, I need to set the extended field to applicable=Members / read=no-one / write=members.

If I set read=userclass_read, I would expect only people with userclass_read to be able to see the phone field of the other members. However, what I get is that regular members can then view the phone of the admins with userclass_read, but not the other way around, which one may think would be the behaviour.

@CaMer0n CaMer0n modified the milestones: e107 2.1.8, e107 2.1.9 Jul 18, 2018
@CaMer0n CaMer0n removed this from the e107 2.1.9 milestone Aug 22, 2018
@CaMer0n CaMer0n added this to the e107 2.2.0 milestone Aug 22, 2018
@Moc Moc removed the status: testing required Someone needs to confirm this issue's existence and write a test to prevent the fix from regressing. label Jan 10, 2019
@Moc Moc closed this as completed in 23b8514 Jan 10, 2019
@Moc
Copy link
Member

Moc commented Jan 11, 2019

@martyshep Sorry it took so long, but this bug is now fixed.

@martyshep
Copy link
Contributor Author

martyshep commented Jan 11, 2019 via email

@Moc
Copy link
Member

Moc commented Jan 11, 2019

@martyshep You're welcome! 👍 Glad to hear.
Information on how to support e107 can be found here: http://e107.org/community (last section).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A problem that should not be happening
Projects
None yet
Development

No branches or pull requests

3 participants