]> git.lyx.org Git - lyx.git/blobdiff - src/insets/insetgraphics.C
quick fix for commented out figures
[lyx.git] / src / insets / insetgraphics.C
index ea7cfc0242ae9b6949636b572a037b2ca92f06be..609e6773e0d42a8e262ad6a0f70346279349fe7d 100644 (file)
@@ -1,35 +1,35 @@
 /* This file is part of
- * ====================================================== 
- * 
+ * ======================================================
+ *
  *           LyX, The Document Processor
- *      
+ *
  *           Copyright 1995-2002 the LyX Team.
- *           
+ *
  * \author Baruch Even
  * \author Herbert Voss <voss@lyx.org>
  * ====================================================== */
 
 /*
 Known BUGS:
-    
+
     * If the image is from the clipart, and the document is moved to another
       directory, the user is screwed. Need a way to handle it.
       This amounts to a problem of when to use relative or absolute file paths
       We should probably use what the user asks to use... but when he chooses
-      by the file dialog we normally get an absolute path and this may not be 
+      by the file dialog we normally get an absolute path and this may not be
       what the user meant.
 
       Note that browseRelFile in helper_funcs.* provides a file name
       which is relative if it is at reference path (here puffer path)
       level or below, and an absolute path if the file name is not a
       `natural' relative file name. In any case,
-              MakeAbsPath(filename, buf->filePath())
+             MakeAbsPath(filename, buf->filePath())
       is guaranteed to provide the correct absolute path. This is what is
       done know for include insets. Feel free to ask me -- JMarc
       14/01/2002
-       
+
 TODO Before initial production release:
-    
+
     * What advanced features the users want to do?
       Implement them in a non latex dependent way, but a logical way.
       LyX should translate it to latex or any other fitting format.
@@ -53,27 +53,27 @@ TODO Before initial production release:
  * and when reading one should change the version check and the error message.
  * The filename is kept in  the lyx file in a relative way, so as to allow
  * moving the document file and its images with no problem.
- * 
+ *
  *
  * Conversions:
  *   Postscript output means EPS figures.
  *
  *   PDF output is best done with PDF figures if it's a direct conversion
  *   or PNG figures otherwise.
- *     Image format
- *     from        to
- *     EPS         epstopdf
- *     PS          ps2pdf
- *     JPG/PNG     direct
- *     PDF         direct
- *     others      PNG
+ *     Image format
+ *     from        to
+ *     EPS         epstopdf
+ *     PS          ps2pdf
+ *     JPG/PNG     direct
+ *     PDF         direct
+ *     others      PNG
  */
 
-#include <config.h> 
+#include <config.h>
 
 #ifdef __GNUG__
 #pragma implementation
-#endif 
+#endif
 
 #include "insets/insetgraphics.h"
 #include "insets/insetgraphicsParams.h"
@@ -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
 
@@ -120,7 +121,7 @@ string const RemoveExtension(string const & filename)
 {
        return ChangeExtension(filename, string());
 }
+
 } // namespace anon
 
 
@@ -143,17 +144,19 @@ string const unique_id()
 InsetGraphics::InsetGraphics()
        : graphic_label(unique_id()),
          cached_status_(grfx::ErrorUnknown), cache_filled_(false), old_asc(0)
-         
+
 {}
 
 
-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);
 }
 
 
@@ -230,7 +233,7 @@ bool InsetGraphics::drawImage() const
        return pixmap != 0;
 }
 
-       
+
 int InsetGraphics::ascent(BufferView *, LyXFont const &) const
 {
        old_asc = 50;
@@ -268,18 +271,18 @@ int InsetGraphics::width(BufferView *, LyXFont const & font) const
                        int const msg_width = lyxfont::width(msg, msgFont);
                        font_width = std::max(font_width, msg_width);
                }
