[#32949] - [4] Allow period in quoted identifier for db name
- Fixed in Code Base
- 1 Apr 2021
- Medium
- Build: 4.0-dev
- # 32949
- Diff
- PhilETaylor:allowperiodindbname
User tests: Successful: Unsuccessful:
Pull Request for Issue #32941
Summary of Changes
Allow period in database name for mysql - this is allowed by the documentation of Mysql 5.6 as periods are allowed in quoted identifiers and Joomla quotes the db name on use
This fixes a b\c break with Joomla 3 as per commentary in #32941
Testing Instructions
Attempt to install Joomla 4 with a database name with a period in it such as joomla.test4
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
You can install and use Joomla 4 with no issues with a database name with a period in it.
Documentation Changes Required
note: only changed mysql, other db platforms might need a review too.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Installation Language & Strings Libraries |
I've marked the test result for @progreccor .
I have tested this item
Tested. I was able to install J4 to DB with the name joomla.test4
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32949.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
?
?
|
||
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32949.
Thanks all for testing and getting this RTC so quickly. Fingers crossed it makes it into Joomla 4 now.
We're not accepting new features in.... just kidding - thanks merged!
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-04-01 12:52:52 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
|
||
Thanks
thanks!
I have tested and it is work as I expected.
Good!