Customizing a routingProfile, and using onewayAware="false" is not enough

[doronbehar]

I drive my bicycle in a crowded city with lot's of one-way streets. I like the elevation aware navigation of OsmAnd's bicycle profile, but in this city it very often doesn't make sense to avoid driving in the opposite direction of small & quiet one-way streets. Hence I wanted to customize a routing profile to do this. I grabbed [routing.xml from OsmAnd-resources repo](https://github.com/osmandapp/OsmAnd-resources/blob/master/routing/routing.xml], and tried to perform the following minimal diff:

diff --git i/routing/routing.xml w/routing/routing.xml
index 333639609..f448e9711 100644
--- i/routing/routing.xml
+++ w/routing/routing.xml
@@ -1217,7 +1217,7 @@
 		</point>
 	</routingProfile>
 
-	<routingProfile name="bicycle" baseProfile="bicycle" restrictionsAware="true" minSpeed="5" defaultSpeed="10" maxSpeed="33" sharpTurn="6" slightTurn="6" followSpeedLimitations="false" onewayAware="true" heuristicCoefficient="1.4">
+	<routingProfile name="bicycle" baseProfile="bicycle" restrictionsAware="true" minSpeed="5" defaultSpeed="10" maxSpeed="33" sharpTurn="6" slightTurn="6" followSpeedLimitations="false" onewayAware="false" heuristicCoefficient="1.4">
 		<attribute name="penaltyForReverseDirection" value="60" /> <!-- bike: ~ 1 minute to turn back -->
 
 		<!-- Turn penalties for short_way=yes -->

But it turned out to not be enough - OsmAnd with this profile would still consider one-way streets as hard blocks for the routes I gave to it. Example:

After an hour+ of debugging, I found out the following additional diff is working:

diff --git i/routing/routing.xml w/routing/routing.xml
index 333639609..36f70fa29 100644
--- i/routing/routing.xml
+++ w/routing/routing.xml
@@ -1393,13 +1393,13 @@
 			<select value="1" t="bicycle:backward" v="no"/>
 			<select value="-1" t="bicycle:forward" v="no"/>
 
-			<select value="1" t="oneway" v="yes"/>
-			<select value="1" t="oneway" v="1"/>
-			<select value="-1" t="oneway" v="-1"/>
-            <!-- Following line because junction=circular is converted to roundabout, but is not necessarily oneway=yes -->
+			<!-- Mirror pedestrian: don't restrict direction on plain oneway/roundabout tags -->
+			<select value="0" t="oneway" v="yes"/>
+			<select value="0" t="oneway" v="1"/>
+			<select value="0" t="oneway" v="-1"/>
 			<select value="0" t="oneway" v="no"/>
-			<select value="1" t="roundabout" v="yes"/>
-			<select value="1" t="junction" v="roundabout"/>
+			<select value="0" t="roundabout" v="yes"/>
+			<select value="0" t="junction" v="roundabout"/>
 		</way>
 
 		<way attribute="speed" type="speed">

I was wondering maybe the XML comments are missing something, or is this a bug...