[libraries/kpublictransport/release/26.08] src/lib/backends: Work around unreasonably long display names for Amarillo ride shares

Git commit 8765f1bf5755a2ae0982f0f9a5ce02145395c808 by Volker Krause.
Committed on 26/07/2026 at 16:03.
Pushed by vkrause into branch 'release/26.08'.

Work around unreasonably long display names for Amarillo ride shares

Use the operator name instead in those cases, ie. we then show
"Fahrgemeinschaft.de" instead of "very detailed departure location ->
very detailed arrival location", which has no chance of fitting on the
screen, and entirely duplicates information we show elsewhere already
anyway.

(cherry picked from commit 2d3c8b198d458447921aee53fc75b0c1e9dc398b)

M  +13   -7    src/lib/backends/motis2parser.cpp

https://invent.kde.org/libraries/kpublictransport/-/commit/8765f1bf5755a2ae0982f0f9a5ce02145395c808

diff --git a/src/lib/backends/motis2parser.cpp b/src/lib/backends/motis2parser.cpp
index e50d1cc4..fb5cc1b2 100644
--- a/src/lib/backends/motis2parser.cpp
+++ b/src/lib/backends/motis2parser.cpp
@@ -241,15 +241,8 @@ Motis2Parser::MotisRoute Motis2Parser::parseRoute(const QJsonObject &obj) const
 
     res.route.setDirection(obj.value("headsign"_L1).toString());
     res.route.setDestination(parsePlace(obj.value("tripTo"_L1).toObject(), false).stopPoint());
-    res.route.setLine(line);
 
     res.bookingUrl = QUrl(obj.value("agencyFareUrl"_L1).toString());
-
-    // Amarillo rideshare booking URLs
-    if (res.route.line().mode() == Line::RideShare) {
-        res.bookingUrl = QUrl(obj.value("routeUrl"_L1).toString());
-    }
-
     if (obj.contains("ticketUrls"_L1)) {
         const auto urls = obj.value("ticketUrls"_L1).toObject();
         // prefer web url even on android
@@ -262,6 +255,19 @@ Motis2Parser::MotisRoute Motis2Parser::parseRoute(const QJsonObject &obj) const
 #endif
     }
 
+    // Amarillo rideshare workarounds
+    if (line.mode() == Line::RideShare) {
+        // routeUrl has the booking URL
+        if (res.bookingUrl.isEmpty()) {
+            res.bookingUrl = QUrl(obj.value("routeUrl"_L1).toString());
+        }
+        // deal with the unreasonably long display names
+        if (line.name().size() > 20 && line.name().contains(" -> "_L1) && !line.operatorName().isEmpty() && line.operatorName().size() < line.name().size()) {
+            line.setName(line.operatorName());
+        }
+    }
+
+    res.route.setLine(line);
     return res;
 }