Make coordinate-based OJP journey queries work

This is also the prerequisite for implementing parsing of walking sections
and turn description.

autotests/data/ojp-request/journey-arrival.xml

@@ -6,7 +6,13 @@
    <ojp:OJPTripRequest>
     <ojp:Origin>
      <ojp:PlaceRef>
-      <ojp:StopPlaceRef>8501687</ojp:StopPlaceRef>
+      <ojp:GeoPosition>
+       <siri:Longitude>7.78</siri:Longitude>
+       <siri:Latitude>46.1</siri:Latitude>
+      </ojp:GeoPosition>
+      <ojp:LocationName>
+       <ojp:Text>Randa</ojp:Text>
+      </ojp:LocationName>
      </ojp:PlaceRef>
     </ojp:Origin>
     <ojp:Destination>

autotests/data/ojp-request/journey-departure.xml

@@ -6,7 +6,13 @@
    <ojp:OJPTripRequest>
     <ojp:Origin>
      <ojp:PlaceRef>
-      <ojp:StopPlaceRef>8501687</ojp:StopPlaceRef>
+      <ojp:GeoPosition>
+       <siri:Longitude>7.78</siri:Longitude>
+       <siri:Latitude>46.1</siri:Latitude>
+      </ojp:GeoPosition>
+      <ojp:LocationName>
+       <ojp:Text> </ojp:Text>
+      </ojp:LocationName>
      </ojp:PlaceRef>
      <ojp:DepArrTime>2020-09-06T20:54:00Z</ojp:DepArrTime>
     </ojp:Origin>

autotests/ojprequesttest.cpp

@@ -110,7 +110,7 @@ private Q_SLOTS:
         QTest::addColumn<QString>("refFileName");
 
         Location from;
-        from.setIdentifier(QStringLiteral("uic"), QStringLiteral("8501687"));
+        from.setCoordinate(46.1, 7.78);
         Location to;
         to.setIdentifier(QStringLiteral("uic"), QStringLiteral("8500010"));
         JourneyRequest req;
@@ -122,6 +122,8 @@ private Q_SLOTS:
         req.setIncludeIntermediateStops(true);
         req.setIncludePaths(false);
         QTest::newRow("journey-departure") << req << s(SOURCE_DIR "/data/ojp-request/journey-departure.xml");
+        from.setName(QStringLiteral("Randa"));
+        req.setFrom(from);
         req.setIncludeIntermediateStops(false);
         req.setIncludePaths(true);
         req.setDateTimeMode(JourneyRequest::Arrival);

src/lib/backends/openjourneyplannerbackend.cpp

@@ -34,9 +34,7 @@ bool OpenJourneyPlannerBackend::needsLocationQuery(const Location &loc, Abstract
     if (!loc.identifier(QStringLiteral("uic")).isEmpty()) {// ### TODO configure identifier type
         return false;
     }
-    // TODO according to the docs this is supposed to work?
-    //return !loc.hasCoordinate() || type != AbstractBackend::QueryType::Journey;
-    return true;
+    return !loc.hasCoordinate() || type != AbstractBackend::QueryType::Journey;
 }
 
 bool OpenJourneyPlannerBackend::queryLocation(const LocationRequest &request, LocationReply *reply, QNetworkAccessManager *nam) const

src/lib/backends/openjourneyplannerrequestbuilder.cpp

@@ -168,6 +168,10 @@ void OpenJourneyPlannerRequestBuilder::writePlaceRef(QXmlStreamWriter &w, const
         w.writeTextElement(siriNS(), QStringLiteral("Longitude"), QString::number(loc.longitude()));
         w.writeTextElement(siriNS(), QStringLiteral("Latitude"), QString::number(loc.latitude()));
         w.writeEndElement(); // </ojp:GeoPosition>
+
+        w.writeStartElement(ojpNS(), QStringLiteral("LocationName"));
+        w.writeTextElement(ojpNS(), QStringLiteral("Text"), loc.name().isEmpty() ? QStringLiteral(" ") : loc.name());
+        w.writeEndElement(); // </ojp:LocationName>
     }
     w.writeEndElement(); // </ojp:PlaceRef>
 }