Feeling Lucky
[#127] - Use InnoDB and utf8mb4_unicode_ci in the patchtester too
- Closed
- 2 Mar 2016
- Medium
- Build: master
- # 127
- Diff
- andrepereiradasilva:patch-1
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Summary of Changes
Create patchtester tables with InnoDB and utf8mb4_unicode_ci.
Testing Instructions
- Install patchtester beta 4 with this patch integrated on a joomla latest staging (clean install).
com_patchtester-beta-4-ut8mb4.zip - Check if everything is working fine.
Observations
To not cause issues added a drop to existing tables first.
Not an expert in databases, so please check if anything more is needed.
andrepereiradasilva
- comment
- 2 Mar 2016
done and done
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-03-02 13:48:14 |
| Closed_By | ⇒ | mbabker |
Two things:
The
DROP TABLEquery isn't needed. Updates between the beta tagged releases aren't fully supported right now, similar to the core CMS (that'll change after a stable tag, it's mostly because I didn't do SQL deltas as needed for other database changes).The install script should now check for 3.5.0 as the minimum instead of 3.3.0 if we're going to add the utf8mb4 stuff, that's checked here.