[#25774] - [4.0] Installation - create a name
- Fixed in Code Base
- 4 Aug 2019
- Medium
- Build: 4.0-dev
- # 25774
- Diff
- brianteeman:sat1
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
For historical reasons that I bet no one can remember Joomla doesn't currently let you enter the real name of the Super User when installing Joomla. As a result every site on installation has a user with the name "Super User". This regularly causes issues as it gets confused with the usergroup "Super Users". That might be a translation issue or simply because there is just one character difference I don't know and it doesnt really matter.
As today there is no valid reason to force the name to be "Super User" this PR prompts for the real name as well as the username during installation.
To test.
Apply this PR using patchtester.
delete configuration.php
re-install joomla
(Please no bikeshedding about other issues in the installer)
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Installation Language & Strings |
oops I meant to do that - even if the entire required thing is foobar
| Labels |
Added:
?
?
|
||
It makes sense to me that this is failing the tests.
Do I submit the pr to fix that here https://github.com/joomla-projects/joomla-browser/blob/develop/src/JoomlaBrowser.php
v4.0.0 branch rather than develop but yes. relevent code is here https://github.com/joomla-projects/joomla-browser/blob/v4.0.0/src/JoomlaBrowser.php#L241-L247
See joomla-projects/joomla-browser#186 for the tests
| Category | Installation Language & Strings | ⇒ | Installation Language & Strings Unit Tests |
merged. can you update the composer dependency here please?
composer install
composer update joomla-projects/joomla-browser
| Labels |
Added:
?
|
||
| Category | Installation Language & Strings Unit Tests | ⇒ | External Library Composer Change Installation Language & Strings Unit Tests |
Updated as requested
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25774.
Tested new install with MySQL 5.7 and PostgreSQL 10.9.
Thanks for this PR. It was time that it was done. One step less to do after a new installation.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25774.
Looks like you made a slight mistake in the system tests jform_admin_name but in the xml there your is is jform_admin_user you need to either update the tests or the xml so they match
| Labels |
Added:
?
|
||
I assume I do so I did
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25774.
You did. Looks happy now!
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-08-04 20:52:26 |
| Closed_By | ⇒ | wilsonge |
Thanks!
Great - thanks
Make the real name required?