Changeset 1574 in josm for trunk/src/org/openstreetmap/josm/io

Timestamp:

2009-05-05T16:44:44+02:00 (17 years ago)

Author:

stoecker

Message:

close #2214 - patch by Henrik Niehaus - GPX export metadata handling

Location:

trunk/src/org/openstreetmap/josm/io

Files:

• GpxReader.java (modified) (5 diffs)
• GpxWriter.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/io/GpxReader.java

@@ -41 +41 @@
      */
     public GpxData data;
-    public enum state { init, metadata, wpt, rte, trk, ext, author, link, trkseg }
+    public enum state { init, metadata, wpt, rte, trk, ext, author, link, trkseg, copyright}
 
     private class Parser extends DefaultHandler {
@@ -106 +106 @@
                     currentState = state.link;
                     currentLink = new GpxLink(atts.getValue("href"));
+                } else if (qName.equals("email")) {
+                    currentData.attr.put(GpxData.META_AUTHOR_EMAIL, atts.getValue("id") + "@" + atts.getValue("domain"));
                 }
                 break;
@@ -129 +131 @@
                     states.push(currentState);
                     currentState = state.ext;
+                } else if (qName.equals("copyright")) {
+                    states.push(currentState);
+                    currentState = state.copyright;
+                    currentData.attr.put(GpxData.META_COPYRIGHT_AUTHOR, atts.getValue("author"));
+                } else if (qName.equals("link")) {
+                    states.push(currentState);
+                    currentState = state.link;
+                    currentLink = new GpxLink(atts.getValue("href"));
                 }
                 break;