-               
+
                return std::max(50, font_width + 15);
        }
 }
 
 
 void InsetGraphics::draw(BufferView * bv, LyXFont const & font,
-                         int baseline, float & x, bool) const
+                        int baseline, float & x, bool) const
 {
        int oasc = old_asc;
        grfx::ImageStatus old_status_ = cached_status_;
-       
+
        int ldescent = descent(bv, font);
        int lascent  = ascent(bv, font);
        int lwidth   = width(bv, font);
@@ -307,18 +310,18 @@ void InsetGraphics::draw(BufferView * bv, LyXFont const & font,
        // This will draw the graphics. If the graphics has not been loaded yet,
        // we draw just a rectangle.
        Painter & paint = bv->painter();
-       
+
        if (drawImage()) {
 //             lyxerr << "IG(" << this << "): " << old_x << endl;
                paint.image(old_x + 2, baseline - lascent,
-                           lwidth - 4, lascent + ldescent,
+                           lwidth - 4, lascent + ldescent,
                            *cached_image_.get());
 
        } else {
 
                paint.rectangle(old_x + 2, baseline - lascent,
-                               lwidth - 4,
-                               lascent + ldescent);
+                               lwidth - 4,
+                               lascent + ldescent);
 
                // Print the file name.
                LyXFont msgFont(font);
@@ -326,7 +329,7 @@ void InsetGraphics::draw(BufferView * bv, LyXFont const & font,
                string const justname = OnlyFilename (params().filename);
                if (!justname.empty()) {
                        msgFont.setSize(LyXFont::SIZE_FOOTNOTE);
-                       paint.text(old_x + 8, 
+                       paint.text(old_x + 8,
                                   baseline - lyxfont::maxAscent(msgFont) - 4,
                                   justname, msgFont);
                }
@@ -344,7 +347,7 @@ void InsetGraphics::draw(BufferView * bv, LyXFont const & font,
        if (old_status_ != grfx::ErrorUnknown && old_status_ != cached_status_) {
                bv->getLyXText()->status(bv, LyXText::CHANGED_IN_DRAW);
        }
+
        // Reset the cache, ready for the next draw request
        cached_status_ = grfx::ErrorUnknown;
        cached_image_.reset(0);
@@ -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;
 
@@ -409,7 +413,7 @@ void InsetGraphics::readInsetGraphics(Buffer const * buf, LyXLex & lex)
                lex.next();
 
                string const token = lex.getString();
-               lyxerr[Debug::GRAPHICS] << "Token: '" << token << '\'' 
+               lyxerr[Debug::GRAPHICS] << "Token: '" << token << '\''
                                    << std::endl;
 
                if (token.empty()) {
@@ -428,25 +432,25 @@ void InsetGraphics::readInsetGraphics(Buffer const * buf, LyXLex & lex)
                        // TODO: Possibly open up a dialog?
                }
                else {
-                       if (! params_.Read(buf, lex, token))
-                               lyxerr << "Unknown token, " << token << ", skipping." 
+                       if (! params_.Read(lex, token))
+                               lyxerr << "Unknown token, " << token << ", skipping."
                                        << std::endl;
                }
        }
 }
 
 // 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%");
        bool finished = false;
-       // set the display default      
-       if (lyxrc.display_graphics == "mono") 
+       // set the display default
+       if (lyxrc.display_graphics == "mono")
            params_.display = InsetGraphicsParams::MONOCHROME;
-       else if (lyxrc.display_graphics == "gray") 
+       else if (lyxrc.display_graphics == "gray")
            params_.display = InsetGraphicsParams::GRAYSCALE;
-       else if (lyxrc.display_graphics == "color") 
+       else if (lyxrc.display_graphics == "color")
            params_.display = InsetGraphicsParams::COLOR;
        else
            params_.display = InsetGraphicsParams::NONE;
@@ -455,16 +459,14 @@ void InsetGraphics::readFigInset(Buffer const * buf, LyXLex & lex)
 
                string const token = lex.getString();
                lyxerr[Debug::GRAPHICS] << "Token: " << token << endl;
-               
+
                if (token.empty())
                        continue;
                else if (token == "\\end_inset") {
                        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,15 +486,16 @@ 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()) {
-                               case 1: params_.display = InsetGraphicsParams::MONOCHROME; 
+                               case 1: params_.display = InsetGraphicsParams::MONOCHROME;
                                    break;
-                               case 2: params_.display = InsetGraphicsParams::GRAYSCALE; 
+                               case 2: params_.display = InsetGraphicsParams::GRAYSCALE;
                                    break;
-                               case 3: params_.display = InsetGraphicsParams::COLOR; 
+                               case 3: params_.display = InsetGraphicsParams::COLOR;
                                    break;
                                }
                } else if (token == "subfigure") {
@@ -571,7 +574,7 @@ string findTargetFormat(string const & suffix)
 {
        // lyxrc.pdf_mode means:
        // Are we creating a PDF or a PS file?
-       // (Should actually mean, are we using latex or pdflatex).      
+       // (Should actually mean, are we using latex or pdflatex).
        lyxerr[Debug::GRAPHICS] << "decideOutput: lyxrc.pdf_mode = "
                            << lyxrc.pdf_mode << std::endl;
        if (lyxrc.pdf_mode) {
@@ -612,11 +615,15 @@ string const InsetGraphics::prepareFile(Buffer const *buf) const
        bool const zipped = zippedFile(filename_);
 
        if ((zipped && params().noUnzip) || buf->niceFile) {
-               lyxerr[Debug::GRAPHICS] << "don't unzip file or export latex" 
+               lyxerr[Debug::GRAPHICS] << "don't unzip file or export latex"
                                    << filename_ << endl;
                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;
 }
 
@@ -707,8 +726,8 @@ int InsetGraphics::linuxdoc(Buffer const *, ostream &) const
 // See also the docbook guide at http://www.docbook.org/
 int InsetGraphics::docbook(Buffer const *, ostream & os) const
 {
-       // In DocBook v5.0, the graphic tag will be eliminated from DocBook, will 
-       // need to switch to MediaObject. However, for now this is sufficient and 
+       // In DocBook v5.0, the graphic tag will be eliminated from DocBook, will
+       // need to switch to MediaObject. However, for now this is sufficient and
        // easier to use.
        os << "<graphic fileref=\"&" << graphic_label << ";\">";
        return 0;
@@ -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);
 }