]> git.lyx.org Git - lyx.git/blobdiff - src/insets/insetgraphics.C
quick fix for commented out figures
[lyx.git] / src / insets / insetgraphics.C
index cbcbe2f0fcda1156c499bca7c5b9aeb2a6ae819d..609e6773e0d42a8e262ad6a0f70346279349fe7d 100644 (file)
@@ -99,6 +99,7 @@ TODO Before initial production release:
 #include "support/LAssert.h"
 #include "support/filetools.h"
 #include "support/lyxalgo.h" // lyx::count
+#include "support/path.h"
 
 #include <algorithm> // For the std::max
 
@@ -147,13 +148,15 @@ InsetGraphics::InsetGraphics()
 {}
 
 
-InsetGraphics::InsetGraphics(InsetGraphics const & ig, bool same_id)
+InsetGraphics::InsetGraphics(InsetGraphics const & ig,
+                            string const & filepath,
+                            bool same_id)
        : Inset(ig, same_id),
          SigC::Object(),
          graphic_label(unique_id()),
          cached_status_(grfx::ErrorUnknown), cache_filled_(false), old_asc(0)
 {
-       setParams(ig.params());
+       setParams(ig.params(), filepath);
 }
 
 
@@ -355,10 +358,10 @@ void InsetGraphics::draw(BufferView * bv, LyXFont const & font,
 // Update the inset after parameters changed (read from file or changed in
 // dialog. The grfx::GCache makes the decisions about whether or not to draw
 // (interogates lyxrc, ascertains whether file exists etc)
-void InsetGraphics::updateInset() const
+void InsetGraphics::updateInset(string const & filepath) const
 {
        grfx::GCache & gc = grfx::GCache::get();
-       gc.update(*this);
+       gc.update(*this, filepath);
 }
 
 
@@ -380,10 +383,10 @@ Inset::EDITABLE InsetGraphics::editable() const
 }
 
 
-void InsetGraphics::write(Buffer const * buf, ostream & os) const
+void InsetGraphics::write(Buffer const *, ostream & os) const
 {
        os << "Graphics FormatVersion " << VersionNumber << '\n';
-       params().Write(buf, os);
+       params().Write(os);
 }
 
 
@@ -392,16 +395,17 @@ void InsetGraphics::read(Buffer const * buf, LyXLex & lex)
        string const token = lex.getString();
 
        if (token == "Graphics")
-               readInsetGraphics(buf, lex);
+               readInsetGraphics(lex);
        else if (token == "Figure") // Compatibility reading of FigInset figures.
-               readFigInset(buf, lex);
+               readFigInset(lex);
        else
                lyxerr[Debug::GRAPHICS] << "Not a Graphics or Figure inset!\n";
 
-       updateInset();
+       updateInset(buf->filePath());
 }
 
-void InsetGraphics::readInsetGraphics(Buffer const * buf, LyXLex & lex)
+
+void InsetGraphics::readInsetGraphics(LyXLex & lex)
 {
        bool finished = false;
 
@@ -428,7 +432,7 @@ void InsetGraphics::readInsetGraphics(Buffer const * buf, LyXLex & lex)
                        // TODO: Possibly open up a dialog?
                }
                else {
-                       if (! params_.Read(buf, lex, token))
+                       if (! params_.Read(lex, token))
                                lyxerr << "Unknown token, " << token << ", skipping."
                                        << std::endl;
                }
@@ -436,7 +440,7 @@ void InsetGraphics::readInsetGraphics(Buffer const * buf, LyXLex & lex)
 }
 
 // FormatVersion < 1.0  (LyX < 1.2)
-void InsetGraphics::readFigInset(Buffer const * buf, LyXLex & lex)
+void InsetGraphics::readFigInset(LyXLex & lex)
 {
        std::vector<string> const oldUnits =
                getVectorFromString("pt,cm,in,p%,c%");
@@ -462,9 +466,7 @@ void InsetGraphics::readFigInset(Buffer const * buf, LyXLex & lex)
                        finished = true;
                } else if (token == "file") {
                        if (lex.next()) {
-                               string const name = lex.getString();
-                               string const path = buf->filePath();
-                               params_.filename = MakeAbsPath(name, path);
+                               params_.filename = lex.getString();
                        }
                } else if (token == "extra") {
                        if (lex.next());
@@ -484,7 +486,8 @@ void InsetGraphics::readFigInset(Buffer const * buf, 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()) {
@@ -617,6 +620,10 @@ string const InsetGraphics::prepareFile(Buffer const *buf) const
                return filename_;
        }
 
+       // Enable these helper functions to find the file if it is stored as
+       // a relative path.
+       Path p(buf->filePath());
+
        if (zipped)
                filename_ = unzipFile(filename_);
 
@@ -644,13 +651,17 @@ string const InsetGraphics::prepareFile(Buffer const *buf) const
 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.
-       }
+       // 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;                    
+
        // These variables collect all the latex code that should be before and
        // after the actual includegraphics command.
        string before;
@@ -662,23 +673,31 @@ 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.
        int const newlines =
                int(lyx::count(latex_str.begin(), latex_str.end(),'\n') + 1);
 
-       // lyxerr << "includegraphics: " << newlines << " lines of text"
-       //        << endl;
        return newlines;
 }
 
@@ -730,7 +749,8 @@ void InsetGraphics::validate(LaTeXFeatures & features) const
 }
 
 
-bool InsetGraphics::setParams(InsetGraphicsParams const & p)
+bool InsetGraphics::setParams(InsetGraphicsParams const & p,
+                             string const & filepath)
 {
        // If nothing is changed, just return and say so.
        if (params() == p && !p.filename.empty()) {
@@ -741,7 +761,7 @@ bool InsetGraphics::setParams(InsetGraphicsParams const & p)
        params_ = p;
 
        // Update the inset with the new parameters.
-       updateInset();
+       updateInset(filepath);
 
        // We have changed data, report it.
        return true;
@@ -754,7 +774,7 @@ InsetGraphicsParams const & InsetGraphics::params() const
 }
 
 
-Inset * InsetGraphics::clone(Buffer const &, bool same_id) const
+Inset * InsetGraphics::clone(Buffer const & buffer, bool same_id) const
 {
-       return new InsetGraphics(*this, same_id);
+       return new InsetGraphics(*this, buffer.filePath(), same_id);
 }