Changeset 5998 in josm


Ignore:
Timestamp:
2013-06-10T00:55:26+02:00 (7 years ago)
Author:
Don-vip
Message:

fix #8775 - IllegalComponentStateException when remembering position of dialogs not displayed on screen

Location:
trunk/src/org/openstreetmap/josm/gui
Files:
13 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/actionsupport/DeleteFromRelationConfirmationDialog.java

    r5983 r5998  
    169169            ).applySafe(this);
    170170            setCanceled(false);
    171         } else if(!visible && isShowing()) {
     171        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    172172            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
    173173        }
  • trunk/src/org/openstreetmap/josm/gui/conflict/tags/CombinePrimitiveResolverDialog.java

    r5988 r5998  
    374374            setCanceled(false);
    375375            btnApply.requestFocusInWindow();
    376         } else {
     376        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    377377            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
    378378        }
  • trunk/src/org/openstreetmap/josm/gui/conflict/tags/PasteTagsConflictResolverDialog.java

    r4387 r5998  
    319319                    WindowGeometry.centerOnScreen(new Dimension(400,300))
    320320            ).applySafe(this);
    321         } else {
     321        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    322322            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
    323323        }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/ConflictResolutionDialog.java

    r5297 r5998  
    5656                new Dimension(600, 400))).applySafe(this);
    5757        } else {
    58             new WindowGeometry(this).remember(geom);
     58            if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
     59                new WindowGeometry(this).remember(geom);
     60            }
    5961            unregisterListeners();
    6062        }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManager.java

    r5958 r5998  
    301301            model.init();
    302302
    303         } else if (!visible && isShowing()){
     303        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    304304            model.tearDown();
    305305            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/ChangesetQueryDialog.java

    r4864 r5998  
    152152            setCanceled(false);
    153153            startUserInput();
    154         } else if (!visible && isShowing()){
     154        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    155155            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
    156156            pnlAdvancedQueries.rememberSettings();
  • trunk/src/org/openstreetmap/josm/gui/download/DownloadDialog.java

    r5460 r5998  
    366366                    )
    367367            ).applySafe(this);
    368         } else if (!visible && isShowing()){
     368        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    369369            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
    370370        }
  • trunk/src/org/openstreetmap/josm/gui/help/HelpBrowser.java

    r5938 r5998  
    209209                    )
    210210            ).applySafe(this);
    211         } else if (!visible && isShowing()){
     211        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    212212            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
    213213        }
  • trunk/src/org/openstreetmap/josm/gui/io/CloseChangesetDialog.java

    r5200 r5998  
    103103                    WindowGeometry.centerInWindow(Main.parent, new Dimension(300,300))
    104104            ).applySafe(this);
    105         } else {
     105        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    106106            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
    107107        }
  • trunk/src/org/openstreetmap/josm/gui/io/UploadDialog.java

    r5842 r5998  
    365365            ).applySafe(this);
    366366            startUserInput();
    367         } else if (!visible && isShowing()){
     367        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    368368            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
    369369        }
  • trunk/src/org/openstreetmap/josm/gui/io/UploadSelectionDialog.java

    r4619 r5998  
    170170                    )
    171171            ).applySafe(this);
    172         } else if (!visible && isShowing()){
     172        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    173173            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
    174174        }
  • trunk/src/org/openstreetmap/josm/gui/oauth/OAuthAuthorizationWizard.java

    r5899 r5998  
    295295            ).applySafe(this);
    296296            initFromPreferences();
    297         } else if (!visible && isShowing()){
     297        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    298298            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
    299299        }
  • trunk/src/org/openstreetmap/josm/gui/preferences/PreferenceDialog.java

    r5631 r5998  
    121121                    )
    122122            ).applySafe(this);
    123         } else if (!visible && isShowing()){
     123        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
    124124            new WindowGeometry(this).remember(getClass().getName() + ".geometry");
    125125        }
Note: See TracChangeset for help on using the changeset viewer.