Skip to content

Commit b494120

Browse files
committed
fix: handle potential null values in workout description checks
1 parent e5aa23a commit b494120

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

boot/src/main/kotlin/org/freekode/tp2intervals/app/workout/WorkoutService.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -436,8 +436,8 @@ class WorkoutService(
436436

437437
return externalData.trainingPeaksId != null &&
438438
externalData.trainerRoadId != null &&
439-
workout.details.description.contains(
439+
workout.details.description?.contains(
440440
ExternalData.DESCRIPTION_SEPARATOR
441-
)
441+
) == true
442442
}
443443
}

boot/src/main/kotlin/org/freekode/tp2intervals/infrastructure/platform/trainingpeaks/workout/TrainingPeaksWorkoutRepository.kt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -196,9 +196,9 @@ class TrainingPeaksWorkoutRepository(
196196
}
197197

198198
require(
199-
workout.details.description.contains(
199+
workout.details.description?.contains(
200200
ExternalData.DESCRIPTION_SEPARATOR
201-
)
201+
) == true
202202
) {
203203
"Refusing to delete a workout not managed by the application"
204204
}
@@ -226,9 +226,9 @@ class TrainingPeaksWorkoutRepository(
226226

227227
return externalData.trainingPeaksId != null &&
228228
externalData.trainerRoadId != null &&
229-
workout.details.description.contains(
229+
workout.details.description?.contains(
230230
ExternalData.DESCRIPTION_SEPARATOR
231-
)
231+
) == true
232232
}
233233

234234
}

0 commit comments

Comments
 (0)