make sure included file is valid before saving it

1 files changed, 3 insertions, 1 deletions

diff --git a/src/MenuCreator.cc b/src/MenuCreator.cc
index d973a66..a04860f 100644
--- a/src/MenuCreator.cc
+++ b/src/MenuCreator.cc
@@ -466,7 +466,6 @@ FbTk::Menu *MenuCreator::createFromFile(const string &filename, int screen_numbe
 bool MenuCreator::createFromFile(const string &filename,
                                  FbTk::Menu &inject_into, bool require_begin) {
     string real_filename = FbTk::StringUtil::expandFilename(filename);
-    Fluxbox::instance()->saveMenuFilename(real_filename.c_str());
 
     FbMenuParser parser(real_filename);
     if (!parser.isLoaded())
@@ -476,6 +475,9 @@ bool MenuCreator::createFromFile(const string &filename,
     if (require_begin && !getStart(parser, label, m_stringconvertor))
         return false;
 
+    // save menu filename, so we can check if it changes
+    Fluxbox::instance()->saveMenuFilename(real_filename.c_str());
+
     startFile();
     parseMenu(parser, inject_into, m_stringconvertor);
     endFile();