]> git.lyx.org Git - lyx.git/blobdiff - src/format.C
minimal effort implementation of:
[lyx.git] / src / format.C
index 5215e34bb31dc4b9a2ac4341b3c52b349349a12d..a8fd1f2828bb1923f7050c5701d6f162500e84fa 100644 (file)
 #include "frontends/Alert.h" //to be removed?
 
 #include "support/filetools.h"
-#include "support/path.h"
+#include "support/lstrings.h"
+#include "support/os.h"
 #include "support/systemcall.h"
 
+#include <boost/filesystem/operations.hpp>
+
+using lyx::support::absolutePath;
 using lyx::support::bformat;
 using lyx::support::compare_ascii_no_case;
 using lyx::support::contains;
-using lyx::support::MakeDisplayPath;
-using lyx::support::OnlyFilename;
-using lyx::support::OnlyPath;
-using lyx::support::Path;
-using lyx::support::QuoteName;
+using lyx::support::libScriptSearch;
+using lyx::support::makeDisplayPath;
+using lyx::support::onlyPath;
+using lyx::support::quoteName;
 using lyx::support::subst;
 using lyx::support::Systemcall;
+using lyx::support::token;
 
 using std::string;
 using std::distance;
 
+namespace fs = boost::filesystem;
+namespace os = lyx::support::os;
+
 extern LyXServerSocket * lyxsocket;
 
 namespace {
@@ -83,9 +90,12 @@ bool operator<(Format const & a, Format const & b)
        return compare_ascii_no_case(a.prettyname(), b.prettyname()) < 0;
 }
 
+
 Format::Format(string const & n, string const & e, string const & p,
-              string const & s, string const & v, string const & ed)
-       : name_(n), extension_(e), prettyname_(p), shortcut_(s), viewer_(v), editor_(ed)
+              string const & s, string const & v, string const & ed,
+               bool d)
+       : name_(n), extension_(e), prettyname_(p), shortcut_(s), viewer_(v),
+         editor_(ed), document_(d)
 {}
 
 
@@ -133,13 +143,13 @@ string Formats::getFormatFromFile(string const & filename) const
                return format;
 
        // try to find a format from the file extension.
-       string const ext(lyx::support::GetExtension(filename));
-        if (!ext.empty()) {
+       string const ext(lyx::support::getExtension(filename));
+       if (!ext.empty()) {
                // this is ambigous if two formats have the same extension,
                // but better than nothing
                Formats::const_iterator cit =
                        find_if(formatlist.begin(), formatlist.end(),
-                               FormatExtensionsEqual(ext));
+                               FormatExtensionsEqual(ext));
                if (cit != formats.end()) {
                        lyxerr[Debug::GRAPHICS]
                                << "\twill guess format from file extension: "
@@ -150,6 +160,39 @@ string Formats::getFormatFromFile(string const & filename) const
        return string();
 }
 
+namespace {
+
+string fixCommand(string const & cmd, string const & ext, 
+                 os::auto_open_mode mode)
+{
+       // configure.py says we do not want a viewer/editor
+       if (cmd.empty())
+               return cmd;
+
+       // Does the OS manage this format?
+       if (os::canAutoOpenFile(ext, mode))
+               return "auto";
+
+       // if configure.py found nothing, clear the command
+       if (token(cmd, ' ', 0) == "auto")
+               return string();
+
+       // use the command found by configure.py
+       return cmd;
+}
+
+}
+
+void Formats::setAutoOpen()
+{
+       FormatList::iterator fit = formatlist.begin();
+       FormatList::iterator const fend = formatlist.end();
+       for ( ; fit != fend ; ++fit) {
+               fit->setViewer(fixCommand(fit->viewer(), fit->extension(), os::VIEW));
+               fit->setEditor(fixCommand(fit->editor(), fit->extension(), os::EDIT));
+       }
+}
+
 
 int Formats::getNumber(string const & name) const
 {
@@ -166,22 +209,24 @@ int Formats::getNumber(string const & name) const
 void Formats::add(string const & name)
 {
        if (!getFormat(name))
-               add(name, name, name, string(), string(), string());
+               add(name, name, name, string(), string(), string(), true);
 }
 
 
 void Formats::add(string const & name, string const & extension,
                  string const & prettyname, string const & shortcut,
-                 string const & viewer, string const & editor)
+                 string const & viewer, string const & editor, bool document)
 {
        FormatList::iterator it =
                find_if(formatlist.begin(), formatlist.end(),
                        FormatNamesEqual(name));
        if (it == formatlist.end())
                formatlist.push_back(Format(name, extension, prettyname,
-                                           shortcut, viewer, editor));
+                                           shortcut, viewer, editor,
+                                           document));
        else
-               *it = Format(name, extension, prettyname, shortcut, viewer, editor);
+               *it = Format(name, extension, prettyname, shortcut, viewer,
+                            editor, document);
 }
 
 
