[#26799] - [4.0] [com_redirect] Fix error on saving automatically created redirect links with MySQL DB in strict mode
- Fixed in Code Base
- 25 Oct 2019
- Medium
- Build: 4.0-dev
- # 26799
- Diff
- richard67:patch-1
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #26782 .
Summary of Changes
Set also modified date for new redirect links because that database column does not have a default value anymore.
Testing Instructions
See issue #26782 .
Has to be tested on MySQL 8 or an earlier version where strict mode is switched on.
Expected result
No error, redirect link is created.
Actual result
Error "500 PLG_SYSTEM_REDIRECT_ERROR_UPDATING_DATABASE".
Documentation Changes Required
None.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
I have tested this item
That works, thanks!
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26799.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26799.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26799.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-10-25 15:05:37 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
?
|
||
Thanks!
Possible we have broken things in other places but that's just kinda life - we can fix it as it comes. That's why we have alpha's and beta's and rc's to catch the edge cases.
I'll work on weekend on it and make PRs if necessary.
@wilsonge I fear that we have such problems elsewhere, too, where we have removed default values from not nullable datetime columns. I'll have to check on weekend whereever inserts are done in such tables in that way, i.e. not using the table class of that component's table but the
insertObjectmethod of the Db driver. Or am I wrong?