Ogranicz zdjęcia do dzisiejszych kursów

app/build.gradle.kts

@@ -12,8 +12,8 @@ android {
         applicationId = "pl.firmatpp.kierowca"
         minSdk = 26
         targetSdk = 35
-        versionCode = 9
-        versionName = "1.0.8"
+        versionCode = 11
+        versionName = "1.0.10"
         testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
 
         buildConfigField("String", "API_BASE_URL", "\"https://api-intranet.firmatpp.pl/api/\"")

app/src/main/java/pl/firmatpp/kierowca/data/DriverRepository.kt

@@ -62,6 +62,10 @@ class DriverRepository(
         )
     }
 
+    suspend fun deletePhoto(photoId: String) {
+        api.deletePhoto(authHeader(requireToken()), photoId)
+    }
+
     suspend fun logout() {
         val token = tokenStore.read()
         if (token != null) {

app/src/main/java/pl/firmatpp/kierowca/data/api/MobileDriverApi.kt

@@ -10,6 +10,7 @@ import pl.firmatpp.kierowca.data.model.RouteResponse
 import pl.firmatpp.kierowca.data.model.VerifyOtpBody
 import pl.firmatpp.kierowca.data.model.VerifyOtpResponse
 import retrofit2.http.Body
+import retrofit2.http.DELETE
 import retrofit2.http.GET
 import retrofit2.http.Header
 import retrofit2.http.Multipart
@@ -52,4 +53,10 @@ interface MobileDriverApi {
         @Part("longitude") longitude: RequestBody?,
         @Part("locationAccuracyMeters") locationAccuracyMeters: RequestBody?,
     ): PhotoUploadResponse
+
+    @DELETE("mobile/driver/photos/{photoId}")
+    suspend fun deletePhoto(
+        @Header("Authorization") authorization: String,
+        @Path("photoId") photoId: String,
+    ): Map<String, Boolean>
 }

app/src/main/java/pl/firmatpp/kierowca/data/model/DriverModels.kt

@@ -69,15 +69,25 @@ data class DriverRouteDto(
     val distanceKm: Double?,
     val notes: String?,
     val truck: TruckDto?,
+    val originNavigation: NavigationPointDto? = null,
+    val destinationNavigation: NavigationPointDto? = null,
     val photos: List<RoutePhotoDto> = emptyList(),
 )
 
 data class TruckDto(
     val id: String,
     val registration: String?,
+    val trailerRegistration: String? = null,
     val name: String?,
 )
 
+data class NavigationPointDto(
+    val id: String,
+    val label: String?,
+    val latitude: Double?,
+    val longitude: Double?,
+)
+
 data class RoutePhotoDto(
     val id: String,
     val routeId: String,
@@ -90,6 +100,8 @@ data class RoutePhotoDto(
     val longitude: Double?,
     val locationAccuracyMeters: Double?,
     val createdAt: String?,
+    val canDelete: Boolean = false,
+    val deleteUntil: String? = null,
 )
 
 data class PhotoUploadResponse(

app/src/main/java/pl/firmatpp/kierowca/ui/DriverApp.kt

@@ -44,6 +44,7 @@ import androidx.compose.foundation.lazy.grid.GridCells
 import androidx.compose.foundation.lazy.grid.LazyVerticalGrid
 import androidx.compose.foundation.lazy.grid.items
 import androidx.compose.foundation.lazy.items
+import androidx.compose.foundation.lazy.rememberLazyListState
 import androidx.compose.foundation.shape.RoundedCornerShape
 import androidx.compose.material.icons.Icons
 import androidx.compose.material.icons.outlined.AddPhotoAlternate
@@ -51,8 +52,10 @@ import androidx.compose.material.icons.outlined.ArrowBack
 import androidx.compose.material.icons.outlined.CameraAlt
 import androidx.compose.material.icons.outlined.CalendarToday
 import androidx.compose.material.icons.outlined.Factory
+import androidx.compose.material.icons.outlined.Delete
 import androidx.compose.material.icons.outlined.LocationOn
 import androidx.compose.material.icons.outlined.LocalShipping
+import androidx.compose.material.icons.outlined.Navigation
 import androidx.compose.material.icons.outlined.Person
 import androidx.compose.material.icons.outlined.Phone
 import androidx.compose.material.ExperimentalMaterialApi
@@ -114,6 +117,7 @@ import java.util.Locale
 import pl.firmatpp.kierowca.R
 import pl.firmatpp.kierowca.data.PhotoUploadMetadata
 import pl.firmatpp.kierowca.data.model.DriverRouteDto
+import pl.firmatpp.kierowca.data.model.NavigationPointDto
 import pl.firmatpp.kierowca.data.model.RoutePhotoDto
 import pl.firmatpp.kierowca.domain.OtpCodeExtractor
 import pl.firmatpp.kierowca.domain.RouteDisplayMapper
@@ -168,7 +172,7 @@ fun DriverApp(viewModel: DriverViewModel) {
                 onRoute = viewModel::openRoute,
             )
             DriverScreen.Profile -> ProfileScreen(state, viewModel::refreshRoutes, viewModel::openProfile, viewModel::logout)
-            DriverScreen.Detail -> DetailScreen(state, viewModel::back, viewModel::uploadPhoto, viewModel::openPhoto)
+            DriverScreen.Detail -> DetailScreen(state, viewModel::back, viewModel::uploadPhoto, viewModel::openPhoto, viewModel::deletePhoto, viewModel::refreshSelectedRoute)
             DriverScreen.Photo -> PhotoScreen(state, viewModel::back)
         }
 
@@ -349,8 +353,16 @@ private fun RouteDateSelector(selectedDate: String, minDate: String, maxDate: St
             }.toList()
         }.getOrElse { listOf(LocalDate.parse(selectedDate)) }
     }
