From 6a764ffac94d6e0b04d7846c69bdfdd64b89e10f Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Wed, 15 May 2024 11:29:05 +0100
Subject: [PATCH 01/16] refactor(Updates to include location groups and
 location group stops):

---
 pom.xml                                       |  2 +-
 .../datatools/manager/DataManager.java        |  5 ++--
 .../jobs/feedmerge/MergeFeedsResult.java      |  2 +-
 .../jobs/feedmerge/MergeLineContext.java      | 13 +++++-----
 .../feedmerge/ReferenceTableDiscovery.java    | 24 +++++++++----------
 .../manager/jobs/FlexMergeFeedsJobTest.java   |  4 ++--
 .../areas.txt                                 |  2 +-
 .../fake-agency-with-flex-version-1/areas.txt |  4 ----
 .../location_group_stops.txt                  |  2 ++
 .../location_groups.txt                       |  2 ++
 .../locations.geojson                         |  8 +++----
 .../stop_areas.txt                            |  4 ----
 .../fake-agency-with-flex-version-2/areas.txt |  4 ----
 .../location_group_stops.txt                  |  2 ++
 .../location_groups.txt                       |  2 ++
 .../locations.geojson                         |  8 +++----
 .../stop_areas.txt                            |  4 ----
 17 files changed, 40 insertions(+), 52 deletions(-)
 delete mode 100644 src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/areas.txt
 create mode 100644 src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/location_group_stops.txt
 create mode 100644 src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/location_groups.txt
 delete mode 100644 src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/stop_areas.txt
 delete mode 100644 src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/areas.txt
 create mode 100644 src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/location_group_stops.txt
 create mode 100644 src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/location_groups.txt
 delete mode 100644 src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/stop_areas.txt

diff --git a/pom.xml b/pom.xml
index b525edaba..19d91f679 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,7 @@
         <dependency>
             <groupId>com.github.ibi-group</groupId>
             <artifactId>gtfs-lib</artifactId>
-            <version>2bd924c6e5f5244a279c9d94298c14c0b3d47b35</version>
+            <version>e623aef54deba8c95d34e2f7c6760bc055612193</version>
             <!-- Latest dev-flex build on jitpack.io -->
             <!-- Exclusions added in order to silence SLF4J warnings about multiple bindings:
                 http://www.slf4j.org/codes.html#multiple_bindings
