aboutsummaryrefslogtreecommitdiffstats
path: root/apps/settings/lib/Controller/UsersController.php
Commit message (Collapse)AuthorAgeFilesLines
* fix: correctly count disabled users for subadminsbackport/48437/stable30Maksim Sukharev2024-09-301-4/+4
| | | | Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
* refactor(settings): Replace security annotations with respective attributesprovokateurin2024-07-271-10/+11
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* feat(users): Enable features for delegated user adminsLouis Chemineau2024-07-221-1/+1
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* feat(users): Add support for admin delegation for users and groups managementLouis Chemineau2024-07-221-2/+8
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* chore(settings): Rename user groupsChristopher Ng2024-07-111-1/+1
| | | | | | | - "Active accounts" -> "All accounts" - "Recent accounts" -> "Recently active" Signed-off-by: Christopher Ng <chrng8@gmail.com>
* chore(l10n): Translate group namesChristopher Ng2024-07-111-2/+2
| | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* feat: Add Recent accounts sectionChristopher Ng2024-07-111-1/+7
| | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* chore: Add SPDX headerAndy Scherzinger2024-06-031-29/+3
| | | | Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
* feat(ldap): sync additional properties to profile and SABJake Nabasny2024-05-301-1/+8
| | | | | | | | | | | | | | | Synced from LDAP to profile: - Date of birth Synced from LDAP to SAB (via the profile): - Biography - Date of birth Original code by Jake Nabasny (GitHub: @slapcat) Co-authored-by: Jake Nabasny <jake@nabasny.com> Co-authored-by: Richard Steinmetz <richard@steinmetz.cloud> Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
* fix: Apply new coding standard to all filesCôme Chilliet2024-04-021-2/+2
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* feat(settings): Allow to sort groups in the account management alphabeticallyFerdinand Thiessen2024-03-271-3/+9
| | | | | | | We can do this purly in the frontend - but when enforced from the backend using the existing system config, we need to follow the requirement. We then show a warning about the configuration. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge branch 'master' into fix/pages-titles-appsAndrii Rublov2024-02-251-70/+27
|\ | | | | Signed-off-by: Andrii Rublov <github.a@rublov.me>
| * Merge branch 'master' into fix/42480/user-admin-not-adminJohn Molakvoæ2024-02-241-63/+23
| |\ | | | | | | Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| | * fix(settings): Use initial state instead of custom state for server dataFerdinand Thiessen2024-02-231-67/+24
| | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
| * | fix(admin role): fix old and wrong way to determine whether user is adminArthur Schiwon2024-02-191-7/+4
| |/ | | | | | | | | | | | | | | | | - fixes Settings knowing who is an admin of non-local group backend groups - obsoletes and removes a little old, deprecated code - double checks proper parameter type on Group\Manager::isAdmin - also fixes legacy OC_User code to check whether user is an admin Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* / [Bugfix]: Titles of pages (dashboard, apps-settings, users-settings) #43327Andrii Rublov2024-02-231-1/+1
|/ | | | Signed-off-by: Andrii Rublov <airublev@outlook.com>
* feat: rename users to account or personVincent Petry2024-02-131-2/+2
| | | | | | Replace translated text in most locations Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* refactor: Migrate IgnoreOpenAPI attributes to OpenAPIprovokateurin2024-01-181-2/+2
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* chore: apply changes from Nextcloud coding standards 1.1.1Joas Schilling2023-11-231-14/+14
| | | | | Signed-off-by: Joas Schilling <coding@schilljs.com> Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* Merge pull request #39175 from ↵Christoph Wurst2023-09-271-1/+1
|\ | | | | | | | | shdehnavi/settings_app_users_controller_square_bracket_syntax use square bracket syntax to modify array
| * use square bracket syntax to modify arrayHamid Dehnavi2023-07-051-1/+1
| | | | | | | | Signed-off-by: Hamid Dehnavi <hamid.dev.pro@gmail.com>
* | settings: Add OpenAPI specjld31032023-07-101-0/+3
|/ | | | Signed-off-by: jld3103 <jld3103yt@gmail.com>
* Create headings for settings pagesjulia.kirschenheuter2023-03-071-1/+1
| | | | Signed-off-by: julia.kirschenheuter <julia.kirschenheuter@nextcloud.com>
* Fix case sensitivity of email when saving settingsCarl Schwan2022-12-161-1/+1
| | | | | | | | | Otherwise we detect a email change all the time and since email are immutable in ldap this prevent updating other fields. Related: https://github.com/nextcloud/server/pull/33813 Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Rename mastodon to fediverseCarl Schwan2022-11-221-7/+9
| | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Add mastodon personal info fieldCarl Schwan2022-11-211-2/+7
| | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* use specific email getter where necessaryArthur Schiwon2021-09-091-2/+2
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* refs #21045 add app config to disable unlimited quota and to set max quotaJulien Veyssier2021-07-291-1/+7
| | | | | | | | | avoid unlimited quota as default_quota fallback value if unlimited quota is not allowed avoid getting/setting/displaying unlimited default quota if not allowed implement tests for unlimited quota restrictions Signed-off-by: Julien Veyssier <eneiluj@posteo.net> Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
* Update php licensesJohn Molakvoæ (skjnldsv)2021-06-041-3/+3
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* do not use private AccountManager in UsersControllerArthur Schiwon2021-05-251-98/+68
| | | | | | | | - extends IAccountProperty for verificationData getters and setters - implementation thereof ^ - and of course adaption of UsersController Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Merge pull request #26259 from ↵Joas Schilling2021-04-261-0/+3
|\ | | | | | | | | nextcloud/feature/noid/validate-website-to-be-valid Validate the website field input to be a valid URL
| * Validate the website field input to be a valid URLJoas Schilling2021-04-221-0/+3
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Fix deleting properties of user settings when not given explicitlyDaniel Calviño Sánchez2021-04-231-13/+39
| | | | | | | | | | | | | | | | | | | | The controller can receive an optional subset of the properties of the user settings; values not given are set to "null" by default. However, those null values overwrote the previously existing values, so in practice any value not given was deleted from the user settings. Now only non null values overwrite the previous values. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Fix TypeError when "email" is not given in the controller requestDaniel Calviño Sánchez2021-04-231-1/+1
| | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Respect additional user settings not covered by the controllerDaniel Calviño Sánchez2021-04-231-7/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "AccountManager::updateUser()" wipes previous user data with whichever user data is given (except for some adjustments, like resetting the verified status when needed). As the controller overrode the properties those properties would lose some of their attributes even if they are not affected by the changes made by the controller. Now the controller only modifies the attributes set ("value" and "scope") to prevent that. Note that with this change the controller no longer removes the "verified" status, but this is not a problem because, as mentioned, "AccountManager::updateUser()" resets them when needed (for example, when the value of the website property changes). This change is a previous step to fix overwritting properties with null values, and it will prevent the controller from making unexpected changes if more attributes are added in the future. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Use constants from interface rather than classDaniel Calviño Sánchez2021-04-231-2/+2
|/ | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Make extra user profile fields always editableVincent Petry2021-03-261-10/+5
| | | | | | | | The fields for phone number, address, website and twitter are now editable regardless whether federated sharing and the lookup server are enabled or not. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Rename some parameters and methods to make the API more clearJoas Schilling2021-03-101-1/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Also clear the knownUser when changing via the settings endpointJoas Schilling2021-03-101-1/+23
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Bump nextcloud/coding-standard from 0.3.0 to 0.5.0dependabot-preview[bot]2021-02-181-1/+1
| | | | | | | | | | Bumps [nextcloud/coding-standard](https://github.com/nextcloud/coding-standard) from 0.3.0 to 0.5.0. - [Release notes](https://github.com/nextcloud/coding-standard/releases) - [Changelog](https://github.com/nextcloud/coding-standard/blob/master/CHANGELOG.md) - [Commits](https://github.com/nextcloud/coding-standard/compare/v0.3.0...v0.5.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Fix updating user settingsChristoph Wurst2021-01-051-26/+27
| | | | | | | | | | | The requests dispatched from the personal settings page don't always send all the possible params but just a subset. Hence requests currently fail on master e.g. when you update your email. This makes the arguments optional again. Regression of da9462b482c77b71d6623be43a6aa1eda3d7cc06 Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Update all license headers for Nextcloud 21Christoph Wurst2020-12-161-0/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Make the throwing optional, so background tasks don't breakJoas Schilling2020-12-071-1/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Also translate the phone number when coming in via the accounts manager API ↵Joas Schilling2020-12-071-22/+7
| | | | | | directly Signed-off-by: Joas Schilling <coding@schilljs.com>
* Validate and standardize the phone number on savingJoas Schilling2020-12-071-2/+34
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Also return the phone number and twitter handle on the APIJoas Schilling2020-12-071-0/+4
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Make code strictJoas Schilling2020-12-071-48/+50
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Simplify the check if admin can change password based on encryption statusMorris Jobke2020-11-051-4/+2
| | | | | | | | | Found by Psalm: ``` /home/runner/work/server/server/apps/settings/lib/Controller/UsersController.php:324:8:error - RedundantCondition: Type true for $isEncryptionModuleLoaded is never falsy ``` Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* save email as lower caseRobin Appelman2020-10-271-1/+2
| | | | | | email addresses are case insensitive Signed-off-by: Robin Appelman <robin@icewind.nl>
* Format code to a single space around binary operatorsChristoph Wurst2020-10-051-2/+2
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>