Task #57572
Task #57578: TceformsUpdateWizard improvements (Migrate all file relations from tt_content.image and pages.media)
TceformsUpdateWizard has to handle SQL errors
| Status: | Resolved | Start date: | 2014-04-02 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assigned To: | Markus Klein | % Done: | 100% |
|
| Category: | Install Tool | Spent time: | - | |
| Target version: | next-patchlevel | |||
| TYPO3 Version: | 6.2 | Complexity: | easy | |
| PHP Version: | Sprint Focus: |
Description
There might be situations where the SELECT query fails. This only leads to a PHP warning currently, as the foreach loop does not work on the NULL value.
Fix that to issue a better warning including the SQL error.
Related issues
Associated revisions
[TASK] TceformsUpdateWizard: Throw exception if database query fails
Inform the user, if the database SELECT query fails during the wizard.
Resolves: #57572
Releases: 6.2
Change-Id: I0908aff5d58d6d309b471c810d924f26fc1ca577
Reviewed-on: https://review.typo3.org/29093
Reviewed-by: Alexander Opitz
Tested-by: Alexander Opitz
Reviewed-by: Steffen Müller
Tested-by: Steffen Müller
History
#1 Updated by Gerrit Code Review over 1 year ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29093
#2 Updated by Markus Klein over 1 year ago
- Status changed from Under Review to Accepted
- Parent task set to #57578
#3 Updated by Markus Klein over 1 year ago
- Status changed from Accepted to Under Review
#4 Updated by Markus Klein over 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset f797f4ae069c2fd2e7d06857d5c2289991a553cd.
#5 Updated by Steffen Krüchten over 1 year ago
Patch now delivers a correct error response. For me an illegal mix of UTF-8 collations. Pointed me in the right direction. After sorting out my collations beforehand migration went smooth. Problem solved for me, thank you very much for the quick response!