+    val listState = rememberLazyListState()
 
-    LazyRow(horizontalArrangement = Arrangement.spacedBy(8.dp)) {
+    LaunchedEffect(dates, selectedDate) {
+        val selectedIndex = dates.indexOfFirst { it.toString() == selectedDate }
+        if (selectedIndex >= 0) {
+            listState.scrollToItem(selectedIndex)
+        }
+    }
+
+    LazyRow(state = listState, horizontalArrangement = Arrangement.spacedBy(8.dp)) {
         items(dates, key = { it.toString() }) { date ->
             DateSelectorChip(
                 date = date,
@@ -566,16 +578,19 @@ private fun routeStatusColor(status: String): Color {
     }
 }
 
-@OptIn(ExperimentalMaterial3Api::class)
+@OptIn(ExperimentalMaterial3Api::class, ExperimentalMaterialApi::class)
 @Composable
 private fun DetailScreen(
     state: DriverUiState,
     onBack: () -> Unit,
     onUpload: (Uri, String, PhotoUploadMetadata) -> Unit,
     onPhoto: (RoutePhotoDto) -> Unit,
+    onDeletePhoto: (RoutePhotoDto) -> Unit,
+    onRefresh: () -> Unit,
 ) {
     val context = LocalContext.current
     var cameraUri by remember { mutableStateOf<Uri?>(null) }
+    val pullRefreshState = rememberPullRefreshState(state.refreshing, onRefresh)
     val cameraLauncher = rememberLauncherForActivityResult(ActivityResultContracts.TakePicture()) { ok ->
         val capturedUri = cameraUri
         if (ok && capturedUri != null) onUpload(capturedUri, "camera", cameraPhotoMetadata(context))
@@ -590,6 +605,7 @@ private fun DetailScreen(
         if (uri != null) onUpload(uri, "gallery", PhotoUploadMetadata())
     }
     val route = state.selectedRoute
+    val canManagePhotos = canManageRoutePhotos(state.selectedDate)
 
     Scaffold(
         topBar = {
@@ -598,34 +614,45 @@ private fun DetailScreen(
         containerColor = TppColors.Surface,
     ) { padding ->
         if (route == null) return@Scaffold
-        LazyColumn(
-            Modifier.fillMaxSize().padding(padding),
-            contentPadding = PaddingValues(16.dp),
-            verticalArrangement = Arrangement.spacedBy(24.dp),
-        ) {
-            item { ManifestSection(route) }
-            item {
-                CargoDocumentationSection(
-                    photos = route.photos,
-                    imageAuthHeader = state.imageAuthHeader,
-                    allowGalleryUploads = state.allowGalleryUploads,
-                    onPhoto = onPhoto,
-                    onCamera = {
-                        val newUri = createCameraUri(context)
-                        cameraUri = newUri
-                        val missingPermissions = cameraCapturePermissions(context)
-                        if (missingPermissions.isEmpty()) {
-                            cameraLauncher.launch(newUri)
-                        } else {
-                            cameraPermissionLauncher.launch(missingPermissions)
-                        }
-                    },
-                    onGallery = {
-                        pickerLauncher.launch(PickVisualMediaRequest(ActivityResultContracts.PickVisualMedia.ImageOnly))
-                    },
-                )
+        Box(Modifier.fillMaxSize().padding(padding).pullRefresh(pullRefreshState)) {
+            LazyColumn(
+                Modifier.fillMaxSize(),
+                contentPadding = PaddingValues(16.dp),
+                verticalArrangement = Arrangement.spacedBy(24.dp),
+            ) {
+                item { ManifestSection(route, onNavigate = { openNavigation(context, it) }) }
+                item {
+                    CargoDocumentationSection(
+                        photos = route.photos,
+                        imageAuthHeader = state.imageAuthHeader,
+                        allowGalleryUploads = state.allowGalleryUploads,
+                        canManagePhotos = canManagePhotos,
+                        onPhoto = onPhoto,
+                        onDeletePhoto = onDeletePhoto,
+                        onCamera = {
+                            val newUri = createCameraUri(context)
+                            cameraUri = newUri
+                            val missingPermissions = cameraCapturePermissions(context)
+                            if (missingPermissions.isEmpty()) {
+                                cameraLauncher.launch(newUri)
+                            } else {
+                                cameraPermissionLauncher.launch(missingPermissions)
+                            }
+                        },
+                        onGallery = {
+                            pickerLauncher.launch(PickVisualMediaRequest(ActivityResultContracts.PickVisualMedia.ImageOnly))
+                        },
+                    )
+                }
+                item { ErrorText(state.error) }
             }
-            item { ErrorText(state.error) }
+            PullRefreshIndicator(
+                refreshing = state.refreshing,
+                state = pullRefreshState,
+                modifier = Modifier.align(Alignment.TopCenter),
+                backgroundColor = Color.White,
+                contentColor = TppColors.Forest,
+            )
         }
     }
 }
@@ -653,7 +680,7 @@ private fun DetailHeader(onBack: () -> Unit) {
 }
 
 @Composable
-private fun ManifestSection(route: DriverRouteDto) {
+private fun ManifestSection(route: DriverRouteDto, onNavigate: (NavigationPointDto) -> Unit) {
     val stripColor = routeStatusColor(route.status)
     Card(
         colors = CardDefaults.cardColors(containerColor = Color.White),
@@ -676,7 +703,6 @@ private fun ManifestSection(route: DriverRouteDto) {
                             color = TppColors.Ink,
                         )
                     }
-                    RouteStartChip(route.startsAt)
                 }
             }
             HorizontalDivider(color = TppColors.Outline.copy(alpha = 0.65f))
@@ -687,6 +713,8 @@ private fun ManifestSection(route: DriverRouteDto) {
                 icon = Icons.Outlined.Factory,
                 iconBackground = TppColors.Panel,
                 iconTint = TppColors.ContainerGreen,
+                navigationPoint = route.originNavigation,
+                onNavigate = onNavigate,
             )
             HorizontalDivider(color = TppColors.Outline.copy(alpha = 0.45f))
             RoutePointBlock(
@@ -697,29 +725,14 @@ private fun ManifestSection(route: DriverRouteDto) {
                 iconBackground = TppColors.Panel.copy(alpha = 0.72f),
                 iconTint = TppColors.Forest,
                 framed = true,
+                navigationPoint = route.destinationNavigation,
+                onNavigate = onNavigate,
             )
             RouteFactsFooter(route)
         }
     }
 }
 
-@Composable
-private fun RouteStartChip(startsAt: String) {
-    Row(
-        modifier = Modifier.background(TppColors.Panel, RoundedCornerShape(8.dp)).padding(horizontal = 10.dp, vertical = 8.dp),
-        verticalAlignment = Alignment.CenterVertically,
-        horizontalArrangement = Arrangement.spacedBy(6.dp),
-    ) {
-        Icon(Icons.Outlined.CalendarToday, contentDescription = null, tint = TppColors.Muted, modifier = Modifier.size(18.dp))
-        Text(
-            startsAt.ifBlank { "Dziś" },
-            color = TppColors.Muted,
-            fontFamily = FontFamily.Monospace,
-            fontWeight = FontWeight.SemiBold,
-        )
-    }
-}
-
 @Composable
 private fun RoutePointBlock(
     label: String,
@@ -729,10 +742,13 @@ private fun RoutePointBlock(
     iconBackground: Color,
     iconTint: Color,
     framed: Boolean = false,
+    navigationPoint: NavigationPointDto? = null,
+    onNavigate: (NavigationPointDto) -> Unit = {},
 ) {
     Row(
         Modifier.fillMaxWidth().padding(20.dp),
         horizontalArrangement = Arrangement.spacedBy(16.dp),
+        verticalAlignment = Alignment.CenterVertically,
     ) {
         Box(
             Modifier.size(48.dp).background(iconBackground, RoundedCornerShape(24.dp)),
@@ -760,6 +776,14 @@ private fun RoutePointBlock(
                 Text(subtitle, color = TppColors.Muted, style = MaterialTheme.typography.bodyLarge)
             }
         }
+        if (navigationPoint?.hasCoordinates() == true) {
+            IconButton(
+                onClick = { onNavigate(navigationPoint) },
+                modifier = Modifier.size(48.dp).background(TppColors.Forest, RoundedCornerShape(24.dp)),
+            ) {
+                Icon(Icons.Outlined.Navigation, contentDescription = "Nawiguj", tint = Color.White)
+            }
+        }
     }
 }
 
@@ -790,7 +814,9 @@ private fun CargoDocumentationSection(
     photos: List<RoutePhotoDto>,
     imageAuthHeader: String?,
     allowGalleryUploads: Boolean,
+    canManagePhotos: Boolean,
     onPhoto: (RoutePhotoDto) -> Unit,
+    onDeletePhoto: (RoutePhotoDto) -> Unit,
     onCamera: () -> Unit,
     onGallery: () -> Unit,
 ) {
@@ -801,12 +827,18 @@ private fun CargoDocumentationSection(
             Box(Modifier.width(116.dp).height(3.dp).background(TppColors.ContainerGreen))
         }
         Column(verticalArrangement = Arrangement.spacedBy(12.dp)) {
-            if (allowGalleryUploads) {
-                CargoActionButton("Dodaj zdjęcie z galerii", Icons.Outlined.AddPhotoAlternate, TppColors.ContainerGreen, onGallery)
+            if (canManagePhotos) {
+                if (allowGalleryUploads) {
+                    CargoActionButton("Dodaj zdjęcie z galerii", Icons.Outlined.AddPhotoAlternate, TppColors.ContainerGreen, onGallery)
+                }
+                CargoActionButton("Zrób nowe zdjęcie", Icons.Outlined.CameraAlt, TppColors.Forest, onCamera)
+            } else {
+                Box(Modifier.fillMaxWidth().background(TppColors.Panel, RoundedCornerShape(4.dp)).padding(14.dp)) {
+                    Text("Zdjęcia można dodawać tylko dla dzisiejszych kursów.", color = TppColors.Muted, fontFamily = FontFamily.Monospace)
+                }
             }
-            CargoActionButton("Zrób nowe zdjęcie", Icons.Outlined.CameraAlt, TppColors.Forest, onCamera)
         }
-        PhotoGrid(photos, imageAuthHeader, onPhoto)
+        PhotoGrid(photos, imageAuthHeader, onPhoto, onDeletePhoto)
     }
 }
 
@@ -829,6 +861,7 @@ private fun PhotoGrid(
     photos: List<RoutePhotoDto>,
     imageAuthHeader: String?,
     onPhoto: (RoutePhotoDto) -> Unit,
+    onDeletePhoto: (RoutePhotoDto) -> Unit,
 ) {
     val rows = ceil((photos.size.coerceAtLeast(1) / 2f).toDouble()).toInt()
     val gridHeight = (rows * 172).coerceIn(172, 520).dp
@@ -855,12 +888,31 @@ private fun PhotoGrid(
                     horizontalArrangement = Arrangement.spacedBy(12.dp),
                 ) {
                     items(photos, key = { it.id }) { photo ->
-                        AsyncImage(
-                            model = imageRequest(photo.url, imageAuthHeader),
-                            contentDescription = "Zdjecie ladunku",
-                            contentScale = ContentScale.Crop,
-                            modifier = Modifier.aspectRatio(1f).clickable { onPhoto(photo) },
-                        )
+                        Box(Modifier.aspectRatio(1f)) {
+                            AsyncImage(
+                                model = imageRequest(photo.url, imageAuthHeader),
+                                contentDescription = "Zdjecie ladunku",
+                                contentScale = ContentScale.Crop,
+                                modifier = Modifier.fillMaxSize().clickable { onPhoto(photo) },
+                            )
+                            if (photo.canDelete) {
+                                IconButton(
+                                    onClick = { onDeletePhoto(photo) },
+                                    modifier = Modifier
+                                        .align(Alignment.TopEnd)
+                                        .padding(6.dp)
+                                        .size(34.dp)
+                                        .background(TppColors.Error, RoundedCornerShape(17.dp)),
+                                ) {
+                                    Icon(
+                                        Icons.Outlined.Delete,
+                                        contentDescription = "Usuń zdjęcie",
+                                        tint = Color.White,
+                                        modifier = Modifier.size(18.dp),
+                                    )
+                                }
+                            }
+                        }
                     }
                 }
             }
@@ -880,14 +932,21 @@ private fun EmptyPhotoState() {
 
 private fun DriverRouteDto.truckLabel(): String {
     val registration = truck?.registration.orEmpty()
+    val trailerRegistration = truck?.trailerRegistration.orEmpty()
     val name = truck?.name.orEmpty()
 
-    return when {
+    val vehicle = when {
         registration.isNotBlank() && name.isNotBlank() -> "$registration ($name)"
         registration.isNotBlank() -> registration
         name.isNotBlank() -> name
         else -> "-"
     }
+
+    return if (trailerRegistration.isNotBlank()) {
+        "$vehicle\nNaczepa: $trailerRegistration"
+    } else {
+        vehicle
+    }
 }
 
 private fun DriverRouteDto.distanceLabel(): String =
@@ -945,6 +1004,26 @@ private fun createCameraUri(context: Context): Uri {
     return FileProvider.getUriForFile(context, "${context.packageName}.fileprovider", file)
 }
 
+private fun NavigationPointDto.hasCoordinates(): Boolean = latitude != null && longitude != null
+
+private fun openNavigation(context: Context, point: NavigationPointDto) {
+    val latitude = point.latitude ?: return
+    val longitude = point.longitude ?: return
+    val googleMapsIntent = Intent(Intent.ACTION_VIEW, Uri.parse("google.navigation:q=$latitude,$longitude"))
+        .setPackage("com.google.android.apps.maps")
+
+    if (googleMapsIntent.resolveActivity(context.packageManager) != null) {
+        context.startActivity(googleMapsIntent)
+        return
+    }
+
+    val label = Uri.encode(point.label?.takeIf { it.isNotBlank() } ?: "Cel")
+    val geoIntent = Intent(Intent.ACTION_VIEW, Uri.parse("geo:0,0?q=$latitude,$longitude($label)"))
+    if (geoIntent.resolveActivity(context.packageManager) != null) {
+        context.startActivity(geoIntent)
+    }
+}
+
 private fun cameraCapturePermissions(context: Context): Array<String> = buildList {
     if (!hasPermission(context, Manifest.permission.CAMERA)) {
         add(Manifest.permission.CAMERA)

app/src/main/java/pl/firmatpp/kierowca/ui/DriverUiRules.kt

@@ -0,0 +1,6 @@
+package pl.firmatpp.kierowca.ui
+
+import java.time.LocalDate
+
+fun canManageRoutePhotos(selectedDate: String, today: LocalDate = LocalDate.now()): Boolean =
+    runCatching { LocalDate.parse(selectedDate).isEqual(today) }.getOrDefault(false)

app/src/main/java/pl/firmatpp/kierowca/ui/DriverViewModel.kt

@@ -113,6 +113,30 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
         _state.update { it.copy(screen = DriverScreen.Detail, selectedRoute = response.route, error = null) }
     }
 
+    fun refreshSelectedRoute() {
+        val routeId = _state.value.selectedRoute?.id ?: return
+
+        viewModelScope.launch {
+            _state.update { it.copy(refreshing = true, error = null) }
+
+            runCatching { repository.route(routeId) }
+                .onSuccess { response ->
+                    _state.update {
+                        it.copy(
+                            selectedRoute = response.route,
+                            routes = it.routes.map { route -> if (route.id == routeId) response.route else route },
+                            error = null,
+                        )
+                    }
+                }
+                .onFailure { throwable ->
+                    _state.update { it.copy(error = throwable.message ?: "Wystapil blad.") }
+                }
+
+            _state.update { it.copy(refreshing = false) }
+        }
+    }
+
     fun openProfile() {
         _state.update { it.copy(screen = DriverScreen.Profile, error = null) }
     }
@@ -123,6 +147,12 @@ class DriverViewModel(application: Application) : AndroidViewModel(application)
         openRoute(route.id)
     }
 
+    fun deletePhoto(photo: RoutePhotoDto) = runLoading {
+        val route = _state.value.selectedRoute ?: return@runLoading
+        repository.deletePhoto(photo.id)
+        openRoute(route.id)
+    }
+
     fun openPhoto(photo: RoutePhotoDto) {
         _state.update { it.copy(screen = DriverScreen.Photo, selectedPhoto = photo) }
     }

app/src/test/java/pl/firmatpp/kierowca/ui/DriverUiRulesTest.kt

@@ -0,0 +1,23 @@
+package pl.firmatpp.kierowca.ui
+
+import java.time.LocalDate
+import org.junit.Assert.assertFalse
+import org.junit.Assert.assertTrue
+import org.junit.Test
+
+class DriverUiRulesTest {
+    private val today = LocalDate.parse("2026-06-30")
+
+    @Test
+    fun allowsPhotoActionsOnlyForTodayRouteDate() {
+        assertTrue(canManageRoutePhotos("2026-06-30", today))
+        assertFalse(canManageRoutePhotos("2026-06-29", today))
+        assertFalse(canManageRoutePhotos("2026-07-01", today))
+    }
+
+    @Test
+    fun blocksPhotoActionsWhenDateIsInvalid() {
+        assertFalse(canManageRoutePhotos("", today))
+        assertFalse(canManageRoutePhotos("not-a-date", today))
+    }
+}