quick fix for commented out figures

[lyx.git] / src / insets / insetgraphics.C

diff --git a/src/insets/insetgraphics.C b/src/insets/insetgraphics.C
index 87dbd920d89176a95c0fb90f200c601a689a8e38..609e6773e0d42a8e262ad6a0f70346279349fe7d 100644 (file)
--- a/src/insets/insetgraphics.C
+++ b/src/insets/insetgraphics.C
@@ -486,7 +486,8 @@ void InsetGraphics::readFigInset(LyXLex & lex)
                        if (lex.next())
                                params_.lyxwidth = LyXLength(lex.getString()+"pt");
                        if (lex.next())
-                               params_.lyxheight = LyXLength(lex.getString()+"pt");
+                               params_.lyxheight = LyXLength(lex.getString()+"pt");    
+                       params_.lyxsize_type = InsetGraphicsParams::WH;
                } else if (token == "flags") {
                        if (lex.next())
                                switch (lex.getInteger()) {
@@ -647,71 +648,20 @@ string const InsetGraphics::prepareFile(Buffer const *buf) const
 }
 
 
-namespace {
-
-string const latexify(string const & str)
-{
-       ostringstream out;
-
-       string::const_iterator it  = str.begin();
-       string::const_iterator end = str.end();
-
-       for (; it != end; ++it) {
-               switch (*it) {
-                       
-               case ('$'):
-               case ('&'):
-               case ('%'):
-               case ('#'):
-               case ('_'):
-               case ('{'):
-               case ('}'):
-                       out << '\\' << *it;
-                       break;
-
-               case ('~'):
-               case ('^'):
-                       out << '\\' << *it << "{}";
-                       break;
-
-               case ('\\'):
-                       out << "\textbackslash ";
-                       break;
-
-               default:
-                       out << *it;
-                       break;
-               }
-       }
-       
-       return out.str().c_str();
-}
- 
-} // namespace anon
-
-
 int InsetGraphics::latex(Buffer const *buf, ostream & os,
                         bool /*fragile*/, bool/*fs*/) const
 {
-       // If there is no file specified, just output a message about it in
-       // the latex output.
-       if (params().filename.empty()) {
-               os  << "\\fbox{\\rule[-0.5in]{0pt}{1in}"
-                       << _("empty figure path") << "}\n";
-               return 1; // One end-of-line marker added to the stream.
-       }
-
-       // Enable these helper functions to find the file if it is stored as
-       // a relative path.
-       Path p(buf->filePath());
+       // If there is no file specified or not existing, 
+       // just output a message about it in the latex output.
+       lyxerr[Debug::GRAPHICS] << "[latex]filename = " 
+                               << params().filename << endl;               
+       string const message = 
+           (IsFileReadable(MakeAbsPath(params().filename, buf->filePath())) 
+               && !params().filename.empty()) ?
+                   string() :
+                   string("bb = 0 0 200 100, draft, type=eps]");
+       lyxerr[Debug::GRAPHICS] << "[latex]Messagestring = " << message << endl;                    
 
-       // Ditto if the file is not there.
-       if (!IsFileReadable(params().filename)) {
-               os  << "\\fbox{\\rule[-0.5in]{0pt}{1in}"
-                   << latexify(MakeRelPath(params().filename, buf->filePath()))
-                   << _(" not found") << "}\n";
-               return 1; // One end-of-line marker added to the stream.
-       }
        // These variables collect all the latex code that should be before and
        // after the actual includegraphics command.
        string before;
@@ -723,15 +673,25 @@ int InsetGraphics::latex(Buffer const *buf, ostream & os,
        }
        // We never use the starred form, we use the "clip" option instead.
        before += "\\includegraphics";
+
        // Write the options if there are any.
        string const opts = createLatexOptions();
-       if (!opts.empty()) {
-               before += ("[%\n" + opts +']');
-       }
+       lyxerr[Debug::GRAPHICS] << "[latex]opts = " << opts << endl;                
+       if (!opts.empty() && !message.empty())
+               before += ("[" + opts + ',' + message);
+       else if (!message.empty())
+               before += ('[' + message);
+       else if (!opts.empty())
+               before += ("[" + opts + ']');
+       lyxerr[Debug::GRAPHICS] << "[latex]before = " << before << endl;                    
+       lyxerr[Debug::GRAPHICS] << "[latex]after = " << after << endl;              
+
        // Make the filename relative to the lyx file
        // and remove the extension so the LaTeX will use whatever is
        // appropriate (when there are several versions in different formats)
-       string const latex_str = before + '{' + prepareFile(buf) + '}' + after;
+       string const latex_str = message.empty() ? 
+               (before + '{' + prepareFile(buf) + '}' + after) :
+               (before + '{' + params().filename + " not found!}" + after);
        os << latex_str;
 
        // Return how many newlines we issued.