let the model decide if the apply was successful and should close (#6912)

MjnMixael · web-flow · commit 7c29656c7ca5 · 2025-08-08T10:36:19.000-04:00
diff --git a/qtfred/src/mission/util.cpp b/qtfred/src/mission/util.cpp
@@ -108,8 +108,7 @@ bool rejectOrCloseHandler(__UNUSED QDialog* dialog,
 		}
 
 		if (button == fso::fred::DialogButton::Yes) {
-			model->apply();
-			return true;
+			return model->apply(); // only close if apply was successful
 		}
 		if (button == fso::fred::DialogButton::No) {
 			model->reject();

Original file line number	Diff line number	Diff line change
`@@ -108,8 +108,7 @@ bool rejectOrCloseHandler(__UNUSED QDialog* dialog,`
`108`	`108`	`}`
`109`	`109`
`110`	`110`	`if (button == fso::fred::DialogButton::Yes) {`
`111`		`- model->apply();`
`112`		`- return true;`
	`111`	`+ return model->apply(); // only close if apply was successful`
`113`	`112`	`}`
`114`	`113`	`if (button == fso::fred::DialogButton::No) {`
`115`	`114`	`model->reject();`