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

Timestamp:

2016-01-17T23:24:09+01:00 (8 years ago)

Author:

simon04

Message:

fix #7090 - Upload dialog allow select another user open changeset

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• JosmUserIdentityManager.java (modified) (2 diffs)
• io/OpenChangesetComboBoxModel.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/JosmUserIdentityManager.java

@@ -10 +10 @@
 import org.openstreetmap.josm.data.Preferences.PreferenceChangedListener;
 import org.openstreetmap.josm.data.Preferences.StringSetting;
+import org.openstreetmap.josm.data.osm.User;
 import org.openstreetmap.josm.data.osm.UserInfo;
 import org.openstreetmap.josm.gui.preferences.server.OAuthAccessTokenHolder;
@@ -223 +224 @@
 
     /**
-     * Replies true if the user with name <code>username</code> is the current
-     * user
+     * Replies true if the user with name <code>username</code> is the current user
      *
      * @param username the user name
-     * @return true if the user with name <code>username</code> is the current
-     * user
+     * @return true if the user with name <code>username</code> is the current user
      */
     public boolean isCurrentUser(String username) {
-        if (username == null || this.userName == null) return false;
-        return this.userName.equals(username);
+        return username != null && this.userName != null && this.userName.equals(username);
+    }
+
+    /**
+     * Replies true if the current user is {@link #isFullyIdentified() fully identified} and the {@link #getUserId() user ids} match,
+     * or if the current user is not {@link #isFullyIdentified() fully identified} and the {@link #userName user names} match.
+     *
+     * @param user the user to test
+     * @return true if given user is the current user
+     */
+    public boolean isCurrentUser(User user) {
+        if (user == null) {
+            return false;
+        } else if (isFullyIdentified()) {
+            return getUserId() == user.getId();
+        } else {
+            return isCurrentUser(user.getName());
+        }
     }
 

trunk/src/org/openstreetmap/josm/gui/io/OpenChangesetComboBoxModel.java

@@ -41 +41 @@
     public void refresh() {
         changesets.clear();
-        changesets.addAll(ChangesetCache.getInstance().getOpenChangesets());
+        changesets.addAll(ChangesetCache.getInstance().getOpenChangesetsForCurrentUser());
         fireContentsChanged(this, 0, getSize());
         int idx = changesets.indexOf(selectedChangeset);