Ignore:
Timestamp:
2016-06-24T09:10:57+02:00 (9 years ago)
Author:
donvip
Message:

checkstyle, update to JOSM 10279

File:
1 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/reltoolbox/src/relcontext/RelContextDialog.java

    r30841 r32395  
     1// License: GPL. For details, see LICENSE file.
    12package relcontext;
    23
    … …  
    103104public class RelContextDialog extends ToggleDialog implements EditLayerChangeListener, ChosenRelationListener, SelectionChangedListener {
    104105
    105     public final static String PREF_PREFIX = "reltoolbox";
     106    public static final String PREF_PREFIX = "reltoolbox";
    106107
    107108    private final DefaultTableModel relationsData;
    … …  
    124125        chosenRelation = new ChosenRelation();
    125126        chosenRelation.addChosenRelationListener(this);
    126         MapView.addEditLayerChangeListener(chosenRelation);
     127        Main.getLayerManager().addActiveLayerChangeListener(chosenRelation);
    127128
    128129        popupMenu = new ChosenRelationPopupMenu(chosenRelation);
    … …  
    144145        roleBox.addItemListener(new ItemListener() {
    145146            @Override
    146             public void itemStateChanged( ItemEvent e ) {
    147                 if( e.getStateChange() == ItemEvent.DESELECTED ) return;
     147            public void itemStateChanged(ItemEvent e) {
     148                if (e.getStateChange() == ItemEvent.DESELECTED ) return;
    148149                String memberRole = roleBoxModel.getSelectedMembersRole();
    149150                String selectedRole = roleBoxModel.isAnotherRoleSelected() ? askForRoleName() : roleBoxModel.getSelectedRole();
    150                 if( memberRole != null && selectedRole != null && !memberRole.equals(selectedRole) ) {
     151                if (memberRole != null && selectedRole != null && !memberRole.equals(selectedRole)) {
    151152                    applyRoleToSelection(selectedRole.trim());
    152153                }
    … …  
    177178        roleBox.addPropertyChangeListener("enabled", new PropertyChangeListener() {
    178179            @Override
    179             public void propertyChange( PropertyChangeEvent evt ) {
     180            public void propertyChange(PropertyChangeEvent evt) {
    180181                boolean showRoleBox = roleBox.isEnabled();
    181182                roleBox.setVisible(showRoleBox);
    … …  
    186187        sortAndFixAction.addPropertyChangeListener(new PropertyChangeListener() {
    187188            @Override
    188             public void propertyChange( PropertyChangeEvent evt ) {
     189            public void propertyChange(PropertyChangeEvent evt) {
    189190                sortAndFixButton.setVisible(sortAndFixAction.isEnabled());
    190191            }
    … …  
    194195        downloadChosenRelationAction.addPropertyChangeListener(new PropertyChangeListener() {
    195196            @Override
    196             public void propertyChange( PropertyChangeEvent evt ) {
     197            public void propertyChange(PropertyChangeEvent evt) {
    197198                downloadButton.setVisible(downloadChosenRelationAction.isEnabled());
    198199            }
    199200        });
    200201        downloadButton.setVisible(false);
    201         if( Main.pref.getBoolean(PREF_PREFIX + ".hidetopline", false) )
     202        if (Main.pref.getBoolean(PREF_PREFIX + ".hidetopline", false) ) {
    202203            chosenRelationPanel.setVisible(false);
     204        }
    203205
    204206        // [+][Multi] [X]Adm [X]Tags [X]1
    … …  
    209211        final JButton multipolygonButton = new JButton(createMultipolygonAction);
    210212        bottomLine.add(multipolygonButton, GBC.std());
    211 //        bottomLine.add(sizeButton(new JButton(new MultipolygonSettingsAction()), 16, 0), GBC.std().fill(GBC.VERTICAL));
     213        //        bottomLine.add(sizeButton(new JButton(new MultipolygonSettingsAction()), 16, 0), GBC.std().fill(GBC.VERTICAL));
    212214        bottomLine.add(Box.createHorizontalGlue(), GBC.std().fill());
    213215        findRelationAction = new FindRelationAction(chosenRelation);
    … …  
    217219        multipolygonButton.addMouseListener(new MouseAdapter() {
    218220            @Override
    219             public void mousePressed( MouseEvent e ) {
     221            public void mousePressed(MouseEvent e) {
    220222                checkPopup(e);
    221223            }
    222224
    223225            @Override
    224             public void mouseReleased( MouseEvent e ) {
     226            public void mouseReleased(MouseEvent e) {
    225227                checkPopup(e);
    226228            }
    227229
    228             private void checkPopup( MouseEvent e ) {
    229                 if( e.isPopupTrigger() )
     230            private void checkPopup(MouseEvent e) {
     231                if (e.isPopupTrigger() ) {
    230232                    multiPopupMenu.show(e.getComponent(), e.getX(), e.getY());
     233                }
    231234            }
    232235        });
    … …  
    237240    private static final Color CHOSEN_RELATION_COLOR = new Color(255, 255, 128);
    238241
    239     private void configureRelationsTable( final JTable relationsTable ) {
     242    private void configureRelationsTable(final JTable relationsTable) {
    240243        relationsTable.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
    241244        relationsTable.setTableHeader(null);
    242245        relationsTable.addMouseListener(new MouseAdapter() {
    243246            @Override
    244             public void mouseClicked( MouseEvent e ) {
     247            public void mouseClicked(MouseEvent e) {
    245248                Point p = e.getPoint();
    246249                int row = relationsTable.rowAtPoint(p);
    247                 if( SwingUtilities.isLeftMouseButton(e) && row >= 0 ) {
     250                if (SwingUtilities.isLeftMouseButton(e) && row >= 0) {
    248251                    Relation relation = (Relation)relationsData.getValueAt(row, 0);
    249                     if( e.getClickCount() > 1 ) {
     252                    if (e.getClickCount() > 1) {
    250253                        Main.main.getEditLayer().data.setSelected(relation);
    251254                    }
    … …  
    254257
    255258            @Override
    256             public void mousePressed( MouseEvent e ) {
     259            public void mousePressed(MouseEvent e) {
    257260                checkPopup(e);
    258261            }
    259262
    260263            @Override
    261             public void mouseReleased( MouseEvent e ) {
     264            public void mouseReleased(MouseEvent e) {
    262265                checkPopup(e);
    263266            }
    264267
    265             public void checkPopup( MouseEvent e ) {
    266                 if( e.isPopupTrigger() ) {
     268            public void checkPopup(MouseEvent e) {
     269                if (e.isPopupTrigger()) {
    267270                    Point p = e.getPoint();
    268271                    int row = relationsTable.rowAtPoint(p);
    … …  
    280283        columns.getColumn(0).setCellRenderer(new OsmPrimitivRenderer() {
    281284            @Override
    282             protected String getComponentToolTipText( OsmPrimitive value ) {
     285            protected String getComponentToolTipText(OsmPrimitive value) {
    283286                return null;
    284287            }
    285288
    286289            @Override
    287             public Component getTableCellRendererComponent( JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column ) {
     290            public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) {
    288291                Component c = super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
    289                 if( !isSelected && value instanceof Relation && chosenRelation.isSame(value) )
     292                if (!isSelected && value instanceof Relation && chosenRelation.isSame(value) ) {
    290293                    c.setBackground(CHOSEN_RELATION_COLOR);
    291                 else
     294                } else {
    292295                    c.setBackground(table.getBackground());
     296                }
    293297                return c;
    294298            }
    … …  
    297301        columns.getColumn(1).setCellRenderer(new DefaultTableCellRenderer() {
    298302            @Override
    299             public Component getTableCellRendererComponent( JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column ) {
     303            public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) {
    300304                Component c = super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
    301                 if( !isSelected && chosenRelation.isSame(table.getValueAt(row, 0)) )
     305                if (!isSelected && chosenRelation.isSame(table.getValueAt(row, 0)) ) {
    302306                    c.setBackground(CHOSEN_RELATION_COLOR);
    303                 else
     307                } else {
    304308                    c.setBackground(table.getBackground());
     309                }
    305310                return c;
    306311            }
    … …  
    310315        relationsTable.getSelectionModel().addListSelectionListener(new ListSelectionListener() {
    311316            @Override
    312             public void valueChanged( ListSelectionEvent e ) {
     317            public void valueChanged(ListSelectionEvent e) {
    313318                int selectedRow = relationsTable.getSelectedRow();
    314                 if( selectedRow >= 0 ) {
     319                if (selectedRow >= 0) {
    315320                    chosenRelation.set((Relation)relationsData.getValueAt(selectedRow, 0));
    316321                    relationsTable.clearSelection();
    … …  
    320325    }
    321326
    322     private JComponent sizeButton( JComponent b, int width, int height ) {
     327    private JComponent sizeButton(JComponent b, int width, int height) {
    323328        Dimension pref = b.getPreferredSize();
    324329        b.setPreferredSize(new Dimension(width <= 0 ? pref.width : width, height <= 0 ? pref.height : height));
    … …  
    346351
    347352    @Override
    348     public void chosenRelationChanged( Relation oldRelation, Relation newRelation ) {
    349         if( chosenRelationPanel != null && Main.pref.getBoolean(PREF_PREFIX + ".hidetopline", false) )
     353    public void chosenRelationChanged(Relation oldRelation, Relation newRelation) {
     354        if (chosenRelationPanel != null && Main.pref.getBoolean(PREF_PREFIX + ".hidetopline", false) ) {
    350355            chosenRelationPanel.setVisible(newRelation != null);
    351         if( Main.main.getCurrentDataSet() != null )
     356        }
     357        if (Main.main.getCurrentDataSet() != null ) {
    352358            selectionChanged(Main.main.getCurrentDataSet().getSelected());
     359        }
    353360        roleBoxModel.update();
    354361        // ?
    … …  
    356363
    357364    @Override
    358     public void selectionChanged( Collection<? extends OsmPrimitive> newSelection ) {
    359         if( !isVisible() || relationsData == null )
     365    public void selectionChanged(Collection<? extends OsmPrimitive> newSelection) {
     366        if (!isVisible() || relationsData == null )
    360367            return;
    361368        roleBoxModel.update();
    … …  
    363370        relationsData.setRowCount(0);
    364371        sortAndFixAction.chosenRelationChanged(chosenRelation.get(), chosenRelation.get());
    365         if( newSelection == null )
     372        if (newSelection == null )
    366373            return;
    367374
    368375        Set<Relation> relations = new TreeSet<>(
    369376                DefaultNameFormatter.getInstance().getRelationComparator());
    370         for( OsmPrimitive element : newSelection ) {
    371             for( OsmPrimitive ref : element.getReferrers() ) {
    372                 if( ref instanceof Relation && !ref.isIncomplete() && !ref.isDeleted() ) {
     377        for (OsmPrimitive element : newSelection) {
     378            for (OsmPrimitive ref : element.getReferrers()) {
     379                if (ref instanceof Relation && !ref.isIncomplete() && !ref.isDeleted()) {
    373380                    relations.add((Relation) ref);
    374381                }
    … …  
    376383        }
    377384
    378         for( Relation rel : relations ) {
     385        for (Relation rel : relations) {
    379386            String role = null;
    380             for( RelationMember m : rel.getMembers() ) {
    381                 for( OsmPrimitive element : newSelection ) {
    382                     if( m.getMember().equals(element) ) {
    383                         if( role == null )
     387            for (RelationMember m : rel.getMembers()) {
     388                for (OsmPrimitive element : newSelection) {
     389                    if (m.getMember().equals(element)) {
     390                        if (role == null ) {
    384391                            role = m.getRole();
    385                         else if( !role.equals(m.getRole()) ) {
     392                        } else if (!role.equals(m.getRole())) {
    386393                            role = tr("<different>");
    387394                            break;
    … …  
    392399            relationsData.addRow(new Object[] {rel, role == null ? "" : role});
    393400        }
    394         for( OsmPrimitive element : newSelection )
    395             if( element instanceof Relation && !chosenRelation.isSame(element) )
     401        for (OsmPrimitive element : newSelection )
     402            if (element instanceof Relation && !chosenRelation.isSame(element) ) {
    396403                relationsData.addRow(new Object[] {element, ""});
     404            }
    397405    }
    398406
    … …  
    406414
    407415    @Override
    408     public void editLayerChanged( OsmDataLayer oldLayer, OsmDataLayer newLayer ) {
     416    public void editLayerChanged(OsmDataLayer oldLayer, OsmDataLayer newLayer) {
    409417        updateSelection();
    410418    }
    … …  
    428436        ClassLoader classLoader = RelContextDialog.class.getClassLoader();
    429437        try (
    430             InputStream possibleRolesStream = classLoader.getResourceAsStream(POSSIBLE_ROLES_FILE);
    431             BufferedReader r = new BufferedReader(new InputStreamReader(possibleRolesStream));
    432         ) {
    433             while( r.ready() ) {
     438                InputStream possibleRolesStream = classLoader.getResourceAsStream(POSSIBLE_ROLES_FILE);
     439                BufferedReader r = new BufferedReader(new InputStreamReader(possibleRolesStream));
     440                ) {
     441            while(r.ready()) {
    434442                String line = r.readLine();
    435443                StringTokenizer t = new StringTokenizer(line, " ,;:\"");
    436                 if( t.hasMoreTokens() ) {
     444                if (t.hasMoreTokens()) {
    437445                    String type = t.nextToken();
    438446                    List<String> roles = new ArrayList<>();
    439                     while( t.hasMoreTokens() )
     447                    while(t.hasMoreTokens() ) {
    440448                        roles.add(t.nextToken());
     449                    }
    441450                    result.put(type, roles);
    442451                }
    443452            }
    444         } catch( Exception e ) {
     453        } catch(Exception e) {
    445454            Main.error("[RelToolbox] Error reading possible roles file.");
    446455            Main.error(e);
    … …  
    453462
    454463        List<String> items = new ArrayList<>();
    455         for( String role : roleBoxModel.getRoles() ) {
    456             if( role.length() > 1 )
     464        for (String role : roleBoxModel.getRoles()) {
     465            if (role.length() > 1 ) {
    457466                items.add(role);
     467            }
    458468        }
    459469        final AutoCompletingComboBox role = new AutoCompletingComboBox();
    … …  
    477487        role.getEditor().addActionListener(new ActionListener() {
    478488            @Override
    479             public void actionPerformed( ActionEvent e ) {
     489            public void actionPerformed(ActionEvent e) {
    480490                dlg.setVisible(false);
    481491                optionPane.setValue(JOptionPane.OK_OPTION);
    … …  
    486496
    487497        Object answer = optionPane.getValue();
    488         if( answer == null || answer == JOptionPane.UNINITIALIZED_VALUE
    489                 || (answer instanceof Integer && (Integer)answer != JOptionPane.OK_OPTION) ) {
     498        if (answer == null || answer == JOptionPane.UNINITIALIZED_VALUE
     499                || (answer instanceof Integer && (Integer)answer != JOptionPane.OK_OPTION))
    490500            return null;
    491         }
    492501
    493502        return role.getEditor().getItem().toString().trim();
    … …  
    496505    private class ChosenRelationMouseAdapter extends MouseAdapter {
    497506        @Override
    498         public void mouseClicked( MouseEvent e ) {
    499             if( e.isControlDown() || !(e.getComponent() instanceof JComboBox ) ) // do not use left click handler on combo box
    500             if( SwingUtilities.isLeftMouseButton(e) && chosenRelation.get() != null && Main.main.getEditLayer() != null ) {
    501                 Main.main.getEditLayer().data.setSelected(chosenRelation.get());
    502             }
    503         }
    504 
    505         @Override
    506         public void mousePressed( MouseEvent e ) {
     507        public void mouseClicked(MouseEvent e) {
     508            if (e.isControlDown() || !(e.getComponent() instanceof JComboBox ) ) // do not use left click handler on combo box
     509                if (SwingUtilities.isLeftMouseButton(e) && chosenRelation.get() != null && Main.main.getEditLayer() != null) {
     510                    Main.main.getEditLayer().data.setSelected(chosenRelation.get());
     511                }
     512        }
     513
     514        @Override
     515        public void mousePressed(MouseEvent e) {
    507516            checkPopup(e);
    508517        }
    509518
    510519        @Override
    511         public void mouseReleased( MouseEvent e ) {
     520        public void mouseReleased(MouseEvent e) {
    512521            checkPopup(e);
    513522        }
    514523
    515         private void checkPopup( MouseEvent e ) {
    516             if( e.isPopupTrigger() && chosenRelation.get() != null ) {
     524        private void checkPopup(MouseEvent e) {
     525            if (e.isPopupTrigger() && chosenRelation.get() != null) {
    517526                popupMenu.show(e.getComponent(), e.getX(), e.getY() - 5);
    518527            }
    … …  
    521530
    522531    private class ChosenRelationPopupMenu extends JPopupMenu {
    523         public ChosenRelationPopupMenu( ChosenRelation chosenRelation ) {
     532        public ChosenRelationPopupMenu(ChosenRelation chosenRelation) {
    524533            add(new SelectMembersAction(chosenRelation));
    525534            add(new SelectRelationAction(chosenRelation));
    … …  
    535544    }
    536545
    537     protected void applyRoleToSelection( String role ) {
    538         if( chosenRelation != null && chosenRelation.get() != null && Main.main.getCurrentDataSet() != null && !Main.main.getCurrentDataSet().selectionEmpty() ) {
     546    protected void applyRoleToSelection(String role) {
     547        if (chosenRelation != null && chosenRelation.get() != null && Main.main.getCurrentDataSet() != null && !Main.main.getCurrentDataSet().selectionEmpty()) {
    539548            Collection<OsmPrimitive> selected = Main.main.getCurrentDataSet().getSelected();
    540549            Relation r = chosenRelation.get();
    541550            List<Command> commands = new ArrayList<>();
    542             for( int i = 0; i < r.getMembersCount(); i++ ) {
     551            for (int i = 0; i < r.getMembersCount(); i++) {
    543552                RelationMember m = r.getMember(i);
    544                 if( selected.contains(m.getMember()) ) {
    545                     if( !role.equals(m.getRole()) ) {
    546 //                        r.setMember(i, new RelationMember(role, m.getMember()));
     553                if (selected.contains(m.getMember())) {
     554                    if (!role.equals(m.getRole())) {
     555                        //                        r.setMember(i, new RelationMember(role, m.getMember()));
    547556                        commands.add(new ChangeRelationMemberRoleCommand(r, i, role));
    548557                    }
    549558                }
    550559            }
    551             if( !commands.isEmpty() ) {
    552 //                Main.main.undoRedo.add(new ChangeCommand(chosenRelation.get(), r));
     560            if (!commands.isEmpty()) {
     561                //                Main.main.undoRedo.add(new ChangeCommand(chosenRelation.get(), r));
    553562                Main.main.undoRedo.add(new SequenceCommand(tr("Change relation member roles to {0}", role), commands));
    554563            }
    … …  
    567576        }
    568577    }
    569 /*
     578    /*
    570579    private class MultipolygonSettingsAction extends AbstractAction {
    571580        public MultipolygonSettingsAction() {
    … …  
    575584        }
    576585
    577         public void actionPerformed( ActionEvent e ) {
     586        public void actionPerformed(ActionEvent e) {
    578587            Component c = e.getSource() instanceof Component ? (Component)e.getSource() : Main.parent;
    579588            multiPopupMenu.show(c, 0, 0);
    … …  
    591600        }
    592601
    593         protected final JCheckBoxMenuItem addMenuItem( String property, String title ) {
     602        protected final JCheckBoxMenuItem addMenuItem(String property, String title) {
    594603            String fullProperty = PREF_PREFIX + ".multipolygon." + property;
    595604            JCheckBoxMenuItem item = new JCheckBoxMenuItem(tr(title));
    … …  
    602611
    603612        @Override
    604         public void actionPerformed( ActionEvent e ) {
     613        public void actionPerformed(ActionEvent e) {
    605614            String property = e.getActionCommand();
    606             if( property != null && property.length() > 0 && e.getSource() instanceof JCheckBoxMenuItem ) {
     615            if (property != null && property.length() > 0 && e.getSource() instanceof JCheckBoxMenuItem) {
    607616                boolean value = ((JCheckBoxMenuItem)e.getSource()).isSelected();
    608617                Main.pref.put(property, value);
    … …  
    617626            super(tr("Change role"), (String)null, tr("Enter role for selected members"),
    618627                    Shortcut.registerShortcut("reltoolbox:changerole", tr("Relation Toolbox: {0}", tr("Enter role for selected members")),
    619                     KeyEvent.VK_R, Shortcut.ALT_CTRL), false, "relcontext/enterrole", true);
     628                            KeyEvent.VK_R, Shortcut.ALT_CTRL), false, "relcontext/enterrole", true);
    620629            chosenRelation.addChosenRelationListener(this);
    621630            updateEnabledState();
    … …  
    623632
    624633        @Override
    625         public void actionPerformed( ActionEvent e ) {
    626             if( roleBoxModel.membersRole != null ) {
     634        public void actionPerformed(ActionEvent e) {
     635            if (roleBoxModel.membersRole != null) {
    627636                String role = askForRoleName();
    628                 if( role != null )
     637                if (role != null ) {
    629638                    applyRoleToSelection(role);
    630             }
    631         }
    632 
    633         @Override
    634         public void chosenRelationChanged( Relation oldRelation, Relation newRelation ) {
     639                }
     640            }
     641        }
     642
     643        @Override
     644        public void chosenRelationChanged(Relation oldRelation, Relation newRelation) {
    635645            setEnabled(newRelation != null);
    636646        }
    … …  
    645655        private final String ANOTHER_ROLE = tr("another...");
    646656
    647         public RoleComboBoxModel( JComboBox<String> combobox ) {
     657        public RoleComboBoxModel(JComboBox<String> combobox) {
    648658            super();
    649659            this.combobox = combobox;
    … …  
    653663        public void update() {
    654664            membersRole = getSelectedMembersRoleIntl();
    655             if( membersRole == null ) {
    656                 if( combobox.isEnabled() )
     665            if (membersRole == null) {
     666                if (combobox.isEnabled() ) {
    657667                    combobox.setEnabled(false);
     668                }
    658669                return;
    659670            }
    660             if( !combobox.isEnabled() )
     671            if (!combobox.isEnabled() ) {
    661672                combobox.setEnabled(true);
     673            }
    662674
    663675            List<String> items = new ArrayList<>();
    664             if( chosenRelation != null && chosenRelation.get() != null ) {
    665                 if( chosenRelation.isMultipolygon() ) {
     676            if (chosenRelation != null && chosenRelation.get() != null) {
     677                if (chosenRelation.isMultipolygon()) {
    666678                    items.add("outer");
    667679                    items.add("inner");
    668680                }
    669                 if( chosenRelation.get().get("type") != null ) {
     681                if (chosenRelation.get().get("type") != null) {
    670682                    List<String> values = possibleRoles.get(chosenRelation.get().get("type"));
    671                     if( values != null )
     683                    if (values != null ) {
    672684                        items.addAll(values);
    673                 }
    674                 for( RelationMember m : chosenRelation.get().getMembers() )
    675                     if( m.getRole().length() > 0 && !items.contains(m.getRole()) )
     685                    }
     686                }
     687                for (RelationMember m : chosenRelation.get().getMembers() )
     688                    if (m.getRole().length() > 0 && !items.contains(m.getRole()) ) {
    676689                        items.add(m.getRole());
     690                    }
    677691            }
    678692            items.add(EMPTY_ROLE);
    679             if( !items.contains(membersRole) )
     693            if (!items.contains(membersRole) ) {
    680694                items.add(0, membersRole);
     695            }
    681696            items.add(ANOTHER_ROLE);
    682697            roles = Collections.unmodifiableList(items);
    683698
    684             if( membersRole != null )
     699            if (membersRole != null ) {
    685700                setSelectedItem(membersRole);
    686             else
     701            } else {
    687702                fireContentsChanged(this, -1, -1);
     703            }
    688704            combobox.repaint();
    689705        }
    … …  
    699715        private String getSelectedMembersRoleIntl() {
    700716            String role = null;
    701             if( chosenRelation != null && chosenRelation.get() != null && Main.main.getCurrentDataSet() != null && !Main.main.getCurrentDataSet().selectionEmpty() ) {
     717            if (chosenRelation != null && chosenRelation.get() != null && Main.main.getCurrentDataSet() != null && !Main.main.getCurrentDataSet().selectionEmpty()) {
    702718                Collection<OsmPrimitive> selected = Main.main.getCurrentDataSet().getSelected();
    703                 for( RelationMember m : chosenRelation.get().getMembers() ) {
    704                     if( selected.contains(m.getMember()) ) {
    705                         if( role == null )
     719                for (RelationMember m : chosenRelation.get().getMembers()) {
     720                    if (selected.contains(m.getMember())) {
     721                        if (role == null ) {
    706722                            role = m.getRole();
    707                         else if( m.getRole() != null && !role.equals(m.getRole()) ) {
     723                        } else if (m.getRole() != null && !role.equals(m.getRole())) {
    708724                            role = tr("<different>");
    709725                            break;
    … …  
    725741
    726742        @Override
    727         public String getElementAt( int index ) {
     743        public String getElementAt(int index) {
    728744            return getRole(index);
    729745        }
    730746
    731         public String getRole( int index ) {
     747        public String getRole(int index) {
    732748            return roles.get(index);
    733749        }
    734750
    735751        @Override
    736         public void setSelectedItem( Object anItem ) {
     752        public void setSelectedItem(Object anItem) {
    737753            int newIndex = anItem == null ? -1 : roles.indexOf(anItem);
    738             if( newIndex != selectedIndex ) {
     754            if (newIndex != selectedIndex) {
    739755                selectedIndex = newIndex;
    740756                fireContentsChanged(this, -1, -1);
Note: See TracChangeset for help on using the changeset viewer.