]> git.lyx.org Git - lyx.git/blobdiff - src/insets/insetgraphics.C
fix #832
[lyx.git] / src / insets / insetgraphics.C
index f440b269910f5ed49788200954c7e2705eda27e7..f423210c58408ddf1d1db3f68e7cbf003925f4ac 100644 (file)
@@ -1,34 +1,16 @@
-/* This file is part of
- * ======================================================
- *
- *           LyX, The Document Processor
- *
- *           Copyright 1995-2002 the LyX Team.
+/**
+ * \file insetgraphics.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
  * \author Baruch Even
- * \author Herbert Voss <voss@lyx.org>
- * ====================================================== */
+ * \author Herbert Voss
+ *
+ * Full author contact details are available in file CREDITS
+ */
 
 /*
-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
-      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())
-      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:
+TODO
 
     * What advanced features the users want to do?
       Implement them in a non latex dependent way, but a logical way.
@@ -36,7 +18,6 @@ TODO Before initial production release:
     * Add a way to roll the image file into the file format.
     * When loading, if the image is not found in the expected place, try
       to find it in the clipart, or in the same directory with the image.
-    * Keep a tab on the image file, if it changes, update the lyx view.
     * The image choosing dialog could show thumbnails of the image formats
       it knows of, thus selection based on the image instead of based on
       filename.
@@ -71,50 +52,64 @@ TODO Before initial production release:
 
 #include <config.h>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
-
 #include "insets/insetgraphics.h"
 #include "insets/insetgraphicsParams.h"
 
-#include "graphics/GraphicsCache.h"
+#include "graphics/GraphicsLoader.h"
 #include "graphics/GraphicsImage.h"
+#include "graphics/GraphicsParams.h"
 
-#include "frontends/LyXView.h"
 #include "lyxtext.h"
 #include "buffer.h"
 #include "BufferView.h"
 #include "converter.h"
-#include "frontends/Painter.h"
-#include "lyxrc.h"
-#include "font.h"    // For the lyxfont class.
 #include "debug.h"
+#include "format.h"
+#include "funcrequest.h"
 #include "gettext.h"
 #include "LaTeXFeatures.h"
+#include "Lsstream.h"
+#include "lyxlex.h"
+#include "lyxrc.h"
+#include "Lsstream.h"
 
-#include "frontends/Dialogs.h"
+#include "frontends/lyx_gui.h"
 #include "frontends/Alert.h"
-#include "frontends/controllers/helper_funcs.h" // getVectorFromString
+#include "frontends/Dialogs.h"
+#include "frontends/font_metrics.h"
+#include "frontends/LyXView.h"
+#include "frontends/Painter.h"
 
 #include "support/LAssert.h"
 #include "support/filetools.h"
 #include "support/lyxalgo.h" // lyx::count
+#include "support/lyxlib.h" // float_equal
 #include "support/path.h"
+#include "support/tostr.h"
+#include "support/systemcall.h"
+#include "support/os.h"
+
+#include <boost/weak_ptr.hpp>
+#include <boost/bind.hpp>
+#include <boost/signals/trackable.hpp>
+#include "support/BoostFormat.h"
 
 #include <algorithm> // For the std::max
 
 extern string system_tempdir;
+// set by Exporters
+extern bool pdf_mode;
 
 using std::ostream;
 using std::endl;
 
+
+namespace {
+
 ///////////////////////////////////////////////////////////////////////////
 int const VersionNumber = 1;
 ///////////////////////////////////////////////////////////////////////////
 
-namespace {
-
 // This function is a utility function
 // ... that should be with ChangeExtension ...
 inline
@@ -123,29 +118,78 @@ string const RemoveExtension(string const & filename)
        return ChangeExtension(filename, string());
 }
 
-} // namespace anon
 
-
-namespace {
-
-string const unique_id()
+string const uniqueID()
 {
        static unsigned int seed = 1000;
+       return "graph" + tostr(++seed);
+}
 
-       ostringstream ost;
-       ost << "graph" << ++seed;
 
-       // Needed if we use lyxstring.
-       return ost.str().c_str();
+string findTargetFormat(string const & suffix)
+{
+       // pdf_mode means:
+       // Are we creating a PDF or a PS file?
+       // (Should actually mean, are we using latex or pdflatex).
+       if (pdf_mode) {
+               lyxerr[Debug::GRAPHICS] << "findTargetFormat: PDF mode\n";
+               if (contains(suffix, "ps") || suffix == "pdf")
+                       return "pdf";
+               if (suffix == "jpg")    // pdflatex can use jpeg
+                       return suffix;
+               return "png";         // and also png
+       }
+       // If it's postscript, we always do eps.
+       lyxerr[Debug::GRAPHICS] << "findTargetFormat: PostScript mode\n";
+       if (suffix != "ps")     // any other than ps
+               return "eps";         // is changed to eps
+       return suffix;          // let ps untouched
 }
 
 } // namespace anon
 
 
-InsetGraphics::InsetGraphics()
-       : graphic_label(unique_id()),
-         cached_status_(grfx::ErrorUnknown), cache_filled_(false), old_asc(0)
+struct InsetGraphics::Cache : boost::signals::trackable
+{
+       ///
+       Cache(InsetGraphics &);
+       ///
+       void update(string const & file_with_path);
+
+       ///
+       int old_ascent;
+       ///
+       grfx::Loader loader;
+       ///
+       unsigned long checksum;
+       ///
+       boost::weak_ptr<BufferView> view;
+
+private:
+       ///
+       InsetGraphics & parent_;
+};
+
+
+InsetGraphics::Cache::Cache(InsetGraphics & p)
+       : old_ascent(0), checksum(0), parent_(p)
+{
+       loader.connect(boost::bind(&InsetGraphics::statusChanged, &parent_));
+}
+
 
+void InsetGraphics::Cache::update(string const & file_with_path)
+{
+       lyx::Assert(!file_with_path.empty());
+
+       string const path = OnlyPath(file_with_path);
+       loader.reset(file_with_path, parent_.params().as_grfxParams(path));
+}
+
+
+InsetGraphics::InsetGraphics()
+       : graphic_label(uniqueID()),
+         cache_(new Cache(*this))
 {}
 
 
@@ -153,94 +197,102 @@ 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)
+         graphic_label(uniqueID()),
+         cache_(new Cache(*this))
 {
        setParams(ig.params(), filepath);
 }
 
 
-InsetGraphics::~InsetGraphics()
+Inset * InsetGraphics::clone(Buffer const & buffer, bool same_id) const
 {
-       cached_image_.reset();
-       grfx::GCache & gc = grfx::GCache::get();
-       gc.remove(*this);
+       return new InsetGraphics(*this, buffer.filePath(), same_id);
+}
 
-       // Emits the hide signal to the dialog connected (if any)
-       hideDialog();
+
+InsetGraphics::~InsetGraphics()
+{
+       InsetGraphicsMailer mailer(*this);
+       mailer.hideDialog();
 }
 
 
-string const InsetGraphics::statusMessage() const
+dispatch_result InsetGraphics::localDispatch(FuncRequest const & cmd)
 {
-       string msg;
-
-       switch (cached_status_) {
-       case grfx::WaitingToLoad:
-               msg = _("Waiting for draw request to start loading...");
-               break;
-       case grfx::Loading:
-               msg = _("Loading...");
-               break;
-       case grfx::Converting:
-               msg = _("Converting to loadable format...");
-               break;
-       case grfx::ScalingEtc:
-               msg = _("Loaded. Scaling etc...");
-               break;
-       case grfx::ErrorNoFile:
-               msg = _("No file found!");
-               break;
-       case grfx::ErrorLoading:
-               msg = _("Error loading file into memory");
-               break;
-       case grfx::ErrorConverting:
-               msg = _("Error converting to loadable format");
-               break;
-       case grfx::ErrorScalingEtc:
-               msg = _("Error scaling etc");
-               break;
-       case grfx::ErrorUnknown:
-               msg = _("No image");
-               break;
-       case grfx::Loaded:
-               msg = _("Loaded but not displaying");
-               break;
+       switch (cmd.action) {
+       case LFUN_INSET_MODIFY: {
+               InsetGraphicsParams p;
+               InsetGraphicsMailer::string2params(cmd.argument, p);
+               if (!p.filename.empty()) {
+                       string const filepath = cmd.view()->buffer()->filePath();
+                       setParams(p, filepath);
+                       cmd.view()->updateInset(this);
+               }
+               return DISPATCHED;
        }
 
-       return msg;
+       case LFUN_INSET_DIALOG_UPDATE: 
+               InsetGraphicsMailer(*this).updateDialog(cmd.view());
+               return DISPATCHED;
+
+       case LFUN_INSET_EDIT:
+       case LFUN_MOUSE_RELEASE:
+               InsetGraphicsMailer(*this).showDialog(cmd.view());
+               return DISPATCHED;
+
+       default:
+               return Inset::localDispatch(cmd);
+       }
 }
 
 
-void InsetGraphics::setCache() const
+string const InsetGraphics::statusMessage() const
 {
-       if (cache_filled_)
-               return;
-
-       grfx::GCache & gc = grfx::GCache::get();
-       cached_status_ = gc.status(*this);
-       cached_image_  = gc.image(*this);
+       using namespace grfx;
+
+       switch (cache_->loader.status()) {
+               case WaitingToLoad:
+                       return _("Not shown.");
+               case Loading:
+                       return _("Loading...");
+               case Converting:
+                       return _("Converting to loadable format...");
+               case Loaded:
+                       return _("Loaded into memory. Must now generate pixmap.");
+               case ScalingEtc:
+                       return _("Scaling etc...");
+               case Ready:
+                       return _("Ready to display");
+               case ErrorNoFile:
+                       return _("No file found!");
+               case ErrorConverting:
+                       return _("Error converting to loadable format");
+               case ErrorLoading:
+                       return _("Error loading file into memory");
+               case ErrorGeneratingPixmap:
+                       return _("Error generating the pixmap");
+               case ErrorUnknown:
+                       return _("No image");
+       }
+       return string();
 }
 
 
-bool InsetGraphics::drawImage() const
+bool InsetGraphics::imageIsDrawable() const
 {
-       setCache();
-       Pixmap const pixmap =
-               (cached_status_ == grfx::Loaded && cached_image_.get() != 0) ?
-               cached_image_->getPixmap() : 0;
+       if (!cache_->loader.image() || cache_->loader.status() != grfx::Ready)
+               return false;
 
-       return pixmap != 0;
+       return cache_->loader.image()->isDrawable();
 }
 
 
 int InsetGraphics::ascent(BufferView *, LyXFont const &) const
 {
-       old_asc = 50;
-       if (drawImage())
-               old_asc = cached_image_->getHeight();
-       return old_asc;
+       cache_->old_ascent = 50;
+       if (imageIsDrawable())
+               cache_->old_ascent = cache_->loader.image()->getHeight();
+       return cache_->old_ascent;
 }
 
 
@@ -252,37 +304,54 @@ int InsetGraphics::descent(BufferView *, LyXFont const &) const
 
 int InsetGraphics::width(BufferView *, LyXFont const & font) const
 {
-       if (drawImage())
-               return cached_image_->getWidth();
-       else {
-               int font_width = 0;
+       if (imageIsDrawable())
+               return cache_->loader.image()->getWidth() + 2 * TEXT_TO_INSET_OFFSET;
 
-               LyXFont msgFont(font);
-               msgFont.setFamily(LyXFont::SANS_FAMILY);
+       int font_width = 0;
 
-               string const justname = OnlyFilename (params().filename);
-               if (!justname.empty()) {
-                       msgFont.setSize(LyXFont::SIZE_FOOTNOTE);
-                       font_width = lyxfont::width(justname, msgFont);
-               }
+       LyXFont msgFont(font);
+       msgFont.setFamily(LyXFont::SANS_FAMILY);
 
-               string const msg = statusMessage();
-               if (!msg.empty()) {
-                       msgFont.setSize(LyXFont::SIZE_TINY);
-                       int const msg_width = lyxfont::width(msg, msgFont);
-                       font_width = std::max(font_width, msg_width);
-               }
+       string const justname = OnlyFilename (params().filename);
+       if (!justname.empty()) {
+               msgFont.setSize(LyXFont::SIZE_FOOTNOTE);
+               font_width = font_metrics::width(justname, msgFont);
+       }
 
-               return std::max(50, font_width + 15);
+       string const msg = statusMessage();
+       if (!msg.empty()) {
+               msgFont.setSize(LyXFont::SIZE_TINY);
+               int const msg_width = font_metrics::width(msg, msgFont);
+               font_width = std::max(font_width, msg_width);
        }
+
+       return std::max(50, font_width + 15);
+}
+
+
+BufferView * InsetGraphics::view() const
+{
+       return cache_->view.lock().get();
 }
 
 
 void InsetGraphics::draw(BufferView * bv, LyXFont const & font,
-                        int baseline, float & x, bool) const
+                        int baseline, float & x) const
 {
-       int oasc = old_asc;
-       grfx::ImageStatus old_status_ = cached_status_;
+       // MakeAbsPath returns params().filename unchanged if it absolute
+       // already.
+       string const file_with_path =
+               MakeAbsPath(params().filename, bv->buffer()->filePath());
+
+       // A 'paste' operation creates a new inset with the correct filepath,
+       // but then the 'old' inset stored in the 'copy' operation is actually
+       // added to the buffer.
+       // Thus, we should ensure that the filepath is correct.
+       if (file_with_path != cache_->loader.filename())
+               cache_->update(file_with_path);
+
+       cache_->view = bv->owner()->view();
+       int oasc = cache_->old_ascent;
 
        int ldescent = descent(bv, font);
        int lascent  = ascent(bv, font);
@@ -293,34 +362,35 @@ void InsetGraphics::draw(BufferView * bv, LyXFont const & font,
        // This is not a nice thing to do and should be fixed properly somehow.
        // But I still don't know the best way to go. So let's do this like this
        // for now (Jug 20020311)
-       if (lascent != oasc) {
+       if (lascent != oasc)
                return;
-       }
 
        // Make sure now that x is updated upon exit from this routine
        int old_x = int(x);
        x += lwidth;
 
-       // Initiate the loading of the graphics file
-       if (cached_status_ == grfx::WaitingToLoad) {
-               grfx::GCache & gc = grfx::GCache::get();
-               gc.startLoading(*this);
-       }
+       grfx::Params const & gparams = params().as_grfxParams();
+
+       if (gparams.display != grfx::NoDisplay &&
+                       cache_->loader.status() == grfx::WaitingToLoad)
+               cache_->loader.startLoading();
+
+       if (!cache_->loader.monitoring())
+               cache_->loader.startMonitoring();
 
        // This will draw the graphics. If the graphics has not been loaded yet,
        // we draw just a rectangle.
        Painter & paint = bv->painter();
 
-       if (drawImage()) {
-               paint.image(old_x + 2, baseline - lascent,
-                           lwidth - 4, lascent + ldescent,
-                           *cached_image_.get());
+       if (imageIsDrawable()) {
+               paint.image(old_x + TEXT_TO_INSET_OFFSET, baseline - lascent,
+                           lwidth - 2 * TEXT_TO_INSET_OFFSET, lascent + ldescent,
+                           *cache_->loader.image());
 
        } else {
 
-               paint.rectangle(old_x + 2, baseline - lascent,
-                               lwidth - 4,
-                               lascent + ldescent);
+               paint.rectangle(old_x + TEXT_TO_INSET_OFFSET, baseline - lascent,
+                               lwidth - 2 * TEXT_TO_INSET_OFFSET, lascent + ldescent);
 
                // Print the file name.
                LyXFont msgFont(font);
@@ -328,8 +398,8 @@ 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,
-                                  baseline - lyxfont::maxAscent(msgFont) - 4,
+                       paint.text(old_x + TEXT_TO_INSET_OFFSET + 6,
+                                  baseline - font_metrics::maxAscent(msgFont) - 4,
                                   justname, msgFont);
                }
 
@@ -337,42 +407,9 @@ void InsetGraphics::draw(BufferView * bv, LyXFont const & font,
                string const msg = statusMessage();
                if (!msg.empty()) {
                        msgFont.setSize(LyXFont::SIZE_TINY);
-                       paint.text(old_x + 8, baseline - 4, msg, msgFont);
+                       paint.text(old_x + TEXT_TO_INSET_OFFSET + 6, baseline - 4, msg, msgFont);
                }
        }
-
-       // the status message may mean we changed size, so indicate
-       // we need a row redraw
-       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();
-       cache_filled_ = false;
-}
-
-
-// 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(string const & filepath) const
-{
-       grfx::GCache & gc = grfx::GCache::get();
-       gc.update(*this, filepath);
-}
-
-
-void InsetGraphics::edit(BufferView *bv, int, int, unsigned int)
-{
-       bv->owner()->getDialogs()->showGraphics(this);
-}
-
-
-void InsetGraphics::edit(BufferView * bv, bool)
-{
-       edit(bv, 0, 0, 0);
 }
 
 
@@ -384,7 +421,7 @@ Inset::EDITABLE InsetGraphics::editable() const
 
 void InsetGraphics::write(Buffer const *, ostream & os) const
 {
-       os << "Graphics FormatVersion " << VersionNumber << '\n';
+       os << "Graphics\n";
        params().Write(os);
 }
 
@@ -395,12 +432,10 @@ void InsetGraphics::read(Buffer const * buf, LyXLex & lex)
 
        if (token == "Graphics")
                readInsetGraphics(lex);
-       else if (token == "Figure") // Compatibility reading of FigInset figures.
-               readFigInset(lex);
        else
-               lyxerr[Debug::GRAPHICS] << "Not a Graphics or Figure inset!\n";
+               lyxerr[Debug::GRAPHICS] << "Not a Graphics inset!\n";
 
-       updateInset(buf->filePath());
+       cache_->update(MakeAbsPath(params().filename, buf->filePath()));
 }
 
 
@@ -438,91 +473,6 @@ void InsetGraphics::readInsetGraphics(LyXLex & lex)
        }
 }
 
-// FormatVersion < 1.0  (LyX < 1.2)
-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")
-           params_.display = InsetGraphicsParams::MONOCHROME;
-       else if (lyxrc.display_graphics == "gray")
-           params_.display = InsetGraphicsParams::GRAYSCALE;
-       else if (lyxrc.display_graphics == "color")
-           params_.display = InsetGraphicsParams::COLOR;
-       else
-           params_.display = InsetGraphicsParams::NONE;
-       while (lex.isOK() && !finished) {
-               lex.next();
-
-               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()) {
-                               params_.filename = lex.getString();
-                       }
-               } else if (token == "extra") {
-                       if (lex.next());
-                       // kept for backwards compability. Delete in 0.13.x
-               } else if (token == "subcaption") {
-                       if (lex.eatLine())
-                               params_.subcaptionText = lex.getString();
-               } else if (token == "label") {
-                       if (lex.next());
-                       // kept for backwards compability. Delete in 0.13.x
-               } else if (token == "angle") {
-                       if (lex.next()) {
-                               params_.rotate = true;
-                               params_.rotateAngle = lex.getFloat();
-                       }
-               } else if (token == "size") {
-                       if (lex.next())
-                               params_.lyxwidth = LyXLength(lex.getString()+"pt");
-                       if (lex.next())
-                               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;
-                                   break;
-                               case 2: params_.display = InsetGraphicsParams::GRAYSCALE;
-                                   break;
-                               case 3: params_.display = InsetGraphicsParams::COLOR;
-                                   break;
-                               }
-               } else if (token == "subfigure") {
-                       params_.subcaption = true;
-               } else if (token == "width") {
-                   if (lex.next()) {
-                       int i = lex.getInteger();
-                       if (lex.next()) {
-                           if (i == 5) {
-                               params_.scale = lex.getInteger();
-                               params_.size_type = InsetGraphicsParams::SCALE;
-                           } else {
-                               params_.width = LyXLength(lex.getString()+oldUnits[i]);
-                               params_.size_type = InsetGraphicsParams::WH;
-                           }
-                       }
-                   }
-               } else if (token == "height") {
-                   if (lex.next()) {
-                       int i = lex.getInteger();
-                       if (lex.next()) {
-                           params_.height = LyXLength(lex.getString()+oldUnits[i]);
-                           params_.size_type = InsetGraphicsParams::WH;
-                       }
-                   }
-               }
-       }
-}
 
 string const InsetGraphics::createLatexOptions() const
 {
@@ -531,25 +481,27 @@ string const InsetGraphics::createLatexOptions() const
        // before writing it to the output stream.
        ostringstream options;
        if (!params().bb.empty())
-           options << "  bb=" << strip(params().bb) << ",\n";
+           options << "  bb=" << rtrim(params().bb) << ",\n";
        if (params().draft)
            options << "  draft,\n";
        if (params().clip)
            options << "  clip,\n";
-       if (params().size_type == InsetGraphicsParams::WH) {
-           if (!params().width.zero())
-               options << "  width=" << params().width.asLatexString() << ",\n";
-           if (!params().height.zero())
-               options << "  height=" << params().height.asLatexString() << ",\n";
-       } else if (params().size_type == InsetGraphicsParams::SCALE) {
-           if (params().scale > 0)
-               options << "  scale=" << double(params().scale)/100.0 << ",\n";
+       if (!lyx::float_equal(params().scale, 0.0, 0.05)) {
+               if (!lyx::float_equal(params().scale, 100.0, 0.05))
+                       options << "  scale=" << params().scale / 100.0
+                               << ",\n";
+       } else {
+               if (!params().width.zero())
+                       options << "  width=" << params().width.asLatexString() << ",\n";
+               if (!params().height.zero())
+                       options << "  height=" << params().height.asLatexString() << ",\n";
+               if (params().keepAspectRatio)
+                       options << "  keepaspectratio,\n";
        }
-       if (params().keepAspectRatio)
-           options << "  keepaspectratio,\n";
-       // Make sure it's not very close to zero, a float can be effectively
-       // zero but not exactly zero.
-       if (!lyx::float_equal(params().rotateAngle, 0, 0.001) && params().rotate) {
+
+       // Make sure rotation angle is not very close to zero;
+       // a float can be effectively zero but not exactly zero.
+       if (!lyx::float_equal(params().rotateAngle, 0, 0.001)) {
            options << "  angle=" << params().rotateAngle << ",\n";
            if (!params().rotateOrigin.empty()) {
                options << "  origin=" << params().rotateOrigin[0];
@@ -562,68 +514,89 @@ string const InsetGraphics::createLatexOptions() const
                options << ",\n";
            }
        }
+
        if (!params().special.empty())
            options << params().special << ",\n";
-       string opts = options.str().c_str();
-       return opts.substr(0,opts.size()-2);    // delete last ",\n"
-}
 
-namespace {
-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).
-       if (lyxrc.pdf_mode) {
-               lyxerr[Debug::GRAPHICS] << "findTargetFormat: PDF mode\n";
-               if (contains(suffix,"ps") || suffix == "pdf")
-                       return "pdf";
-               else if (suffix == "jpg")       // pdflatex can use jpeg
-                       return suffix;
-               else
-                       return "png";           // and also png
-       }
-       // If it's postscript, we always do eps.
-       lyxerr[Debug::GRAPHICS] << "findTargetFormat: PostScript mode\n";
-       if (suffix != "ps")                     // any other than ps
-           return "eps";                       // is changed to eps
-       else
-           return suffix;                      // let ps untouched
+       string opts = STRCONV(options.str());
+       // delete last ",\n"
+       return opts.substr(0, opts.size() - 2);
 }
 
-} // Anon. namespace
 
-
-string const InsetGraphics::prepareFile(Buffer const *buf) const
+string const InsetGraphics::prepareFile(Buffer const * buf) const
 {
        // LaTeX can cope if the graphics file doesn't exist, so just return the
        // filename.
        string const orig_file = params().filename;
-       string const orig_file_with_path =
+       string orig_file_with_path =
                MakeAbsPath(orig_file, buf->filePath());
-       lyxerr[Debug::GRAPHICS] << "prepareFile: " << orig_file << endl
-                   << "  with path: " << orig_file_with_path << endl;
+       lyxerr[Debug::GRAPHICS] << "[InsetGraphics::prepareFile] orig_file = "
+                   << orig_file << "\n\twith path: "
+                   << orig_file_with_path << endl;
 
        if (!IsFileReadable(orig_file_with_path))
                return orig_file;
 
-       // If the file is compressed and we have specified that it should not be
-       // uncompressed, then just return its name and let LaTeX do the rest!
        bool const zipped = zippedFile(orig_file_with_path);
-       if (zipped)
-               lyxerr[Debug::GRAPHICS] << "it's a zipped file\n";
+
+       // If the file is compressed and we have specified that it
+       // should not be uncompressed, then just return its name and
+       // let LaTeX do the rest!
        if (zipped && params().noUnzip) {
-               lyxerr[Debug::GRAPHICS] << "pass file unzipped to LaTeX\n";
-               return orig_file;
+               lyxerr[Debug::GRAPHICS]
+                       << "\tpass zipped file to LaTeX but with full path.\n";
+               // LaTeX needs an absolue path, otherwise the
+               // coresponding *.eps.bb file isn't found
+               return orig_file_with_path;
        }
 
-       // "nice" means that the buffer is exported to LaTeX format but not
-       //        run through the LaTeX compiler.
-       // if (nice)
-       //     No conversion of the graphics file is needed.
-       //     Return the original filename without any extension.
-       if (buf->niceFile)
-               return RemoveExtension(orig_file);
+       // Ascertain whether the file has changed.
+       unsigned long const new_checksum = cache_->loader.checksum();
+       bool const file_has_changed = cache_->checksum != new_checksum;
+       if (file_has_changed)
+               cache_->checksum = new_checksum;
+
+       // temp_file will contain the file for LaTeX to act on if, for example,
+       // we move it to a temp dir or uncompress it.
+       string temp_file = orig_file;
+
+       if (zipped) {
+               // Uncompress the file if necessary.
+               // If it has been uncompressed in a previous call to
+               // prepareFile, do nothing.
+               temp_file = MakeAbsPath(OnlyFilename(temp_file), buf->tmppath);
+               lyxerr[Debug::GRAPHICS]
+                       << "\ttemp_file: " << temp_file << endl;
+               if (file_has_changed || !IsFileReadable(temp_file)) {
+                       bool const success = lyx::copy(orig_file_with_path,
+                                                      temp_file);
+                       lyxerr[Debug::GRAPHICS]
+                               << "\tCopying zipped file from "
+                               << orig_file_with_path << " to " << temp_file
+                               << (success ? " succeeded\n" : " failed\n");
+               } else
+                       lyxerr[Debug::GRAPHICS]
+                               << "\tzipped file " << temp_file
+                               << " exists! Maybe no tempdir ...\n";
+               orig_file_with_path = unzipFile(temp_file);
+               lyxerr[Debug::GRAPHICS]
+                       << "\tunzipped to " << orig_file_with_path << endl;
+       }
+
+       string const from = getExtFromContents(orig_file_with_path);
+       string const to   = findTargetFormat(from);
+       lyxerr[Debug::GRAPHICS]
+               << "\t we have: from " << from << " to " << to << '\n';
+
+       if (from == to && !lyxrc.use_tempdir) {
+               // No conversion is needed. LaTeX can handle the
+               // graphic file as is.
+               // This is true even if the orig_file is compressed.
+               if (formats.getFormat(to)->extension() == GetExtension(orig_file))
+                       return RemoveExtension(orig_file_with_path);
+               return orig_file_with_path;
+       }
 
        // We're going to be running the exported buffer through the LaTeX
        // compiler, so must ensure that LaTeX can cope with the graphics
@@ -636,90 +609,93 @@ string const InsetGraphics::prepareFile(Buffer const *buf) const
        // we manipulate the original file "any.dir/file.ext"
        // to "any_dir_file.ext"! changing the dots in the
        // dirname is important for the use of ChangeExtension
-       string temp_file(orig_file);
+       lyxerr[Debug::GRAPHICS]
+               << "\tthe orig file is: " << orig_file_with_path << endl;
+
        if (lyxrc.use_tempdir) {
-               string const ext_tmp = GetExtension(orig_file);
+               string const ext_tmp = GetExtension(orig_file_with_path);
                // without ext and /
                temp_file = subst(
-                       ChangeExtension(temp_file, string()), "/", "_");
-               // without . and again with ext
+                       ChangeExtension(orig_file_with_path, string()), "/", "_");
+               // without dots and again with ext
                temp_file = ChangeExtension(
                        subst(temp_file, ".", "_"), ext_tmp);
                // now we have any_dir_file.ext
                temp_file = MakeAbsPath(temp_file, buf->tmppath);
-       }
-       lyxerr[Debug::GRAPHICS]
-               << "InsetGraphics::prepareFile. The temp file is: "
-               << temp_file << endl;
-
-       // If we are using a temp dir, then copy the file into it.
-       if (lyxrc.use_tempdir && !IsFileReadable(temp_file)) {
-               bool const success = lyx::copy(orig_file_with_path, temp_file);
                lyxerr[Debug::GRAPHICS]
-                       << "InsetGraphics::prepareFile. Copying from "
-                       << orig_file << " to " << temp_file
-                       << (success ? " succeeded\n" : " failed\n");
-               if (!success) {
-                       Alert::alert(_("Cannot copy file"), orig_file,
-                                       _("into tempdir"));
-                       return orig_file;
-               }
-       }
-
-       // Uncompress the file if necessary. If it has been uncompressed in
-       // a previous call to prepareFile, do nothing.
-       if (zipped) {
-               // What we want to end up with:
-               string const temp_file_unzipped =
-                       ChangeExtension(temp_file, string());
-
-               if (!IsFileReadable(temp_file_unzipped)) {
-                       // unzipFile generates a random filename, so move this
-                       // file where we want it to go.
-                       string const tmp = unzipFile(temp_file);
-                       lyx::copy(tmp, temp_file_unzipped);
-                       lyx::unlink(tmp);
+                       << "\tchanged to: " << temp_file << endl;
 
+               // if the file doen't exists, copy it into the tempdir
+               if (file_has_changed || !IsFileReadable(temp_file)) {
+                       bool const success = lyx::copy(orig_file_with_path, temp_file);
                        lyxerr[Debug::GRAPHICS]
-                               << "InsetGraphics::prepareFile. Unzipped to "
-                               << temp_file_unzipped << endl;
+                               << "\tcopying from " << orig_file_with_path << " to "
+                               << temp_file
+                               << (success ? " succeeded\n" : " failed\n");
+                       if (!success) {
+#if USE_BOOST_FORMAT
+                               boost::format fmt(_("Could not copy the file\n%1$s\ninto the temporary directory."));
+                               fmt % orig_file_with_path;
+                               string str = fmt.str();
+#else
+                               string str = _("Could not copy the file\n");
+                               str += orig_file_with_path;
+                               str += _("\ninto the temporary directory.");
+#endif
+                               Alert::error(_("Graphics display failed"), str);
+                               return orig_file;
+                       }
                }
 
-               // We have an uncompressed file where we expect it,
-               // so rename temp_file and continue.
-               temp_file = temp_file_unzipped;
-       }
-
-       // Ascertain the graphics format that LaTeX requires.
-       // Make again an absolute path, maybe that we have no
-       // tempdir. Than temp_file=orig_file
-       string const from = lyxrc.use_tempdir ?
-               getExtFromContents(temp_file) :
-               getExtFromContents(MakeAbsPath(temp_file, buf->filePath()));
-       string const to   = findTargetFormat(from);
-
-       // No conversion is needed. LaTeX can handle the graphics file as it is.
-       // This is true even if the orig_file is compressed.
-       if (from == to) {
-               return orig_file;
+               if (from == to) {
+                       // No conversion is needed. LaTeX can handle the
+                       // graphic file as is.
+                       if (formats.getFormat(to)->extension() == GetExtension(orig_file))
+                               return RemoveExtension(temp_file);
+                       return temp_file;
+               }
        }
 
        string const outfile_base = RemoveExtension(temp_file);
-
        lyxerr[Debug::GRAPHICS]
-               << "InsetGraphics::prepareFile. The original file is "
-               << orig_file << "\n"
-               << "A copy has been made and convert is to be called with:\n"
+               << "\tThe original file is " << orig_file << "\n"
+               << "\tA copy has been made and convert is to be called with:\n"
                << "\tfile to convert = " << temp_file << '\n'
                << "\toutfile_base = " << outfile_base << '\n'
                << "\t from " << from << " to " << to << '\n';
 
-       converters.convert(buf, temp_file, outfile_base, from, to);
+       // if no special converter defined, than we take the default one
+       // from ImageMagic: convert from:inname.from to:outname.to
+       if (!converters.convert(buf, temp_file, outfile_base, from, to)) {
+               string const command =
+                       LibFileSearch("scripts", "convertDefault.sh") +
+                               ' ' + from + ':' + temp_file + ' ' +
+                               to + ':' + outfile_base + '.' + to;
+               lyxerr[Debug::GRAPHICS]
+                       << "No converter defined! I use convertDefault.sh:\n\t"
+                       << command << endl;
+               Systemcall one;
+               one.startscript(Systemcall::Wait, command);
+               if (!IsFileReadable(ChangeExtension(outfile_base, to))) {
+#if USE_BOOST_FORMAT
+                       boost::format fmt(_("No information for converting %1$s format files to %1$s.\n"
+                               "Try defining a convertor in the preferences."));
+                       fmt % from % to;
+                       string str = fmt.str();
+#else
+                       string str = _("No information for converting ");
+                       str += from + _(" format files to ") + to;
+                       str += _(".\nTry defining a convertor in the preferences.");
+#endif
+                       Alert::error(_("Could not convert image"), str);
+               }
+       }
+
        return RemoveExtension(temp_file);
 }
 
 
-int InsetGraphics::latex(Buffer const *buf, ostream & os,
+int InsetGraphics::latex(Buffer const * buf, ostream & os,
                         bool /*fragile*/, bool/*fs*/) const
 {
        // If there is no file specified or not existing,
@@ -730,20 +706,20 @@ int InsetGraphics::latex(Buffer const *buf, ostream & os,
 
        // A missing (e)ps-extension is no problem for LaTeX, so
        // we have to test three different cases
-       string const file_(MakeAbsPath(params().filename, buf->filePath()));
+       string const file_ = MakeAbsPath(params().filename, buf->filePath());
        bool const file_exists =
                !file_.empty() &&
                (IsFileReadable(file_) ||               // original
                 IsFileReadable(file_ + ".eps") ||      // original.eps
                 IsFileReadable(file_ + ".ps"));        // original.ps
        string const message = file_exists ?
-               string() : string("bb = 0 0 200 100, draft, type=eps]");
+               string() : string("bb = 0 0 200 100, draft, type=eps");
        // if !message.empty() than there was no existing file
        // "filename(.(e)ps)" found. In this case LaTeX
        // draws only a rectangle with the above bb and the
        // not found filename in it.
        lyxerr[Debug::GRAPHICS]
-               << "InsetGraphics::latex. Message = \"" << message << '\"' << endl;
+               << "\tMessage = \"" << message << '\"' << endl;
 
        // These variables collect all the latex code that should be before and
        // after the actual includegraphics command.
@@ -759,33 +735,35 @@ int InsetGraphics::latex(Buffer const *buf, ostream & os,
 
        // Write the options if there are any.
        string const opts = createLatexOptions();
-       lyxerr[Debug::GRAPHICS]
-               << "InsetGraphics::latex. Opts = " << opts << endl;
+       lyxerr[Debug::GRAPHICS] << "\tOpts = " << opts << endl;
 
        if (!opts.empty() && !message.empty())
-               before += ("[" + opts + ',' + message);
-       else if (!message.empty())
-               before += ('[' + message);
-       else if (!opts.empty())
-               before += ("[" + opts + ']');
+               before += ("[%\n" + opts + ',' + message + ']');
+       else if (!opts.empty() || !message.empty())
+               before += ("[%\n" + opts + message + ']');
 
        lyxerr[Debug::GRAPHICS]
-               << "InsetGraphics::latex. Before = " << before
-               << "\nafter = " << after << endl;
+               << "\tBefore = " << before
+               << "\n\tafter = " << after << endl;
+
+
+       // "nice" means that the buffer is exported to LaTeX format but not
+       //        run through the LaTeX compiler.
+       if (buf->niceFile) {
+               os << before <<'{' << params().filename << '}' << after;
+               return 1;
+       }
 
        // 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 = message.empty() ?
-               (before + '{' + prepareFile(buf) + '}' + after) :
+               (before + '{' + os::external_path(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);
-
-       return newlines;
+       return int(lyx::count(latex_str.begin(), latex_str.end(),'\n') + 1);
 }
 
 
@@ -796,7 +774,7 @@ int InsetGraphics::ascii(Buffer const *, ostream & os, int) const
        // 1. Convert file to ascii using gifscii
        // 2. Read ascii output file and add it to the output stream.
        // at least we send the filename
-       os << '<' << _("Graphic file:") << params().filename << ">\n";
+       os << '<' << bformat(_("Graphics file: %1$s"), params().filename) << ">\n";
        return 0;
 }
 
@@ -811,7 +789,8 @@ int InsetGraphics::linuxdoc(Buffer const *, ostream &) const
 // For explanation on inserting graphics into DocBook checkout:
 // http://linuxdoc.org/LDP/LDP-Author-Guide/inserting-pictures.html
 // See also the docbook guide at http://www.docbook.org/
-int InsetGraphics::docbook(Buffer const *, ostream & os) const
+int InsetGraphics::docbook(Buffer const *, ostream & os,
+                          bool /*mixcont*/) 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
@@ -825,7 +804,7 @@ void InsetGraphics::validate(LaTeXFeatures & features) const
 {
        // If we have no image, we should not require anything.
        if (params().filename.empty())
-               return ;
+               return;
 
        features.includeFile(graphic_label, RemoveExtension(params().filename));
 
@@ -836,19 +815,25 @@ void InsetGraphics::validate(LaTeXFeatures & features) const
 }
 
 
+void InsetGraphics::statusChanged()
+{
+       if (!cache_->view.expired())
+               cache_->view.lock()->updateInset(this);
+}
+
+
 bool InsetGraphics::setParams(InsetGraphicsParams const & p,
                              string const & filepath)
 {
        // If nothing is changed, just return and say so.
-       if (params() == p && !p.filename.empty()) {
+       if (params() == p && !p.filename.empty())
                return false;
-       }
 
        // Copy the new parameters.
        params_ = p;
 
        // Update the inset with the new parameters.
-       updateInset(filepath);
+       cache_->update(MakeAbsPath(params().filename, filepath));
 
        // We have changed data, report it.
        return true;
@@ -861,7 +846,52 @@ InsetGraphicsParams const & InsetGraphics::params() const
 }
 
 
-Inset * InsetGraphics::clone(Buffer const & buffer, bool same_id) const
+string const InsetGraphicsMailer::name_("graphics");
+
+InsetGraphicsMailer::InsetGraphicsMailer(InsetGraphics & inset)
+       : inset_(inset)
+{}
+
+
+string const InsetGraphicsMailer::inset2string() const
 {
-       return new InsetGraphics(*this, buffer.filePath(), same_id);
+       return params2string(inset_.params());
+}
+
+
+void InsetGraphicsMailer::string2params(string const & in,
+                                       InsetGraphicsParams & params)
+{
+       params = InsetGraphicsParams();
+
+       if (in.empty())
+               return;
+
+       istringstream data(STRCONV(in));
+       LyXLex lex(0,0);
+       lex.setStream(data);
+
+       if (lex.isOK()) {
+               lex.next();
+               string const token = lex.getString();
+               if (token != name_)
+                       return;
+       }
+
+       if (lex.isOK()) {
+               InsetGraphics inset;
+               inset.readInsetGraphics(lex);
+               params = inset.params();
+       }
+}
+
+
+string const
+InsetGraphicsMailer::params2string(InsetGraphicsParams const & params)
+{
+       ostringstream data;
+       data << name_ << ' ';
+       params.Write(data);
+       data << "\\end_inset\n";
+       return STRCONV(data.str());
 }