SoftwareUpdater: Move "complete" logic to ProgressApplyingChangesDone.

This is more like how pkgman does it. Behaviorally, users should
not notice a difference.
This commit is contained in:
Augustin Cavalier
2026-03-03 17:45:42 -05:00
parent 61c7d7e4ee
commit ff0cd4274c
+14 -14
View File
@@ -369,20 +369,6 @@ void
UpdateManager::ProgressTransactionCommitted(InstalledRepository& repository,
const BCommitTransactionResult& result)
{
_SetCurrentStep(ACTION_STEP_COMPLETE);
BString header(B_TRANSLATE("Updates completed"));
BString detail;
if (BPackageRoster().IsRebootNeeded()) {
detail = B_TRANSLATE("A reboot is necessary to complete the "
"update process.");
fStatusWindow->PostMessage(kMsgShowReboot);
} else {
detail = B_TRANSLATE("Updates have been successfully installed.");
}
_FinalUpdate(header.String(), detail.String());
if (fVerbose) {
const char* repositoryName = repository.Name().String();
@@ -410,6 +396,20 @@ UpdateManager::ProgressApplyingChangesDone(InstalledRepository& repository)
{
if (fVerbose)
printf("[%s] Done.\n", repository.Name().String());
_SetCurrentStep(ACTION_STEP_COMPLETE);
BString header(B_TRANSLATE("Updates completed"));
BString detail;
if (BPackageRoster().IsRebootNeeded()) {
detail = B_TRANSLATE("A reboot is necessary to complete the "
"update process.");
fStatusWindow->PostMessage(kMsgShowReboot);
} else {
detail = B_TRANSLATE("Updates have been successfully installed.");
}
_FinalUpdate(header.String(), detail.String());
}