diff --git a/src/main/java/com/conveyal/datatools/manager/DataManager.java b/src/main/java/com/conveyal/datatools/manager/DataManager.java
index a201c2912..9c74f8527 100644
--- a/src/main/java/com/conveyal/datatools/manager/DataManager.java
+++ b/src/main/java/com/conveyal/datatools/manager/DataManager.java
@@ -210,12 +210,11 @@ static void registerRoutes() throws IOException {
             // NOTE: fare_attributes controller handles updates to nested table fare_rules.
             new EditorControllerImpl(EDITOR_API_PREFIX, Table.FARE_ATTRIBUTES, DataManager.GTFS_DATA_SOURCE);
             new EditorControllerImpl(EDITOR_API_PREFIX, Table.FEED_INFO, DataManager.GTFS_DATA_SOURCE);
-            // NOTE: AREA and STOP_AREAS are additions for GTFS Flex but belong to the Fares v2 specification:
-            new EditorControllerImpl(EDITOR_API_PREFIX, Table.STOP_AREAS, DataManager.GTFS_DATA_SOURCE);
-            new EditorControllerImpl(EDITOR_API_PREFIX, Table.AREA, DataManager.GTFS_DATA_SOURCE);
             // NOTE: Booking rules, locations and location shapes are GTFS Flex additions.
             new EditorControllerImpl(EDITOR_API_PREFIX, Table.BOOKING_RULES, DataManager.GTFS_DATA_SOURCE);
             new EditorControllerImpl(EDITOR_API_PREFIX, Table.LOCATIONS, DataManager.GTFS_DATA_SOURCE);
+            new EditorControllerImpl(EDITOR_API_PREFIX, Table.LOCATION_GROUP, DataManager.GTFS_DATA_SOURCE);
+            new EditorControllerImpl(EDITOR_API_PREFIX, Table.LOCATION_GROUP_STOPS, DataManager.GTFS_DATA_SOURCE);
             new EditorControllerImpl(EDITOR_API_PREFIX, Table.LOCATION_SHAPES, DataManager.GTFS_DATA_SOURCE);
             new EditorControllerImpl(EDITOR_API_PREFIX, Table.ROUTES, DataManager.GTFS_DATA_SOURCE);
             // NOTE: Patterns controller handles updates to nested tables shapes, pattern stops, and frequencies.
diff --git a/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeFeedsResult.java b/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeFeedsResult.java
index 46f980d09..b9f848b4c 100644
--- a/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeFeedsResult.java
+++ b/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeFeedsResult.java
@@ -40,7 +40,7 @@ public class MergeFeedsResult implements Serializable {
      * Track various table ids for resolving foreign references.
      */
     public Set<String> stopIds = new HashSet<>();
-    public Set<String> stopAreaIds = new HashSet<>();
+    public Set<String> locationGroupStopIds = new HashSet<>();
 
     /**
      * Track the set of route IDs to end up in the merged feed in order to determine which route_attributes
diff --git a/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeLineContext.java b/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeLineContext.java
index 15d9a77c2..c8ff7a1f9 100644
--- a/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeLineContext.java
+++ b/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeLineContext.java
@@ -246,8 +246,8 @@ private void setForeignReferenceKeyValues() {
             case "stops":
                 mergeFeedsResult.stopIds.add(getIdWithScope(keyValue));
                 break;
-            case "stop_areas":
-                mergeFeedsResult.stopAreaIds.add(getIdWithScope(keyValue));
+            case "location_group_stops":
+                mergeFeedsResult.locationGroupStopIds.add(getIdWithScope(keyValue));
                 break;
             default:
                 // nothing.
@@ -272,16 +272,15 @@ private Table getReferenceTable(FieldContext fieldContext, Field field) {
                     getTableScopedValue(Table.CALENDAR_DATES, fieldContext.getValue())
                 );
             case STOP_TIMES_STOP_ID_KEY:
-                return ReferenceTableDiscovery.getStopTimeStopIdReferenceTable(
+                return ReferenceTableDiscovery.getStopTimeReferenceTable(
                     fieldContext.getValueToWrite(),
                     mergeFeedsResult,
                     feedMergeContext.locationIds
                 );
-            case STOP_AREA_STOP_ID_KEY:
-                return ReferenceTableDiscovery.getStopAreaAreaIdReferenceTable(
+            case LOCATION_GROUP_STOPS_STOP_ID_KEY:
+                return ReferenceTableDiscovery.getLocationGroupStopReferenceTable(
                     fieldContext.getValueToWrite(),
-                    mergeFeedsResult,
-                    feedMergeContext.locationIds
+                    mergeFeedsResult
                 );
             default:
                 return null;
diff --git a/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/ReferenceTableDiscovery.java b/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/ReferenceTableDiscovery.java
index 00e5f9afe..1aa8ac4f5 100644
--- a/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/ReferenceTableDiscovery.java
+++ b/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/ReferenceTableDiscovery.java
@@ -27,10 +27,10 @@ public enum ReferenceTableKey {
                 Table.SCHEDULE_EXCEPTIONS.name
             )
         ),
-        STOP_AREA_STOP_ID_KEY(
+        LOCATION_GROUP_STOPS_STOP_ID_KEY(
             String.join(
                 REF_TABLE_SEPARATOR,
-                Table.STOP_AREAS.name,
+                Table.LOCATION_GROUP_STOPS.name,
                 "stop_id",
                 Table.STOPS.name,
                 Table.LOCATIONS.name
@@ -43,7 +43,7 @@ public enum ReferenceTableKey {
                 "stop_id",
                 Table.STOPS.name,
                 Table.LOCATIONS.name,
-                Table.STOP_AREAS.name
+                Table.LOCATION_GROUP_STOPS.name
             )
         );
 
@@ -112,25 +112,23 @@ public static Table getTripServiceIdReferenceTable(
     }
 
     /**
-     * Define the reference table for a stop area's area id. This will either be a stop or location.
+     * Define the reference table for a location group stop: stop id. This will either be a stop or null.
      */
-    public static Table getStopAreaAreaIdReferenceTable(
+    public static Table getLocationGroupStopReferenceTable(
         String fieldValue,
-        MergeFeedsResult mergeFeedsResult,
-        Set<String> locationIds
+        MergeFeedsResult mergeFeedsResult
     ) {
         if (mergeFeedsResult.stopIds.contains(fieldValue)) {
             return Table.STOPS;
-        } else if (locationIds.contains(fieldValue)) {
-            return Table.LOCATIONS;
         }
         return null;
     }
 
     /**
-     * Define the reference table for a stop time's stop id. This will either be a stop, location or stop area.
+     * Define the reference table for a stop time's stop id. This will either be a stop, location or location group.
+     * TODO: In later PR's this will be redundant as a stop time: stop id will only reference a stop.
      */
-    public static Table getStopTimeStopIdReferenceTable(
+    public static Table getStopTimeReferenceTable(
         String fieldValue,
         MergeFeedsResult mergeFeedsResult,
         Set<String> locationIds
@@ -139,8 +137,8 @@ public static Table getStopTimeStopIdReferenceTable(
             return Table.STOPS;
         } else if (locationIds.contains(fieldValue)) {
             return Table.LOCATIONS;
-        } else if (mergeFeedsResult.stopAreaIds.contains(fieldValue)) {
-            return Table.STOP_AREAS;
+        } else if (mergeFeedsResult.locationGroupStopIds.contains(fieldValue)) {
+            return Table.LOCATION_GROUP_STOPS;
         }
         return null;
     }
diff --git a/src/test/java/com/conveyal/datatools/manager/jobs/FlexMergeFeedsJobTest.java b/src/test/java/com/conveyal/datatools/manager/jobs/FlexMergeFeedsJobTest.java
index eb021c3b2..99dfc6ec3 100644
--- a/src/test/java/com/conveyal/datatools/manager/jobs/FlexMergeFeedsJobTest.java
+++ b/src/test/java/com/conveyal/datatools/manager/jobs/FlexMergeFeedsJobTest.java
@@ -97,7 +97,6 @@ void canMergeRegional() throws SQLException {
 
         // Ensure the feed has the row counts we expect.
         assertRowCount(r1.agency, r2.agency, merged.agency, "Agency");
-        assertRowCount(r1.area, r2.area, merged.area, "Area");
         assertRowCount(r1.attributions, r2.attributions, merged.attributions, "Attributions");
         assertRowCount(r1.bookingRules, r2.bookingRules, merged.bookingRules, "Booking rules");
         assertRowCount(r1.calendar, r2.calendar, merged.calendar, "Calendar");
@@ -106,11 +105,12 @@ void canMergeRegional() throws SQLException {
         assertRowCount(r1.fareRules, r2.fareRules, merged.fareRules, "Fare rules");
         assertRowCount(r1.frequencies, r2.frequencies, merged.frequencies, "Frequencies");
         assertRowCount(r1.locations, r2.locations, merged.locations, "Locations");
+        assertRowCount(r1.locationGroup, r2.locationGroup, merged.locationGroup, "Location Groups");
+        assertRowCount(r1.locationGroupStops, r2.locationGroupStops, merged.locationGroupStops, "Location Group Stops");
         assertRowCount(r1.locationShapes, r2.locationShapes, merged.locationShapes, "Location shapes");
         assertRowCount(r1.routes, r2.routes, merged.routes, "Routes");
         assertRowCount(r1.shapes, r2.shapes, merged.shapes, "Shapes");
         assertRowCount(r1.stops, r2.stops, merged.stops, "Stops");
-        assertRowCount(r1.stopAreas, r2.stopAreas, merged.stopAreas, "Stop areas");
         assertRowCount(r1.stopTimes, r2.stopTimes, merged.stopTimes, "Stop times");
         assertRowCount(r1.trips, r2.trips, merged.trips, "Trips");
         assertRowCount(r1.translations, r2.translations, merged.translations, "Translations");
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-for-field-normalizing/areas.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-for-field-normalizing/areas.txt
index 6eefe8872..7e2111957 100644
--- a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-for-field-normalizing/areas.txt
+++ b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-for-field-normalizing/areas.txt
@@ -1,2 +1,2 @@
 area_id,area_name
-Area One,"This is area one"
+Area One,"This is locationGroup one"
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/areas.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/areas.txt
deleted file mode 100644
index 3f8aefce7..000000000
--- a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/areas.txt
+++ /dev/null
@@ -1,4 +0,0 @@
-area_id,area_name
-1,"Area referencing a stop"
-2,"Area referencing a location"
-3,"Multi areas"
\ No newline at end of file
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/location_group_stops.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/location_group_stops.txt
new file mode 100644
index 000000000..f964d1401
--- /dev/null
+++ b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/location_group_stops.txt
@@ -0,0 +1,2 @@
+location_group_id,stop_id
+1,123
\ No newline at end of file
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/location_groups.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/location_groups.txt
new file mode 100644
index 000000000..b9ac54dbd
--- /dev/null
+++ b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/location_groups.txt
@@ -0,0 +1,2 @@
+location_group_id,location_group_name
+1,"Area referencing a stop"
\ No newline at end of file
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/locations.geojson b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/locations.geojson
index efae1fe8c..948f5c0be 100644
--- a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/locations.geojson
+++ b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/locations.geojson
@@ -5,7 +5,7 @@
       "id": "area_1469-2",
       "properties": {
         "stop_name": "Templeboy to Ballisodare",
-        "stop_desc": "Templeboy to Ballisodare Door-to-door pickup area"
+        "stop_desc": "Templeboy to Ballisodare Door-to-door pickup locationGroup"
       },
       "type": "Feature",
       "geometry": {
@@ -25,7 +25,7 @@
       "id": "area_1469-1",
       "properties": {
         "stop_name": "Dromore West to Templeboy",
-        "stop_desc": "Dromore West to Templeboy Door-to-door pickup area"
+        "stop_desc": "Dromore West to Templeboy Door-to-door pickup locationGroup"
       },
       "type": "Feature",
       "geometry": {
@@ -172,7 +172,7 @@
 	  "id": "area_246",
       "properties": {
         "stop_name": "Moville to Letterkenny",
-        "stop_desc": "Moville to Letterkenny Door-to-door pickup area"
+        "stop_desc": "Moville to Letterkenny Door-to-door pickup locationGroup"
       },
       "type": "Feature",
       "geometry": {
@@ -187,7 +187,7 @@
       "id": "area_251",
       "properties": {
         "stop_name": "Muff to Moville",
-        "stop_desc": "Muff to Moville Door-to-door pickup area"
+        "stop_desc": "Muff to Moville Door-to-door pickup locationGroup"
       },
       "type": "Feature",
       "geometry": {
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/stop_areas.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/stop_areas.txt
deleted file mode 100644
index 3f900c131..000000000
--- a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/stop_areas.txt
+++ /dev/null
@@ -1,4 +0,0 @@
-area_id,stop_id
-1,123
-2,area_1469-1
-3,123
\ No newline at end of file
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/areas.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/areas.txt
deleted file mode 100644
index 3f8aefce7..000000000
--- a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/areas.txt
+++ /dev/null
@@ -1,4 +0,0 @@
-area_id,area_name
-1,"Area referencing a stop"
-2,"Area referencing a location"
-3,"Multi areas"
\ No newline at end of file
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/location_group_stops.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/location_group_stops.txt
new file mode 100644
index 000000000..f964d1401
--- /dev/null
+++ b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/location_group_stops.txt
@@ -0,0 +1,2 @@
+location_group_id,stop_id
+1,123
\ No newline at end of file
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/location_groups.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/location_groups.txt
new file mode 100644
index 000000000..b9ac54dbd
--- /dev/null
+++ b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/location_groups.txt
@@ -0,0 +1,2 @@
+location_group_id,location_group_name
+1,"Area referencing a stop"
\ No newline at end of file
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/locations.geojson b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/locations.geojson
index efae1fe8c..948f5c0be 100644
--- a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/locations.geojson
+++ b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/locations.geojson
@@ -5,7 +5,7 @@
       "id": "area_1469-2",
       "properties": {
         "stop_name": "Templeboy to Ballisodare",
-        "stop_desc": "Templeboy to Ballisodare Door-to-door pickup area"
+        "stop_desc": "Templeboy to Ballisodare Door-to-door pickup locationGroup"
       },
       "type": "Feature",
       "geometry": {
@@ -25,7 +25,7 @@
       "id": "area_1469-1",
       "properties": {
         "stop_name": "Dromore West to Templeboy",
-        "stop_desc": "Dromore West to Templeboy Door-to-door pickup area"
+        "stop_desc": "Dromore West to Templeboy Door-to-door pickup locationGroup"
       },
       "type": "Feature",
       "geometry": {
@@ -172,7 +172,7 @@
 	  "id": "area_246",
       "properties": {
         "stop_name": "Moville to Letterkenny",
-        "stop_desc": "Moville to Letterkenny Door-to-door pickup area"
+        "stop_desc": "Moville to Letterkenny Door-to-door pickup locationGroup"
       },
       "type": "Feature",
       "geometry": {
@@ -187,7 +187,7 @@
       "id": "area_251",
       "properties": {
         "stop_name": "Muff to Moville",
-        "stop_desc": "Muff to Moville Door-to-door pickup area"
+        "stop_desc": "Muff to Moville Door-to-door pickup locationGroup"
       },
       "type": "Feature",
       "geometry": {
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/stop_areas.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/stop_areas.txt
deleted file mode 100644
index 3f900c131..000000000
--- a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/stop_areas.txt
+++ /dev/null
@@ -1,4 +0,0 @@
-area_id,stop_id
-1,123
-2,area_1469-1
-3,123
\ No newline at end of file

From 777ba4e4b203024612ad9b72edb0e71852e20d75 Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Wed, 15 May 2024 12:00:37 +0100
Subject: [PATCH 02/16] refactor(Remove areas from gtfs test file and unit
 tests):

---
 .../datatools/manager/jobs/NormalizeFieldTransformJobTest.java | 3 +--
 .../datatools/gtfs/fake-agency-for-field-normalizing/areas.txt | 2 --
 2 files changed, 1 insertion(+), 4 deletions(-)
 delete mode 100644 src/test/resources/com/conveyal/datatools/gtfs/fake-agency-for-field-normalizing/areas.txt

diff --git a/src/test/java/com/conveyal/datatools/manager/jobs/NormalizeFieldTransformJobTest.java b/src/test/java/com/conveyal/datatools/manager/jobs/NormalizeFieldTransformJobTest.java
index 85f8f3599..a5e7c8863 100644
--- a/src/test/java/com/conveyal/datatools/manager/jobs/NormalizeFieldTransformJobTest.java
+++ b/src/test/java/com/conveyal/datatools/manager/jobs/NormalizeFieldTransformJobTest.java
@@ -102,8 +102,7 @@ void canNormalizeField(TransformationCase transformationCase) throws IOException
     private static Stream<Arguments> createNormalizedFieldCases() {
         return Stream.of(
             Arguments.of(new TransformationCase("routes", "route_long_name", "Route", "Rte")),
-            Arguments.of(new TransformationCase("booking_rules", "pickup_message", "Message", "Msg")),
-            Arguments.of(new TransformationCase("areas", "area_name", "Area", "Place"))
+            Arguments.of(new TransformationCase("booking_rules", "pickup_message", "Message", "Msg"))
         );
     }
 
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-for-field-normalizing/areas.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-for-field-normalizing/areas.txt
deleted file mode 100644
index 7e2111957..000000000
--- a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-for-field-normalizing/areas.txt
+++ /dev/null
@@ -1,2 +0,0 @@
-area_id,area_name
-Area One,"This is locationGroup one"

From 4e17e9c54c2d5b5f19d57d3613d9fac25508fef8 Mon Sep 17 00:00:00 2001
From: miles-grant-ibi <miles.grant@ibigroup.com>
Date: Tue, 21 May 2024 11:12:52 -0400
Subject: [PATCH 03/16] temp: enable docker build

---
 .github/workflows/build-docker.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/build-docker.yml b/.github/workflows/build-docker.yml
index e0702e703..aa6290c21 100644
--- a/.github/workflows/build-docker.yml
+++ b/.github/workflows/build-docker.yml
@@ -3,7 +3,7 @@ name: Create and publish a Docker image
 
 on:
   push:
-    branches: ['dev', 'master', 'dev-flex']
+    branches: ['dev', 'master', 'dev-flex', 'feature/DT-423-rename-stop-areas-location-groups']
 
 env:
   REGISTRY: ghcr.io

From 38f9fd8d7009fcdb321d5eb61bab3342f493b4b5 Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Mon, 24 Jun 2024 13:58:35 +0100
Subject: [PATCH 04/16] refactor(Bumped GTFS-Lib version and fixed failing flex
 merge test):

---
 pom.xml                                            |  2 +-
 .../fake-agency-with-flex-version-1/stop_times.txt | 14 +++++++-------
 .../fake-agency-with-flex-version-2/stop_times.txt | 14 +++++++-------
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/pom.xml b/pom.xml
index a6995f9e5..f7d4290ba 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,7 @@
         <dependency>
             <groupId>com.github.ibi-group</groupId>
             <artifactId>gtfs-lib</artifactId>
-            <version>e623aef54deba8c95d34e2f7c6760bc055612193</version>
+            <version>57506da336673b3eff1092b39ce550d5cbe31a4f</version>
             <!-- Latest dev-flex build on jitpack.io -->
             <!-- Exclusions added in order to silence SLF4J warnings about multiple bindings:
                 http://www.slf4j.org/codes.html#multiple_bindings
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/stop_times.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/stop_times.txt
index f82e627d8..b46178992 100644
--- a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/stop_times.txt
+++ b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-1/stop_times.txt
@@ -1,7 +1,7 @@
-trip_id,arrival_time,departure_time,stop_id,stop_sequence,stop_headsign,pickup_type,drop_off_type,continuous_pickup,continuous_drop_off,shape_dist_traveled,timepoint,pickup_booking_rule_id,drop_off_booking_rule_id,start_pickup_drop_off_window,end_pickup_drop_off_window,mean_duration_factor,mean_duration_offset,safe_duration_factor,safe_duration_offset
-a30277f8-e50a-4a85-9141-b1e0da9d429d,07:00:00,07:00:00,4u6g,1,Test stop headsign,0,0,1,1,0.0000000,1,"1","1",07:00:00,07:00:00,1,60,1,60
-a30277f8-e50a-4a85-9141-b1e0da9d429d,07:01:00,07:01:00,johv,2,Test stop headsign 2,0,0,1,1,341.4491961,1,"1","1",07:01:00,07:01:00,1,60,1,60
-frequency-trip,08:00:00,08:00:00,4u6g,1,Test stop headsign frequency trip,0,0,1,1,0.0000000,1,"1","1",07:02:00,07:02:00,1,60,1,60
-frequency-trip,08:29:00,08:29:00,1234,2,Test stop headsign frequency trip,0,0,1,1,341.4491961,1,"1","1",07:03:00,07:03:00,1,60,1,60
-frequency-trip,08:31:00,08:31:00,area_251,3,,0,0,1,1,341.4491961,1,"1","1",07:03:00,07:03:00,1,60,1,60
-frequency-trip,,,3,4,,1,1,1,1,341.4491961,1,"1","1",07:04:00,07:05:00,1,60,1,60
+trip_id,arrival_time,departure_time,stop_id,location_group_id,location_id,stop_sequence,stop_headsign,pickup_type,drop_off_type,continuous_pickup,continuous_drop_off,shape_dist_traveled,timepoint,pickup_booking_rule_id,drop_off_booking_rule_id,start_pickup_drop_off_window,end_pickup_drop_off_window
+a30277f8-e50a-4a85-9141-b1e0da9d429d,07:00:00,07:00:00,4u6g,,,1,Test stop headsign,0,0,1,1,0.0000000,1,"1","1",07:00:00,07:00:00,1
+a30277f8-e50a-4a85-9141-b1e0da9d429d,07:01:00,07:01:00,johv,,,2,Test stop headsign 2,0,0,1,1,341.4491961,1,"1","1",07:01:00,07:01:00
+frequency-trip,08:00:00,08:00:00,4u6g,,,1,Test stop headsign frequency trip,0,0,1,1,0.0000000,1,"1","1",07:02:00,07:02:00
+frequency-trip,08:29:00,08:29:00,1234,,,2,Test stop headsign frequency trip,0,0,1,1,341.4491961,1,"1","1",07:03:00,07:03:00
+frequency-trip,08:31:00,08:31:00,,,area_251,3,,0,0,1,1,341.4491961,1,"1","1",07:03:00,07:03:00
+frequency-trip,,,,1,,4,,1,1,1,1,341.4491961,1,"1","1",07:04:00,07:05:00
diff --git a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/stop_times.txt b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/stop_times.txt
index f82e627d8..b46178992 100644
--- a/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/stop_times.txt
+++ b/src/test/resources/com/conveyal/datatools/gtfs/fake-agency-with-flex-version-2/stop_times.txt
@@ -1,7 +1,7 @@
-trip_id,arrival_time,departure_time,stop_id,stop_sequence,stop_headsign,pickup_type,drop_off_type,continuous_pickup,continuous_drop_off,shape_dist_traveled,timepoint,pickup_booking_rule_id,drop_off_booking_rule_id,start_pickup_drop_off_window,end_pickup_drop_off_window,mean_duration_factor,mean_duration_offset,safe_duration_factor,safe_duration_offset
-a30277f8-e50a-4a85-9141-b1e0da9d429d,07:00:00,07:00:00,4u6g,1,Test stop headsign,0,0,1,1,0.0000000,1,"1","1",07:00:00,07:00:00,1,60,1,60
-a30277f8-e50a-4a85-9141-b1e0da9d429d,07:01:00,07:01:00,johv,2,Test stop headsign 2,0,0,1,1,341.4491961,1,"1","1",07:01:00,07:01:00,1,60,1,60
-frequency-trip,08:00:00,08:00:00,4u6g,1,Test stop headsign frequency trip,0,0,1,1,0.0000000,1,"1","1",07:02:00,07:02:00,1,60,1,60
-frequency-trip,08:29:00,08:29:00,1234,2,Test stop headsign frequency trip,0,0,1,1,341.4491961,1,"1","1",07:03:00,07:03:00,1,60,1,60
-frequency-trip,08:31:00,08:31:00,area_251,3,,0,0,1,1,341.4491961,1,"1","1",07:03:00,07:03:00,1,60,1,60
-frequency-trip,,,3,4,,1,1,1,1,341.4491961,1,"1","1",07:04:00,07:05:00,1,60,1,60
+trip_id,arrival_time,departure_time,stop_id,location_group_id,location_id,stop_sequence,stop_headsign,pickup_type,drop_off_type,continuous_pickup,continuous_drop_off,shape_dist_traveled,timepoint,pickup_booking_rule_id,drop_off_booking_rule_id,start_pickup_drop_off_window,end_pickup_drop_off_window
+a30277f8-e50a-4a85-9141-b1e0da9d429d,07:00:00,07:00:00,4u6g,,,1,Test stop headsign,0,0,1,1,0.0000000,1,"1","1",07:00:00,07:00:00,1
+a30277f8-e50a-4a85-9141-b1e0da9d429d,07:01:00,07:01:00,johv,,,2,Test stop headsign 2,0,0,1,1,341.4491961,1,"1","1",07:01:00,07:01:00
+frequency-trip,08:00:00,08:00:00,4u6g,,,1,Test stop headsign frequency trip,0,0,1,1,0.0000000,1,"1","1",07:02:00,07:02:00
+frequency-trip,08:29:00,08:29:00,1234,,,2,Test stop headsign frequency trip,0,0,1,1,341.4491961,1,"1","1",07:03:00,07:03:00
+frequency-trip,08:31:00,08:31:00,,,area_251,3,,0,0,1,1,341.4491961,1,"1","1",07:03:00,07:03:00
+frequency-trip,,,,1,,4,,1,1,1,1,341.4491961,1,"1","1",07:04:00,07:05:00

From 874d08e65cbc7f768e26fe90054ddfedc2ef281e Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Wed, 3 Jul 2024 13:38:20 +0100
Subject: [PATCH 05/16] refactor(pom.xml): Bumped GTFS-lib version

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index f7d4290ba..458d24f9d 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,7 @@
         <dependency>
             <groupId>com.github.ibi-group</groupId>
             <artifactId>gtfs-lib</artifactId>
-            <version>57506da336673b3eff1092b39ce550d5cbe31a4f</version>
+            <version>458639c188efe0407e7f24b5cac7760b039891ef</version>
             <!-- Latest dev-flex build on jitpack.io -->
             <!-- Exclusions added in order to silence SLF4J warnings about multiple bindings:
                 http://www.slf4j.org/codes.html#multiple_bindings

From 41de8f0fa8065efa54be9486c8a74861f028afe1 Mon Sep 17 00:00:00 2001
From: miles-grant-ibi <miles.grant@ibigroup.com>
Date: Fri, 5 Jul 2024 14:51:28 -0400
Subject: [PATCH 06/16] update gtfs.yml to latest spec

---
 src/main/resources/gtfs/gtfs.yml | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/src/main/resources/gtfs/gtfs.yml b/src/main/resources/gtfs/gtfs.yml
index 1f1de715b..5ab584bf1 100644
--- a/src/main/resources/gtfs/gtfs.yml
+++ b/src/main/resources/gtfs/gtfs.yml
@@ -581,6 +581,16 @@
     inputType: GTFS_STOP
     columnWidth: 6
     helpContent: "The stop_id field contains an ID that uniquely identifies a stop. Multiple routes may use the same stop. The stop_id is referenced from the stops.txt file. If location_type is used in stops.txt, all stops referenced in stop_times.txt must have location_type of 0."
+  - name: "location_id"
+    required: false
+    inputType: GTFS_ID
+    columnWidth: 6
+    helpContent: "The location_id field contains an ID that uniquely identifies a location. Multiple routes may use the same stop. The stop_id is referenced from the stops.txt file. If location_type is used in stops.txt, all stops referenced in stop_times.txt must have location_type of 0."
+  - name: "location_group_id"
+    required: false
+    inputType: TEXT
+    columnWidth: 6
+    helpContent: "The location_group_id"
   - name: "stop_sequence"
     required: true
     inputType: POSITIVE_INT

From 1655d8f31b3b396dc1017a104b237cc42ca21cb0 Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Mon, 8 Jul 2024 16:30:49 +0100
Subject: [PATCH 07/16] refactor(pom.xml): Bumped GTFS-Lib version

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 458d24f9d..9c0736972 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,7 @@
         <dependency>
             <groupId>com.github.ibi-group</groupId>
             <artifactId>gtfs-lib</artifactId>
-            <version>458639c188efe0407e7f24b5cac7760b039891ef</version>
+            <version>0dd268bf27afc5efa925d06bdbfab17bfc14d832</version>
             <!-- Latest dev-flex build on jitpack.io -->
             <!-- Exclusions added in order to silence SLF4J warnings about multiple bindings:
                 http://www.slf4j.org/codes.html#multiple_bindings

From 04a9819bb22ad2110d3c1384c6ebdc66a6dbc340 Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Tue, 9 Jul 2024 16:41:40 +0100
Subject: [PATCH 08/16] refactor(pom.xml): Bumped GTFS-Lib version

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 9c0736972..a47139f36 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,7 @@
         <dependency>
             <groupId>com.github.ibi-group</groupId>
             <artifactId>gtfs-lib</artifactId>
-            <version>0dd268bf27afc5efa925d06bdbfab17bfc14d832</version>
+            <version>20346cde341313485be6572afc91e14bd9a20b6a</version>
             <!-- Latest dev-flex build on jitpack.io -->
             <!-- Exclusions added in order to silence SLF4J warnings about multiple bindings:
                 http://www.slf4j.org/codes.html#multiple_bindings

From 0c490d630057eed70f52570cecf321bb5972851e Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Thu, 11 Jul 2024 14:55:45 +0100
Subject: [PATCH 09/16] refactor(Bumped gtfs-lib version):

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index a47139f36..96b4368cc 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,7 @@
         <dependency>
             <groupId>com.github.ibi-group</groupId>
             <artifactId>gtfs-lib</artifactId>
-            <version>20346cde341313485be6572afc91e14bd9a20b6a</version>
+            <version>3b25aa17d8cf5117b2127ca06ef45dda5c3d1f71</version>
             <!-- Latest dev-flex build on jitpack.io -->
             <!-- Exclusions added in order to silence SLF4J warnings about multiple bindings:
                 http://www.slf4j.org/codes.html#multiple_bindings

From a62e3db7c641afb3dbec8d1d793283e597918c16 Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Mon, 9 Sep 2024 16:34:53 +0100
Subject: [PATCH 10/16] refactor(pom.xml): Bumped GTFS-Lib version

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 96b4368cc..049b547b7 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,7 @@
         <dependency>
             <groupId>com.github.ibi-group</groupId>
             <artifactId>gtfs-lib</artifactId>
-            <version>3b25aa17d8cf5117b2127ca06ef45dda5c3d1f71</version>
+            <version>abd7b9dfb9fd6a84a69b0f296e61597a63a10cf5</version>
             <!-- Latest dev-flex build on jitpack.io -->
             <!-- Exclusions added in order to silence SLF4J warnings about multiple bindings:
                 http://www.slf4j.org/codes.html#multiple_bindings

From 45d70360b9d7b58afd1861b9136cbeac21c0eda1 Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Mon, 9 Sep 2024 16:58:57 +0100
Subject: [PATCH 11/16] refactor(pom.xml): Included common-lang3 required by
 GTFS-Lib

---
 pom.xml | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/pom.xml b/pom.xml
index 049b547b7..2e99905d8 100644
--- a/pom.xml
+++ b/pom.xml
@@ -455,6 +455,11 @@
             <artifactId>gson</artifactId>
             <version>2.8.9</version>
         </dependency>
+        <dependency>
+            <groupId>org.apache.commons</groupId>
+            <artifactId>commons-lang3</artifactId>
+            <version>3.17.0</version>
+        </dependency>
     </dependencies>
 
 </project>

From 40dfe4015be09c802fc37e083bf7a35b9f1dbf21 Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Tue, 17 Sep 2024 09:57:24 +0100
Subject: [PATCH 12/16] refactor(Refined reference table discovery):

---
 .../jobs/feedmerge/MergeLineContext.java      |  7 +---
 .../feedmerge/ReferenceTableDiscovery.java    | 34 +++----------------
 2 files changed, 5 insertions(+), 36 deletions(-)

diff --git a/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeLineContext.java b/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeLineContext.java
index c8ff7a1f9..51b5a255c 100644
--- a/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeLineContext.java
+++ b/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/MergeLineContext.java
@@ -272,13 +272,8 @@ private Table getReferenceTable(FieldContext fieldContext, Field field) {
                     getTableScopedValue(Table.CALENDAR_DATES, fieldContext.getValue())
                 );
             case STOP_TIMES_STOP_ID_KEY:
-                return ReferenceTableDiscovery.getStopTimeReferenceTable(
-                    fieldContext.getValueToWrite(),
-                    mergeFeedsResult,
-                    feedMergeContext.locationIds
-                );
             case LOCATION_GROUP_STOPS_STOP_ID_KEY:
-                return ReferenceTableDiscovery.getLocationGroupStopReferenceTable(
+                return ReferenceTableDiscovery.getStopReferenceTable(
                     fieldContext.getValueToWrite(),
                     mergeFeedsResult
                 );
diff --git a/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/ReferenceTableDiscovery.java b/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/ReferenceTableDiscovery.java
index 1aa8ac4f5..285a8912d 100644
--- a/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/ReferenceTableDiscovery.java
+++ b/src/main/java/com/conveyal/datatools/manager/jobs/feedmerge/ReferenceTableDiscovery.java
@@ -3,7 +3,6 @@
 import com.conveyal.gtfs.loader.Field;
 import com.conveyal.gtfs.loader.Table;
 
-import java.util.Set;
 import java.util.stream.Collectors;
 
 import static com.conveyal.datatools.manager.jobs.feedmerge.MergeLineContext.SERVICE_ID;
@@ -32,8 +31,7 @@ public enum ReferenceTableKey {
                 REF_TABLE_SEPARATOR,
                 Table.LOCATION_GROUP_STOPS.name,
                 "stop_id",
-                Table.STOPS.name,
-                Table.LOCATIONS.name
+                Table.STOPS.name
             )
         ),
         STOP_TIMES_STOP_ID_KEY(
@@ -41,9 +39,7 @@ public enum ReferenceTableKey {
                 REF_TABLE_SEPARATOR,
                 Table.STOP_TIMES.name,
                 "stop_id",
-                Table.STOPS.name,
-                Table.LOCATIONS.name,
-                Table.LOCATION_GROUP_STOPS.name
+                Table.STOPS.name
             )
         );
 
@@ -112,33 +108,11 @@ public static Table getTripServiceIdReferenceTable(
     }
 
     /**
-     * Define the reference table for a location group stop: stop id. This will either be a stop or null.
+     * Defines the reference table as a stop if the field value matches a stop id.
      */
-    public static Table getLocationGroupStopReferenceTable(
-        String fieldValue,
-        MergeFeedsResult mergeFeedsResult
-    ) {
-        if (mergeFeedsResult.stopIds.contains(fieldValue)) {
-            return Table.STOPS;
-        }
-        return null;
-    }
-
-    /**
-     * Define the reference table for a stop time's stop id. This will either be a stop, location or location group.
-     * TODO: In later PR's this will be redundant as a stop time: stop id will only reference a stop.
-     */
-    public static Table getStopTimeReferenceTable(
-        String fieldValue,
-        MergeFeedsResult mergeFeedsResult,
-        Set<String> locationIds
-    ) {
+    public static Table getStopReferenceTable(String fieldValue, MergeFeedsResult mergeFeedsResult) {
         if (mergeFeedsResult.stopIds.contains(fieldValue)) {
             return Table.STOPS;
-        } else if (locationIds.contains(fieldValue)) {
-            return Table.LOCATIONS;
-        } else if (mergeFeedsResult.locationGroupStopIds.contains(fieldValue)) {
-            return Table.LOCATION_GROUP_STOPS;
         }
         return null;
     }

From 8e2d5208e4c8a2760c353702393388d6259acad4 Mon Sep 17 00:00:00 2001
From: miles-grant-ibi <miles.grant@ibigroup.com>
Date: Tue, 17 Sep 2024 10:32:30 -0400
Subject: [PATCH 13/16] update gtfs-lib

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 2e99905d8..fd7f71367 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,7 @@
         <dependency>
             <groupId>com.github.ibi-group</groupId>
             <artifactId>gtfs-lib</artifactId>
-            <version>abd7b9dfb9fd6a84a69b0f296e61597a63a10cf5</version>
+            <version>7c9a09cb463e932773b2611e49ac1f18fd4b295d</version>
             <!-- Latest dev-flex build on jitpack.io -->
             <!-- Exclusions added in order to silence SLF4J warnings about multiple bindings:
                 http://www.slf4j.org/codes.html#multiple_bindings

From a892b31d4bcd22b617b2a6ac05ece30fa3ee0573 Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Wed, 18 Sep 2024 08:20:37 +0100
Subject: [PATCH 14/16] refactor(pom.xml): Bumped GTFS-lib version

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index fd7f71367..9e040308b 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,7 @@
         <dependency>
             <groupId>com.github.ibi-group</groupId>
             <artifactId>gtfs-lib</artifactId>
-            <version>7c9a09cb463e932773b2611e49ac1f18fd4b295d</version>
+            <version>26824f52fd7f8e74069ebfaa197754cefb6e5a03</version>
             <!-- Latest dev-flex build on jitpack.io -->
             <!-- Exclusions added in order to silence SLF4J warnings about multiple bindings:
                 http://www.slf4j.org/codes.html#multiple_bindings

From 2426091b0dc82ccc8ab4a7f36ab1efdbd5db1ad7 Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Tue, 22 Oct 2024 10:27:14 +0100
Subject: [PATCH 15/16] refactor(Addressed PR feedback):

---
 .github/workflows/build-docker.yml |  2 +-
 pom.xml                            |  2 +-
 src/main/resources/gtfs/gtfs.yml   | 10 ----------
 3 files changed, 2 insertions(+), 12 deletions(-)

diff --git a/.github/workflows/build-docker.yml b/.github/workflows/build-docker.yml
index aa6290c21..e0702e703 100644
--- a/.github/workflows/build-docker.yml
+++ b/.github/workflows/build-docker.yml
@@ -3,7 +3,7 @@ name: Create and publish a Docker image
 
 on:
   push:
-    branches: ['dev', 'master', 'dev-flex', 'feature/DT-423-rename-stop-areas-location-groups']
+    branches: ['dev', 'master', 'dev-flex']
 
 env:
   REGISTRY: ghcr.io
diff --git a/pom.xml b/pom.xml
index 9e040308b..0f430e5b4 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,7 @@
         <dependency>
             <groupId>com.github.ibi-group</groupId>
             <artifactId>gtfs-lib</artifactId>
-            <version>26824f52fd7f8e74069ebfaa197754cefb6e5a03</version>
+            <version>43bd72f5a3d0ddd8da9061dbc57f9e88cecf972e</version>
             <!-- Latest dev-flex build on jitpack.io -->
             <!-- Exclusions added in order to silence SLF4J warnings about multiple bindings:
                 http://www.slf4j.org/codes.html#multiple_bindings
diff --git a/src/main/resources/gtfs/gtfs.yml b/src/main/resources/gtfs/gtfs.yml
index 5ab584bf1..1f1de715b 100644
--- a/src/main/resources/gtfs/gtfs.yml
+++ b/src/main/resources/gtfs/gtfs.yml
@@ -581,16 +581,6 @@
     inputType: GTFS_STOP
     columnWidth: 6
     helpContent: "The stop_id field contains an ID that uniquely identifies a stop. Multiple routes may use the same stop. The stop_id is referenced from the stops.txt file. If location_type is used in stops.txt, all stops referenced in stop_times.txt must have location_type of 0."
-  - name: "location_id"
-    required: false
-    inputType: GTFS_ID
-    columnWidth: 6
-    helpContent: "The location_id field contains an ID that uniquely identifies a location. Multiple routes may use the same stop. The stop_id is referenced from the stops.txt file. If location_type is used in stops.txt, all stops referenced in stop_times.txt must have location_type of 0."
-  - name: "location_group_id"
-    required: false
-    inputType: TEXT
-    columnWidth: 6
-    helpContent: "The location_group_id"
   - name: "stop_sequence"
     required: true
     inputType: POSITIVE_INT

From 10d924c72b5e18346ba2146ddb90601d35eda486 Mon Sep 17 00:00:00 2001
From: Robin Beer <robin.beer@arcadis.com>
Date: Tue, 22 Oct 2024 10:39:01 +0100
Subject: [PATCH 16/16] refactor(pom.xml): Bumped GTFS-lib version

Reinstated methods in gtfs-lib required by MergeFeedJob
---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 0f430e5b4..d23073715 100644
--- a/pom.xml
+++ b/pom.xml
@@ -271,7 +271,7 @@
         <dependency>
             <groupId>com.github.ibi-group</groupId>
             <artifactId>gtfs-lib</artifactId>
-            <version>43bd72f5a3d0ddd8da9061dbc57f9e88cecf972e</version>
+            <version>c9908447dad0720cd606c3c753752e6850f7b572</version>
             <!-- Latest dev-flex build on jitpack.io -->
             <!-- Exclusions added in order to silence SLF4J warnings about multiple bindings:
                 http://www.slf4j.org/codes.html#multiple_bindings