From 793ffd942ece6fee0b4005cab4f5aa193a4cea11 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9mi=20Verschelde?= Date: Fri, 22 Feb 2019 10:21:34 +0100 Subject: [PATCH] Improve error message on project export failure It's still not enough and we need better validation/error checking, but it should help with people assume corrupted templates when it's their config which is invalid. --- editor/project_export.cpp | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/editor/project_export.cpp b/editor/project_export.cpp index ec64d76b184..831ebde3a6c 100644 --- a/editor/project_export.cpp +++ b/editor/project_export.cpp @@ -972,10 +972,15 @@ void ProjectExportDialog::_export_project_to_path(const String &p_path) { Error err = platform->export_project(current, export_debug->is_pressed(), p_path, 0); if (err != OK) { - error_dialog->set_text(TTR("Export templates for this platform are missing/corrupted:") + " " + platform->get_name()); + if (err == ERR_FILE_NOT_FOUND) { + error_dialog->set_text(vformat(TTR("Failed to export the project for platform '%s'.\nExport templates seem to be missing or invalid."), platform->get_name())); + } else { // Assume misconfiguration. FIXME: Improve error handling and preset config validation. + error_dialog->set_text(vformat(TTR("Failed to export the project for platform '%s'.\nThis might be due to a configuration issue in the export preset or your export settings."), platform->get_name())); + } + + ERR_PRINTS(vformat("Failed to export the project for platform '%s'.", platform->get_name())); error_dialog->show(); error_dialog->popup_centered_minsize(Size2(300, 80)); - ERR_PRINT("Failed to export project"); } }