[LyX/2.2.x] If we cannot find the input format for a graphic, we cannot convert it.

[Richard Heck]

commit e1bad0610337da2d4948c0eb5fe988e8f9ac73af
Author: Richard Heck <rgh...@lyx.org>
Date:   Sat Jul 30 21:51:25 2016 -0400

    If we cannot find the input format for a graphic, we cannot convert it.
    
    Also includes some minor code cleanup.
    
    (cherry picked from commit e1f597e3da26865cb01eab5bef6ad5d9e862ff7a)
---
 src/insets/InsetGraphics.cpp |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/insets/InsetGraphics.cpp b/src/insets/InsetGraphics.cpp
index 323eaac..04faae7 100644
--- a/src/insets/InsetGraphics.cpp
+++ b/src/insets/InsetGraphics.cpp
@@ -873,15 +873,13 @@ string InsetGraphics::prepareHTMLFile(OutputParams const 
& runparams) const
        if (params().filename.empty())
                return string();
 
-       string const orig_file = params().filename.absFileName();
+       if (!params().filename.isReadableFile())
+               return string();
 
        // The master buffer. This is useful when there are multiple levels
        // of include files
        Buffer const * masterBuffer = buffer().masterBuffer();
 
-       if (!params().filename.isReadableFile())
-               return string();
-
        // We place all temporary files in the master buffer's temp dir.
        // This is possible because we use mangled file names.
        // FIXME We may want to put these files in some special temporary
@@ -897,14 +895,16 @@ string InsetGraphics::prepareHTMLFile(OutputParams const 
& runparams) const
        if (status == FAILURE)
                return string();
 
-       string output_file = onlyFileName(temp_file.absFileName());
-
        string const from = formats.getFormatFromFile(temp_file);
-       if (from.empty())
+       if (from.empty()) {
                LYXERR(Debug::GRAPHICS, "\tCould not get file format.");
+               return string();
+       }
 
        string const to   = findTargetFormat(from, runparams);
        string const ext  = formats.extension(to);
+       string const orig_file = params().filename.absFileName();
+       string output_file = onlyFileName(temp_file.absFileName());
        LYXERR(Debug::GRAPHICS, "\t we have: from " << from << " to " << to);
        LYXERR(Debug::GRAPHICS, "\tthe orig file is: " << orig_file);