Skip to content

Commit 56ee716

Browse files
committed
Change property order to be more logical in ImporterRoute.
1 parent 629c4fa commit 56ee716

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

src/main/java/fi/hsl/jore/importer/feature/batch/route/RouteExportMapper.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,12 +31,12 @@ public RouteExportMapper(final IJsonbConverter jsonConverter) {
3131
@Override
3232
public ImporterRoute mapRow(final ResultSet resultSet, final int rowNumber) throws SQLException {
3333
return ImporterRoute.of(
34+
resultSet.getString("route_number"),
35+
getOptionalShort(resultSet, "hidden_variant"),
3436
UUID.fromString(resultSet.getString("direction_id")),
3537
DirectionType.of(resultSet.getString("direction_type")),
3638
jsonConverter.fromJson(resultSet.getString("name"), MultilingualString.class),
3739
UUID.fromString(resultSet.getString("line_jore4_id")),
38-
resultSet.getString("route_number"),
39-
getOptionalShort(resultSet, "hidden_variant"),
4040
DATE_RANGE_CONVERTER.from(resultSet.getString("valid_date_range")),
4141
LegacyHslMunicipalityCode.valueOf(resultSet.getString("legacy_hsl_municipality_code"))
4242
);

src/main/java/fi/hsl/jore/importer/feature/network/route/dto/ImporterRoute.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,13 @@ public interface ImporterRoute {
3232

3333
LegacyHslMunicipalityCode legacyHslMunicipalityCode();
3434

35-
static ImporterRoute of(final UUID directionId,
35+
static ImporterRoute of(final String routeNumber,
36+
final Optional<Short> hiddenVariant,
37+
final UUID directionId,
3638
final DirectionType directionType,
3739
final MultilingualString name,
3840
final UUID lineJore4Id,
39-
final String routeNumber,
40-
final Optional<Short> hiddenVariant,
41-
final DateRange validDateRange,
41+
final DateRange validDateRange,
4242
final LegacyHslMunicipalityCode legacyHslMunicipalityCode) {
4343
return ImmutableImporterRoute.builder()
4444
.directionId(directionId)

src/test/java/fi/hsl/jore/importer/feature/batch/route/RouteExportProcessorTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,12 @@ class RouteExportProcessorTest {
3333
private static final LegacyHslMunicipalityCode ROUTE_LEGACY_HSL_MUNICIPALITY_CODE = LegacyHslMunicipalityCode.HELSINKI;
3434

3535
private static final ImporterRoute INPUT = ImporterRoute.of(
36+
ROUTE_NUMBER,
37+
ROUTE_HIDDEN_VARIANT,
3638
IMPORTER_ROUTE_DIRECTION_ID,
3739
IMPORTER_ROUTE_DIRECTION,
3840
createMultilingualString(FINNISH_NAME, SWEDISH_NAME),
3941
LINE_JORE4_ID,
40-
ROUTE_NUMBER,
41-
ROUTE_HIDDEN_VARIANT,
4242
DateRange.between(VALIDITY_PERIOD_START_DAY, VALIDITY_PERIOD_END_DAY),
4343
ROUTE_LEGACY_HSL_MUNICIPALITY_CODE
4444
);

0 commit comments

Comments
 (0)