Rationalize the handling of InsetLayout's preamble and requries. That is,

[features.git] / src / insets / InsetLayout.cpp

diff --git a/src/insets/InsetLayout.cpp b/src/insets/InsetLayout.cpp
index f71fd582a2c0711f219a9f7df11bb1dfb122186d..ca1429793b47e46d6cc6b26f5540f86a05ff6bd2 100644 (file)
--- a/src/insets/InsetLayout.cpp
+++ b/src/insets/InsetLayout.cpp
@@ -14,7 +14,7 @@
 
 #include "InsetLayout.h"
 
-#include "Color.h"
+#include "ColorSet.h"
 #include "Font.h"
 #include "Lexer.h"
 #include "TextClass.h"
@@ -31,12 +31,13 @@ using std::vector;
 namespace lyx {
 
 InsetLayout::InsetLayout() :
-       name_(from_ascii("undefined")), labelstring_(from_ascii("UNDEFINED")),
-       decoration_(InsetLayout::Default),
-       font_(sane_font), labelfont_(sane_font), bgcolor_(Color_error), 
+       name_(from_ascii("undefined")), lyxtype_(STANDARD),
+       labelstring_(from_ascii("UNDEFINED")), decoration_(DEFAULT),
+       latextype_(NOLATEXTYPE), font_(sane_font), 
+       labelfont_(sane_font), bgcolor_(Color_error), 
        multipar_(false), custompars_(false), forceplain_(true), 
        passthru_(false), needprotect_(false), freespacing_(false), 
-       keepempty_(false), forceltr_(false)
+       keepempty_(false), forceltr_(false), intoc_(false)
 { 
        labelfont_.setColor(Color_error);
 }
@@ -46,26 +47,31 @@ namespace {
 
 InsetLayout::InsetDecoration translateDecoration(std::string const & str) 
 {
-       if (str == "classic" || str == "Classic")
-               return InsetLayout::Classic;
-       if (str == "minimalistic" || str == "Minimalistic")
-               return InsetLayout::Minimalistic;
-       if (str == "conglomerate" || str == "Conglomerate")
-               return InsetLayout::Conglomerate;
-       return InsetLayout::Default;
+       if (support::compare_ascii_no_case(str, "classic") == 0)
+               return InsetLayout::CLASSIC;
+       if (support::compare_ascii_no_case(str, "minimalistic") == 0)
+               return InsetLayout::MINIMALISTIC;
+       if (support::compare_ascii_no_case(str, "conglomerate") == 0)
+               return InsetLayout::CONGLOMERATE;
+       return InsetLayout::DEFAULT;
 }
 
+InsetLayout::InsetLaTeXType translateLaTeXType(std::string const & str)
+{
+       if (support::compare_ascii_no_case(str, "command") == 0)
+               return InsetLayout::COMMAND;
+       if (support::compare_ascii_no_case(str, "environment") == 0)
+               return InsetLayout::ENVIRONMENT;
+       if (support::compare_ascii_no_case(str, "none") == 0)
+               return InsetLayout::NOLATEXTYPE;
+       return InsetLayout::ILT_ERROR;
 }
 
+} // namespace anon
 
-bool InsetLayout::read(Lexer & lex, TextClass & tclass)
-{
-       name_ = support::subst(lex.getDocString(), '_', ' ');
-       // FIXME We need to check for name_.empty() here, and
-       // take the same sort of action as in TextClass::read()
-       // if it is empty. Or, better, we could read name_ there,
-       // take action there, etc.
 
+bool InsetLayout::read(Lexer & lex, TextClass const & tclass)
+{
        enum {
                IL_BGCOLOR,
                IL_COPYSTYLE,
@@ -75,6 +81,7 @@ bool InsetLayout::read(Lexer & lex, TextClass & tclass)
                IL_FORCELTR,
                IL_FORCEPLAIN,
                IL_FREESPACING,
+               IL_INTOC,
                IL_LABELFONT,
                IL_LABELSTRING,
                IL_LATEXNAME,
@@ -101,6 +108,7 @@ bool InsetLayout::read(Lexer & lex, TextClass & tclass)
                { "forceltr", IL_FORCELTR },
                { "forceplain", IL_FORCEPLAIN },
                { "freespacing", IL_FREESPACING },
+               { "intoc", IL_INTOC },
                { "keepempty", IL_KEEPEMPTY },
                { "labelfont", IL_LABELFONT },
                { "labelstring", IL_LABELSTRING },
@@ -119,7 +127,7 @@ bool InsetLayout::read(Lexer & lex, TextClass & tclass)
 
        FontInfo font = inherit_font;
        labelfont_ = inherit_font;
-       bgcolor_ = Color_background;
+       bgcolor_ = Color_none;
        bool getout = false;
        // whether we've read the CustomPars or ForcePlain tag
        // for issuing a warning in case MultiPars comes later
@@ -136,12 +144,26 @@ bool InsetLayout::read(Lexer & lex, TextClass & tclass)
                        break;
                }
                switch (le) {
-               case IL_LYXTYPE:
-                       lex >> lyxtype_;
+               // FIXME
+               // Perhaps a more elegant way to deal with the next two would be the
+               // way this sort of thing is handled in Layout::read(), namely, by
+               // using the Lexer.
+               case IL_LYXTYPE: {
+                       string lt;
+                       lex >> lt;
+                       lyxtype_ = translateLyXType(lt);
+                       if (lyxtype_  == NOLYXTYPE)
+                               LYXERR0("Unknown LyXType `" << lt << "'.");
                        break;
-               case IL_LATEXTYPE:
-                       lex >> latextype_;
+               }
+               case IL_LATEXTYPE:  {
+                       string lt;
+                       lex >> lt;
+                       latextype_ = translateLaTeXType(lt);
+                       if (latextype_  == ILT_ERROR)
+                               LYXERR0("Unknown LaTeXType `" << lt << "'.");
                        break;
+               }
                case IL_LABELSTRING:
                        lex >> labelstring_;
                        break;
@@ -162,6 +184,9 @@ bool InsetLayout::read(Lexer & lex, TextClass & tclass)
                case IL_FORCELTR:
                        lex >> forceltr_;
                        break;
+               case IL_INTOC:
+                       lex >> intoc_;
+                       break;
                case IL_MULTIPAR:
                        lex >> multipar_;
                        // the defaults for these depend upon multipar_
@@ -231,7 +256,7 @@ bool InsetLayout::read(Lexer & lex, TextClass & tclass)
                        bgcolor_ = lcolor.getFromLyXName(tmp);
                        break;
                case IL_PREAMBLE:
-                       preamble_ = lex.getLongString("EndPreamble");
+                       preamble_ = from_utf8(lex.getLongString("EndPreamble"));
                        break;
                case IL_REQUIRES: {
                        lex.eatLine();
@@ -258,4 +283,21 @@ bool InsetLayout::read(Lexer & lex, TextClass & tclass)
        return true;
 }
 
+
+InsetLayout::InsetLyXType translateLyXType(std::string const & str) 
+{
+       
+       if (support::compare_ascii_no_case(str, "charstyle") == 0)
+               return InsetLayout::CHARSTYLE;
+       if (support::compare_ascii_no_case(str, "custom") == 0)
+               return InsetLayout::CUSTOM;
+       if (support::compare_ascii_no_case(str, "element") == 0)
+               return InsetLayout::ELEMENT;
+       if (support::compare_ascii_no_case(str, "end") == 0)
+               return InsetLayout::END;
+       if (support::compare_ascii_no_case(str, "standard") == 0)
+               return InsetLayout::STANDARD;
+       return InsetLayout::NOLYXTYPE;
+}
+
 } //namespace lyx