Changeset 14860 in josm


Ignore:
Timestamp:
2019-03-09T17:27:17+01:00 (6 weeks ago)
Author:
GerdP
Message:

fix some checkstyle and sonar issues, improve readability, no functional change expected

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/dialogs/validator/ValidatorTreePanel.java

    r14857 r14860  
    1212import java.util.List;
    1313import java.util.Map;
     14import java.util.Map.Entry;
    1415import java.util.Set;
    1516import java.util.function.Consumer;
     
    164165
    165166        // Remember the currently expanded rows
    166         Set<Object> oldSelectedRows = new HashSet<>();
     167        Set<Object> oldExpandedRows = new HashSet<>();
    167168        Enumeration<TreePath> expanded = getExpandedDescendants(new TreePath(getRoot()));
    168169        if (expanded != null) {
     
    172173                Object userObject = node.getUserObject();
    173174                if (userObject instanceof Severity) {
    174                     oldSelectedRows.add(userObject);
     175                    oldExpandedRows.add(userObject);
    175176                } else if (userObject instanceof String) {
    176177                    String msg = (String) userObject;
     
    179180                        msg = msg.substring(0, index);
    180181                    }
    181                     oldSelectedRows.add(msg);
     182                    oldExpandedRows.add(msg);
    182183                }
    183184            }
     
    195196
    196197        final List<TreePath> expandedPaths = new ArrayList<>();
    197         errorsBySeverityMessageDescription.forEach((severity, errorsByMessageDescription) -> {
     198        for (Entry<Severity, Map<String, Map<String, List<TestError>>>> entry: errorsBySeverityMessageDescription.entrySet()) {
     199            Severity severity = entry.getKey();
     200            Map<String, Map<String, List<TestError>>> errorsByMessageDescription = entry.getValue();
     201
    198202            // Severity node
    199203            final DefaultMutableTreeNode severityNode = new GroupTreeNode(severity);
    200204            rootNode.add(severityNode);
    201205
    202             if (oldSelectedRows.contains(severity)) {
    203                 expandedPaths.add(new TreePath(new Object[] {rootNode, severityNode}));
    204             }
    205 
    206206            final Map<String, List<TestError>> errorsWithEmptyMessageByDescription = errorsByMessageDescription.get("");
    207207            if (errorsWithEmptyMessageByDescription != null) {
    208                 errorsWithEmptyMessageByDescription.forEach((description, errors) -> {
    209                     final String msg = tr("{0} ({1})", description, errors.size());
     208                errorsWithEmptyMessageByDescription.forEach((description, noDescriptionErrors) -> {
     209                    final String msg = tr("{0} ({1})", description, noDescriptionErrors.size());
    210210                    final DefaultMutableTreeNode messageNode = new DefaultMutableTreeNode(msg);
    211211                    severityNode.add(messageNode);
    212212
    213                     if (oldSelectedRows.contains(description)) {
    214                         expandedPaths.add(new TreePath(new Object[] {rootNode, severityNode, messageNode}));
     213                    if (oldExpandedRows.contains(description)) {
     214                        expandedPaths.add(new TreePath(new Object[] { rootNode, severityNode, messageNode }));
    215215                    }
    216 
    217                     errors.stream().map(DefaultMutableTreeNode::new).forEach(messageNode::add);
     216                    // add the matching errors to the current node
     217                    noDescriptionErrors.stream().map(DefaultMutableTreeNode::new).forEach(messageNode::add);
    218218                });
    219219            }
     
    228228                    groupNode = new GroupTreeNode(message);
    229229                    severityNode.add(groupNode);
    230                     if (oldSelectedRows.contains(message)) {
     230                    if (oldExpandedRows.contains(message)) {
    231231                        expandedPaths.add(new TreePath(new Object[] {rootNode, severityNode, groupNode}));
    232232                    }
     
    235235                }
    236236
    237                 errorsByDescription.forEach((description, errors) -> {
     237                errorsByDescription.forEach((description, errorsWithDescription) -> {
    238238                    boolean emptyDescription = description == null || description.isEmpty();
    239239                    // Message node
    240240                    final String msg;
    241241                    if (groupNode != null) {
    242                         msg = tr("{0} ({1})", description, errors.size());
     242                        msg = tr("{0} ({1})", description, errorsWithDescription.size());
    243243                    } else if (emptyDescription) {
    244                         msg = tr("{0} ({1})", message, errors.size());
     244                        msg = tr("{0} ({1})", message, errorsWithDescription.size());
    245245                    } else {
    246                         msg = tr("{0} - {1} ({2})", message, description, errors.size());
     246                        msg = tr("{0} - {1} ({2})", message, description, errorsWithDescription.size());
    247247                    }
    248248                    final DefaultMutableTreeNode messageNode = new DefaultMutableTreeNode(msg);
     
    253253                    }
    254254
    255                     if (oldSelectedRows.contains(description) || (emptyDescription && oldSelectedRows.contains(message))) {
     255                    if (oldExpandedRows.contains(description) || (emptyDescription && oldExpandedRows.contains(message))) {
    256256                        if (groupNode != null) {
    257257                            expandedPaths.add(new TreePath(new Object[] {rootNode, severityNode, groupNode, messageNode}));
     
    261261                    }
    262262
    263                  // add the matching errors to the current node
    264                     errors.stream().map(DefaultMutableTreeNode::new).forEach(messageNode::add);
     263                    // add the matching errors to the current node
     264                    errorsWithDescription.stream().map(DefaultMutableTreeNode::new).forEach(messageNode::add);
    265265                });
    266266            });
    267         });
     267        }
    268268
    269269        valTreeModel.setRoot(rootNode);
Note: See TracChangeset for help on using the changeset viewer.