* Use JMarc's browseRelFiles rather than browseFiles.

[lyx.git] / src / frontends / controllers / ControlGraphics.C

diff --git a/src/frontends/controllers/ControlGraphics.C b/src/frontends/controllers/ControlGraphics.C
index 9bf1949d3fd87d0502cb13146cd16aadc38cb494..b1de9231ad39bbedf71bc4f6dfd806e4603419c1 100644 (file)
--- a/src/frontends/controllers/ControlGraphics.C
+++ b/src/frontends/controllers/ControlGraphics.C
@@ -1,5 +1,5 @@
 /* This file is part of
- * ====================================================== 
+ * ======================================================
  *
  *           LyX, The Document Processor
  *
@@ -12,18 +12,19 @@
  * \author Herbert Voss <voss@perce.de>
  */
 
-#include <config.h> 
+#include <config.h>
 #include <fstream>
 
 #ifdef __GNUG__
 #pragma implementation
-#endif 
+#endif
 
 #include "ViewBase.h"
 #include "ButtonControllerBase.h"
 #include "ControlGraphics.h"
 #include "ControlInset.tmpl"
 #include "buffer.h"
+#include "BufferView.h"
 #include "Dialogs.h"
 #include "LyXView.h"
 #include "gettext.h"
@@ -33,10 +34,9 @@
 #include "insets/insetgraphicsParams.h" // need operator!=()
 
 #include "support/FileInfo.h"  // for FileInfo
-#include "helper_funcs.h"      // for browseFile
+#include "helper_funcs.h"
 #include "support/lstrings.h"
 #include "support/filetools.h" // for AddName, zippedFile
-#include "BufferView.h"
 
 using std::pair;
 using std::make_pair;
@@ -58,7 +58,7 @@ InsetGraphicsParams const ControlGraphics::getParams(string const &)
 InsetGraphicsParams const
 ControlGraphics::getParams(InsetGraphics const & inset)
 {
-       return inset.getParams();
+       return inset.params();
 }
 
 
@@ -81,38 +81,41 @@ void ControlGraphics::applyParamsNoInset()
 extern string system_lyxdir;
 extern string user_lyxdir;
 
+
 string const ControlGraphics::Browse(string const & in_name)
 {
-       string const title = N_("Graphics");
-       // FIXME: currently we need the second '|' to prevent mis-interpretation 
+       string const title = N_("Select graphics file");
+       // FIXME: currently we need the second '|' to prevent mis-interpretation
        string const pattern = "*.(ps|eps|png|jpeg|jpg|gif|gz)|";
 
-       // Does user clipart directory exist?
-       string clipdir = AddName (user_lyxdir, "clipart");
-       FileInfo fileInfo(clipdir);
-       if (!(fileInfo.isOK() && fileInfo.isDir()))
-               // No - bail out to system clipart directory
-               clipdir = AddName (system_lyxdir, "clipart");
+       // Does user clipart directory exist?
+       string clipdir = AddName (user_lyxdir, "clipart");
+       FileInfo fileInfo(clipdir);
+       if (!(fileInfo.isOK() && fileInfo.isDir()))
+               // No - bail out to system clipart directory
+               clipdir = AddName (system_lyxdir, "clipart");
        pair<string, string> dir1(N_("Clipart|#C#c"), clipdir);
        pair<string, string> dir2(N_("Documents|#o#O"), string(lyxrc.document_path));
        // Show the file browser dialog
-       return browseFile(&lv_, in_name, title, pattern, dir1, dir2);
+       return browseRelFile(&lv_, in_name, lv_.buffer()->filePath(),
+                            title, pattern, dir1, dir2);
 }
 
+
 string const ControlGraphics::readBB(string const & file)
 {
 // in a file it's an entry like %%BoundingBox:23 45 321 345
 // the first number can following without a space, so we have
-// to check a bit more. 
+// to check a bit more.
 // on the other hand some plot programs write the bb at the
 // end of the file. Than we have in the header a
-//     %%BoundingBox: (atend)
+//     %%BoundingBox: (atend)
 // In this case we must check until the end.
        string file_ = file;
        if (zippedFile(file_))
            file_ = unzipFile(file_);
        std::ifstream is(file_.c_str());
-       if (!contains(getExtFromContents(file_),"ps"))  // bb exists? 
+       if (!contains(getExtFromContents(file_),"ps"))  // bb exists?
            return string();
        while (is) {
                string s;
@@ -121,7 +124,7 @@ string const ControlGraphics::readBB(string const & file)
                        string a, b, c, d;
                        is >> a >> b >> c >> d;
                        if (is && !contains(a,"atend")) { // bb at the end?
-                               if (s != "%%BoundingBox:") 
+                               if (s != "%%BoundingBox:")
                                    return (s.substr(14)+" "+a+" "+b+" "+c+" ");
                                else
                                    return (a+" "+b+" "+c+" "+d+" ");
@@ -132,3 +135,9 @@ string const ControlGraphics::readBB(string const & file)
 }
 
 
+bool ControlGraphics::isFilenameValid(string const & fname) const
+{
+       // It may be that the filename is relative.
+       string const name = MakeAbsPath(fname, lv_.buffer()->filePath());
+       return IsFileReadable(name);
+}