Add a comment, per Pavel's suggestion.

git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@26874 a592a061-630c-0410-9148-cb99ea01b6c8

diff --git a/src/frontends/qt4/GuiDocument.cpp b/src/frontends/qt4/GuiDocument.cpp
index a7a7a603e4c9c8feede9dcbf60054e301f85b15e..085e647c3a3ca07c72b29b1865a52ac3155eebc5 100644 (file)
--- a/src/frontends/qt4/GuiDocument.cpp
+++ b/src/frontends/qt4/GuiDocument.cpp
@@ -1351,11 +1351,17 @@ void GuiDocument::classChanged()
                }
                bp_.useClassDefaults();
        }
+       // With the introduction of modules came a distinction between the base 
+       // class  and the document class. The former corresponds to the main layout 
+       // file; the  latter is that plus the modules (or the document-specific layout,
+       // or  whatever else there could be). Our parameters come from the document 
+       // class.  So when we set the base class, we also need to recreate the document 
+       // class. Otherwise, we still have the old one.
+       bp_.makeDocumentClass();
        // FIXME There's a little bug here connected with auto_reset, namely,
        // that, if the preceding is skipped and the user has changed the
        // modules before changing the class, those changes will be lost on
        // update. But maybe that's what we want?
-       bp_.makeDocumentClass();
        updateSelectedModules();
        paramsToDialog(bp_);
 }