Changeset 9917 in josm


Ignore:
Timestamp:
2016-03-03T23:06:36+01:00 (4 years ago)
Author:
Don-vip
Message:

better handling of HelpAwareOptionPane.showOptionDialog in headless mode

Location:
trunk/src/org/openstreetmap/josm
Files:
7 edited

Legend:

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

    r9136 r9917  
    8585                )
    8686        };
    87         int ret = HelpAwareOptionPane.showOptionDialog(
     87        return 0 == HelpAwareOptionPane.showOptionDialog(
    8888                Main.parent,
    8989                msg,
     
    9595                HelpUtil.ht("/WarningMessages#ToManyBrowsersToOpen")
    9696        );
    97         return ret == 0;
    9897    }
    9998
  • trunk/src/org/openstreetmap/josm/actions/SimplifyWayAction.java

    r9230 r9917  
    7878                                )
    7979        };
    80         int ret = HelpAwareOptionPane.showOptionDialog(
     80        return 0 == HelpAwareOptionPane.showOptionDialog(
    8181                Main.parent,
    8282                tr(
     
    9191                        HelpUtil.ht("/Action/SimplifyWay#ConfirmSimplifyAll")
    9292                );
    93         return ret == 0;
    9493    }
    9594
  • trunk/src/org/openstreetmap/josm/gui/HelpAwareOptionPane.java

    r9512 r9917  
    66import java.awt.Component;
    77import java.awt.Dialog.ModalityType;
     8import java.awt.GraphicsEnvironment;
    89import java.awt.event.ActionEvent;
    910import java.awt.event.KeyEvent;
     
    241242        );
    242243
     244        if (!GraphicsEnvironment.isHeadless()) {
     245            doShowOptionDialog(parentComponent, title, options, defaultOption, helpTopic, buttons, pane);
     246        }
     247        return (Integer) pane.getValue();
     248    }
     249
     250    private static void doShowOptionDialog(Component parentComponent, String title, final ButtonSpec[] options,
     251            final ButtonSpec defaultOption, final String helpTopic, final List<JButton> buttons,
     252            final JOptionPane pane) {
    243253        final JDialog dialog = new JDialog(
    244254                JOptionPane.getFrameForComponent(parentComponent),
     
    302312        }
    303313        dialog.setVisible(true);
    304         return (Integer) pane.getValue();
    305314    }
    306315
  • trunk/src/org/openstreetmap/josm/gui/io/AbstractUploadTask.java

    r9246 r9917  
    243243                options[0],
    244244                "/Action/Upload#NodeStillInUseInWay"
    245 );
     245        );
    246246        if (ret == 0) {
    247247            DownloadReferrersAction.downloadReferrers(Main.main.getEditLayer(), Arrays.asList(conflict.a));
  • trunk/src/org/openstreetmap/josm/gui/layer/gpx/DownloadAlongPanel.java

    r9804 r9917  
    55
    66import java.awt.Dimension;
    7 import java.awt.GraphicsEnvironment;
    87import java.awt.GridBagLayout;
    98
     
    186185        });
    187186
    188         int ret = 0;
    189         if (!GraphicsEnvironment.isHeadless()) {
    190             ret = HelpAwareOptionPane.showOptionDialog(Main.parent, this, title,
     187        int ret = HelpAwareOptionPane.showOptionDialog(Main.parent, this, title,
    191188                    JOptionPane.QUESTION_MESSAGE, null, options, options[0], helpTopic);
    192         }
    193189        if (0 == ret) {
    194190            rememberSettings();
  • trunk/src/org/openstreetmap/josm/gui/oauth/AdvancedOAuthPropertiesPanel.java

    r9355 r9917  
    171171                )
    172172        };
    173         int ret = HelpAwareOptionPane.showOptionDialog(
     173        return 0 == HelpAwareOptionPane.showOptionDialog(
    174174                this,
    175175                tr(
     
    184184                HelpUtil.ht("/Dialog/OAuthAuthorisationWizard")
    185185        );
    186 
    187         return ret == 0; // OK button clicked
    188186    }
    189187
  • trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java

    r9870 r9917  
    88import java.awt.Component;
    99import java.awt.Font;
    10 import java.awt.GraphicsEnvironment;
    1110import java.awt.GridBagConstraints;
    1211import java.awt.GridBagLayout;
     
    856855                plugins.size()))
    857856          .append("</html>");
    858         if (!GraphicsEnvironment.isHeadless()) {
    859             HelpAwareOptionPane.showOptionDialog(
    860                     parent,
    861                     sb.toString(),
    862                     tr("Warning"),
    863                     JOptionPane.WARNING_MESSAGE,
    864                     HelpUtil.ht("/Plugin/Loading#MissingPluginInfos")
    865             );
    866         }
     857        HelpAwareOptionPane.showOptionDialog(
     858                parent,
     859                sb.toString(),
     860                tr("Warning"),
     861                JOptionPane.WARNING_MESSAGE,
     862                HelpUtil.ht("/Plugin/Loading#MissingPluginInfos")
     863        );
    867864    }
    868865
     
    11091106                )
    11101107        };
    1111         int ret = -1;
    1112         if (!GraphicsEnvironment.isHeadless()) {
    1113             ret = HelpAwareOptionPane.showOptionDialog(
     1108        return 0 == HelpAwareOptionPane.showOptionDialog(
    11141109                    parent,
    11151110                    reason,
     
    11211116                    null // FIXME: add help topic
    11221117            );
    1123         }
    1124         return ret == 0;
    11251118    }
    11261119
Note: See TracChangeset for help on using the changeset viewer.