Changeset 33545 in osm for applications/editors

Timestamp:

2017-08-26T20:45:43+02:00 (7 years ago)

Author:

zverik

Message:

Upgrade API for geochat

Location:

applications/editors/josm/plugins/geochat

Files:

• build.xml (modified) (1 diff)
• src/geochat/ChatMessage.java (modified) (1 diff)
• src/geochat/ChatPaneManager.java (modified) (2 diffs)
• src/geochat/ChatServerConnection.java (modified) (8 diffs)
• src/geochat/GeoChatPanel.java (modified) (3 diffs)
• src/geochat/GeoChatPopupAdapter.java (modified) (2 diffs)
• src/geochat/JPanelTextField.java (modified) (2 diffs)
• src/geochat/JsonQueryUtil.java (modified) (3 diffs)

applications/editors/josm/plugins/geochat/build.xml

@@ -5 +5 @@
     <property name="commit.message" value="[josm_geochat] copypaste from keyboard, font size advanced parameters"/>
     <!-- enter the *lowest* JOSM version this plugin is currently compatible with -->
-    <property name="plugin.main.version" value="10580"/>
+    <property name="plugin.main.version" value="12643"/>
 
     <property name="plugin.author" value="Ilya Zverev"/>

applications/editors/josm/plugins/geochat/src/geochat/ChatMessage.java

@@ -84 +84 @@
         }
         final ChatMessage other = (ChatMessage) obj;
-        if (this.id != other.id) {
-            return false;
-        }
-        return true;
+        return this.id == other.id;
     }
 

applications/editors/josm/plugins/geochat/src/geochat/ChatPaneManager.java

@@ -25 +25 @@
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.gui.util.GuiHelper;
+import org.openstreetmap.josm.tools.Logging;
 
 /**
@@ -116 +117 @@
                     doc.insertString(doc.getLength(), nline, attrs);
                 } catch (BadLocationException ex) {
-                    Main.warn(ex);
+                    Logging.warn(ex);
                 }
                 thepane.setCaretPosition(doc.getLength());

applications/editors/josm/plugins/geochat/src/geochat/ChatServerConnection.java

@@ -23 +23 @@
 import org.openstreetmap.josm.data.coor.LatLon;
 import org.openstreetmap.josm.data.projection.Projection;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapView;
+import org.openstreetmap.josm.tools.Logging;
 
 /**
@@ -122 +124 @@
                     }
                 } catch (InterruptedException e) {
-                    Main.warn(e);
+                    Logging.warn(e);
                 }
                 autoLogin(userName);
@@ -164 +166 @@
             });
         } catch (UnsupportedEncodingException e) {
-            Main.error(e);
+            Logging.error(e);
         }
     }
@@ -266 +268 @@
             });
         } catch (UnsupportedEncodingException e) {
-            Main.error(e);
+            Logging.error(e);
         }
     }
@@ -274 +276 @@
      */
     private static LatLon getPosition() {
-        if (Main.map == null || Main.map.mapView == null)
+        if (!MainApplication.isDisplayingMapView())
             return null;
         if (getCurrentZoom() < 10)
             return null;
         Projection proj = Main.getProjection();
-        return proj.eastNorth2latlon(Main.map.mapView.getCenter());
+        return proj.eastNorth2latlon(MainApplication.getMap().mapView.getCenter());
     }
 
@@ -294 +296 @@
 
     public static int getCurrentZoom() {
-        if (Main.map == null || Main.map.mapView == null) {
+        if (!MainApplication.isDisplayingMapView()) {
             return 1;
         }
-        MapView mv = Main.map.mapView;
+        MapView mv = MainApplication.getMap().mapView;
         LatLon topLeft = mv.getLatLon(0, 0);
         LatLon botRight = mv.getLatLon(mv.getWidth(), mv.getHeight());
@@ -434 +436 @@
                     result.add(cm);
                 } catch (JsonException e) {
-                    Main.trace(e);
+                    Logging.trace(e);
                 }
             }
@@ -450 +452 @@
                     result.put(name, new LatLon(lat, lon));
                 } catch (JsonException e) {
-                    Main.trace(e);
+                    Logging.trace(e);
                 }
             }

