Bug 12450 fixed: xlfont() did not return font names as specified int the doc 31/11031/2
Calixte DENIZET [Wed, 27 Mar 2013 08:31:47 +0000 (09:31 +0100)]
Change-Id: I42c1e8192841d15f94bb5002cb5931d416d30825

scilab/CHANGES_5.4.X
scilab/modules/renderer/src/java/org/scilab/modules/renderer/utils/textRendering/FontManager.java

index b88e23e..b184155 100644 (file)
@@ -591,7 +591,9 @@ Bug fixes
 
 * Bug #12418 fixed - Continuation was incorrectly supported in bvode.
 
-* Bug #12442 fixed - pathconvert() help page is not clear enough.
+* Bug #12442 fixed - pathconvert() help page was not clear about input arguments.
+
+* Bug #12450 fixed - xlfont() did not return font names as specified in the documentation.
 
 
                     Changes between version 5.3.3 and 5.4.0
index e0954b8..feeb8c8 100644 (file)
@@ -68,7 +68,7 @@ public class FontManager {
     /**
      * A list of fonts.
      */
-    private static class FontList extends ArrayList < Font > {
+    private static class FontList extends ArrayList<Font> {
 
         private static final long serialVersionUID = 1L;
 
@@ -78,7 +78,6 @@ public class FontManager {
         public FontList() {
             super();
         }
-
     }
 
     private static FontList sciFonts;
@@ -359,7 +358,7 @@ public class FontManager {
     public final boolean isAvailableFontName(String fontname) {
         int i = getSizeAvailableFontsName();
         String[] availableFontsName = getAvailableFontsName();
-        for (i = 0; i < getSizeAvailableFontsName(); i++) {
+        for (i = 0; i < availableFontsName.length; i++) {
             if (fontname.compareTo(availableFontsName[i]) == 0) {
                 return true;
             }
@@ -378,6 +377,12 @@ public class FontManager {
         for (int i = 0; i < nbFonts; i++) {
             Font res = getFontFromIndex(i);
             fontnames[i] = res.getFamily();
+            if (res.isBold()) {
+                fontnames[i] += " Bold";
+            }
+            if (res.isItalic()) {
+                fontnames[i] += " Italic";
+            }
         }
 
         return fontnames;
@@ -430,5 +435,4 @@ public class FontManager {
         sciFonts.add(createFont(SANSSERIF, true, true));  /* scilab font_style 9 */
         sciFonts.add(createFont(SANSSERIF, true, true));  /* scilab font_style 10 */
     }
-
 }