Helptools error reports: print out the expected file path on fileref error 77/9677/3
Clément DAVID [Tue, 13 Nov 2012 14:32:40 +0000 (15:32 +0100)]
Change-Id: I11254eb44c720b578c8274a65bde2fe13d5500b6

scilab/modules/helptools/src/java/org/scilab/modules/helptools/HTMLDocbookTagConverter.java
scilab/modules/helptools/src/java/org/scilab/modules/helptools/image/ImageConverter.java

index 4a0dda6..26d6872 100644 (file)
@@ -1263,8 +1263,9 @@ public class HTMLDocbookTagConverter extends DocbookTagConverter implements Temp
 
         try {
             String path = new File(new URI(currentFileName)).getParent();
-            if (!ImageConverter.imageExists(path, fileref)) {
-                throw new SAXException("The given fileref is not on an existing image file:\n" + fileref);
+            File file = ImageConverter.imageExists(path, fileref);
+            if (file != null) {
+                throw new SAXException("The given fileref is not on an existing image file:\n" + fileref + " [" + file + "]");
             }
 
             return ImageConverter.getImageByFile(attributes, path, fileref, outName, imageDir);
index db0e132..5dbb977 100644 (file)
@@ -153,15 +153,19 @@ public final class ImageConverter {
      * Test if an image file exists.
      * @param path of the parsed file
      * @param image the image name
-     * @return true if the image exists
+     * @return null if the image exists, the expected file path otherwise.
      */
-    public static boolean imageExists(String path, String image) {
+    public static File imageExists(String path, String image) {
         File f = new File(image);
         if (!f.isAbsolute()) {
             f = new File(path + File.separator + image);
         }
 
-        return f.exists();
+        if (f.exists()) {
+            return null;
+        } else {
+            return f;
+        }
     }
 
     /**