Bad workaround for #659

[jmarrec]

Temporary workaround for #659 on mac. Just disabling the messagebox Invalid weather file object, weather file object has been reset. Please choose another weather file. Hopefully we find an actual fix, but in the meantime it's more important to be able to load a file than to see this warning messagebox.

[macumber]

The MessageBox in OpenStudioApp::versionUpdateMessageBox is not parented, the one in OSDocument::weatherFileReset is, I wonder if that makes any difference?

Went looking for any related Qt bugs:

https://bugreports.qt.io/browse/QTBUG-116622
https://bugreports.qt.io/browse/QTBUG-114546

[jmarrec]

@macumber I tried this, doesn't work.

--- a/src/openstudio_app/OpenStudioApp.cpp
+++ b/src/openstudio_app/OpenStudioApp.cpp
@@ -1076,7 +1076,13 @@ void OpenStudioApp::childEvent(QChildEvent* event) {

 void OpenStudioApp::versionUpdateMessageBox(const osversion::VersionTranslator& translator, bool successful, const QString& fileName,
                                             const openstudio::path& tempModelDir) {
-  QMessageBox messageBox;
+
+  QWidget* parent = nullptr;
+
+  if (this->currentDocument()) {
+    parent = this->currentDocument()->mainWindow();
+  }
+  QMessageBox messageBox(parent);

I also tried a few more things including OSAppBase::mainWidget as I said here: #659 (comment)