Feeling Lucky
?
[#26724] - [4.0] Fix user creation error no group
- Fixed in Code Base
- 19 Oct 2019
- Medium
- Build: 4.0-dev
- # 26724
- Diff
- richard67:4.0-dev-com-users-group-model-sql
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Pull Request for Issue "popstgresql and mysql 8 system test fail since #25090 has been merged" .
Summary of Changes
See the diff: With the changes of PR #25090 , the group by was missing in the 2nd changed statement. The join syntax of the 1st statement I change to the one used in the 2nd statement for consistenty and because that's the new J4 way to do it.
Testing Instructions
Code review, or install a clean 4.0-dev on Postgresql and then try to create a user.
Expected result
All ok.
Actual result
Documentation Changes Required
None.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_users |
| Labels |
Added:
?
|
||
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-10-19 17:42:07 |
| Closed_By | ⇒ | wilsonge |
richard67
- comment
- 19 Oct 2019
Have tested with real test and it worked, too.
System tests passing for mysql8 and postgres which they weren't before. Good enough for me :)
Thankyou!