Re: [PATCH] update layout lists before setting layout

[Abdelrazak Younes]

Jean-Marc Lasgouttes wrote:
When creating a new document, I see:
Trying to select non existent layout type Standard
This is because LyXView::setBuffer sets the layout before updating
layout list.

Abdel, is the following patch OK?

Hum I atalked too fast, sorry.

updateLayoutChoice() is doing different things if there is a visible 
Buffer. This is guaranteed only after the check on BufferView::buffer() 
so I think the aatched patch is more correct.

Abdel.

Index: LyXView.C
===================================================================
--- LyXView.C   (revision 14828)
+++ LyXView.C   (working copy)
@@ -145,12 +145,12 @@
                // require bv_->text.
                getDialogs().updateBufferDependent(true);
                connectBuffer(*work_area_->bufferView().buffer());
+               updateLayoutChoice();
                setLayout(work_area_->bufferView().firstLayout());
        }
 
        updateMenubar();
        updateToolbars();
-       updateLayoutChoice();
        updateWindowTitle();
        redrawWorkArea();
 }