[#35163] - [3.10] Add missing mssql updates
- Fixed in Code Base
- 22 Aug 2021
- Medium
- Build: 3.10-dev
- # 35163
- Diff
- zero-24:styles_mssql
User tests: Successful: Unsuccessful:
Summary of Changes
Add missing mssql updates for #__template_styles
Testing Instructions
code review or testing mssql
@richard67 nothing urgent but please take a look that we didnt broken the mssql syntax here.
Actual result BEFORE applying this Pull Request
The structure is not 1:1 with mysql and postgressql
Expected result AFTER applying this Pull Request
the structure is 1:1 while it does not have an impact as the colums are not used nor are you allowed to upgrade from 3.10 to 4 using mssql
Documentation Changes Required
none.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | MS SQL SQL Administration com_admin Installation |
| Labels |
Added:
?
|
||
I have tested this item
Code review.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35163.
The reason why this PR is needed is that people using MS SQL Server might have to migrate their database to a supported one when being on 3.10, if they haven't done this before, and then the columns might be missing or the migration might fail.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35163.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35163.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-08-22 10:47:42 |
| Closed_By | ⇒ | zero-24 | |
| Labels |
Added:
?
|
||
Great thanks guys :)
@zero-24 I've allowed myself to commit my suggested change.