]> git.lyx.org Git - lyx.git/blobdiff - src/insets/insetgraphics.C
use the lyxdot replacement mechanism also in insetexternal
[lyx.git] / src / insets / insetgraphics.C
index 53a8a7f81733dc2da753615275838bb96d3f813a..44a37fd723496af1c9d4f2acaa2690984183e62f 100644 (file)
@@ -58,42 +58,51 @@ TODO
 #include "cursor.h"
 #include "debug.h"
 #include "dispatchresult.h"
+#include "exporter.h"
 #include "format.h"
 #include "funcrequest.h"
+#include "FuncStatus.h"
 #include "gettext.h"
 #include "LaTeXFeatures.h"
 #include "lyx_main.h"
+#include "lyxlength.h"
 #include "lyxlex.h"
-#include "lyxrc.h"
 #include "metricsinfo.h"
+#include "mover.h"
 #include "outputparams.h"
+#include "sgml.h"
 
 #include "frontends/Alert.h"
 #include "frontends/LyXView.h"
 
+#include "support/convert.h"
 #include "support/filetools.h"
 #include "support/lyxalgo.h" // lyx::count
-#include "support/lyxlib.h" // float_equal
+#include "support/lyxlib.h" // lyx::sum
+#include "support/lstrings.h"
 #include "support/os.h"
 #include "support/systemcall.h"
-#include "support/tostr.h"
-#include "support/std_sstream.h"
 
 #include <boost/bind.hpp>
 #include <boost/tuple/tuple.hpp>
 
+#include <sstream>
+
 namespace support = lyx::support;
+
 using lyx::support::AbsolutePath;
 using lyx::support::bformat;
 using lyx::support::ChangeExtension;
+using lyx::support::compare_timestamps;
 using lyx::support::contains;
 using lyx::support::FileName;
 using lyx::support::float_equal;
 using lyx::support::GetExtension;
-using lyx::support::getExtFromContents;
 using lyx::support::IsFileReadable;
-using lyx::support::LibFileSearch;
+using lyx::support::latex_path;
+using lyx::support::OnlyFilename;
 using lyx::support::rtrim;
+using lyx::support::subst;
 using lyx::support::Systemcall;
 using lyx::support::unzipFile;
 using lyx::support::unzippedFileName;
@@ -110,10 +119,6 @@ using std::ostringstream;
 
 namespace {
 
-///////////////////////////////////////////////////////////////////////////
-int const VersionNumber = 1;
-///////////////////////////////////////////////////////////////////////////
-
 // This function is a utility function
 // ... that should be with ChangeExtension ...
 inline
@@ -123,54 +128,49 @@ string const RemoveExtension(string const & filename)
 }
 
 
-string const uniqueID()
-{
-       static unsigned int seed = 1000;
-       return "graph" + tostr(++seed);
-}
-
-
-string findTargetFormat(string const & suffix, OutputParams const & runparams)
+string findTargetFormat(string const & format, OutputParams const & runparams)
 {
-       // Are we using latex or pdflatex).
+       // Are we using latex or pdflatex?
        if (runparams.flavor == OutputParams::PDFLATEX) {
                lyxerr[Debug::GRAPHICS] << "findTargetFormat: PDF mode" << endl;
-               if (contains(suffix, "ps") || suffix == "pdf")
+               // Convert postscript to pdf
+               if (format == "eps" || format == "ps")
                        return "pdf";
-               if (suffix == "jpg")    // pdflatex can use jpeg
-                       return suffix;
-               return "png";         // and also png
+               // pdflatex can use jpeg, png and pdf directly
+               if (format == "jpg" || format == "pdf")
+                       return format;
+               // Convert everything else to png
+               return "png";
        }
        // If it's postscript, we always do eps.
        lyxerr[Debug::GRAPHICS] << "findTargetFormat: PostScript mode" << endl;
-       if (suffix != "ps")     // any other than ps
-               return "eps";         // is changed to eps
-       return suffix;          // let ps untouched
+       if (format != "ps")
+               // any other than ps is changed to eps
+               return "eps";
+       // let ps untouched
+       return format;
 }
 
 } // namespace anon
 
 
 InsetGraphics::InsetGraphics()
