Changeset 32417 in osm for applications/editors/josm


Ignore:
Timestamp:
2016-06-27T13:43:11+02:00 (8 years ago)
Author:
darya
Message:

fix problem in construction test

Location:
applications/editors/josm/plugins/pt_assistant/src/org/openstreetmap/josm/plugins/pt_assistant/validation
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/pt_assistant/src/org/openstreetmap/josm/plugins/pt_assistant/validation/RoadTypeTest.java

    r32415 r32417  
    4949                                // be a route of public_transport:version 2
    5050                                boolean isCorrectRoadType = true;
     51                                boolean isUnderConstruction = false;
     52
    5153                                if (r.hasTag("route", "bus") || r.hasTag("route", "share_taxi")) {
    5254                                        if (!RouteUtils.isWaySuitableForBuses(way)) {
    5355                                                isCorrectRoadType = false;
    5456                                        }
     57                                        if (way.hasTag("highway", "construction") && way.hasKey("construction")) {
     58                                                isUnderConstruction = true;
     59                                        }
    5560                                } else if (r.hasTag("route", "trolleybus")) {
    5661                                        if (!(RouteUtils.isWaySuitableForBuses(way) && way.hasTag("trolley_wire", "yes"))) {
    5762                                                isCorrectRoadType = false;
    5863                                        }
     64                                        if (way.hasTag("highway", "construction") && way.hasKey("construction")) {
     65                                                isUnderConstruction = true;
     66                                        }
    5967                                } else if (r.hasTag("route", "tram")) {
    6068                                        if (!way.hasTag("railway", "tram")) {
    6169                                                isCorrectRoadType = false;
    6270                                        }
     71                                        if (way.hasTag("railway", "construction") && way.hasKey("construction")) {
     72                                                isUnderConstruction = true;
     73                                        }
    6374                                } else if (r.hasTag("route", "subway")) {
    6475                                        if (!r.hasTag("railway", "subway")) {
    6576                                                isCorrectRoadType = false;
    6677                                        }
     78                                        if (way.hasTag("railway", "construction") && way.hasKey("construction")) {
     79                                                isUnderConstruction = true;
     80                                        }
    6781                                } else if (r.hasTag("route", "light_rail")) {
    6882                                        if (!r.hasTag("raiilway", "subway")) {
    6983                                                isCorrectRoadType = false;
    7084                                        }
     85                                        if (way.hasTag("railway", "construction") && way.hasKey("construction")) {
     86                                                isUnderConstruction = true;
     87                                        }
    7188                                } else if (r.hasTag("route", "light_rail")) {
    7289                                        if (!r.hasTag("railway", "light_rail")) {
    7390                                                isCorrectRoadType = false;
    7491                                        }
     92                                        if (way.hasTag("railway", "construction") && way.hasKey("construction")) {
     93                                                isUnderConstruction = true;
     94                                        }
    7595                                } else if (r.hasTag("route", "train")) {
    7696                                        if (!r.hasTag("railway", "train")) {
    7797                                                isCorrectRoadType = false;
    7898                                        }
    79                                 }
    80 
    81                                 if (!isCorrectRoadType) {
     99                                        if (way.hasTag("railway", "construction") && way.hasKey("construction")) {
     100                                                isUnderConstruction = true;
     101                                        }
     102                                }
     103
     104                                if (!isCorrectRoadType && !isUnderConstruction) {
    82105
    83106                                        List<Relation> primitives = new ArrayList<>(1);
    … …  
    90113                                }
    91114                               
    92                                 if ((way.hasTag("highway", "construction") || way.hasTag("railway", "construction")) && way.hasKey("construction")) {
     115                                if (isUnderConstruction) {
    93116                                        List<Relation> primitives = new ArrayList<>(1);
    94117                                        primitives.add(r);

  • applications/editors/josm/plugins/pt_assistant/src/org/openstreetmap/josm/plugins/pt_assistant/validation/WayChecker.java

    r32415 r32417  
    5353
    5454                                boolean isCorrectRoadType = true;
     55                                boolean isUnderConstruction = false;
    5556                                if (relation.hasTag("route", "bus") || relation.hasTag("route", "share_taxi")) {
    5657                                        if (!isWaySuitableForBuses(way)) {
    5758                                                isCorrectRoadType = false;
    5859                                        }
     60                                        if (way.hasTag("highway", "construction") && way.hasKey("construction")) {
     61                                                isUnderConstruction = true;
     62                                        }
    5963                                } else if (relation.hasTag("route", "trolleybus")) {
    6064                                        if (!(isWaySuitableForBuses(way) && way.hasTag("trolley_wire", "yes"))) {
    6165                                                isCorrectRoadType = false;
    6266                                        }
     67                                        if (way.hasTag("highway", "construction") && way.hasKey("construction")) {
     68                                                isUnderConstruction = true;
     69                                        }
    6370                                } else if (relation.hasTag("route", "tram")) {
    6471                                        if (!way.hasTag("railway", "tram")) {
    6572                                                isCorrectRoadType = false;
    6673                                        }
     74                                        if (way.hasTag("railway", "construction") && way.hasKey("construction")) {
     75                                                isUnderConstruction = true;
     76                                        }
    6777                                } else if (relation.hasTag("route", "subway")) {
    6878                                        if (!relation.hasTag("railway", "subway")) {
    6979                                                isCorrectRoadType = false;
    7080                                        }
     81                                        if (way.hasTag("railway", "construction") && way.hasKey("construction")) {
     82                                                isUnderConstruction = true;
     83                                        }
    7184                                } else if (relation.hasTag("route", "light_rail")) {
    7285                                        if (!relation.hasTag("raiilway", "subway")) {
    7386                                                isCorrectRoadType = false;
    7487                                        }
     88                                        if (way.hasTag("railway", "construction") && way.hasKey("construction")) {
     89                                                isUnderConstruction = true;
     90                                        }
    7591                                } else if (relation.hasTag("route", "light_rail")) {
    7692                                        if (!relation.hasTag("railway", "light_rail")) {
    7793                                                isCorrectRoadType = false;
    7894                                        }
     95                                        if (way.hasTag("railway", "construction") && way.hasKey("construction")) {
     96                                                isUnderConstruction = true;
     97                                        }
    7998                                } else if (relation.hasTag("route", "train")) {
    8099                                        if (!relation.hasTag("railway", "train")) {
    81100                                                isCorrectRoadType = false;
    82101                                        }
    83                                 }
    84 
    85                                 if (!isCorrectRoadType) {
     102                                        if (way.hasTag("railway", "construction") && way.hasKey("construction")) {
     103                                                isUnderConstruction = true;
     104                                        }
     105                                }
     106
     107                                if (!isCorrectRoadType && !isUnderConstruction) {
    86108
    87109                                        List<Relation> primitives = new ArrayList<>(1);
    … …  
    96118                                }
    97119                               
    98                                 if ((way.hasTag("highway", "construction") || way.hasTag("railway", "construction")) && way.hasKey("construction")) {
     120                                if (isUnderConstruction) {
    99121                                        List<Relation> primitives = new ArrayList<>(1);
    100122                                        primitives.add(relation);
Note: See TracChangeset for help on using the changeset viewer.