Bug #84146
closed
"Reports" says update wizard incomplete
Added by Gone With the Wind over 6 years ago.
Updated almost 6 years ago.
Description
The Reports tool says that the update wizard is incomplete and recommends to switch to the install tool and check the update wizard there.
But the install tool's update wizard shows 100%.
This happened after an update from 8.7.9 to 8.7.10, php 7.1.13 on at least 3 different installations.
Images included.
Files
- Status changed from New to Accepted
- Target version changed from 9.2 to 8.7.11
- Status changed from Accepted to Needs Feedback
Robert, it seems that at least one of the upgrade wizards in the context of the Reports modules finds that it hasn't finished the upgrade.
Can you modify \TYPO3\CMS\Install\Report\InstallStatusReport::getRemainingUpdatesStatus() to make $message show $identifier and $className?
It could be that more wizards think they are not finished. In that case the code could be:
if ($updateObject->shouldRenderWizard()) {
// at least one wizard was found
$value = $languageService->getLL('status_updateIncomplete');
$severity = Status::WARNING;
$url = BackendUtility::getModuleUrl('system_extinstall');
$message .= $identifier . ':' . $className . ', ';
}
From there the affected wizards can be checked further.
Jigal - do your TYPO3 installations n ot show the same problem?
Message after implementing your script:
Update Incomplete
This installation is not configured for the TYPO3 version it is running. If you did so intentionally, this message can be safely ignored. If you are unsure, visit the Update Wizard section of the Install Tool to see how TYPO3 would change.
TYPO3\CMS\Install\Updates\DatabaseRowsUpdateWizard:TYPO3\CMS\Install\Updates\DatabaseRowsUpdateWizard,
Robert,
Finally I found an installation that shows this behaviour. It was in a different wizard. A patch will follow soon.
- Status changed from Needs Feedback to Under Review
Thanks a lot, Jigal, for your quick fix!
Looking into it...
Thanks a lot, Jigal, for your quick fix!
The changed code works fine on my 8.7.10 installation. But the file itself (that is: your file) is a bit different in the import statement section (your file has less import statements than 8.7.10). Do I still vote +1?
A side note: There is a grammar error around line 62:
// 1 = required, don't has to be writable
should be:
// 1 = required, doesn't have to be writable
Can you take this fix along with this one or should I create a separate issue?
Robert. Thanks for your testing! I may agree this fixes an issue for v8, but we're currently first discussing the patch for core master branch (v9). If that is sorted out, we later back port to v8. This is the reason the patch did not apply cleanly to v8 - a master variant has been pushed first. For master however, my testing says the v9 master core branch is not affected by the bug (maybe has been fixed meanwhile). If that turns out to be true and we can still reproduce the issue for v8, we may then decide to abandon the master patch and only fix it in v8 in a separate patch.
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
- Status changed from Resolved to Closed
- Related to Bug #86632: Reports complaining that "Update incomplete" added
Also available in: Atom
PDF