@@ -215,23 +260,39 @@ void Formats::setViewer(string const & name, string const & command)
 bool Formats::view(Buffer const & buffer, string const & filename,
                   string const & format_name) const
 {
-       if (filename.empty())
+       BOOST_ASSERT(absolutePath(filename));
+       if (filename.empty() || !fs::exists(filename)) {
+               Alert::error(_("Cannot view file"),
+                       bformat(_("File does not exist: %1$s"),
+                               filename));
                return false;
+       }
 
        Format const * format = getFormat(format_name);
        if (format && format->viewer().empty() &&
            format->isChildFormat())
                format = getFormat(format->parentFormat());
        if (!format || format->viewer().empty()) {
-// I believe this is the wrong place to show alerts, it should be done by
-// the caller (this should be "utility" code)
+// FIXME: I believe this is the wrong place to show alerts, it should be done
+// by the caller (this should be "utility" code)
                Alert::error(_("Cannot view file"),
                        bformat(_("No information for viewing %1$s"),
                                prettyName(format_name)));
                return false;
        }
+       // viewer is 'auto'
+       if (format->viewer() == "auto") {
+               if (os::autoOpenFile(filename, os::VIEW))
+                       return true;
+               else {
+                       Alert::error(_("Cannot view file"),
+                               bformat(_("Auto-view file %1$s failed"),
+                                       filename));
+                       return false;
+               }
+       }
 
-       string command = format->viewer();
+       string command = libScriptSearch(format->viewer());
 
        if (format_name == "dvi" &&
            !lyxrc.view_dvi_paper_option.empty()) {
@@ -247,21 +308,19 @@ bool Formats::view(Buffer const & buffer, string const & filename,
        if (!contains(command, token_from))
                command += ' ' + token_from;
 
-       command = subst(command, token_from,
-                       QuoteName(OnlyFilename(filename)));
-       command = subst(command, token_path, QuoteName(OnlyPath(filename)));
-       command = subst(command, token_socket, QuoteName(lyxsocket->address()));
+       command = subst(command, token_from, quoteName(filename));
+       command = subst(command, token_path, quoteName(onlyPath(filename)));
+       command = subst(command, token_socket, quoteName(lyxsocket->address()));
        lyxerr[Debug::FILES] << "Executing command: " << command << std::endl;
        buffer.message(_("Executing command: ") + command);
 
-       Path p(OnlyPath(filename));
        Systemcall one;
        int const res = one.startscript(Systemcall::DontWait, command);
 
        if (res) {
                Alert::error(_("Cannot view file"),
                             bformat(_("An error occurred whilst running %1$s"),
-                              MakeDisplayPath(command, 50)));
+                              makeDisplayPath(command, 50)));
                return false;
        }
        return true;
@@ -269,44 +328,58 @@ bool Formats::view(Buffer const & buffer, string const & filename,
 
 
 bool Formats::edit(Buffer const & buffer, string const & filename,
-                        string const & format_name) const
+                        string const & format_name) const
 {
-       if (filename.empty())
+       BOOST_ASSERT(absolutePath(filename));
+       if (filename.empty() || !fs::exists(filename)) {
+               Alert::error(_("Cannot edit file"),
+                       bformat(_("File does not exist: %1$s"),
+                               filename));
                return false;
+       }
 
        Format const * format = getFormat(format_name);
        if (format && format->editor().empty() &&
            format->isChildFormat())
                format = getFormat(format->parentFormat());
        if (!format || format->editor().empty()) {
-// I believe this is the wrong place to show alerts, it should be done by
-// the caller (this should be "utility" code)
+// FIXME: I believe this is the wrong place to show alerts, it should
+// be done by the caller (this should be "utility" code)
                Alert::error(_("Cannot edit file"),
                        bformat(_("No information for editing %1$s"),
                                prettyName(format_name)));
                return false;
        }
+       // editor is 'auto'
+       if (format->editor() == "auto") {
+               if (os::autoOpenFile(filename, os::EDIT))
+                       return true;
+               else {
+                       Alert::error(_("Cannot edit file"),
+                               bformat(_("Auto-edit file %1$s failed"),
+                                       filename));
+                       return false;
+               }
+       }
 
        string command = format->editor();
 
        if (!contains(command, token_from))
                command += ' ' + token_from;
 
-       command = subst(command, token_from,
-                       QuoteName(OnlyFilename(filename)));
-       command = subst(command, token_path, QuoteName(OnlyPath(filename)));
-       command = subst(command, token_socket, QuoteName(lyxsocket->address()));
+       command = subst(command, token_from, quoteName(filename));
+       command = subst(command, token_path, quoteName(onlyPath(filename)));
+       command = subst(command, token_socket, quoteName(lyxsocket->address()));
        lyxerr[Debug::FILES] << "Executing command: " << command << std::endl;
        buffer.message(_("Executing command: ") + command);
 
-       Path p(OnlyPath(filename));
        Systemcall one;
        int const res = one.startscript(Systemcall::DontWait, command);
 
        if (res) {
                Alert::error(_("Cannot edit file"),
                             bformat(_("An error occurred whilst running %1$s"),
-                              MakeDisplayPath(command, 50)));
+                              makeDisplayPath(command, 50)));
                return false;
        }
        return true;