@@ -184 +194 @@
             switch (currentState) {
             case metadata:
-                if (qName.equals("name") || qName.equals("desc") ||
-                        qName.equals("time") || qName.equals("keywords")) {
-                    currentData.attr.put(qName, accumulator.toString());
+                if (qName.equals("name")) {
+                    currentData.attr.put(GpxData.META_NAME, accumulator.toString());
+                } else if (qName.equals("desc")) {
+                    currentData.attr.put(GpxData.META_DESC, accumulator.toString());
+                } else if (qName.equals("time")) {
+                    currentData.attr.put(GpxData.META_TIME, accumulator.toString());
+                } else if (qName.equals("keywords")) {
+                    currentData.attr.put(GpxData.META_KEYWORDS, accumulator.toString());
                 } else if (qName.equals("metadata")) {
                     currentState = states.pop();
                 }
-                //TODO: parse copyright, bounds, extensions
+                //TODO: parse bounds, extensions
                 break;
             case author:
                 if (qName.equals("author")) {
                     currentState = states.pop();
-                } else if (qName.equals("name") || qName.equals("email")) {
-                    currentData.attr.put("author" + qName, accumulator.toString());
+                } else if (qName.equals("name")) {
+                    currentData.attr.put(GpxData.META_AUTHOR_NAME, accumulator.toString());
+                } else if (qName.equals("email")) {
+                    // do nothing, has been parsed on startElement
                 } else if (qName.equals("link")) {
-                    currentData.attr.put("authorlink", currentLink);
+                    currentData.attr.put(GpxData.META_AUTHOR_LINK, currentLink);
+                }
+                break;
+            case copyright:
+                if (qName.equals("copyright")) {
+                    currentState = states.pop();
+                } else if (qName.equals("year")) {
+                    currentData.attr.put(GpxData.META_COPYRIGHT_YEAR, accumulator.toString());
+                } else if (qName.equals("license")) {
+                    currentData.attr.put(GpxData.META_COPYRIGHT_LICENSE, accumulator.toString());
                 }
                 break;
@@ -207 +233 @@
                     currentLink.type = accumulator.toString();
                 } else if (qName.equals("link")) {
-                    // <link>URL</link>
-                    if (currentLink.uri == null)
-                        currentLink.uri = accumulator.toString();
-
                     currentState = states.pop();
                 }
                 if (currentState == state.author) {
-                    currentData.attr.put("authorlink", currentLink);
-                } else if (currentState != state.link) {
+                    currentData.attr.put(GpxData.META_AUTHOR_LINK, currentLink);
+                } else if (currentState == state.metadata) {
                     Map<String, Object> attr = getAttr();
-                    if (!attr.containsKey("link")) {
-                        attr.put("link", new LinkedList<GpxLink>());
+                    if (!attr.containsKey(GpxData.META_LINKS)) {
+                        attr.put(GpxData.META_LINKS, new LinkedList<GpxLink>());
                     }
-                    ((Collection<GpxLink>) attr.get("link")).add(currentLink);
+                    ((Collection<GpxLink>) attr.get(GpxData.META_LINKS)).add(currentLink);
                 }
                 break;

trunk/src/org/openstreetmap/josm/io/GpxWriter.java

@@ -45 +45 @@
         this.data = data;
         out.println("<?xml version='1.0' encoding='UTF-8'?>");
-        out.println("<gpx version=\"1.1\" creator=\"JOSM GPX export\" xmlns=\"http://www.topografix.com/GPX/1/1\">");
+        out.println("<gpx version=\"1.1\" creator=\"JOSM GPX export\" xmlns=\"http://www.topografix.com/GPX/1/1\"\n" +
+                        "    xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\" \n" + 
+                        "    xsi:schemaLocation=\"http://www.topografix.com/GPX/1/1 http://www.topografix.com/GPX/1/1/gpx.xsd\">");
         indent = "  ";
         writeMetaData();
@@ -56 +58 @@
 
     private void writeAttr(Map<String, Object> attr) {
-        boolean hasAuthor = false;
+        // FIXME this loop is evil, because it does not assure the
+        // correct element order specified by the xml schema.
+        // for now it works, but future extension could get very complex and unmaintainable 
         for (Map.Entry<String, Object> ent : attr.entrySet()) {
             String k = ent.getKey();
-            if (k.indexOf("author") == 0) {
-                hasAuthor = true;
-            } else if (k.equals("link")) {
+            if (k.equals("link")) {
                 for (GpxLink link : (Collection<GpxLink>) ent.getValue()) {
                     gpxLink(link);
@@ -69 +71 @@
             }
         }
-
-        if (hasAuthor) {
-            open("author");
-            simpleTag("name", (String) attr.get("authorname"));
-            simpleTag("email", (String) attr.get("authoremail"));
-            gpxLink((GpxLink) attr.get("authorlink"));
+    }
+
+    private void writeMetaData() {
+        Map<String, Object> attr = data.attr;
+        openln("metadata");
+        
+        // write the description
+        if (attr.containsKey(GpxData.META_DESC)) simpleTag("desc", (String)attr.get(GpxData.META_DESC));
+        
+        // write the author details
+        if (attr.containsKey(GpxData.META_AUTHOR_NAME) 
+                || attr.containsKey(GpxData.META_AUTHOR_EMAIL)) {
+            openln("author");
+            // write the name
+            simpleTag("name", (String) attr.get(GpxData.META_AUTHOR_NAME));
+            // write the email address
+            if(attr.containsKey(GpxData.META_AUTHOR_EMAIL)) {
+                String[] tmp = ((String)attr.get(GpxData.META_AUTHOR_EMAIL)).split("@");
+                if(tmp.length == 2) {
+                    inline("email", "id=\"" + tmp[0] + "\" domain=\""+tmp[1]+"\"");
+                }
+            }
+            // write the author link
+            gpxLink((GpxLink) attr.get(GpxData.META_AUTHOR_LINK));
             closeln("author");
         }
 
-        // TODO: copyright
-    }
-
-    private void writeMetaData() {
-        openln("metadata");
-        writeAttr(data.attr);
+        // write the copyright details
+        if(attr.containsKey(GpxData.META_COPYRIGHT_LICENSE) 
+                || attr.containsKey(GpxData.META_COPYRIGHT_YEAR)) {
+            openAtt("copyright", "author=\""+ attr.get(GpxData.META_COPYRIGHT_AUTHOR) +"\"");
+            if(attr.containsKey(GpxData.META_COPYRIGHT_YEAR)) {
+                simpleTag("year", (String) attr.get(GpxData.META_COPYRIGHT_YEAR));
+            }
+            if(attr.containsKey(GpxData.META_COPYRIGHT_LICENSE)) {
+                simpleTag("license", encode((String) attr.get(GpxData.META_COPYRIGHT_LICENSE)));
+            }
+            closeln("copyright");
+        }
+        
+        // write links
+        if(attr.containsKey(GpxData.META_LINKS)) {
+            for (GpxLink link : (Collection<GpxLink>) attr.get(GpxData.META_LINKS)) {
+                gpxLink(link);
+            }
+        }
+        
+        // write keywords 
+        if (attr.containsKey(GpxData.META_KEYWORDS)) simpleTag("keywords", (String)attr.get(GpxData.META_KEYWORDS));
 
         data.recalculateBounds();