commit 8dde86d2de44786fc9f13a101037dec32eb6922a
Author: Juergen Spitzmueller <sp...@lyx.org>
Date:   Wed Feb 21 10:17:10 2018 +0100

    Fix list of viewable formats in view/update menu.
    
    Also fix list in (currently unused) Export toolbar button.
    
    Fixes: #11044
    (cherry picked from commit ab6bd5e3190e61fa46eb2fd057cdf7f71faf3f38)
---
 src/frontends/qt4/Menus.cpp    |    2 +-
 src/frontends/qt4/Toolbars.cpp |   10 +++++++---
 status.23x                     |    2 ++
 3 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/frontends/qt4/Menus.cpp b/src/frontends/qt4/Menus.cpp
index d08517f..4ced589 100644
--- a/src/frontends/qt4/Menus.cpp
+++ b/src/frontends/qt4/Menus.cpp
@@ -1123,7 +1123,7 @@ void MenuDefinition::expandFormats(MenuItem::Kind const 
kind, Buffer const * buf
                                add(MenuItem(MenuItem::Command, toqstr(lbl), 
FuncRequest(action)));
                                continue;
                        }
-               // fall through
+                       break;
                case MenuItem::ExportFormats:
                        if (!f->inExportMenu())
                                continue;
diff --git a/src/frontends/qt4/Toolbars.cpp b/src/frontends/qt4/Toolbars.cpp
index 2c0d2ab..32d438f 100644
--- a/src/frontends/qt4/Toolbars.cpp
+++ b/src/frontends/qt4/Toolbars.cpp
@@ -194,9 +194,13 @@ ToolbarInfo & ToolbarInfo::read(Lexer & lex)
                case TO_IMPORTFORMATS:
                case TO_UPDATEFORMATS:
                case TO_VIEWFORMATS: {
-                       FormatList formats = (code == TO_IMPORTFORMATS) ?
-                               theConverters().importableFormats() :
-                               theConverters().exportableFormats(true);
+                       FormatList formats;
+                       if (code == TO_IMPORTFORMATS)
+                               formats = theConverters().importableFormats();
+                       else if (code == TO_EXPORTFORMATS)
+                               formats = 
theConverters().exportableFormats(false);
+                       else
+                               formats = 
theConverters().exportableFormats(true);
                        sort(formats.begin(), formats.end());
                        for (Format const * f : formats) {
                                if (f->dummy())
diff --git a/status.23x b/status.23x
index 42bc947..407f06c 100644
--- a/status.23x
+++ b/status.23x
@@ -60,6 +60,8 @@ What's new
 - When entering a math macro name, Escape properly cancels the
   operation (bug 9251).
 
+- Fix list of viewable formats in view/update menu (bug 11044).
+
 
 * INTERNALS
 

Reply via email to