applications/editors/josm/plugins/geochat/src/geochat/GeoChatPanel.java

@@ -37 +37 @@
 import org.openstreetmap.josm.data.Bounds;
 import org.openstreetmap.josm.data.coor.LatLon;
-import org.openstreetmap.josm.gui.JosmUserIdentityManager;
-import org.openstreetmap.josm.gui.MapView;
-import org.openstreetmap.josm.gui.Notification;
+import org.openstreetmap.josm.gui.*;
 import org.openstreetmap.josm.gui.dialogs.ToggleDialog;
 import org.openstreetmap.josm.gui.layer.MapViewPaintable;
 import org.openstreetmap.josm.gui.util.GuiHelper;
 import org.openstreetmap.josm.tools.GBC;
+import org.openstreetmap.josm.tools.Logging;
 
 /**
@@ -156 +155 @@
             }
         } catch (IOException e) {
-            Main.warn("Failed to logout from geochat server: " + e.getMessage());
+            Logging.warn("Failed to logout from geochat server: " + e.getMessage());
         }
         super.destroy();
@@ -317 +316 @@
         this.users = newUsers;
         updateTitleAlarm();
-        if (userLayerActive && Main.map.mapView != null)
-            Main.map.mapView.repaint();
+        if (userLayerActive && MainApplication.isDisplayingMapView())
+            MainApplication.getMap().mapView.repaint();
     }
 

applications/editors/josm/plugins/geochat/src/geochat/GeoChatPopupAdapter.java

@@ -13 +13 @@
 import javax.swing.JPopupMenu;
 
-import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.gui.MainApplication;
 
 /**
@@ -114 +114 @@
         ToggleUserLayerAction() {
             super(tr("Show users on map"));
-            putValue(SELECTED_KEY, Boolean.valueOf(panel.userLayerActive));
+            putValue(SELECTED_KEY, panel.userLayerActive);
         }
 
         @Override
         public void actionPerformed(ActionEvent e) {
-            if (Main.map == null || Main.map.mapView == null)
+            if (!MainApplication.isDisplayingMapView())
                 return;
-            boolean wasAdded = Main.map.mapView.addTemporaryLayer(panel);
+            boolean wasAdded = MainApplication.getMap().mapView.addTemporaryLayer(panel);
             if (!wasAdded)
-                Main.map.mapView.removeTemporaryLayer(panel);
+                MainApplication.getMap().mapView.removeTemporaryLayer(panel);
             panel.userLayerActive = wasAdded;
-            putValue(SELECTED_KEY, Boolean.valueOf(panel.userLayerActive));
-            Main.map.mapView.repaint();
+            putValue(SELECTED_KEY, panel.userLayerActive);
+            MainApplication.getMap().mapView.repaint();
         }
     }

applications/editors/josm/plugins/geochat/src/geochat/JPanelTextField.java

@@ -16 +16 @@
 import javax.swing.text.DefaultEditorKit;
 
-import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.widgets.PopupMenuLauncher;
 
@@ -87 +87 @@
                 }
             } else if (code == KeyEvent.VK_ESCAPE) {
-                if (Main.map != null && Main.map.mapView != null)
-                    Main.map.mapView.requestFocus();
+                if (MainApplication.isDisplayingMapView())
+                    MainApplication.getMap().mapView.requestFocus();
             }
 

applications/editors/josm/plugins/geochat/src/geochat/JsonQueryUtil.java

@@ -14 +14 @@
 
 import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.gui.MainApplication;
+import org.openstreetmap.josm.tools.Logging;
 
 /**
@@ -70 +72 @@
      */
     public static void queryAsync(String query, JsonQueryCallback callback) {
-        Main.worker.submit(new JsonQueryUtil(query, callback));
+        MainApplication.worker.submit(new JsonQueryUtil(query, callback));
     }
 
@@ -78 +80 @@
             obj = query(query);
         } catch (IOException e) {
-            Main.warn(e.getClass().getName() + " while connecting to a chat server: " + e.getMessage());
+            Logging.warn(e.getClass().getName() + " while connecting to a chat server: " + e.getMessage());
             obj = null;
         }