ensuring walkability with foot="yes" fix #14124

eclipse-sumo · Dec 1, 2023 · 744f119 · 744f119
1 parent 5f0720b
commit 744f119
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/netimport/NIImporter_OpenStreetMap.cpp b/src/netimport/NIImporter_OpenStreetMap.cpp
@@ -743,6 +743,11 @@ NIImporter_OpenStreetMap::insertEdge(Edge* e, int index, NBNode* from, NBNode* t
                     || (myImportSidewalks && (sidewalkType & WAY_FORWARD) != 0 && defaultPermissions != SVC_PEDESTRIAN)) {
                 nbe->addSidewalk(sidewalkWidth * offsetFactor);
             }
+            if (!addBackward && (e->myExtraAllowed & SVC_PEDESTRIAN) != 0 && (nbe->getPermissions(0) & SVC_PEDESTRIAN) == 0) {
+                // pedestrians are explicitly allowed (maybe through foot="yes") but dídn't get a sidewalk (maybe through sidewalk="no")
+                // since we do not have a backward edge, we need to make sure they can at least walk somewhere, see #14124
+                nbe->setPermissions(nbe->getPermissions(0) | SVC_PEDESTRIAN, 0);
+            }
             nbe->updateParameters(e->getParametersMap());
             nbe->setDistance(distanceStart);