Ignore:
Timestamp:
2009-11-15T15:28:15+01:00 (14 years ago)
Author:
stoecker
Message:

cleanup of mappaint error text handling

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/MapPaintVisitor.java

    r2452 r2460  
    217217            return;
    218218
    219         if(fillAreas > dist) {
    220             w.clearErrors();
    221         }
    222 
    223219        if(wayStyle==null)
    224220        {
    … …  
    246242                drawArea(w, data.isSelected(w) ? selectedColor : areaStyle.color);
    247243                if(!w.isClosed()) {
    248                     w.putError(tr("Area style way is not closed."), true);
     244                    putError(w, tr("Area style way is not closed."), true);
    249245                }
    250246            }
    … …  
    479475                if(errs != null)
    480476                {
    481                     errs.putError(tr("multipolygon way ''{0}'' is not closed.",
     477                    putError(errs, tr("multipolygon way ''{0}'' is not closed.",
    482478                            w.getDisplayName(DefaultNameFormatter.getInstance())), true);
    483479                }
    … …  
    556552        //    System.out.println("Restriction: " + r.keys.get("name") + " restriction " + r.keys.get("restriction"));
    557553
    558         r.clearErrors();
    559 
    560554        Way fromWay = null;
    561555        Way toWay = null;
    … …  
    569563
    570564            if (m.getMember().isDeleted()) {
    571                 r.putError(tr("Deleted member ''{0}'' in relation.",
     565                putError(r, tr("Deleted member ''{0}'' in relation.",
    572566                        m.getMember().getDisplayName(DefaultNameFormatter.getInstance())), true);
    573567            } else if(m.getMember().incomplete)
    … …  
    580574                    if(w.getNodesCount() < 2)
    581575                    {
    582                         r.putError(tr("Way ''{0}'' with less than two points.",
     576                        putError(r, tr("Way ''{0}'' with less than two points.",
    583577                                w.getDisplayName(DefaultNameFormatter.getInstance())), true);
    584578                    }
    585579                    else if("from".equals(m.getRole())) {
    586580                        if(fromWay != null) {
    587                             r.putError(tr("More than one \"from\" way found."), true);
     581                            putError(r, tr("More than one \"from\" way found."), true);
    588582                        } else {
    589583                            fromWay = w;
    … …  
    591585                    } else if("to".equals(m.getRole())) {
    592586                        if(toWay != null) {
    593                             r.putError(tr("More than one \"to\" way found."), true);
     587                            putError(r, tr("More than one \"to\" way found."), true);
    594588                        } else {
    595589                            toWay = w;
    … …  
    597591                    } else if("via".equals(m.getRole())) {
    598592                        if(via != null) {
    599                             r.putError(tr("More than one \"via\" found."), true);
     593                            putError(r, tr("More than one \"via\" found."), true);
    600594                        } else {
    601595                            via = w;
    602596                        }
    603597                    } else {
    604                         r.putError(tr("Unknown role ''{0}''.", m.getRole()), true);
     598                        putError(r, tr("Unknown role ''{0}''.", m.getRole()), true);
    605599                    }
    606600                }
    … …  
    611605                    {
    612606                        if(via != null) {
    613                             r.putError(tr("More than one \"via\" found."), true);
     607                            putError(r, tr("More than one \"via\" found."), true);
    614608                        } else {
    615609                            via = n;
    616610                        }
    617611                    } else {
    618                         r.putError(tr("Unknown role ''{0}''.", m.getRole()), true);
     612                        putError(r, tr("Unknown role ''{0}''.", m.getRole()), true);
    619613                    }
    620614                } else {
    621                     r.putError(tr("Unknown member type for ''{0}''.", m.getMember().getDisplayName(DefaultNameFormatter.getInstance())), true);
     615                    putError(r, tr("Unknown member type for ''{0}''.", m.getMember().getDisplayName(DefaultNameFormatter.getInstance())), true);
    622616                }
    623617            }
    … …  
    625619
    626620        if (fromWay == null) {
    627             r.putError(tr("No \"from\" way found."), true);
     621            putError(r, tr("No \"from\" way found."), true);
    628622            return;
    629623        }
    630624        if (toWay == null) {
    631             r.putError(tr("No \"to\" way found."), true);
     625            putError(r, tr("No \"to\" way found."), true);
    632626            return;
    633627        }
    634628        if (via == null) {
    635             r.putError(tr("No \"via\" node or way found."), true);
     629            putError(r, tr("No \"via\" node or way found."), true);
    636630            return;
    637631        }
    … …  
    642636            viaNode = (Node) via;
    643637            if(!fromWay.isFirstLastNode(viaNode)) {
    644                 r.putError(tr("The \"from\" way doesn't start or end at a \"via\" node."), true);
     638                putError(r, tr("The \"from\" way doesn't start or end at a \"via\" node."), true);
    645639                return;
    646640            }
    647641            if(!toWay.isFirstLastNode(viaNode)) {
    648                 r.putError(tr("The \"to\" way doesn't start or end at a \"via\" node."), true);
     642                putError(r, tr("The \"to\" way doesn't start or end at a \"via\" node."), true);
    649643            }
    650644        }
    … …  
    673667                viaNode = lastNode;
    674668            } else {
    675                 r.putError(tr("The \"from\" way doesn't start or end at the \"via\" way."), true);
     669                putError(r, tr("The \"from\" way doesn't start or end at the \"via\" way."), true);
    676670                return;
    677671            }
    678672            if(!toWay.isFirstLastNode(viaNode == firstNode ? lastNode : firstNode)) {
    679                 r.putError(tr("The \"to\" way doesn't start or end at the \"via\" way."), true);
     673                putError(r, tr("The \"to\" way doesn't start or end at the \"via\" way."), true);
    680674            }
    681675        }
    … …  
    800794
    801795        if (nodeStyle == null) {
    802             r.putError(tr("Style for restriction {0} not found.", r.get("restriction")), true);
     796            putError(r, tr("Style for restriction {0} not found.", r.get("restriction")), true);
    803797            return;
    804798        }
    … …  
    894888                if(c > 1 && pdOuter.way != null && pdOuter.way.isClosed())
    895889                {
    896                     r.putError(tr("Intersection between ways ''{0}'' and ''{1}''.",
     890                    putError(r, tr("Intersection between ways ''{0}'' and ''{1}''.",
    897891                            pdOuter.way.getDisplayName(DefaultNameFormatter.getInstance()), wInner.getDisplayName(DefaultNameFormatter.getInstance())), true);
    898892                }
    … …  
    906900            if(!incomplete)
    907901            {
    908                 r.putError(tr("Inner way ''{0}'' is outside.",
     902                putError(r, tr("Inner way ''{0}'' is outside.",
    909903                        wInner.getDisplayName(DefaultNameFormatter.getInstance())), true);
    910904            }
    … …  
    922916        Boolean drawn = false;
    923917
    924         r.clearErrors();
    925 
    926918        for (RelationMember m : r.getMembers())
    927919        {
    928920            if (m.getMember().isDeleted()) {
    929                 r.putError(tr("Deleted member ''{0}'' in relation.",
     921                putError(r, tr("Deleted member ''{0}'' in relation.",
    930922                        m.getMember().getDisplayName(DefaultNameFormatter.getInstance())), true);
    931923            } else if(m.getMember().incomplete) {
    … …  
    935927                    Way w = m.getWay();
    936928                    if(w.getNodesCount() < 2) {
    937                         r.putError(tr("Way ''{0}'' with less than two points.",
     929                        putError(r, tr("Way ''{0}'' with less than two points.",
    938930                                w.getDisplayName(DefaultNameFormatter.getInstance())), true);
    939931                    }
    … …  
    943935                        outer.add(w);
    944936                    } else {
    945                         r.putError(tr("No useful role ''{0}'' for Way ''{1}''.",
     937                        putError(r, tr("No useful role ''{0}'' for Way ''{1}''.",
    946938                                m.getRole(), w.getDisplayName(DefaultNameFormatter.getInstance())), true);
    947939                        if(!m.hasRole()) {
    … …  
    955947                else
    956948                {
    957                     r.putError(tr("Non-Way ''{0}'' in multipolygon.",
     949                    putError(r, tr("Non-Way ''{0}'' in multipolygon.",
    958950                            m.getMember().getDisplayName(DefaultNameFormatter.getInstance())), true);
    959951                }
    … …  
    999991            if(outerclosed.size() == 0 && outerjoin.size() == 0)
    1000992            {
    1001                 r.putError(tr("No outer way for multipolygon ''{0}''.",
     993                putError(r, tr("No outer way for multipolygon ''{0}''.",
    1002994                        r.getDisplayName(DefaultNameFormatter.getInstance())), true);
    1003995                visible = true; /* prevent killing remaining ways */
    … …  
    10591051                    if(wayStyle.equals(innerStyle))
    10601052                    {
    1061                         r.putError(tr("Style for inner way ''{0}'' equals multipolygon.",
     1053                        putError(r, tr("Style for inner way ''{0}'' equals multipolygon.",
    10621054                                wInner.getDisplayName(DefaultNameFormatter.getInstance())), false);
    10631055                        if(!data.isSelected(r)) {
    … …  
    10891081                            && !wayStyle.equals(outerStyle))
    10901082                    {
    1091                         r.putError(tr("Style for outer way ''{0}'' mismatches.",
     1083                        putError(r, tr("Style for outer way ''{0}'' mismatches.",
    10921084                                wOuter.getDisplayName(DefaultNameFormatter.getInstance())), true);
    10931085                    }
    … …  
    14641456        maxEN = nc.getEastNorth(nc.getWidth() - 1, 0);
    14651457
     1458        data.clearErrors();
    14661459
    14671460        ++paintid;
    … …  
    16451638        drawOrderNumber(p1, p2, orderNumber);
    16461639    }
     1640
     1641    public void putError(OsmPrimitive p, String text, boolean isError)
     1642    {
     1643        data.addError(p, isError ? tr("Error: {0}", text) : tr("Warning: {0}", text));
     1644    }
    16471645}
Note: See TracChangeset for help on using the changeset viewer.