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

disable allow_user_to_change_display_name, admin can't change name in user panel #642

Closed
focparty opened this issue Jul 29, 2016 · 20 comments · Fixed by #8824
Closed

disable allow_user_to_change_display_name, admin can't change name in user panel #642

focparty opened this issue Jul 29, 2016 · 20 comments · Fixed by #8824
Assignees
Milestone

Comments

@focparty
Copy link

Steps to reproduce

  1. allow_user_to_change_display_name=false in config.php
  2. user panel: changing name, doesnt save

Expected behaviour

admin should be able to change user name

Actual behaviour

see point 2.

Server configuration

Operating system: OVH shared hosting

Web server: OVH shared hosting

Database: mysql

PHP version: 5.6

Nextcloud version: (see Nextcloud admin page) 9.0.53

Updated from an older Nextcloud/ownCloud or fresh install: new install

Where did you install Nextcloud from: nextcloud

@nickvergessen
Copy link
Member

Well that is the way the config is designed at the moment, however I can see cases where this would not be the intended behaviour.

@blizzz
Copy link
Member

blizzz commented Dec 7, 2016

allow_user_to_change_display_name=false in config.php

admin should be able to change user name

@focparty Display Name and Username are two different things. Could you elaborate?

@GitHubUser4234
Copy link
Contributor

As #1302 has been closed as duplicate of this issue, I'd like to throw in my support for enabling email modification independently from OC_User::canUserChangeDisplayName. There are in fact use cases where changing the display name should be disabled while changing email should be enabled.

After some discussion with @blizzz , it might be necessary to consider that user_ldap overwrites the email value in oc_preferences in case emails are present in LDAP. So for example the user_ldap backend could provide the email set-ability depending on whether an LDAP email attribute is set or not.

@MorrisJobke It would be great to know what you have in mind about how to implement this :)

@githubos1
Copy link

I suggest:
'allow_user_to_change_display_name' => false,

should only prevent USERS from changing their DisplayName Settings and not automatically also their e-mail settings, so maybe an additional setting could be implemented like:
'allow_user_to_change_email_address' => false, ???

MUCH MORE IMPORTANT: this very important feature (when set to "false" should not impact ADMINS from changing or setting UserScreenName and UserEmail as well as it functions right now (I'm using 12.0.0. beta3 !!)

Regards,

Oliver

@OutbackMatt
Copy link

11.0.3 seems to NOT be working from ./index.php/settings/users
I can't change any details on this screen...

@MorrisJobke
Copy link
Member

ettings/users
I can't change any details on this screen...

Have you set allow_user_to_change_display_name to false? If so: remove this line from the config and it should work again. That's currently a limitation we have.

@OutbackMatt
Copy link

The config that is built automatically doesn't include that line at all.
Specifically when I add that line = true then there is no change.

I can't make changes to user full name or add an email address, all I can do is allocate the user to groups and set them as admin for those groups.

Ideally, I'd like a screen similar to my own ./index.php/settings/personal page that would all of these settings to be changed for each user by the server admin

@MorrisJobke
Copy link
Member

I can't make changes to user full name or add an email address, all I can do is allocate the user to groups and set them as admin for those groups.

Yes. This is sadly how it is implemented currently. 🙈 Somebody™ needs to fix it, but as this is a rarely used feature currently other issues are higher priotized. We are happy about contributions that fix this bug.

@radiantwave
Copy link

I really need this feature to control if the users can change the displayname or e-mail. And this independently.

In my usecase I need to disallow the displaynamechange for the users. The E-Mail should be still changeable.

At the moment I use a workaround that removes the form from the template. But this is only a visual removal!

@radiantwave
Copy link

This is not a Milestone anymore?

@sduensin
Copy link

sduensin commented Oct 8, 2017

Still the behavior in 12.0.3. :-(

@t0mcat1337
Copy link

Any news about that? Me too stumbled accross this issue... took me hours to figure out, why I suddenly couldn't change / set users mail addresses in ADMIN backend (technically: the request http://<nextcloud.home>/index.php/settings/users//mailAddress always failed with a HTTP 403)
By chance I removed the previously added "allow_user_to_change_display_name=false" line from config and changing mail addresses in ADMIN works again as it should.
Can't really understand why setting "allow_user_to_change_display_name" to "false" has this impact...
The variable name indicates USER preventing changing their DISPLAY_NAME, and NOT that also ADMINS may not change users MAIL ADDRESS ... IMHO this config var is wrong implemented or wrong named...

@KB7777
Copy link
Contributor

KB7777 commented Feb 26, 2018

Hello.

I need this very badly to deploy my third NC instance for my company. :(
Is there any news about it?

Regards.

@skjnldsv skjnldsv added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of labels Apr 9, 2018
@revolter
Copy link

revolter commented May 8, 2018

Is allow_user_to_change_display_name able to be changed from the Admin page instead of the config/config.php file?

@radiantwave
Copy link

Would be much better.
allow_user_to_change_display_name should be a setting in the UI.

@nickvergessen
Copy link
Member

I don't think so, because it's normally a one-time setting.

@revolter
Copy link

revolter commented May 8, 2018

Regardless, it should be in the UI.

In my case, I am an admin and I can access the Admin panel, but I don't have access to the Nextcloud server files on our server. So I could only go to http://<our_ip>:8080/nextcloud/index.php/settings/admin in the browser and change it there.

@radiantwave
Copy link

radiantwave commented May 8, 2018

And I hope that the suggestions from @githubos1 has been seen..

'allow_user_to_change_email_address'
'allow_user_to_change_display_name'

really should be two separate settings..

I'm wishing this feature since I was using Owncloud..

@GoetheG
Copy link

GoetheG commented Apr 12, 2019

Hi there. @MorrisJobke: Please reopen this ticket.

@dan-nkl, @revolter: I totally agree with you. Same problem with me. I am using a Nextcloud hoster for my company and I dot not have access to the config file. In order to be able to block users from 1. changing their screen name and 2. changing their e-mail address I really need this function to be part of the user interface.

If the company behind Nextcloud really has the goal to be used by many people, then you need to add many more functions to the user interface. Just take a look at Rocket.Chat and the amount of settings it offers for administrators with no access to any config file.

@MorrisJobke
Copy link
Member

@GoetheG Please open a new issue for this. We then can evaluate again if this makes sense or not (also from a maintainability point of view)

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.