Changeset 12743 in josm


Ignore:
Timestamp:
2017-09-05T21:13:21+02:00 (3 months ago)
Author:
Don-vip
Message:

see #15229 - see #15182 - deprecate gui.JosmUserIdentityManager - replaced by data.UserIdentityManager

Location:
trunk
Files:
1 added
18 edited
1 moved

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/ChangesetQueryTask.java

    r12620 r12743  
    1010
    1111import org.openstreetmap.josm.Main;
     12import org.openstreetmap.josm.data.UserIdentityManager;
    1213import org.openstreetmap.josm.data.osm.UserInfo;
    13 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    1414import org.openstreetmap.josm.gui.util.GuiHelper;
    1515import org.openstreetmap.josm.io.ChangesetQuery;
     
    4848
    4949            UserInfo info = userInfoReader.fetchUserInfo(getProgressMonitor().createSubTaskMonitor(1, false));
    50             JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     50            UserIdentityManager im = UserIdentityManager.getInstance();
    5151            im.setFullyIdentified(im.getUserName(), info);
    5252        }
     
    5555        protected void realRun() throws SAXException, IOException, OsmTransferException {
    5656            try {
    57                 JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     57                UserIdentityManager im = UserIdentityManager.getInstance();
    5858                if (query.isRestrictedToPartiallyIdentifiedUser() && im.isCurrentUser(query.getUserName())) {
    5959                    // if we query changesets for the current user, make sure we query against
     
    6363                        fullyIdentifyCurrentUser();
    6464                    }
    65                     query = query.forUser(JosmUserIdentityManager.getInstance().getUserId());
     65                    query = query.forUser(UserIdentityManager.getInstance().getUserId());
    6666                }
    6767                if (isCanceled())
  • trunk/src/org/openstreetmap/josm/data/osm/ChangesetCache.java

    r12537 r12743  
    1515import org.openstreetmap.josm.data.Preferences.PreferenceChangeEvent;
    1616import org.openstreetmap.josm.data.Preferences.PreferenceChangedListener;
    17 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
     17import org.openstreetmap.josm.data.UserIdentityManager;
    1818import org.openstreetmap.josm.gui.util.GuiHelper;
    1919import org.openstreetmap.josm.tools.SubclassFilteredCollection;
     
    244244
    245245    /**
    246      * If the current user {@link JosmUserIdentityManager#isAnonymous() is known}, the {@link #getOpenChangesets() open changesets}
    247      * for the {@link JosmUserIdentityManager#isCurrentUser(User) current user} are returned. Otherwise,
     246     * If the current user {@link UserIdentityManager#isAnonymous() is known}, the {@link #getOpenChangesets() open changesets}
     247     * for the {@link UserIdentityManager#isCurrentUser(User) current user} are returned. Otherwise,
    248248     * the unfiltered {@link #getOpenChangesets() open changesets} are returned.
    249249     *
     
    251251     */
    252252    public List<Changeset> getOpenChangesetsForCurrentUser() {
    253         if (JosmUserIdentityManager.getInstance().isAnonymous()) {
     253        if (UserIdentityManager.getInstance().isAnonymous()) {
    254254            return getOpenChangesets();
    255255        } else {
    256256            return new ArrayList<>(SubclassFilteredCollection.filter(getOpenChangesets(),
    257                     object -> JosmUserIdentityManager.getInstance().isCurrentUser(object.getUser())));
     257                    object -> UserIdentityManager.getInstance().isCurrentUser(object.getUser())));
    258258        }
    259259    }
  • trunk/src/org/openstreetmap/josm/data/osm/NoteData.java

    r12734 r12743  
    1010import java.util.Map;
    1111
     12import org.openstreetmap.josm.data.UserIdentityManager;
    1213import org.openstreetmap.josm.data.coor.LatLon;
    1314import org.openstreetmap.josm.data.notes.Note;
    1415import org.openstreetmap.josm.data.notes.Note.State;
    1516import org.openstreetmap.josm.data.notes.NoteComment;
    16 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    1717import org.openstreetmap.josm.tools.ListenerList;
    1818import org.openstreetmap.josm.tools.Logging;
     
    234234
    235235    private static User getCurrentUser() {
    236         JosmUserIdentityManager userMgr = JosmUserIdentityManager.getInstance();
     236        UserIdentityManager userMgr = UserIdentityManager.getInstance();
    237237        return User.createOsmUser(userMgr.getUserId(), userMgr.getUserName());
    238238    }
  • trunk/src/org/openstreetmap/josm/gui/JosmUserIdentityManager.java

    r12686 r12743  
    22package org.openstreetmap.josm.gui;
    33
    4 import static org.openstreetmap.josm.tools.I18n.tr;
    5 
    6 import java.text.MessageFormat;
    7 
    8 import org.openstreetmap.josm.Main;
    94import org.openstreetmap.josm.data.Preferences.PreferenceChangeEvent;
    10 import org.openstreetmap.josm.data.Preferences.PreferenceChangedListener;
    11 import org.openstreetmap.josm.data.oauth.OAuthAccessTokenHolder;
     5import org.openstreetmap.josm.data.UserIdentityManager;
    126import org.openstreetmap.josm.data.osm.User;
    137import org.openstreetmap.josm.data.osm.UserInfo;
    14 import org.openstreetmap.josm.data.preferences.StringSetting;
    15 import org.openstreetmap.josm.gui.progress.NullProgressMonitor;
    16 import org.openstreetmap.josm.io.OnlineResource;
    17 import org.openstreetmap.josm.io.OsmApi;
    18 import org.openstreetmap.josm.io.OsmServerUserInfoReader;
    19 import org.openstreetmap.josm.io.OsmTransferException;
    20 import org.openstreetmap.josm.io.auth.CredentialsManager;
    21 import org.openstreetmap.josm.tools.CheckParameterUtil;
    22 import org.openstreetmap.josm.tools.JosmRuntimeException;
    23 import org.openstreetmap.josm.tools.Logging;
    248
    259/**
     
    5034 *   <li>safely search for objects last touched by the current user based on its user id, not on its user name</li>
    5135 * </ul>
    52  *
     36 * @since 2689 (creation)
     37 * @deprecated to be removed end of 2017. Use {@link UserIdentityManager} instead
    5338 */
    54 public final class JosmUserIdentityManager implements PreferenceChangedListener {
     39@Deprecated
     40public final class JosmUserIdentityManager {
    5541
    5642    private static JosmUserIdentityManager instance;
     
    6450        if (instance == null) {
    6551            instance = new JosmUserIdentityManager();
    66             if (OsmApi.isUsingOAuth() && OAuthAccessTokenHolder.getInstance().containsAccessToken() &&
    67                     !Main.isOffline(OnlineResource.OSM_API)) {
    68                 try {
    69                     instance.initFromOAuth();
    70                 } catch (JosmRuntimeException | IllegalArgumentException | IllegalStateException e) {
    71                     Logging.error(e);
    72                     // Fall back to preferences if OAuth identification fails for any reason
    73                     instance.initFromPreferences();
    74                 }
    75             } else {
    76                 instance.initFromPreferences();
    77             }
    78             Main.pref.addPreferenceChangeListener(instance);
     52            UserIdentityManager.getInstance();
    7953        }
    8054        return instance;
    8155    }
    82 
    83     private String userName;
    84     private UserInfo userInfo;
    85     private boolean accessTokenKeyChanged;
    86     private boolean accessTokenSecretChanged;
    8756
    8857    private JosmUserIdentityManager() {
     
    9362     */
    9463    public void setAnonymous() {
    95         userName = null;
    96         userInfo = null;
     64        UserIdentityManager.getInstance().setAnonymous();
    9765    }
    9866
     
    10674     */
    10775    public void setPartiallyIdentified(String userName) {
    108         CheckParameterUtil.ensureParameterNotNull(userName, "userName");
    109         String trimmedUserName = userName.trim();
    110         if (trimmedUserName.isEmpty())
    111             throw new IllegalArgumentException(
    112                     MessageFormat.format("Expected non-empty value for parameter ''{0}'', got ''{1}''", "userName", userName));
    113         this.userName = trimmedUserName;
    114         userInfo = null;
     76        UserIdentityManager.getInstance().setPartiallyIdentified(userName);
    11577    }
    11678
     
    12688     */
    12789    public void setFullyIdentified(String userName, UserInfo userInfo) {
    128         CheckParameterUtil.ensureParameterNotNull(userName, "userName");
    129         String trimmedUserName = userName.trim();
    130         if (trimmedUserName.isEmpty())
    131             throw new IllegalArgumentException(tr("Expected non-empty value for parameter ''{0}'', got ''{1}''", "userName", userName));
    132         CheckParameterUtil.ensureParameterNotNull(userInfo, "userInfo");
    133         this.userName = trimmedUserName;
    134         this.userInfo = userInfo;
     90        UserIdentityManager.getInstance().setFullyIdentified(userName, userInfo);
    13591    }
    13692
     
    14197     */
    14298    public boolean isAnonymous() {
    143         return userName == null && userInfo == null;
     99        return UserIdentityManager.getInstance().isAnonymous();
    144100    }
    145101
     
    150106     */
    151107    public boolean isPartiallyIdentified() {
    152         return userName != null && userInfo == null;
     108        return UserIdentityManager.getInstance().isPartiallyIdentified();
    153109    }
    154110
     
    159115     */
    160116    public boolean isFullyIdentified() {
    161         return userName != null && userInfo != null;
     117        return UserIdentityManager.getInstance().isFullyIdentified();
    162118    }
    163119
     
    168124     */
    169125    public String getUserName() {
    170         return userName;
     126        return UserIdentityManager.getInstance().getUserName();
    171127    }
    172128
     
    178134     */
    179135    public int getUserId() {
    180         if (userInfo == null) return 0;
    181         return userInfo.getId();
     136        return UserIdentityManager.getInstance().getUserId();
    182137    }
    183138
     
    189144     */
    190145    public UserInfo getUserInfo() {
    191         return userInfo;
     146        return UserIdentityManager.getInstance().getUserInfo();
    192147    }
    193148
     
    198153     */
    199154    public User asUser() {
    200         return isAnonymous() ? User.getAnonymous() : User.createOsmUser(userInfo != null ? userInfo.getId() : 0, userName);
     155        return UserIdentityManager.getInstance().asUser();
    201156    }
    202157
     
    207162     */
    208163    public void initFromPreferences() {
    209         String userName = CredentialsManager.getInstance().getUsername();
    210         if (isAnonymous()) {
    211             if (userName != null && !userName.trim().isEmpty()) {
    212                 setPartiallyIdentified(userName);
    213             }
    214         } else {
    215             if (userName != null && !userName.equals(this.userName)) {
    216                 setPartiallyIdentified(userName);
    217             }
    218             // else: same name in the preferences as JOSM already knows about.
    219             // keep the state, be it partially or fully identified
    220         }
     164        UserIdentityManager.getInstance().initFromPreferences();
    221165    }
    222166
     
    228172     */
    229173    public void initFromOAuth() {
    230         try {
    231             UserInfo info = new OsmServerUserInfoReader().fetchUserInfo(NullProgressMonitor.INSTANCE);
    232             setFullyIdentified(info.getDisplayName(), info);
    233         } catch (IllegalArgumentException | OsmTransferException e) {
    234             Logging.error(e);
    235         }
     174        UserIdentityManager.getInstance().initFromOAuth();
    236175    }
    237176
     
    243182     */
    244183    public boolean isCurrentUser(String username) {
    245         return this.userName != null && this.userName.equals(username);
     184        return UserIdentityManager.getInstance().isCurrentUser(username);
    246185    }
    247186
    248187    /**
    249188     * Replies true if the current user is {@link #isFullyIdentified() fully identified} and the {@link #getUserId() user ids} match,
    250      * or if the current user is not {@link #isFullyIdentified() fully identified} and the {@link #userName user names} match.
     189     * or if the current user is not {@link #isFullyIdentified() fully identified} and the {@link #getUserName() user names} match.
    251190     *
    252191     * @param user the user to test
     
    254193     */
    255194    public boolean isCurrentUser(User user) {
    256         if (user == null) {
    257             return false;
    258         } else if (isFullyIdentified()) {
    259             return getUserId() == user.getId();
    260         } else {
    261             return isCurrentUser(user.getName());
    262         }
    263     }
    264 
    265     /* ------------------------------------------------------------------- */
    266     /* interface PreferenceChangeListener                                  */
    267     /* ------------------------------------------------------------------- */
    268     @Override
    269     public void preferenceChanged(PreferenceChangeEvent evt) {
    270         switch (evt.getKey()) {
    271         case "osm-server.username":
    272             String newUserName = null;
    273             if (evt.getNewValue() instanceof StringSetting) {
    274                 newUserName = ((StringSetting) evt.getNewValue()).getValue();
    275             }
    276             if (newUserName == null || newUserName.trim().isEmpty()) {
    277                 setAnonymous();
    278             } else {
    279                 if (!newUserName.equals(userName)) {
    280                     setPartiallyIdentified(newUserName);
    281                 }
    282             }
    283             return;
    284         case "osm-server.url":
    285             String newUrl = null;
    286             if (evt.getNewValue() instanceof StringSetting) {
    287                 newUrl = ((StringSetting) evt.getNewValue()).getValue();
    288             }
    289             if (newUrl == null || newUrl.trim().isEmpty()) {
    290                 setAnonymous();
    291             } else if (isFullyIdentified()) {
    292                 setPartiallyIdentified(getUserName());
    293             }
    294             break;
    295         case "oauth.access-token.key":
    296             accessTokenKeyChanged = true;
    297             break;
    298         case "oauth.access-token.secret":
    299             accessTokenSecretChanged = true;
    300             break;
    301         default: // Do nothing
    302         }
    303 
    304         if (accessTokenKeyChanged && accessTokenSecretChanged) {
    305             accessTokenKeyChanged = false;
    306             accessTokenSecretChanged = false;
    307             if (OsmApi.isUsingOAuth()) {
    308                 getInstance().initFromOAuth();
    309             }
    310         }
     195        return UserIdentityManager.getInstance().isCurrentUser(user);
    311196    }
    312197}
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManager.java

    r12678 r12743  
    4747import org.openstreetmap.josm.actions.downloadtasks.ChangesetQueryTask;
    4848import org.openstreetmap.josm.actions.downloadtasks.PostDownloadHandler;
     49import org.openstreetmap.josm.data.UserIdentityManager;
    4950import org.openstreetmap.josm.data.osm.Changeset;
    5051import org.openstreetmap.josm.data.osm.ChangesetCache;
     
    5354import org.openstreetmap.josm.data.osm.history.HistoryOsmPrimitive;
    5455import org.openstreetmap.josm.gui.HelpAwareOptionPane;
    55 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    5656import org.openstreetmap.josm.gui.MainApplication;
    5757import org.openstreetmap.josm.gui.dialogs.changeset.query.ChangesetQueryDialog;
     
    465465        protected void updateEnabledState() {
    466466            List<Changeset> selected = model.getSelectedChangesets();
    467             JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     467            UserIdentityManager im = UserIdentityManager.getInstance();
    468468            for (Changeset cs: selected) {
    469469                if (cs.isOpen()) {
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/BasicChangesetQueryPanel.java

    r12630 r12743  
    2121import org.openstreetmap.josm.Main;
    2222import org.openstreetmap.josm.data.Bounds;
    23 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
     23import org.openstreetmap.josm.data.UserIdentityManager;
    2424import org.openstreetmap.josm.gui.MainApplication;
    2525import org.openstreetmap.josm.gui.MapView;
     
    158158        // partially identified
    159159        lbl = lblQueries.get(BasicQuery.MY_OPEN_CHANGESETS);
    160         if (JosmUserIdentityManager.getInstance().isAnonymous()) {
     160        if (UserIdentityManager.getInstance().isAnonymous()) {
    161161            rbQueries.get(BasicQuery.MY_OPEN_CHANGESETS).setEnabled(false);
    162162            lbl.setText(tr("<html>Download my open changesets<br><em>Disabled. " +
     
    232232        BasicQuery q = getSelectedQuery();
    233233        ChangesetQuery query = new ChangesetQuery();
    234         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     234        UserIdentityManager im = UserIdentityManager.getInstance();
    235235        if (q == null)
    236236            return query;
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/UserRestrictionPanel.java

    r11330 r12743  
    1919
    2020import org.openstreetmap.josm.Main;
     21import org.openstreetmap.josm.data.UserIdentityManager;
    2122import org.openstreetmap.josm.gui.HelpAwareOptionPane;
    22 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    2323import org.openstreetmap.josm.gui.help.HelpUtil;
    2424import org.openstreetmap.josm.gui.preferences.server.UserNameValidator;
     
    163163     */
    164164    public void startUserInput() {
    165         if (JosmUserIdentityManager.getInstance().isAnonymous()) {
     165        if (UserIdentityManager.getInstance().isAnonymous()) {
    166166            lblRestrictedToMyself.setText(tr("Only changesets owned by myself (disabled. JOSM is currently run by an anonymous user)"));
    167167            rbRestrictToMyself.setEnabled(false);
     
    188188        CheckParameterUtil.ensureParameterNotNull(query, "query");
    189189        if (rbRestrictToMyself.isSelected()) {
    190             JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     190            UserIdentityManager im = UserIdentityManager.getInstance();
    191191            if (im.isPartiallyIdentified()) {
    192192                query.forUser(im.getUserName());
     
    277277        String v = Main.pref.get(PREF_QUERY_TYPE, "mine");
    278278        if ("mine".equals(v)) {
    279             JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     279            UserIdentityManager im = UserIdentityManager.getInstance();
    280280            if (im.isAnonymous()) {
    281281                rbRestrictToUid.setSelected(true);
  • trunk/src/org/openstreetmap/josm/gui/download/BookmarkList.java

    r12634 r12743  
    2626import org.openstreetmap.josm.actions.downloadtasks.ChangesetQueryTask;
    2727import org.openstreetmap.josm.data.Bounds;
     28import org.openstreetmap.josm.data.UserIdentityManager;
    2829import org.openstreetmap.josm.data.coor.LatLon;
    2930import org.openstreetmap.josm.data.osm.Changeset;
     
    3233import org.openstreetmap.josm.data.projection.Projection;
    3334import org.openstreetmap.josm.data.projection.Projections;
    34 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    3535import org.openstreetmap.josm.gui.MainApplication;
    3636import org.openstreetmap.josm.gui.MapViewState;
     
    193193            setName(tr("Home location"));
    194194            setIcon(ImageProvider.get("help", "home", ImageSizes.SMALLICON));
    195             UserInfo info = JosmUserIdentityManager.getInstance().getUserInfo();
     195            UserInfo info = UserIdentityManager.getInstance().getUserInfo();
    196196            if (info == null) {
    197197                throw new IllegalStateException("User not identified");
     
    251251        final DefaultListModel<Bookmark> model = (DefaultListModel<Bookmark>) getModel();
    252252        model.removeAllElements();
    253         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     253        UserIdentityManager im = UserIdentityManager.getInstance();
    254254        // Add home location bookmark first, if user fully identified
    255255        if (im.isFullyIdentified()) {
  • trunk/src/org/openstreetmap/josm/gui/download/BookmarkSelection.java

    r12495 r12743  
    2424import org.openstreetmap.josm.Main;
    2525import org.openstreetmap.josm.data.Bounds;
    26 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
     26import org.openstreetmap.josm.data.UserIdentityManager;
    2727import org.openstreetmap.josm.gui.download.BookmarkList.Bookmark;
    2828import org.openstreetmap.josm.gui.widgets.JMultilineLabel;
     
    290290            putValue(SMALL_ICON, ImageProvider.get("dialogs/changeset", "downloadchangeset"));
    291291            putValue(SHORT_DESCRIPTION, tr("Download bookmarks for my {0} last changesets", BookmarkList.MAX_CHANGESET_BOOKMARKS.get()));
    292             setEnabled(!JosmUserIdentityManager.getInstance().isAnonymous());
     292            setEnabled(!UserIdentityManager.getInstance().isAnonymous());
    293293        }
    294294
  • trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java

    r12636 r12743  
    1111
    1212import org.openstreetmap.josm.Main;
     13import org.openstreetmap.josm.data.UserIdentityManager;
    1314import org.openstreetmap.josm.data.osm.Node;
    1415import org.openstreetmap.josm.data.osm.OsmPrimitive;
     
    3536import org.openstreetmap.josm.data.osm.history.HistoryWay;
    3637import org.openstreetmap.josm.data.osm.visitor.AbstractVisitor;
    37 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    3838import org.openstreetmap.josm.gui.MainApplication;
    3939import org.openstreetmap.josm.gui.layer.Layer;
     
    674674
    675675        private static User getCurrentUser() {
    676             UserInfo info = JosmUserIdentityManager.getInstance().getUserInfo();
     676            UserInfo info = UserIdentityManager.getInstance().getUserInfo();
    677677            return info == null ? User.getAnonymous() : User.createOsmUser(info.getId(), info.getDisplayName());
    678678        }
  • trunk/src/org/openstreetmap/josm/gui/history/VersionInfoPanel.java

    r12636 r12743  
    2525
    2626import org.openstreetmap.josm.Main;
     27import org.openstreetmap.josm.data.UserIdentityManager;
    2728import org.openstreetmap.josm.data.osm.Changeset;
    2829import org.openstreetmap.josm.data.osm.OsmPrimitive;
    2930import org.openstreetmap.josm.data.osm.User;
    3031import org.openstreetmap.josm.data.osm.history.HistoryOsmPrimitive;
    31 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    3232import org.openstreetmap.josm.gui.MainApplication;
    3333import org.openstreetmap.josm.gui.dialogs.ChangesetDialog;
     
    256256            }
    257257        } else {
    258             String username = JosmUserIdentityManager.getInstance().getUserName();
     258            String username = UserIdentityManager.getInstance().getUserName();
    259259            if (username == null) {
    260260                lblUser.setDescription(tr("anonymous"));
  • trunk/src/org/openstreetmap/josm/gui/io/DownloadOpenChangesetsTask.java

    r12620 r12743  
    1414
    1515import org.openstreetmap.josm.Main;
     16import org.openstreetmap.josm.data.UserIdentityManager;
    1617import org.openstreetmap.josm.data.osm.Changeset;
    1718import org.openstreetmap.josm.data.osm.ChangesetCache;
    1819import org.openstreetmap.josm.data.osm.UserInfo;
    1920import org.openstreetmap.josm.gui.ExceptionDialogUtil;
    20 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    2121import org.openstreetmap.josm.gui.PleaseWaitRunnable;
    2222import org.openstreetmap.josm.gui.util.GuiHelper;
     
    6161    @Override
    6262    protected void finish() {
    63         if (JosmUserIdentityManager.getInstance().isAnonymous()) {
     63        if (UserIdentityManager.getInstance().isAnonymous()) {
    6464            String msg = tr("Could not retrieve the list of your open changesets because<br>"
    6565                    + "JOSM does not know your identity.<br>"
     
    9696     */
    9797    protected void refreshUserIdentity() {
    98         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     98        UserIdentityManager im = UserIdentityManager.getInstance();
    9999        try {
    100100            OsmServerUserInfoReader infoReader = new OsmServerUserInfoReader();
     
    122122    protected void realRun() throws SAXException, IOException, OsmTransferException {
    123123        try {
    124             JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     124            UserIdentityManager im = UserIdentityManager.getInstance();
    125125            if (im.isAnonymous()) {
    126126                refreshUserIdentity();
  • trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java

    r12620 r12743  
    1717
    1818import org.openstreetmap.josm.data.Bounds;
     19import org.openstreetmap.josm.data.UserIdentityManager;
    1920import org.openstreetmap.josm.data.coor.LatLon;
    20 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    2121import org.openstreetmap.josm.tools.CheckParameterUtil;
    2222import org.openstreetmap.josm.tools.Logging;
     
    7171     */
    7272    public static ChangesetQuery forCurrentUser() {
    73         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     73        UserIdentityManager im = UserIdentityManager.getInstance();
    7474        if (im.isAnonymous()) {
    7575            throw new IllegalStateException("anonymous user");
  • trunk/src/org/openstreetmap/josm/io/MessageNotifier.java

    r12620 r12743  
    1717
    1818import org.openstreetmap.josm.Main;
     19import org.openstreetmap.josm.data.UserIdentityManager;
    1920import org.openstreetmap.josm.data.osm.UserInfo;
    2021import org.openstreetmap.josm.data.preferences.BooleanProperty;
    2122import org.openstreetmap.josm.data.preferences.IntegerProperty;
    22 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    2323import org.openstreetmap.josm.gui.Notification;
    2424import org.openstreetmap.josm.gui.progress.NullProgressMonitor;
     
    130130     */
    131131    public static boolean isUserEnoughIdentified() {
    132         JosmUserIdentityManager identManager = JosmUserIdentityManager.getInstance();
     132        UserIdentityManager identManager = UserIdentityManager.getInstance();
    133133        if (identManager.isFullyIdentified()) {
    134134            return true;
  • trunk/src/org/openstreetmap/josm/io/OsmServerWriter.java

    r12687 r12743  
    1313import java.util.concurrent.TimeUnit;
    1414
     15import org.openstreetmap.josm.data.UserIdentityManager;
    1516import org.openstreetmap.josm.data.osm.Changeset;
    1617import org.openstreetmap.josm.data.osm.OsmPrimitive;
    1718import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    18 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    1919import org.openstreetmap.josm.gui.progress.NullProgressMonitor;
    2020import org.openstreetmap.josm.gui.progress.ProgressMonitor;
     
    202202                api.openChangeset(changeset, monitor.createSubTaskMonitor(0, false));
    203203                // update the user information
    204                 changeset.setUser(JosmUserIdentityManager.getInstance().asUser());
     204                changeset.setUser(UserIdentityManager.getInstance().asUser());
    205205            } else {
    206206                api.updateChangeset(changeset, monitor.createSubTaskMonitor(0, false));
  • trunk/src/org/openstreetmap/josm/io/auth/CredentialsManager.java

    r12620 r12743  
    77import java.util.Objects;
    88
     9import org.openstreetmap.josm.data.UserIdentityManager;
    910import org.openstreetmap.josm.data.oauth.OAuthToken;
    10 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    1111import org.openstreetmap.josm.io.OsmApi;
    1212import org.openstreetmap.josm.tools.CheckParameterUtil;
     
    122122            String username = credentials.getUserName();
    123123            if (username != null && !username.trim().isEmpty()) {
    124                 JosmUserIdentityManager.getInstance().setPartiallyIdentified(username);
     124                UserIdentityManager.getInstance().setPartiallyIdentified(username);
    125125            }
    126126        }
  • trunk/test/unit/org/openstreetmap/josm/data/UserIdentityManagerTest.java

    r12742 r12743  
    11// License: GPL. For details, see LICENSE file.
    2 package org.openstreetmap.josm.gui;
     2package org.openstreetmap.josm.data;
    33
    44import static org.junit.Assert.assertEquals;
     
    1717
    1818/**
    19  * Unit tests of {@link JosmUserIdentityManager} class.
     19 * Unit tests of {@link UserIdentityManager} class.
    2020 */
    21 public class JosmUserIdentityManagerTest {
     21public class UserIdentityManagerTest {
    2222
    2323    /**
     
    4646    public void testSingletonAccess() {
    4747
    48         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     48        UserIdentityManager im = UserIdentityManager.getInstance();
    4949
    5050        // created ?
    5151        assertNotNull(im);
    5252
    53         JosmUserIdentityManager im2 = JosmUserIdentityManager.getInstance();
     53        UserIdentityManager im2 = UserIdentityManager.getInstance();
    5454
    5555        // only one instance
     
    5858
    5959    /**
    60      * Unit test of {@link JosmUserIdentityManager#setAnonymous}.
     60     * Unit test of {@link UserIdentityManager#setAnonymous}.
    6161     */
    6262    @Test
    6363    public void testSetAnonymous() {
    64         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     64        UserIdentityManager im = UserIdentityManager.getInstance();
    6565
    6666        im.setPartiallyIdentified("test");
     
    7878
    7979    /**
    80      * Unit test of {@link JosmUserIdentityManager#setPartiallyIdentified} - nominal case.
     80     * Unit test of {@link UserIdentityManager#setPartiallyIdentified} - nominal case.
    8181     */
    8282    @Test
    8383    public void testSetPartiallyIdentified() {
    84         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     84        UserIdentityManager im = UserIdentityManager.getInstance();
    8585        im.setPartiallyIdentified("test");
    8686
     
    9898
    9999    /**
    100      * Unit test of {@link JosmUserIdentityManager#setPartiallyIdentified} - null case.
     100     * Unit test of {@link UserIdentityManager#setPartiallyIdentified} - null case.
    101101     */
    102102    @Test(expected = IllegalArgumentException.class)
    103103    public void testSetPartiallyIdentifiedNull() {
    104         JosmUserIdentityManager.getInstance().setPartiallyIdentified(null);
    105     }
    106 
    107     /**
    108      * Unit test of {@link JosmUserIdentityManager#setPartiallyIdentified} - empty case.
     104        UserIdentityManager.getInstance().setPartiallyIdentified(null);
     105    }
     106
     107    /**
     108     * Unit test of {@link UserIdentityManager#setPartiallyIdentified} - empty case.
    109109     */
    110110    @Test(expected = IllegalArgumentException.class)
    111111    public void testSetPartiallyIdentifiedEmpty() {
    112         JosmUserIdentityManager.getInstance().setPartiallyIdentified("");
    113     }
    114 
    115     /**
    116      * Unit test of {@link JosmUserIdentityManager#setPartiallyIdentified} - blank case.
     112        UserIdentityManager.getInstance().setPartiallyIdentified("");
     113    }
     114
     115    /**
     116     * Unit test of {@link UserIdentityManager#setPartiallyIdentified} - blank case.
    117117     */
    118118    @Test(expected = IllegalArgumentException.class)
    119119    public void testSetPartiallyIdentifiedBlank() {
    120         JosmUserIdentityManager.getInstance().setPartiallyIdentified("  \t  ");
    121     }
    122 
    123     /**
    124      * Unit test of {@link JosmUserIdentityManager#setFullyIdentified} - nominal case.
     120        UserIdentityManager.getInstance().setPartiallyIdentified("  \t  ");
     121    }
     122
     123    /**
     124     * Unit test of {@link UserIdentityManager#setFullyIdentified} - nominal case.
    125125     */
    126126    @Test
    127127    public void testSetFullyIdentified() {
    128         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     128        UserIdentityManager im = UserIdentityManager.getInstance();
    129129
    130130        UserInfo userInfo = newUserInfo();
     
    145145
    146146    /**
    147      * Unit test of {@link JosmUserIdentityManager#setFullyIdentified} - null name case.
     147     * Unit test of {@link UserIdentityManager#setFullyIdentified} - null name case.
    148148     */
    149149    @Test(expected = IllegalArgumentException.class)
    150150    public void testSetFullyIdentifiedNullName() {
    151         JosmUserIdentityManager.getInstance().setFullyIdentified(null, newUserInfo());
    152     }
    153 
    154     /**
    155      * Unit test of {@link JosmUserIdentityManager#setFullyIdentified} - empty name case.
     151        UserIdentityManager.getInstance().setFullyIdentified(null, newUserInfo());
     152    }
     153
     154    /**
     155     * Unit test of {@link UserIdentityManager#setFullyIdentified} - empty name case.
    156156     */
    157157    @Test(expected = IllegalArgumentException.class)
    158158    public void testSetFullyIdentifiedEmptyName() {
    159         JosmUserIdentityManager.getInstance().setFullyIdentified("", newUserInfo());
    160     }
    161 
    162     /**
    163      * Unit test of {@link JosmUserIdentityManager#setFullyIdentified} - blank name case.
     159        UserIdentityManager.getInstance().setFullyIdentified("", newUserInfo());
     160    }
     161
     162    /**
     163     * Unit test of {@link UserIdentityManager#setFullyIdentified} - blank name case.
    164164     */
    165165    @Test(expected = IllegalArgumentException.class)
    166166    public void testSetFullyIdentifiedBlankName() {
    167         JosmUserIdentityManager.getInstance().setFullyIdentified(" \t ", newUserInfo());
    168     }
    169 
    170     /**
    171      * Unit test of {@link JosmUserIdentityManager#setFullyIdentified} - null info case.
     167        UserIdentityManager.getInstance().setFullyIdentified(" \t ", newUserInfo());
     168    }
     169
     170    /**
     171     * Unit test of {@link UserIdentityManager#setFullyIdentified} - null info case.
    172172     */
    173173    @Test(expected = IllegalArgumentException.class)
    174174    public void testSetFullyIdentifiedNullInfo() {
    175         JosmUserIdentityManager.getInstance().setFullyIdentified("test", null);
     175        UserIdentityManager.getInstance().setFullyIdentified("test", null);
    176176    }
    177177
     
    181181    @Test
    182182    public void testInitFromPreferences1() {
    183         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     183        UserIdentityManager im = UserIdentityManager.getInstance();
    184184
    185185        // reset it
     
    206206    @Test
    207207    public void testInitFromPreferences2() {
    208         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     208        UserIdentityManager im = UserIdentityManager.getInstance();
    209209
    210210        // reset it
     
    231231    @Test
    232232    public void testInitFromPreferences3() {
    233         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     233        UserIdentityManager im = UserIdentityManager.getInstance();
    234234
    235235        // for this test we disable the listener
     
    258258    @Test
    259259    public void testInitFromPreferences4() {
    260         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     260        UserIdentityManager im = UserIdentityManager.getInstance();
    261261
    262262        // for this test we disable the listener
     
    284284    @Test
    285285    public void testInitFromPreferences5() {
    286         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     286        UserIdentityManager im = UserIdentityManager.getInstance();
    287287
    288288        // for this test we disable the listener
     
    305305    @Test
    306306    public void testApiUrlChanged() {
    307         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     307        UserIdentityManager im = UserIdentityManager.getInstance();
    308308
    309309        // reset it
     
    341341
    342342    @Test
    343     //@Ignore
    344343    public void testUserNameChanged() {
    345         JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
     344        UserIdentityManager im = UserIdentityManager.getInstance();
    346345
    347346        // reset it
  • trunk/test/unit/org/openstreetmap/josm/data/osm/ChangesetCacheTest.java

    r12034 r12743  
    1515import org.junit.Test;
    1616import org.openstreetmap.josm.TestUtils;
    17 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
     17import org.openstreetmap.josm.data.UserIdentityManager;
    1818
    1919/**
     
    259259        assertEquals(3, cache.getChangesets().size());
    260260
    261         JosmUserIdentityManager.getInstance().setAnonymous();
     261        UserIdentityManager.getInstance().setAnonymous();
    262262        assertEquals(2, cache.getOpenChangesetsForCurrentUser().size());
    263263
    264         JosmUserIdentityManager.getInstance().setPartiallyIdentified("foo");
     264        UserIdentityManager.getInstance().setPartiallyIdentified("foo");
    265265        assertEquals(1, cache.getOpenChangesetsForCurrentUser().size());
    266266    }
  • trunk/test/unit/org/openstreetmap/josm/gui/io/DownloadOpenChangesetsTaskTest.java

    r11110 r12743  
    1010import org.junit.Rule;
    1111import org.junit.Test;
    12 import org.openstreetmap.josm.gui.JosmUserIdentityManager;
     12import org.openstreetmap.josm.data.UserIdentityManager;
    1313import org.openstreetmap.josm.testutils.JOSMTestRules;
    1414
     
    3535        assertNull(task.getChangesets());
    3636
    37         assertTrue(JosmUserIdentityManager.getInstance().isAnonymous());
     37        assertTrue(UserIdentityManager.getInstance().isAnonymous());
    3838        task.run();
    3939        assertNull(task.getChangesets());
    4040
    4141        task = new DownloadOpenChangesetsTask(new JPanel());
    42         JosmUserIdentityManager.getInstance().setPartiallyIdentified(System.getProperty("osm.username", "josm_test"));
    43         assertTrue(JosmUserIdentityManager.getInstance().isPartiallyIdentified());
     42        UserIdentityManager.getInstance().setPartiallyIdentified(System.getProperty("osm.username", "josm_test"));
     43        assertTrue(UserIdentityManager.getInstance().isPartiallyIdentified());
    4444        task.run();
    4545        assertNotNull(task.getChangesets());
Note: See TracChangeset for help on using the changeset viewer.