]> git.lyx.org Git - lyx.git/blobdiff - src/insets/InsetGraphics.cpp
Fix wrongly copy-pasted entries in SpellcheckerUi.ui
[lyx.git] / src / insets / InsetGraphics.cpp
index 0fa18359078cd8dfd75c8836b800222b48c8de14..ac76d4e251d1afa5aa4de3d57452cec3fd597c13 100644 (file)
@@ -66,6 +66,7 @@ TODO
 #include "MetricsInfo.h"
 #include "Mover.h"
 #include "OutputParams.h"
+#include "output_xhtml.h"
 #include "sgml.h"
 #include "TocBackend.h"
 
@@ -101,9 +102,10 @@ namespace {
 /// Note that \p format may be unknown (i. e. an empty string)
 string findTargetFormat(string const & format, OutputParams const & runparams)
 {
-       // Are we using latex or XeTeX/pdflatex?
+       // Are we using latex or XeTeX/LuaTeX/pdflatex?
        if (runparams.flavor == OutputParams::PDFLATEX
-           || runparams.flavor == OutputParams::XETEX) {
+           || runparams.flavor == OutputParams::XETEX
+           || runparams.flavor == OutputParams::LUATEX) {
                LYXERR(Debug::GRAPHICS, "findTargetFormat: PDF mode");
                Format const * const f = formats.getFormat(format);
                // Convert vector graphics to pdf
@@ -191,7 +193,7 @@ InsetGraphics::~InsetGraphics()
 
 void InsetGraphics::doDispatch(Cursor & cur, FuncRequest & cmd)
 {
-       switch (cmd.action) {
+       switch (cmd.action()) {
        case LFUN_INSET_EDIT: {
                InsetGraphicsParams p = params();
                if (!cmd.argument().empty())
@@ -204,10 +206,11 @@ void InsetGraphics::doDispatch(Cursor & cur, FuncRequest & cmd)
                InsetGraphicsParams p;
                string2params(to_utf8(cmd.argument()), buffer(), p);
                if (p.filename.empty()) {
-                       cur.noUpdate();
+                       cur.noScreenUpdate();
                        break;
                }
 
+               cur.recordUndo();
                setParams(p);
                // if the inset is part of a graphics group, all the
                // other members should be updated too.
@@ -222,6 +225,7 @@ void InsetGraphics::doDispatch(Cursor & cur, FuncRequest & cmd)
                break;
 
        case LFUN_GRAPHICS_RELOAD:
+               params_.filename.refresh();
                graphic_->reload();
                break;
 
@@ -235,7 +239,7 @@ void InsetGraphics::doDispatch(Cursor & cur, FuncRequest & cmd)
 bool InsetGraphics::getStatus(Cursor & cur, FuncRequest const & cmd,
                FuncStatus & flag) const
 {
-       switch (cmd.action) {
+       switch (cmd.action()) {
        case LFUN_INSET_EDIT:
        case LFUN_INSET_MODIFY:
        case LFUN_INSET_DIALOG_UPDATE:
@@ -391,7 +395,7 @@ docstring InsetGraphics::toDocbookLength(Length const & len) const
                case Length::PPW: // Percent of PageWidth
                case Length::PLW: // Percent of LineWidth
                case Length::PTH: // Percent of TextHeight
-               case Length::PPH: // Percent of Paper
+               case Length::PPH: // Percent of PaperHeight
                        // Sigh, this will go wrong.
                        result << len.value() << "%";
                        break;
@@ -469,7 +473,7 @@ copyFileIfNeeded(FileName const & file_in, FileName const & file_out)
                LYXERR(Debug::GRAPHICS,
                        to_utf8(bformat(_("Could not copy the file\n%1$s\n"
                                                           "into the temporary directory."),
-                                               from_utf8(file_in.absFilename()))));
+                                               from_utf8(file_in.absFileName()))));
        }
 
        GraphicsCopyStatus status = success ? SUCCESS : FAILURE;
@@ -480,20 +484,20 @@ copyFileIfNeeded(FileName const & file_in, FileName const & file_out)
 pair<GraphicsCopyStatus, FileName> const
 copyToDirIfNeeded(DocFileName const & file, string const & dir)
 {
-       string const file_in = file.absFilename();
+       string const file_in = file.absFileName();
        string const only_path = onlyPath(file_in);
        if (rtrim(onlyPath(file_in) , "/") == rtrim(dir, "/"))
                return make_pair(IDENTICAL_PATHS, file_in);
 
-       string mangled = file.mangledFilename();
+       string mangled = file.mangledFileName();
        if (file.isZipped()) {
                // 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
+               // 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(file.unzippedFilename());
+               string const base = removeExtension(file.unzippedFileName());
                string::size_type const ext_len = file_in.length() - base.length();
                mangled[mangled.length() - ext_len] = '.';
        }
@@ -546,9 +550,9 @@ string InsetGraphics::prepareFile(OutputParams const & runparams) const
        if (params().filename.empty())
                return string();
 
-       string const orig_file = params().filename.absFilename();
+       string const orig_file = params().filename.absFileName();
        // this is for dryrun and display purposes, do not use latexFilename
-       string const rel_file = params().filename.relFilename(buffer().filePath());
+       string const rel_file = params().filename.relFileName(buffer().filePath());
 
        // previewing source code, no file copying or file format conversion
        if (runparams.dryrun)
@@ -563,7 +567,7 @@ string InsetGraphics::prepareFile(OutputParams const & runparams) const
        // We are not going to change the extension or using the name of the
        // temporary file, the code is already complicated enough.
        if (runparams.inComment || !params().filename.isReadableFile())
-               return params().filename.outputFilename(masterBuffer->filePath());
+               return params().filename.outputFileName(masterBuffer->filePath());
 
        // We place all temporary files in the master buffer's temp dir.
        // This is possible because we use mangled file names.
@@ -584,17 +588,29 @@ string InsetGraphics::prepareFile(OutputParams const & runparams) const
        // "nice" means that the buffer is exported to LaTeX format but not
        // run through the LaTeX compiler.
        string output_file = runparams.nice ?
-               params().filename.outputFilename(masterBuffer->filePath()) :
-               onlyFilename(temp_file.absFilename());
-
-       if (runparams.nice && !isValidLaTeXFilename(output_file)) {
-               frontend::Alert::warning(_("Invalid filename"),
-                                        _("The following filename is likely to cause trouble "
-                                          "when running the exported file through LaTeX: ") +
-                                           from_utf8(output_file));
+               params().filename.outputFileName(masterBuffer->filePath()) :
+               onlyFileName(temp_file.absFileName());
+
+       if (runparams.nice) {
+               if (!isValidLaTeXFileName(output_file)) {
+                       frontend::Alert::warning(_("Invalid filename"),
+                               _("The following filename will cause troubles "
+                                 "when running the exported file through LaTeX: ") +
+                               from_utf8(output_file));
+               }
+               // only show DVI-specific warning when export format is plain latex
+               if (!isValidDVIFileName(output_file)
+                       && runparams.flavor == OutputParams::LATEX) {
+                               frontend::Alert::warning(_("Problematic filename for DVI"),
+                                        _("The following filename can cause troubles "
+                                              "when running the exported file through LaTeX "
+                                                  "and opening the resulting DVI: ") +
+                                            from_utf8(output_file), true);
+               }
        }
 
        FileName source_file = runparams.nice ? FileName(params().filename) : temp_file;
+       // determine the export format
        string const tex_format = (runparams.flavor == OutputParams::LATEX) ?
                        "latex" : "pdflatex";
 
@@ -619,7 +635,7 @@ string InsetGraphics::prepareFile(OutputParams const & runparams) const
                                // LaTeX needs the bounding box file in the
                                // tmp dir
                                FileName bb_file =
-                                       FileName(changeExtension(temp_file.absFilename(), "bb"));
+                                       FileName(changeExtension(temp_file.absFileName(), "bb"));
                                boost::tie(status, bb_file) =
                                        copyFileIfNeeded(bb_orig_file, bb_file);
                                if (status == FAILURE)
@@ -637,9 +653,9 @@ string InsetGraphics::prepareFile(OutputParams const & runparams) const
                }
 
                FileName const unzipped_temp_file =
-                       FileName(unzippedFileName(temp_file.absFilename()));
+                       FileName(unzippedFileName(temp_file.absFileName()));
                output_file = unzippedFileName(output_file);
-               source_file = FileName(unzippedFileName(source_file.absFilename()));
+               source_file = FileName(unzippedFileName(source_file.absFileName()));
                if (compare_timestamps(unzipped_temp_file, temp_file) > 0) {
                        // temp_file has been unzipped already and
                        // orig_file has not changed in the meantime.
@@ -673,12 +689,12 @@ string InsetGraphics::prepareFile(OutputParams const & runparams) const
                        // the file format from the extension, so we must
                        // change it.
                        FileName const new_file = 
-                               FileName(changeExtension(temp_file.absFilename(), ext));
+                               FileName(changeExtension(temp_file.absFileName(), ext));
                        if (temp_file.moveTo(new_file)) {
                                temp_file = new_file;
                                output_file = changeExtension(output_file, ext);
                                source_file = 
-                                       FileName(changeExtension(source_file.absFilename(), ext));
+                                       FileName(changeExtension(source_file.absFileName(), ext));
                        } else {
                                LYXERR(Debug::GRAPHICS, "Could not rename file `"
                                        << temp_file << "' to `" << new_file << "'.");
@@ -693,7 +709,7 @@ string InsetGraphics::prepareFile(OutputParams const & runparams) const
        }
 
        // so the source and destination formats are different
-       FileName const to_file = FileName(changeExtension(temp_file.absFilename(), ext));
+       FileName const to_file = FileName(changeExtension(temp_file.absFileName(), ext));
        string const output_to_file = changeExtension(output_file, ext);
 
        // Do we need to perform the conversion?
@@ -730,13 +746,13 @@ string InsetGraphics::prepareFile(OutputParams const & runparams) const
 }
 
 
-int InsetGraphics::latex(odocstream & os,
-                        OutputParams const & runparams) const
+void InsetGraphics::latex(otexstream & os,
+                         OutputParams const & runparams) const
 {
        // If there is no file specified or not existing,
        // just output a message about it in the latex output.
        LYXERR(Debug::GRAPHICS, "insetgraphics::latex: Filename = "
-               << params().filename.absFilename());
+               << params().filename.absFileName());
 
        bool const file_exists = !params().filename.empty()
                        && params().filename.isReadableFile();
@@ -780,8 +796,6 @@ int InsetGraphics::latex(odocstream & os,
        os << from_utf8(latex_str);
 
        LYXERR(Debug::GRAPHICS, "InsetGraphics::latex outputting:\n" << latex_str);
-       // Return how many newlines we issued.
-       return int(count(latex_str.begin(), latex_str.end(),'\n'));
 }
 
 
@@ -796,7 +810,7 @@ int InsetGraphics::plaintext(odocstream & os, OutputParams const &) const
        // FIXME: We have no idea what the encoding of the filename is
 
        docstring const str = bformat(buffer().B_("Graphics file: %1$s"),
-                                     from_utf8(params().filename.absFilename()));
+                                     from_utf8(params().filename.absFileName()));
        os << '<' << str << '>';
 
        return 2 + str.size();
@@ -868,7 +882,7 @@ string InsetGraphics::prepareHTMLFile(OutputParams const & runparams) const
        if (params().filename.empty())
                return string();
 
-       string const orig_file = params().filename.absFilename();
+       string const orig_file = params().filename.absFileName();
 
        // The master buffer. This is useful when there are multiple levels
        // of include files
@@ -892,7 +906,7 @@ string InsetGraphics::prepareHTMLFile(OutputParams const & runparams) const
        if (status == FAILURE)
                return string();
 
-       string output_file = onlyFilename(temp_file.absFilename());
+       string output_file = onlyFileName(temp_file.absFileName());
 
        string const from = formats.getFormatFromFile(temp_file);
        if (from.empty())
@@ -910,7 +924,7 @@ string InsetGraphics::prepareHTMLFile(OutputParams const & runparams) const
        }
 
        // so the source and destination formats are different
-       FileName const to_file = FileName(changeExtension(temp_file.absFilename(), ext));
+       FileName const to_file = FileName(changeExtension(temp_file.absFileName(), ext));
        string const output_to_file = changeExtension(output_file, ext);
 
        // Do we need to perform the conversion?
@@ -941,20 +955,34 @@ string InsetGraphics::prepareHTMLFile(OutputParams const & runparams) const
 }
 
 
-docstring InsetGraphics::xhtml(odocstream & os, OutputParams const & op) const
+docstring InsetGraphics::xhtml(XHTMLStream & xs, OutputParams const & op) const
 {
-       string output_file = prepareHTMLFile(op);
+       string const output_file = prepareHTMLFile(op);
+
        if (output_file.empty()) {
-               LYXERR0("InsetGraphics::xhtml: File `" << params().filename 
-                                               << "' not found.");
-               os << "<img src=\"" << params().filename.absFilename() << "\" />";
+               LYXERR0("InsetGraphics::xhtml: Unable to prepare file `" 
+                       << params().filename << "' for output. File missing?");
+               string const attr = "src='" + params().filename.absFileName() 
+                                   + "' alt='image: " + output_file + "'";
+               xs << html::CompTag("img", attr);
                return docstring();
        }
 
-       // FIXME Do we want to do something with the parameters, other than
-       // use them to do another conversion?
-       // FIXME Do the other conversion! Cropping, rotating, etc.
-       os << "<img src=\"" << from_utf8(output_file) << "\" />";
+       // FIXME XHTML 
+       // We aren't doing anything with the crop and rotate parameters, and it would
+       // really be better to do width and height conversion, rather than to output
+       // these parameters here.
+       string imgstyle;
+       if (!params().width.zero())
+               imgstyle += "width:" + params().width.asHTMLString() + ";";
+       if (!params().height.zero())
+               imgstyle += " height:" + params().height.asHTMLString() + ";";
+       if (!imgstyle.empty())
+               imgstyle = "style='" + imgstyle + "' ";
+
+       string const attr = imgstyle + "src='" + output_file + "' alt='image: " 
+                           + output_file + "'";
+       xs << html::CompTag("img", attr);
        return docstring();
 }
 
@@ -966,14 +994,14 @@ void InsetGraphics::validate(LaTeXFeatures & features) const
                return;
 
        features.includeFile(graphic_label,
-                            removeExtension(params().filename.absFilename()));
+                            removeExtension(params().filename.absFileName()));
 
        features.require("graphicx");
 
        if (features.runparams().nice) {
                Buffer const * masterBuffer = features.buffer().masterBuffer();
                string const rel_file = removeExtension(
-                       params().filename.relFilename(masterBuffer->filePath()));
+                       params().filename.relFileName(masterBuffer->filePath()));
                if (contains(rel_file, "."))
                        features.require("lyxdot");
        }
@@ -1010,7 +1038,7 @@ void InsetGraphics::editGraphics(InsetGraphicsParams const & p) const
 }
 
 
-void InsetGraphics::addToToc(DocIterator const & cpit)
+void InsetGraphics::addToToc(DocIterator const & cpit) const
 {
        TocBackend & backend = buffer().tocBackend();
 
@@ -1020,7 +1048,7 @@ void InsetGraphics::addToToc(DocIterator const & cpit)
 }
 
 
-docstring InsetGraphics::contextMenu(BufferView const &, int, int) const
+docstring InsetGraphics::contextMenuName() const
 {
        return from_ascii("context-graphics");
 }