#38665 closed enhancement (fixed)
Add user language preference to 'Add new user' form
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.5 | Priority: | normal |
Severity: | normal | Version: | 4.7 |
Component: | Users | Keywords: | needs-testing has-patch |
Focuses: | administration | Cc: |
Description
Existing users can select an admin language. Perhaps allow preselecting a language on the user edit screen may be helpful.
Attachments (3)
Change History (23)
This ticket was mentioned in Slack in #core by johnbillion. View the logs.
8 years ago
#3
@
8 years ago
- Milestone changed from Awaiting Review to Future Release
- Type changed from feature request to enhancement
#4
@
8 years ago
- Keywords has-patch added; needs-patch removed
38665.diff adds the language dropdown to the 'Add User' form.
Setting $user->locale
is moved outside of the $update
conditional.
Also removes setting $user->locale = ''
. Unsure why that was there in the first place?
The language select is only shown if !is_multisite()
. Is that correct?
#5
@
8 years ago
- Focuses administration added
- Keywords needs-testing added
Need to consider that there's also the ability to add a user to a site from the Network Admin -> Sites -> Edit -> Users screen.
#7
@
5 years ago
- Keywords needs-testing removed
- Milestone changed from Future Release to 5.5
- Owner set to johnbillion
- Status changed from new to reviewing
#8
@
5 years ago
- Keywords needs-refresh removed
Updated the patch with changes which means the UI and behaviour of the dropdown now matches that of the one used when editing a user.
#15
@
5 years ago
- Keywords needs-patch added; has-patch removed
- Resolution fixed deleted
- Status changed from closed to reopened
This has broken the ability for the user to change their language from the user profile editing screen. $user->locale
always gets set to an empty string.
#18
@
5 years ago
Looking at 38665.3.diff, I'd like to suggest to add an inline comment where $user->locale
is set to empty string. That may help avoid the same mistake in the future, as it's not immediately clear why it's needed. See
Also removes setting $user->locale = . Unsure why that was there in the first place?
Makes sense. We did the same thing for sites in #33528.