[#43151] - [5.1.0_Beta2] SQL Error on Update to J5.1.0_Beta2
- Closed
- 28 Mar 2024
- Medium
- Build: 4.4-dev
- # 43151
Steps to reproduce the issue
- Clone a site with J5.0.3
- Download J5.1.0_Beta2 update ZIP
- Start manual update by "Upload and Install" the downloaded file
Expected result
The update runs smoothly.
Actual result
The update fails (or seems to fail) with an SQL error:
The update log reveals more information:
The database needs to be repaired:
Database repair is marked as successful
Frontend and backend seem to work without issues.
System information (as much as possible)
J5.0.3
PHP 8.3.3
MySQL 8.0.33
Additional comments
joomla-cms-bot
- | Labels |
Added:
No Code Attached Yet
|
||
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-03-25 19:41:03 |
| Closed_By | ⇒ | richard67 |
I don't know whether it adds more information to the test, but I repeated the test with some more attention to update sources. When I started the update process to J5.1._beta2, there were two entries for Joomla! Core (IDs 1 and 104 in the screenshot).
For my second try, I deleted the records for update source with ID 1 from the two relevant database tables (#__update_sites and #__update_sites_extensions). After that, the update process finished without any errors.
There was no need to repair the database tables after the update anymore.
| Status | Closed | ⇒ | New |
| Closed_Date | 2024-03-25 19:41:03 | ⇒ | |
| Closed_By | richard67 | ⇒ |
Hmm, my fix was included in Beta 2, and here the update was made to Beta 2, so either the change to the SQL script made by my PR has not been applied before it was run (but it should have been), or there is still an issue with that SQL statement after my fix.
Reopening.
I think the problem is the insert statement before the update statement which I had fixed with my PR. Will check tomorrow, and if my assumption is right I will make a PR.
| Labels |
Added:
bug
|
||
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-03-28 08:56:47 |
| Closed_By | ⇒ | richard67 |
should be fixed by #42988