From d234c7246c1d5334cbd0f86038ad7008aea7bc11 Mon Sep 17 00:00:00 2001
From: Lasse Liehu <lasse.liehu@iki.fi>
Date: Sun, 20 Aug 2023 12:17:30 +0300
Subject: [PATCH] Don't route on proposed ways (#6615)

---
 CHANGELOG.md                     | 2 ++
 features/bicycle/pushing.feature | 9 +++++++++
 features/foot/way.feature        | 6 ++++++
 profiles/bicycle.lua             | 3 ++-
 profiles/foot.lua                | 3 ++-
 5 files changed, 21 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 4c751c7048c..d389493a466 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -30,6 +30,8 @@
       - FIXED: Added a variable to preprocessor guard in file osrm-backend/include/util/range_table.hpp to solve build error. [#6596](https://github.com/Project-OSRM/osrm-backend/pull/6596)
       - FIXED: Ensure required file check in osrm-routed is correctly enforced. [#6655](https://github.com/Project-OSRM/osrm-backend/pull/6655)
       - FIXED: Correct HTTP docs to reflect summary output dependency on steps parameter. [#6655](https://github.com/Project-OSRM/osrm-backend/pull/6655)
+    - Profiles:
+      - FIXED: Bicycle and foot profiles now don't route on proposed ways [#6615](https://github.com/Project-OSRM/osrm-backend/pull/6615)
     - Routing:
       - FIXED: Fix adding traffic signal penalties during compression [#6419](https://github.com/Project-OSRM/osrm-backend/pull/6419)
 # 5.27.1
diff --git a/features/bicycle/pushing.feature b/features/bicycle/pushing.feature
index 5cd41c541df..5423859ad3d 100644
--- a/features/bicycle/pushing.feature
+++ b/features/bicycle/pushing.feature
@@ -78,6 +78,15 @@ Feature: Bike - Accessability of different way types
             | construction | yes  |         |       |
             | construction |      | yes     |       |
 
+    @proposed
+    Scenario: Bike - Don't allow routing on ways still being proposed
+        Then routability should be
+            | highway  | foot | bicycle | proposed | bothw |
+            | primary  |      |         |          | x     |
+            | proposed |      |         |          |       |
+            | proposed | yes  |         | yes      |       |
+            | proposed |      | yes     | yes      |       |
+
     @roundabout
     Scenario: Bike - Don't push bikes against oneway flow on roundabouts
         Then routability should be
diff --git a/features/foot/way.feature b/features/foot/way.feature
index ff3314f1cef..6916adece43 100644
--- a/features/foot/way.feature
+++ b/features/foot/way.feature
@@ -36,3 +36,9 @@ Feature: Foot - Accessability of different way types
             | highway | leisure  | forw |
             | (nil)   | track    |   x  |
 
+    Scenario: Foot - Proposed ways
+        Then routability should be
+            | highway  | foot  | proposed | forw |
+            | footway  |       |          | x    |
+            | proposed |       |          |      |
+            | proposed | yes   | yes      |      |
diff --git a/profiles/bicycle.lua b/profiles/bicycle.lua
index 374cb4fa2c0..19202768ec2 100644
--- a/profiles/bicycle.lua
+++ b/profiles/bicycle.lua
@@ -216,7 +216,8 @@ function setup()
 
     avoid = Set {
       'impassable',
-      'construction'
+      'construction',
+      'proposed'
     }
   }
 end
diff --git a/profiles/foot.lua b/profiles/foot.lua
index 978648954e2..1b16e092c15 100644
--- a/profiles/foot.lua
+++ b/profiles/foot.lua
@@ -69,7 +69,8 @@ function setup()
     },
 
     avoid = Set {
-      'impassable'
+      'impassable',
+      'proposed'
     },
 
     speeds = Sequence {