-       : graphic_label(uniqueID()),
-         graphic_(new RenderGraphic)
-{
-       graphic_->connect(boost::bind(&InsetGraphics::statusChanged, this));
-}
+       : graphic_label(sgml::uniqueID("graph")),
+         graphic_(new RenderGraphic(this))
+{}
 
 
 InsetGraphics::InsetGraphics(InsetGraphics const & ig)
        : InsetOld(ig),
          boost::signals::trackable(),
-         graphic_label(uniqueID()),
-         graphic_(new RenderGraphic(*ig.graphic_))
+         graphic_label(sgml::uniqueID("graph")),
+         graphic_(new RenderGraphic(*ig.graphic_, this))
 {
-       graphic_->connect(boost::bind(&InsetGraphics::statusChanged, this));
        setParams(ig.params());
 }
 
 
-auto_ptr<InsetBase> InsetGraphics::clone() const
+auto_ptr<InsetBase> InsetGraphics::doClone() const
 {
        return auto_ptr<InsetBase>(new InsetGraphics(*this));
 }
@@ -182,23 +182,25 @@ InsetGraphics::~InsetGraphics()
 }
 
 
-void InsetGraphics::statusChanged() const
-{
-       LyX::cref().updateInset(this);
-}
-
-
-void InsetGraphics::priv_dispatch(LCursor & cur, FuncRequest & cmd)
+void InsetGraphics::doDispatch(LCursor & cur, FuncRequest & cmd)
 {
        switch (cmd.action) {
-       case LFUN_INSET_MODIFY: {
+       case LFUN_GRAPHICS_EDIT: {
                Buffer const & buffer = *cur.bv().buffer();
                InsetGraphicsParams p;
                InsetGraphicsMailer::string2params(cmd.argument, buffer, p);
-               if (!p.filename.empty()) {
+               editGraphics(p, buffer);
+               break;
+       }
+
+       case LFUN_INSET_MODIFY: {
+               Buffer const & buffer = cur.buffer();
+               InsetGraphicsParams p;
+               InsetGraphicsMailer::string2params(cmd.argument, buffer, p);
+               if (!p.filename.empty())
                        setParams(p);
-                       cur.bv().update();
-               }
+               else
+                       cur.noUpdate();
                break;
        }
 
@@ -211,12 +213,28 @@ void InsetGraphics::priv_dispatch(LCursor & cur, FuncRequest & cmd)
                break;
 
        default:
-               InsetOld::priv_dispatch(cur, cmd);
+               InsetBase::doDispatch(cur, cmd);
                break;
        }
 }
 
 
+bool InsetGraphics::getStatus(LCursor & cur, FuncRequest const & cmd,
+               FuncStatus & flag) const
+{
+       switch (cmd.action) {
+       case LFUN_GRAPHICS_EDIT:
+       case LFUN_INSET_MODIFY:
+       case LFUN_INSET_DIALOG_UPDATE:
+               flag.enabled(true);
+               return true;
+
+       default:
+               return InsetBase::getStatus(cur, cmd, flag);
+       }
+}
+
+
 void InsetGraphics::edit(LCursor & cur, bool)
 {
        InsetGraphicsMailer(*this).showDialog(&cur.bv());
@@ -237,7 +255,7 @@ void InsetGraphics::draw(PainterInfo & pi, int x, int y) const
 }
 
 
-InsetOld::EDITABLE InsetGraphics::editable() const
+InsetBase::EDITABLE InsetGraphics::editable() const
 {
        return IS_EDITABLE;
 }
@@ -294,46 +312,142 @@ string const InsetGraphics::createLatexOptions() const
        // before writing it to the output stream.
        ostringstream options;
        if (!params().bb.empty())
-           options << "  bb=" << rtrim(params().bb) << ",\n";
+           options << "bb=" << rtrim(params().bb) << ',';
        if (params().draft)
-           options << "  draft,\n";
+           options << "draft,";
        if (params().clip)
-           options << "  clip,\n";
-       if (!float_equal(params().scale, 0.0, 0.05)) {
-               if (!float_equal(params().scale, 100.0, 0.05))
-                       options << "  scale=" << params().scale / 100.0
-                               << ",\n";
+           options << "clip,";
+       double const scl = convert<double>(params().scale);
+       if (!params().scale.empty() && !float_equal(scl, 0.0, 0.05)) {
+               if (!float_equal(scl, 100.0, 0.05))
+                       options << "scale=" << scl / 100.0 << ',';
        } else {
                if (!params().width.zero())
-                       options << "  width=" << params().width.asLatexString() << ",\n";
+                       options << "width=" << params().width.asLatexString() << ',';
                if (!params().height.zero())
-                       options << "  height=" << params().height.asLatexString() << ",\n";
+                       options << "height=" << params().height.asLatexString() << ',';
                if (params().keepAspectRatio)
-                       options << "  keepaspectratio,\n";
+                       options << "keepaspectratio,";
        }
 
        // Make sure rotation angle is not very close to zero;
        // a float can be effectively zero but not exactly zero.
-       if (!float_equal(params().rotateAngle, 0, 0.001)) {
-           options << "  angle=" << params().rotateAngle << ",\n";
+       if (!params().rotateAngle.empty()
+               && !float_equal(convert<double>(params().rotateAngle), 0.0, 0.001)) {
+           options << "angle=" << params().rotateAngle << ',';
            if (!params().rotateOrigin.empty()) {
-               options << "  origin=" << params().rotateOrigin[0];
+               options << "origin=" << params().rotateOrigin[0];
                if (contains(params().rotateOrigin,"Top"))
                    options << 't';
                else if (contains(params().rotateOrigin,"Bottom"))
                    options << 'b';
                else if (contains(params().rotateOrigin,"Baseline"))
                    options << 'B';
-               options << ",\n";
+               options << ',';
            }
        }
 
        if (!params().special.empty())
-           options << params().special << ",\n";
+           options << params().special << ',';
 
        string opts = options.str();
-       // delete last ",\n"
-       return opts.substr(0, opts.size() - 2);
+       // delete last ','
+       return opts.substr(0, opts.size() - 1);
+}
+
+
+string const InsetGraphics::toDocbookLength(LyXLength const & len) const
+{
+       ostringstream result;
+       switch (len.unit()) {
+               case LyXLength::SP: // Scaled point (65536sp = 1pt) TeX's smallest unit.
+                       result << len.value() * 65536.0 * 72 / 72.27 << "pt";
+                       break;
+               case LyXLength::PT: // Point = 1/72.27in = 0.351mm
+                       result << len.value() * 72 / 72.27 << "pt";
+                       break;
+               case LyXLength::BP: // Big point (72bp = 1in), also PostScript point
+                       result << len.value() << "pt";
+                       break;
+               case LyXLength::DD: // Didot point = 1/72 of a French inch, = 0.376mm
+                       result << len.value() * 0.376 << "mm";
+                       break;
+               case LyXLength::MM: // Millimeter = 2.845pt
+                       result << len.value() << "mm";
+                       break;
+               case LyXLength::PC: // Pica = 12pt = 4.218mm
+                       result << len.value() << "pc";
+                       break;
+               case LyXLength::CC: // Cicero = 12dd = 4.531mm
+                       result << len.value() * 4.531 << "mm";
+                       break;
+               case LyXLength::CM: // Centimeter = 10mm = 2.371pc
+                       result << len.value() << "cm";
+                       break;
+               case LyXLength::IN: // Inch = 25.4mm = 72.27pt = 6.022pc
+                       result << len.value() << "in";
+                       break;
+               case LyXLength::EX: // Height of a small "x" for the current font.
+                       // Obviously we have to compromise here. Any better ratio than 1.5 ?
+                       result << len.value() / 1.5 << "em";
+                       break;
+               case LyXLength::EM: // Width of capital "M" in current font.
+                       result << len.value() << "em";
+                       break;
+               case LyXLength::MU: // Math unit (18mu = 1em) for positioning in math mode
+                       result << len.value() * 18 << "em";
+                       break;
+               case LyXLength::PTW: // Percent of TextWidth
+               case LyXLength::PCW: // Percent of ColumnWidth
+               case LyXLength::PPW: // Percent of PageWidth
+               case LyXLength::PLW: // Percent of LineWidth
+               case LyXLength::PTH: // Percent of TextHeight
+               case LyXLength::PPH: // Percent of Paper
+                       // Sigh, this will go wrong.
+                       result << len.value() << "%";
+                       break;
+               default:
+                       result << len.asString();
+                       break;
+       }
+       return result.str();
+}
+
+string const InsetGraphics::createDocBookAttributes() const
+{
+       // Calculate the options part of the command, we must do it to a string
+       // stream since we copied the code from createLatexParams() ;-)
+
+       // FIXME: av: need to translate spec -> Docbook XSL spec (http://www.sagehill.net/docbookxsl/ImageSizing.html)
+       // Right now it only works with my version of db2latex :-)
+
+       ostringstream options;
+       double const scl = convert<double>(params().scale);
+       if (!params().scale.empty() && !float_equal(scl, 0.0, 0.05)) {
+               if (!float_equal(scl, 100.0, 0.05))
+                       options << " scale=\""
+                               << static_cast<int>( (scl) + 0.5 )
+                               << "\" ";
+       } else {
+               if (!params().width.zero()) {
+                       options << " width=\"" << toDocbookLength(params().width)  << "\" ";
+               }
+               if (!params().height.zero()) {
+                       options << " depth=\"" << toDocbookLength(params().height)  << "\" ";
+               }
+               if (params().keepAspectRatio) {
+                       // This will be irrelevant unless both width and height are set
+                       options << "scalefit=\"1\" ";
+               }
+       }
+
+
+       if (!params().special.empty())
+           options << params().special << " ";
+
+       string opts = options.str();
+       // trailing blanks are ok ...
+       return opts;
 }
 
 
@@ -348,26 +462,10 @@ enum CopyStatus {
 
 
 std::pair<CopyStatus, string> const
-copyToDirIfNeeded(string const & file_in, string const & dir)
+copyFileIfNeeded(string const & file_in, string const & file_out)
 {
-       using support::rtrim;
-
        BOOST_ASSERT(AbsolutePath(file_in));
-
-       string const only_path = support::OnlyPath(file_in);
-       if (rtrim(support::OnlyPath(file_in) , "/") == rtrim(dir, "/"))
-               return std::make_pair(IDENTICAL_PATHS, file_in);
-
-       string mangled;
-       if (support::zippedFile(file_in)) {
-               string const ext = GetExtension(file_in);
-               string const unzipped = support::unzippedFileName(file_in);
-               mangled = FileName(unzipped).mangledFilename();
-               mangled += "." + ext;
-       } else
-               mangled = FileName(file_in).mangledFilename();
-
-       string const file_out = support::MakeAbsPath(mangled, dir);
+       BOOST_ASSERT(AbsolutePath(file_out));
 
        unsigned long const checksum_in  = support::sum(file_in);
        unsigned long const checksum_out = support::sum(file_out);
@@ -376,7 +474,8 @@ copyToDirIfNeeded(string const & file_in, string const & dir)
                // Nothing to do...
                return std::make_pair(IDENTICAL_CONTENTS, file_out);
 
-       bool const success = support::copy(file_in, file_out);
+       Mover const & mover = movers(formats.getFormatFromFile(file_in));
+       bool const success = mover.copy(file_in, file_out);
        if (!success) {
                lyxerr[Debug::GRAPHICS]
                        << support::bformat(_("Could not copy the file\n%1$s\n"
@@ -390,13 +489,67 @@ copyToDirIfNeeded(string const & file_in, string const & dir)
 }
 
 
+std::pair<CopyStatus, string> const
+copyToDirIfNeeded(string const & file_in, string const & dir, bool zipped)
+{
+       using support::rtrim;
+
+       BOOST_ASSERT(AbsolutePath(file_in));
+
+       string const only_path = support::OnlyPath(file_in);
+       if (rtrim(support::OnlyPath(file_in) , "/") == rtrim(dir, "/"))
+               return std::make_pair(IDENTICAL_PATHS, file_in);
+
+       string mangled = FileName(file_in).mangledFilename();
+       if (zipped) {
+               // We need to change _eps.gz to .eps.gz. The mangled name is
+               // still unique because of the counter in mangledFilename().
+               // We can't just call mangledFilename() with the zip
+               // extension removed, because base.eps and base.eps.gz may
+               // have different content but would get the same mangled
+               // name in this case.
+               string const base = RemoveExtension(unzippedFileName(file_in));
+               string::size_type const ext_len = file_in.length() - base.length();
+               mangled[mangled.length() - ext_len] = '.';
+       }
+       string const file_out = support::MakeAbsPath(mangled, dir);
+
+       return copyFileIfNeeded(file_in, file_out);
+}
+
+
+string const stripExtensionIfPossible(string const & file)
+{
+       // Remove the extension so the LaTeX will use whatever
+       // is appropriate (when there are several versions in
+       // different formats)
+       // This works only if the filename contains no dots besides
+       // the just removed one. We can fool here by replacing all
+       // dots with a macro whose definition is just a dot ;-)
+       // The automatic format selection does not work if the file
+       // name is escaped.
+       string const latex_name = latex_path(file,
+                                            lyx::support::EXCLUDE_EXTENSION);
+       if (contains(latex_name, '"'))
+               return latex_name;
+       return latex_path(RemoveExtension(file),
+                         lyx::support::PROTECT_EXTENSION,
+                         lyx::support::ESCAPE_DOTS);
+}
+
+
 string const stripExtensionIfPossible(string const & file, string const & to)
 {
        // 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(file))
-               return RemoveExtension(file);
-       return file;
+       string const to_format = formats.getFormat(to)->extension();
+       string const file_format = GetExtension(file);
+       // for latex .ps == .eps
+       if (to_format == file_format ||
+           (to_format == "eps" && file_format ==  "ps") ||
+           (to_format ==  "ps" && file_format == "eps"))
+               return stripExtensionIfPossible(file);
+       return latex_path(file, lyx::support::EXCLUDE_EXTENSION);
 }
 
 } // namespace anon
@@ -405,58 +558,112 @@ string const stripExtensionIfPossible(string const & file, string const & to)
 string const InsetGraphics::prepareFile(Buffer const & buf,
                                        OutputParams const & runparams) const
 {
-       string orig_file = params().filename.absFilename();
-       string const rel_file = params().filename.relFilename(buf.filePath());
+       // The following code depends on non-empty filenames
+       if (params().filename.empty())
+               return string();
 
-       // LaTeX can cope if the graphics file doesn't exist, so just return the
-       // filename.
-       if (!IsFileReadable(orig_file)) {
-               lyxerr[Debug::GRAPHICS]
-                       << "InsetGraphics::prepareFile\n"
-                       << "No file '" << orig_file << "' can be found!" << endl;
-               return rel_file;
-       }
+       string const orig_file = params().filename.absFilename();
+       string const rel_file = params().filename.relFilename(buf.filePath());
 
        // 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 = params().filename.isZipped();
 
-       if (zipped && params().noUnzip) {
-               lyxerr[Debug::GRAPHICS]
-                       << "\tpass zipped file to LaTeX but with full path.\n";
-               // LaTeX needs an absolute path, otherwise the
-               // coresponding *.eps.bb file isn't found
-               return orig_file;
-       }
-
        // 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;
 
+       // The master buffer. This is useful when there are multiple levels
+       // of include files
+       Buffer const * m_buffer = buf.getMasterBuffer();
+
+       // Return the output name if the file does not exist.
+       // We are not going to change the extension or using the name of the
+       // temporary file, the code is already complicated enough.
+       if (!IsFileReadable(orig_file))
+               return params().filename.outputFilename(m_buffer->filePath());
+
        // We place all temporary files in the master buffer's temp dir.
        // This is possible because we use mangled file names.
        // This is necessary for DVI export.
-       string const temp_path = buf.getMasterBuffer()->temppath();
+       string const temp_path = m_buffer->temppath();
 
-       if (zipped) {
-               CopyStatus status;
-               boost::tie(status, temp_file) =
-                       copyToDirIfNeeded(orig_file, temp_path);
+       CopyStatus status;
+       boost::tie(status, temp_file) =
+                       copyToDirIfNeeded(orig_file, temp_path, zipped);
+
+       if (status == FAILURE)
+               return orig_file;
 
-               if (status == FAILURE)
-                       return orig_file;
+       // a relative filename should be relative to the master
+       // buffer.
+       // "nice" means that the buffer is exported to LaTeX format but not
+       //        run through the LaTeX compiler.
+       string const output_file = os::external_path(runparams.nice ?
+               params().filename.outputFilename(m_buffer->filePath()) :
+               OnlyFilename(temp_file));
+       string const source_file = runparams.nice ? orig_file : temp_file;
 
-               orig_file = unzippedFileName(temp_file);
-               if (!IsFileReadable(orig_file)) {
-                       unzipFile(temp_file);
+       if (zipped) {
+               if (params().noUnzip) {
+                       // We don't know whether latex can actually handle
+                       // this file, but we can't check, because that would
+                       // mean to unzip the file and thereby making the
+                       // noUnzip parameter meaningless.
                        lyxerr[Debug::GRAPHICS]
-                               << "\tunzipped to " << orig_file << endl;
+                               << "\tpass zipped file to LaTeX.\n";
+
+                       string const bb_orig_file = ChangeExtension(orig_file, "bb");
+                       if (runparams.nice) {
+                               runparams.exportdata->addExternalFile("latex",
+                                               bb_orig_file,
+                                               ChangeExtension(output_file, "bb"));
+                       } else {
+                               // LaTeX needs the bounding box file in the
+                               // tmp dir
+                               string bb_file = ChangeExtension(temp_file, "bb");
+                               boost::tie(status, bb_file) =
+                                       copyFileIfNeeded(bb_orig_file, bb_file);
+                               if (status == FAILURE)
+                                       return orig_file;
+                               runparams.exportdata->addExternalFile("latex",
+                                               bb_file);
+                       }
+                       runparams.exportdata->addExternalFile("latex",
+                                       source_file, output_file);
+                       runparams.exportdata->addExternalFile("dvi",
+                                       source_file, output_file);
+                       // We can't strip the extension, because we don't know
+                       // the unzipped file format
+                       return latex_path(output_file,
+                                         lyx::support::EXCLUDE_EXTENSION);
+               }
+
+               string const unzipped_temp_file = unzippedFileName(temp_file);
+               if (compare_timestamps(unzipped_temp_file, temp_file) > 0) {
+                       // temp_file has been unzipped already and
+                       // orig_file has not changed in the meantime.
+                       temp_file = unzipped_temp_file;
+                       lyxerr[Debug::GRAPHICS]
+                               << "\twas already unzipped to " << temp_file
+                               << endl;
+               } else {
+                       // unzipped_temp_file does not exist or is too old
+                       temp_file = unzipFile(temp_file);
+                       lyxerr[Debug::GRAPHICS]
+                               << "\tunzipped to " << temp_file << endl;
                }
        }
 
-       string const from = getExtFromContents(orig_file);
+       string const from = formats.getFormatFromFile(temp_file);
+       if (from.empty()) {
+               lyxerr[Debug::GRAPHICS]
+                       << "\tCould not get file format." << endl;
+               return orig_file;
+       }
        string const to   = findTargetFormat(from, runparams);
+       string const ext  = formats.extension(to);
        lyxerr[Debug::GRAPHICS]
                << "\t we have: from " << from << " to " << to << '\n';
 
@@ -467,71 +674,52 @@ string const InsetGraphics::prepareFile(Buffer const & buf,
        lyxerr[Debug::GRAPHICS]
                << "\tthe orig file is: " << orig_file << endl;
 
-       bool conversion_needed = true;
-       CopyStatus status;
-       boost::tie(status, temp_file) =
-                       copyToDirIfNeeded(orig_file, temp_path);
-
-       if (status == FAILURE)
-               return orig_file;
-       else if (status == IDENTICAL_CONTENTS)
-               conversion_needed = false;
-
-       if (from == to)
-               return stripExtensionIfPossible(temp_file, to);
+       if (from == to) {
+               // The extension of temp_file might be != ext!
+               runparams.exportdata->addExternalFile("latex", source_file,
+                                                     output_file);
+               runparams.exportdata->addExternalFile("dvi", source_file,
+                                                     output_file);
+               return stripExtensionIfPossible(output_file, to);
+       }
 
-       string const to_file_base = RemoveExtension(temp_file);
-       string const to_file = ChangeExtension(to_file_base, to);
+       string const to_file = ChangeExtension(temp_file, ext);
+       string const output_to_file = ChangeExtension(output_file, ext);
 
        // Do we need to perform the conversion?
        // Yes if to_file does not exist or if temp_file is newer than to_file
-       if (!conversion_needed ||
-           support::compare_timestamps(temp_file, to_file) < 0) {
+       if (compare_timestamps(temp_file, to_file) < 0) {
                lyxerr[Debug::GRAPHICS]
                        << bformat(_("No conversion of %1$s is needed after all"),
                                   rel_file)
                        << std::endl;
-               return to_file_base;
+               runparams.exportdata->addExternalFile("latex", to_file,
+                                                     output_to_file);
+               runparams.exportdata->addExternalFile("dvi", to_file,
+                                                     output_to_file);
+               return stripExtensionIfPossible(output_file);
        }
 
        lyxerr[Debug::GRAPHICS]
                << "\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'
-               << "\tto_file_base = " << to_file_base << '\n'
                << "\t from " << from << " to " << to << '\n';
 
-       // if no special converter defined, then we take the default one
-       // from ImageMagic: convert from:inname.from to:outname.to
-       if (!converters.convert(&buf, temp_file, to_file_base, from, to)) {
-               string const command =
-                       "sh " + LibFileSearch("scripts", "convertDefault.sh") +
-                               ' ' + from + ':' + temp_file + ' ' +
-                               to + ':' + to_file_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(to_file_base, to))) {
-                       string str = bformat(_("No information for converting %1$s "
-                               "format files to %2$s.\n"
-                               "Try defining a convertor in the preferences."), from, to);
-                       Alert::error(_("Could not convert image"), str);
-               }
+       if (converters.convert(&buf, temp_file, temp_file, from, to, true)) {
+               runparams.exportdata->addExternalFile("latex",
+                               to_file, output_to_file);
+               runparams.exportdata->addExternalFile("dvi",
+                               to_file, output_to_file);
        }
 
-       return to_file_base;
+       return stripExtensionIfPossible(output_file);
 }
 
 
 int InsetGraphics::latex(Buffer const & buf, ostream & os,
                         OutputParams const & runparams) const
 {
-       // The master buffer. This is useful when there are multiple levels
-       // of include files
-       Buffer const * m_buffer = buf.getMasterBuffer();
-
        // If there is no file specified or not existing,
        // just output a message about it in the latex output.
        lyxerr[Debug::GRAPHICS]
@@ -541,21 +729,12 @@ int InsetGraphics::latex(Buffer const & buf, ostream & os,
        string const relative_file =
                params().filename.relFilename(buf.filePath());
 
-       // A missing (e)ps-extension is no problem for LaTeX, so
-       // we have to test three different cases
-#ifdef WITH_WARNINGS
-#warning uh, but can our cache handle it ? no.
-#endif
        string const file_ = params().filename.absFilename();
-       bool const file_exists =
-               !file_.empty() &&
-               (IsFileReadable(file_) ||               // original
-                IsFileReadable(file_ + ".eps") ||      // original.eps
-                IsFileReadable(file_ + ".ps"));        // original.ps
+       bool const file_exists = !file_.empty() && IsFileReadable(file_);
        string const message = file_exists ?
                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
+       // if !message.empty() then there was no existing file
+       // "filename" found. In this case LaTeX
        // draws only a rectangle with the above bb and the
        // not found filename in it.
        lyxerr[Debug::GRAPHICS]
@@ -578,38 +757,26 @@ int InsetGraphics::latex(Buffer const & buf, ostream & os,
        lyxerr[Debug::GRAPHICS] << "\tOpts = " << opts << endl;
 
        if (!opts.empty() && !message.empty())
-               before += ("[%\n" + opts + ',' + message + ']');
+               before += ('[' + opts + ',' + message + ']');
        else if (!opts.empty() || !message.empty())
-               before += ("[%\n" + opts + message + ']');
+               before += ('[' + opts + message + ']');
 
        lyxerr[Debug::GRAPHICS]
                << "\tBefore = " << before
                << "\n\tafter = " << after << endl;
 
-
        string latex_str = before + '{';
-       // "nice" means that the buffer is exported to LaTeX format but not
-       //        run through the LaTeX compiler.
-       if (runparams.nice) {
-               // a relative filename should be relative to the master
-               // buffer.
-               latex_str += params().filename.outputFilename(m_buffer->filePath());
-       } else if (file_exists) {
-               // 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)
-               latex_str += os::external_path(prepareFile(buf, runparams));
-       } else
-               latex_str += relative_file + " not found!";
-
+       // Convert the file if necessary.
+       // Remove the extension so LaTeX will use whatever is appropriate
+       // (when there are several versions in different formats)
+       latex_str += prepareFile(buf, runparams);
        latex_str += '}' + after;
        os << latex_str;
 
        lyxerr[Debug::GRAPHICS] << "InsetGraphics::latex outputting:\n"
                                << latex_str << endl;
        // Return how many newlines we issued.
-       return int(lyx::count(latex_str.begin(), latex_str.end(),'\n') + 1);
+       return int(lyx::count(latex_str.begin(), latex_str.end(),'\n'));
 }
 
 
@@ -634,23 +801,67 @@ int InsetGraphics::linuxdoc(Buffer const & buf, ostream & os,
                                params().filename.relFilename(buf.filePath()):
                                params().filename.absFilename();
 
+       runparams.exportdata->addExternalFile("linuxdoc",
+                                             params().filename.absFilename());
        os << "<eps file=\"" << file_name << "\">\n";
        os << "<img src=\"" << file_name << "\">";
        return 0;
 }
 
 
+namespace {
+
+int writeImageObject(char * format, ostream& os, OutputParams const & runparams,
+                                        string const graphic_label, string const attributes)
+{
+               if (runparams.flavor != OutputParams::XML) {
+                       os << "<![ %output.print." << format << "; [" << std::endl;
+               }
+               os <<"<imageobject><imagedata fileref=\"&"
+                  << graphic_label << ";." << format << "\" " << attributes ;
+               if (runparams.flavor == OutputParams::XML) {
+                       os <<  " role=\"" << format << "\"/>" ;
+               }
+               else {
+                       os << " format=\"" << format << "\">" ;
+               }
+               os << "</imageobject>";
+               if (runparams.flavor != OutputParams::XML) {
+                       os << std::endl << "]]>" ;
+               }
+               return runparams.flavor == OutputParams::XML ? 0 : 2;
+}
+// end anonymous namespace
+}
+
+
 // For explanation on inserting graphics into DocBook checkout:
-// http://en.tldp.org/LDP/LDP-Author-Guide/inserting-pictures.html
+// http://en.tldp.org/LDP/LDP-Author-Guide/html/inserting-pictures.html
 // See also the docbook guide at http://www.docbook.org/
 int InsetGraphics::docbook(Buffer const &, ostream & os,
-                          OutputParams const &) const
+                          OutputParams const & runparams) 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
        // easier to use.
-       os << "<graphic fileref=\"&" << graphic_label << ";\">";
-       return 0;
+       if (runparams.flavor == OutputParams::XML) {
+               runparams.exportdata->addExternalFile("docbook-xml",
+                                                     params().filename.absFilename());
+       } else {
+               runparams.exportdata->addExternalFile("docbook",
+                                                     params().filename.absFilename());
+       }
+       os << "<inlinemediaobject>";
+
+       int r = 0;
+       string attributes = createDocBookAttributes();
+       r += writeImageObject("png", os, runparams, graphic_label, attributes);
+       r += writeImageObject("pdf", os, runparams, graphic_label, attributes);
+       r += writeImageObject("eps", os, runparams, graphic_label, attributes);
+       r += writeImageObject("bmp", os, runparams, graphic_label, attributes);
+
+       os << "</inlinemediaobject>";
+       return r;
 }
 
 
@@ -661,10 +872,21 @@ void InsetGraphics::validate(LaTeXFeatures & features) const
                return;
 
        features.includeFile(graphic_label,
-                            RemoveExtension(params().filename.absFilename()));
+                            RemoveExtension(params().filename.absFilename()));
 
        features.require("graphicx");
 
+       if (features.nice()) {
+               Buffer const * m_buffer = features.buffer().getMasterBuffer();
+               string basename =
+                       params().filename.outputFilename(m_buffer->filePath());
+               basename = RemoveExtension(basename);
+               if(params().filename.isZipped())
+                       basename = RemoveExtension(basename);
+               if (contains(basename, "."))
+                       features.require("lyxdot");
+       }
+
        if (params().subcaption)
                features.require("subfigure");
 }
@@ -693,6 +915,15 @@ InsetGraphicsParams const & InsetGraphics::params() const
 }
 
 
+void InsetGraphics::editGraphics(InsetGraphicsParams const & p,
+                                 Buffer const & buffer) const
+{
+       string const file_with_path = p.filename.absFilename();
+       formats.edit(buffer, file_with_path,
+                    formats.getFormatFromFile(file_with_path));
+}
+
+
 string const InsetGraphicsMailer::name_("graphics");
 
 InsetGraphicsMailer::InsetGraphicsMailer(InsetGraphics & inset)