Changeset 991 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2008-09-19T08:33:02+02:00 (16 years ago)

Author:

stoecker

Message:

improved mappaint loader a lot

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• mappaint/ElemStyleHandler.java (modified) (1 diff)
• mappaint/MapPaintStyles.java (modified) (4 diffs)
• tagging/TaggingPreset.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyleHandler.java

@@ -150 +150 @@
                                 {
                                         if (atts.getQName(count).equals("src"))
-                                                rule.icon.icon = MapPaintStyles.getIcon(atts.getValue(count));
+                                                rule.icon.icon = MapPaintStyles.getIcon(atts.getValue(count), styleName);
                                         else if (atts.getQName(count).equals("annotate"))
                                                 rule.icon.annotate = Boolean.parseBoolean (atts.getValue(count));

trunk/src/org/openstreetmap/josm/gui/mappaint/MapPaintStyles.java

@@ -6 +6 @@
 import java.net.URL;
 import java.util.HashMap;
+import java.util.LinkedList;
+import java.util.List;
 import java.util.Iterator;
 
@@ -13 +15 @@
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.gui.mappaint.ElemStyles;
+import org.openstreetmap.josm.io.MirroredInputStream;
+import org.openstreetmap.josm.tools.ImageProvider;
 import org.xml.sax.InputSource;
 import org.xml.sax.XMLReader;
@@ -19 +23 @@
 public class MapPaintStyles {
 
-        private static String styleDir;
-        private static String imageDir;
-        private static String internalImageDir;
-        private static Boolean isInternal = false;
         private static ElemStyles styles = new ElemStyles();
+        private static String iconDirs;
         
         public static ElemStyles getStyles()
@@ -30 +31 @@
         }
 
-        public static ImageIcon getIcon(String name)
+        public static ImageIcon getIcon(String name, String styleName)
         {
-                try {
-                        if(isInternal)
-                        {
-                                String imageFile = imageDir+name;
-                                File f = new File(imageFile);
-                                if(f.exists())
-                                {
-                                        //open icon from user directory
-                                        return new ImageIcon(imageFile);
-                                }
-                        }
-                        URL path = Main.class.getResource(internalImageDir+name);
-                        if(path == null)
-                                path = Main.class.getResource("/images/styles/"+name);
-                        if(path == null)
-                        {
-                                System.out.println("Mappaint: Icon " + name + " not found, using default icon");
-                                path = Main.class.getResource(internalImageDir+"misc/no_icon.png");
-                        }
-                        return new ImageIcon(Toolkit.getDefaultToolkit().createImage(path));
+                List<String> dirs = new LinkedList<String>();
+                for(String fileset : iconDirs.split(";"))
+                {
+                        String[] a;
+                        if(fileset.indexOf("=") >= 0)
+                                a = fileset.split("=", 2);
+                        else
+                                a = new String[] {"", fileset};
+
+                        /* non-prefixed path is generic path, always take it */
+                        if(a[0].length() == 0 || styleName.equals(a[0]))
+                                dirs.add(a[1]);
                 }
-                catch (Exception e)
+                ImageIcon i = ImageProvider.getIfAvailable(dirs, "mappaint."+styleName, null, name);            
+                if(i == null)
                 {
-                        URL path = Main.class.getResource(internalImageDir+"incomming/amenity.png");
-                        return new ImageIcon(Toolkit.getDefaultToolkit().createImage(path));
+                        System.out.println("Mappaint-Style \""+styleName+"\" icon \"" + name + "\" not found.");
+                        i = ImageProvider.getIfAvailable(dirs, "mappaint."+styleName, null, "misc/no_icon.png");
                 }
+                return i;
         }
 
         public static void readFromPreferences() {
-                String styleName = Main.pref.get("mappaint.style", "standard");
-                // fallback to standard name for internal case, as we only have one internal style
-                String internalStyleName = "standard";
-                styleDir = Main.pref.get("mappaint.styledir", Main.pref.getPreferencesDir()+"plugins/mappaint/"+styleName+"/");
-                String elemStylesFile = styleDir+"elemstyles.xml";
-                imageDir = styleDir+"icons/";
-                internalImageDir = "/images/styles/"+internalStyleName+"/";
+                /* don't prefix icon path, as it should be generic */
+                String internalicon = "resource://images/styles/standard/;resource://images/styles/";
+                String internalfile = "standard=resource://styles/standard/elemstyles.xml";
 
-//              System.out.println("mappaint: Using style: " + styleName);
-//              System.out.println("mappaint: Using style dir: " + styleDir);
-//              System.out.println("mappaint: Using style file: " + elemStylesFile);
+                iconDirs = Main.pref.get("mappaint.iconpaths");
+                iconDirs = iconDirs == null || iconDirs.length() == 0 ? internalicon : iconDirs + ";" + internalicon;
 
-                File f = new File(elemStylesFile);
-                if (f.exists())
+                String file = Main.pref.get("mappaint.sources");
+                file = file == null || file.length() == 0 ? internalfile : internalfile + ";" + file;
+
+                for(String fileset : file.split(";"))
                 {
-                        try// reading file from file system
+                        try
                         {
-//                              System.out.println("mappaint: Using style file: \"" + f + "\"");
+                                String[] a;
+                                if(fileset.indexOf("=") >= 0)
+                                        a = fileset.split("=", 2);
+                                else
+                                        a = new String[] {"standard", fileset};
                                 XMLReader xmlReader = XMLReaderFactory.createXMLReader();
-                                ElemStyleHandler handler = new ElemStyleHandler(styleName);
+                                ElemStyleHandler handler = new ElemStyleHandler(a[0]);
                                 xmlReader.setContentHandler(handler);
                                 xmlReader.setErrorHandler(handler);
-//                              temporary only!
-                                xmlReader.parse(new InputSource(new FileReader(f)));
+                                xmlReader.parse(new InputSource(new MirroredInputStream(a[1])));
                         }
                         catch (Exception e)
                         {
-                                throw new RuntimeException(e);
+                                System.out.println("Mappaint-Style problems: \"" + fileset + "\"");
                         }
                 } 
-                else {// reading the builtin file from the plugin jar file
-                        URL elemStylesPath = Main.class.getResource("/styles/"+internalStyleName+"/elemstyles.xml");
-
-//                      System.out.println("mappaint: Using jar's elemstyles.xml: \"" + elemStylesPath + "\"");
-                        if (elemStylesPath != null)
-                        {
-                                isInternal = true;
-                                try
-                                {
-                                        XMLReader xmlReader = XMLReaderFactory.createXMLReader();
-                                        ElemStyleHandler handler = new ElemStyleHandler(internalStyleName);
-                                        xmlReader.setContentHandler(handler);
-                                        xmlReader.setErrorHandler(handler);
-//                                      temporary only!
-                                        xmlReader.parse(new InputSource(elemStylesPath.openStream()));
-                                }
-                                catch (Exception e)
-                                {
-                                        throw new RuntimeException(e);
-                                }
-                        } else {
-                                System.out.println("mappaint: Couldn't find style: \"" + styleDir + "elemstyles.xml\"");
-                        }
-                }
+                iconDirs = null;
         }
 

trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPreset.java

@@ -405 +405 @@
          */
         public void setIcon(String iconName) {
-                ImageIcon icon = ImageProvider.getIfAvailable(null, iconName);
+                String s = Main.pref.get("taggingpreset.iconpaths");
+                ImageIcon icon = ImageProvider.getIfAvailable((s != null ? s.split(";") : null), "presets", null, iconName);
                 if (icon == null)
                 {