Compare commits
16 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 0ba4fd7eb9 | |||
| 69d4257a18 | |||
| 683169e8b7 | |||
| 77d56ee6be | |||
| 99193af2c5 | |||
| 3c008ec8df | |||
| c869f84a9d | |||
| c3be23417d | |||
| ae10ed0c82 | |||
| 897b685c70 | |||
| 4b20697bb1 | |||
| 66d28761a8 | |||
| ec478531da | |||
| 5ade80a334 | |||
| 34fb06a673 | |||
| dc2a0b2c68 |
@@ -13,6 +13,8 @@ import okhttp3.*
|
|||||||
import okhttp3.HttpUrl.Companion.toHttpUrlOrNull
|
import okhttp3.HttpUrl.Companion.toHttpUrlOrNull
|
||||||
import okhttp3.MediaType.Companion.toMediaType
|
import okhttp3.MediaType.Companion.toMediaType
|
||||||
import okhttp3.RequestBody.Companion.toRequestBody
|
import okhttp3.RequestBody.Companion.toRequestBody
|
||||||
|
import okio.BufferedSink
|
||||||
|
import okio.source
|
||||||
import org.xmlpull.v1.XmlPullParser
|
import org.xmlpull.v1.XmlPullParser
|
||||||
import org.xmlpull.v1.XmlPullParserFactory
|
import org.xmlpull.v1.XmlPullParserFactory
|
||||||
import java.io.InputStream
|
import java.io.InputStream
|
||||||
@@ -84,13 +86,18 @@ open class WebDavProvider(protected val account: CloudAccount) : CloudProvider {
|
|||||||
onProgress: (Long) -> Unit,
|
onProgress: (Long) -> Unit,
|
||||||
): Result<RemoteFile> = runCatching {
|
): Result<RemoteFile> = runCatching {
|
||||||
withContext(Dispatchers.IO) {
|
withContext(Dispatchers.IO) {
|
||||||
val bytes = localStream.readBytes()
|
val body = object : RequestBody() {
|
||||||
val body = bytes.toRequestBody("application/octet-stream".toMediaType())
|
override fun contentType() = "application/octet-stream".toMediaType()
|
||||||
|
override fun contentLength() = sizeBytes
|
||||||
|
override fun writeTo(sink: BufferedSink) {
|
||||||
|
localStream.source().use { source -> sink.writeAll(source) }
|
||||||
|
}
|
||||||
|
}
|
||||||
val req = Request.Builder().url(url(remotePath)).put(body).build()
|
val req = Request.Builder().url(url(remotePath)).put(body).build()
|
||||||
client.newCall(req).execute().use { resp ->
|
client.newCall(req).execute().use { resp ->
|
||||||
if (!resp.isSuccessful) throw Exception("Upload HTTP ${resp.code}")
|
if (!resp.isSuccessful) throw Exception("Upload HTTP ${resp.code}")
|
||||||
}
|
}
|
||||||
onProgress(bytes.size.toLong())
|
onProgress(sizeBytes)
|
||||||
getFileMetadata(remotePath).getOrThrow()
|
getFileMetadata(remotePath).getOrThrow()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -58,6 +58,7 @@ enum class ConflictStrategy(val label: String) {
|
|||||||
enum class DeleteBehavior(val label: String, val description: String) {
|
enum class DeleteBehavior(val label: String, val description: String) {
|
||||||
MIRROR("Mirror deletions", "Delete on target when deleted on source"),
|
MIRROR("Mirror deletions", "Delete on target when deleted on source"),
|
||||||
KEEP("Keep deleted files", "Never delete — only add/update"),
|
KEEP("Keep deleted files", "Never delete — only add/update"),
|
||||||
|
ARCHIVE("Archive deleted", "Move files deleted from phone to _Deleted/ folder on remote"),
|
||||||
}
|
}
|
||||||
|
|
||||||
enum class ScheduleType(val label: String) {
|
enum class ScheduleType(val label: String) {
|
||||||
|
|||||||
@@ -62,13 +62,28 @@ class SyncEngine @Inject constructor(
|
|||||||
else
|
else
|
||||||
LocalAccessor.JavaFile(File(localPath))
|
LocalAccessor.JavaFile(File(localPath))
|
||||||
|
|
||||||
private suspend fun performSync(pair: SyncPair, provider: CloudProvider): SyncResult {
|
private suspend fun performSync(
|
||||||
|
pair: SyncPair,
|
||||||
|
provider: CloudProvider,
|
||||||
|
isRetry: Boolean = false,
|
||||||
|
): SyncResult {
|
||||||
val accessor = makeAccessor(pair.localPath)
|
val accessor = makeAccessor(pair.localPath)
|
||||||
val knownStates = fileStateDao.getForPair(pair.id).associateBy { it.relativePath }
|
var knownStates = fileStateDao.getForPair(pair.id).associateBy { it.relativePath }
|
||||||
val remoteFiles = provider.listFiles(pair.remotePath).getOrThrow()
|
val remoteFiles = provider.listFiles(pair.remotePath).getOrThrow()
|
||||||
.associateBy { it.path.removePrefix(pair.remotePath).trimStart('/') }
|
.associateBy { it.path.removePrefix(pair.remotePath).trimStart('/') }
|
||||||
val localFiles = accessor.walkFiles(pair)
|
val localFiles = accessor.walkFiles(pair)
|
||||||
|
|
||||||
|
// Self-healing: if every known-state path is absent from the current local scan but
|
||||||
|
// the local folder does have files, the localPath was changed without clearing state.
|
||||||
|
// The stale records would cause every old file to look like "DELETE_REMOTE" and every
|
||||||
|
// new file to re-upload indefinitely. Wipe and retry once as a fresh initial sync.
|
||||||
|
if (!isRetry && knownStates.isNotEmpty() && localFiles.isNotEmpty() &&
|
||||||
|
knownStates.keys.none { it in localFiles }) {
|
||||||
|
Timber.w("SyncEngine: stale folder states detected for pair ${pair.id} — resetting")
|
||||||
|
fileStateDao.deleteForPair(pair.id)
|
||||||
|
return performSync(pair, provider, isRetry = true)
|
||||||
|
}
|
||||||
|
|
||||||
val allPaths = (localFiles.keys + remoteFiles.keys + knownStates.keys).toSet()
|
val allPaths = (localFiles.keys + remoteFiles.keys + knownStates.keys).toSet()
|
||||||
val hasPriorSyncState = knownStates.isNotEmpty()
|
val hasPriorSyncState = knownStates.isNotEmpty()
|
||||||
val semaphore = Semaphore(4)
|
val semaphore = Semaphore(4)
|
||||||
@@ -92,11 +107,10 @@ class SyncEngine @Inject constructor(
|
|||||||
|
|
||||||
when (decision) {
|
when (decision) {
|
||||||
SyncDecision.UPLOAD -> {
|
SyncDecision.UPLOAD -> {
|
||||||
var uploadedRemoteFile: RemoteFile? = null
|
|
||||||
val bytes = runCatching {
|
val bytes = runCatching {
|
||||||
ensureRemoteDirs(provider, pair.remotePath, rel)
|
ensureRemoteDirs(provider, pair.remotePath, rel)
|
||||||
accessor.openInputStream(rel)?.use { stream ->
|
accessor.openInputStream(rel)?.use { stream ->
|
||||||
uploadedRemoteFile = provider.uploadFile(stream, "${pair.remotePath}/$rel", local!!.sizeBytes) { }.getOrThrow()
|
provider.uploadFile(stream, "${pair.remotePath}/$rel", local!!.sizeBytes) { }.getOrThrow()
|
||||||
}
|
}
|
||||||
local!!.sizeBytes
|
local!!.sizeBytes
|
||||||
}.getOrElse { e ->
|
}.getOrElse { e ->
|
||||||
@@ -105,8 +119,12 @@ class SyncEngine @Inject constructor(
|
|||||||
return@withPermit FileOutcome(failed = 1)
|
return@withPermit FileOutcome(failed = 1)
|
||||||
}
|
}
|
||||||
logEvent(pair.id, SyncEventType.FILE_UPLOADED, rel, null, bytes)
|
logEvent(pair.id, SyncEventType.FILE_UPLOADED, rel, null, bytes)
|
||||||
|
// Don't store remote metadata from upload response — the server (Nextcloud etc.)
|
||||||
|
// may change mtime/etag during post-upload processing. Leaving remoteModifiedAt
|
||||||
|
// null forces the SKIP reconciliation on the next sync to fill it in from the
|
||||||
|
// directory listing, which is the same source all future syncs will use.
|
||||||
FileOutcome(uploaded = 1, bytesTransferred = bytes,
|
FileOutcome(uploaded = 1, bytesTransferred = bytes,
|
||||||
newState = buildState(pair.id, rel, local!!, remoteAfterTransfer = uploadedRemoteFile))
|
newState = buildState(pair.id, rel, local!!, remoteAfterTransfer = null))
|
||||||
}
|
}
|
||||||
SyncDecision.DOWNLOAD -> {
|
SyncDecision.DOWNLOAD -> {
|
||||||
val bytes = runCatching {
|
val bytes = runCatching {
|
||||||
@@ -126,18 +144,32 @@ class SyncEngine @Inject constructor(
|
|||||||
logEvent(pair.id, SyncEventType.FILE_DOWNLOADED, rel, null, bytes)
|
logEvent(pair.id, SyncEventType.FILE_DOWNLOADED, rel, null, bytes)
|
||||||
FileOutcome(downloaded = 1, bytesTransferred = bytes,
|
FileOutcome(downloaded = 1, bytesTransferred = bytes,
|
||||||
newState = buildState(pair.id, rel,
|
newState = buildState(pair.id, rel,
|
||||||
LocalFileInfo(rel, remote!!.sizeBytes, localMtime), remoteAfterTransfer = remote))
|
LocalFileInfo(rel, remote!!.sizeBytes, localMtime),
|
||||||
|
remoteAfterTransfer = remote,
|
||||||
|
storeLocalMtime = false))
|
||||||
}
|
}
|
||||||
SyncDecision.DELETE_LOCAL -> {
|
SyncDecision.DELETE_LOCAL -> {
|
||||||
accessor.delete(rel)
|
val deleted = accessor.delete(rel)
|
||||||
|
if (!deleted) Timber.w("SyncEngine: DELETE_LOCAL failed (silent) for $rel")
|
||||||
fileStateDao.delete(pair.id, rel)
|
fileStateDao.delete(pair.id, rel)
|
||||||
logEvent(pair.id, SyncEventType.FILE_DELETED, rel, "local", 0)
|
logEvent(pair.id, SyncEventType.FILE_DELETED, rel, "local", 0)
|
||||||
FileOutcome(deleted = 1)
|
FileOutcome(deleted = 1)
|
||||||
}
|
}
|
||||||
SyncDecision.DELETE_REMOTE -> {
|
SyncDecision.DELETE_REMOTE -> {
|
||||||
provider.deleteFile("${pair.remotePath}/$rel")
|
if (pair.deleteBehavior == DeleteBehavior.ARCHIVE) {
|
||||||
|
val archivePath = "${pair.remotePath}/_Deleted/$rel"
|
||||||
|
runCatching {
|
||||||
|
ensureRemoteDirs(provider, "${pair.remotePath}/_Deleted", rel)
|
||||||
|
provider.moveFile("${pair.remotePath}/$rel", archivePath).getOrThrow()
|
||||||
|
}.onFailure { e -> Timber.e(e, "SyncEngine: ARCHIVE failed for $rel") }
|
||||||
|
fileStateDao.delete(pair.id, rel)
|
||||||
|
logEvent(pair.id, SyncEventType.FILE_DELETED, rel, "archived", 0)
|
||||||
|
} else {
|
||||||
|
runCatching { provider.deleteFile("${pair.remotePath}/$rel") }
|
||||||
|
.onFailure { e -> Timber.e(e, "SyncEngine: DELETE_REMOTE failed for $rel") }
|
||||||
fileStateDao.delete(pair.id, rel)
|
fileStateDao.delete(pair.id, rel)
|
||||||
logEvent(pair.id, SyncEventType.FILE_DELETED, rel, "remote", 0)
|
logEvent(pair.id, SyncEventType.FILE_DELETED, rel, "remote", 0)
|
||||||
|
}
|
||||||
FileOutcome(deleted = 1)
|
FileOutcome(deleted = 1)
|
||||||
}
|
}
|
||||||
SyncDecision.CONFLICT -> {
|
SyncDecision.CONFLICT -> {
|
||||||
@@ -203,10 +235,13 @@ class SyncEngine @Inject constructor(
|
|||||||
rel: String,
|
rel: String,
|
||||||
local: LocalFileInfo?,
|
local: LocalFileInfo?,
|
||||||
remoteAfterTransfer: RemoteFile?,
|
remoteAfterTransfer: RemoteFile?,
|
||||||
|
storeLocalMtime: Boolean = true,
|
||||||
) = SyncFileStateEntity(
|
) = SyncFileStateEntity(
|
||||||
syncPairId = pairId,
|
syncPairId = pairId,
|
||||||
relativePath = rel,
|
relativePath = rel,
|
||||||
localModifiedAt = local?.lastModifiedMs?.let { Instant.ofEpochMilli(it) },
|
// When storeLocalMtime=false, leave localModifiedAt null so the SKIP reconciliation
|
||||||
|
// pass on the next sync reads it from the walkFiles cursor (avoids SAF stale-mtime loops).
|
||||||
|
localModifiedAt = if (storeLocalMtime) local?.lastModifiedMs?.let { Instant.ofEpochMilli(it) } else null,
|
||||||
localSizeBytes = local?.sizeBytes ?: 0L,
|
localSizeBytes = local?.sizeBytes ?: 0L,
|
||||||
localHash = null,
|
localHash = null,
|
||||||
remoteModifiedAt = remoteAfterTransfer?.modifiedAt,
|
remoteModifiedAt = remoteAfterTransfer?.modifiedAt,
|
||||||
@@ -236,12 +271,16 @@ internal fun syncDecide(
|
|||||||
|
|
||||||
// Treat null known timestamps as "not yet recorded" — don't treat as changed.
|
// Treat null known timestamps as "not yet recorded" — don't treat as changed.
|
||||||
// The SKIP reconciliation pass will fill them in on the next sync.
|
// The SKIP reconciliation pass will fill them in on the next sync.
|
||||||
|
// Use second-precision for both sides: FAT32 has 2-second mtime resolution, WebDAV
|
||||||
|
// RFC-1123 has 1-second resolution, so millisecond comparison causes phantom "changed"
|
||||||
|
// detections and rewrite loops after a fresh download/upload.
|
||||||
val localChanged = known == null ||
|
val localChanged = known == null ||
|
||||||
(localExists && known.localModifiedAt != null &&
|
(localExists && known.localModifiedAt != null &&
|
||||||
local!!.lastModifiedMs != known.localModifiedAt.toEpochMilli())
|
local!!.lastModifiedMs / 1000 != known.localModifiedAt.epochSecond)
|
||||||
val remoteChanged = known == null ||
|
val remoteChanged = known == null ||
|
||||||
(remoteExists && known.remoteModifiedAt != null &&
|
(remoteExists && known.remoteModifiedAt != null &&
|
||||||
(remote!!.etag != known.remoteEtag || remote.modifiedAt != known.remoteModifiedAt))
|
(remote!!.etag != known.remoteEtag ||
|
||||||
|
remote.modifiedAt.epochSecond != known.remoteModifiedAt.epochSecond))
|
||||||
|
|
||||||
return when {
|
return when {
|
||||||
!localExists && !remoteExists -> SyncDecision.SKIP
|
!localExists && !remoteExists -> SyncDecision.SKIP
|
||||||
@@ -259,21 +298,15 @@ internal fun syncDecide(
|
|||||||
}
|
}
|
||||||
|
|
||||||
!localExists && remoteExists -> when {
|
!localExists && remoteExists -> when {
|
||||||
known == null -> if (!hasPriorSyncState) {
|
known == null -> {
|
||||||
// Initial sync: no history at all — remote files are new, download them.
|
// No state record: could be a new remote file OR a file whose state was lost.
|
||||||
|
// Downloading is always safer than deleting — if the user deleted the local
|
||||||
|
// copy intentionally, the state record will still exist (known != null) and
|
||||||
|
// the else-branch below correctly deletes the remote copy.
|
||||||
when (direction) {
|
when (direction) {
|
||||||
SyncDirection.DOWNLOAD_ONLY, SyncDirection.TWO_WAY -> SyncDecision.DOWNLOAD
|
SyncDirection.DOWNLOAD_ONLY, SyncDirection.TWO_WAY -> SyncDecision.DOWNLOAD
|
||||||
else -> SyncDecision.SKIP
|
else -> SyncDecision.SKIP
|
||||||
}
|
}
|
||||||
} else {
|
|
||||||
// Pair has been synced before but this file has no state record
|
|
||||||
// (e.g. uploaded before state-tracking was fixed). Treat the same
|
|
||||||
// as a known remote-deletion: apply mirror/keep behavior.
|
|
||||||
when {
|
|
||||||
deleteBehavior == DeleteBehavior.KEEP -> SyncDecision.SKIP
|
|
||||||
direction == SyncDirection.UPLOAD_ONLY || direction == SyncDirection.TWO_WAY -> SyncDecision.DELETE_REMOTE
|
|
||||||
else -> SyncDecision.SKIP
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
else -> when {
|
else -> when {
|
||||||
deleteBehavior == DeleteBehavior.KEEP -> SyncDecision.SKIP
|
deleteBehavior == DeleteBehavior.KEEP -> SyncDecision.SKIP
|
||||||
|
|||||||
@@ -1,10 +1,10 @@
|
|||||||
package com.syncflow.ui.addpair
|
package com.syncflow.ui.addpair
|
||||||
|
|
||||||
import android.content.Intent
|
import android.content.Intent
|
||||||
import android.net.Uri
|
|
||||||
import androidx.activity.compose.rememberLauncherForActivityResult
|
import androidx.activity.compose.rememberLauncherForActivityResult
|
||||||
import androidx.activity.result.contract.ActivityResultContracts
|
import androidx.activity.result.contract.ActivityResultContracts
|
||||||
import androidx.compose.animation.AnimatedVisibility
|
import androidx.compose.animation.AnimatedVisibility
|
||||||
|
import androidx.compose.foundation.clickable
|
||||||
import androidx.compose.foundation.layout.*
|
import androidx.compose.foundation.layout.*
|
||||||
import androidx.compose.foundation.rememberScrollState
|
import androidx.compose.foundation.rememberScrollState
|
||||||
import androidx.compose.foundation.text.KeyboardOptions
|
import androidx.compose.foundation.text.KeyboardOptions
|
||||||
@@ -22,6 +22,7 @@ import androidx.compose.ui.text.input.KeyboardType
|
|||||||
import androidx.compose.ui.unit.dp
|
import androidx.compose.ui.unit.dp
|
||||||
import androidx.hilt.navigation.compose.hiltViewModel
|
import androidx.hilt.navigation.compose.hiltViewModel
|
||||||
import com.syncflow.domain.model.*
|
import com.syncflow.domain.model.*
|
||||||
|
import com.syncflow.ui.browser.LocalBrowserDialog
|
||||||
import com.syncflow.ui.browser.RemoteBrowserDialog
|
import com.syncflow.ui.browser.RemoteBrowserDialog
|
||||||
import java.time.DayOfWeek
|
import java.time.DayOfWeek
|
||||||
|
|
||||||
@@ -33,17 +34,29 @@ fun AddPairScreen(onDone: () -> Unit, vm: AddPairViewModel = hiltViewModel()) {
|
|||||||
|
|
||||||
val context = LocalContext.current
|
val context = LocalContext.current
|
||||||
var showRemoteBrowser by remember { mutableStateOf(false) }
|
var showRemoteBrowser by remember { mutableStateOf(false) }
|
||||||
|
var showLocalBrowser by remember { mutableStateOf(false) }
|
||||||
|
|
||||||
val dirPicker = rememberLauncherForActivityResult(ActivityResultContracts.OpenDocumentTree()) { uri: Uri? ->
|
val safLauncher = rememberLauncherForActivityResult(ActivityResultContracts.OpenDocumentTree()) { uri ->
|
||||||
uri?.let {
|
if (uri != null) {
|
||||||
context.contentResolver.takePersistableUriPermission(
|
context.contentResolver.takePersistableUriPermission(
|
||||||
it,
|
uri,
|
||||||
Intent.FLAG_GRANT_READ_URI_PERMISSION or Intent.FLAG_GRANT_WRITE_URI_PERMISSION,
|
Intent.FLAG_GRANT_READ_URI_PERMISSION or Intent.FLAG_GRANT_WRITE_URI_PERMISSION,
|
||||||
)
|
)
|
||||||
vm.update { copy(localPath = it.toString()) }
|
vm.update { copy(localPath = uri.toString()) }
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (showLocalBrowser) {
|
||||||
|
LocalBrowserDialog(
|
||||||
|
initialPath = s.localPath.ifBlank { "" },
|
||||||
|
onSelect = { path ->
|
||||||
|
vm.update { copy(localPath = path) }
|
||||||
|
showLocalBrowser = false
|
||||||
|
},
|
||||||
|
onDismiss = { showLocalBrowser = false },
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
if (showRemoteBrowser && s.selectedAccountId != -1L) {
|
if (showRemoteBrowser && s.selectedAccountId != -1L) {
|
||||||
RemoteBrowserDialog(
|
RemoteBrowserDialog(
|
||||||
accountId = s.selectedAccountId,
|
accountId = s.selectedAccountId,
|
||||||
@@ -107,19 +120,27 @@ fun AddPairScreen(onDone: () -> Unit, vm: AddPairViewModel = hiltViewModel()) {
|
|||||||
|
|
||||||
Spacer(Modifier.height(4.dp))
|
Spacer(Modifier.height(4.dp))
|
||||||
|
|
||||||
// Local folder
|
// Local folder — tap opens Android system picker (SAF), same as Autosync
|
||||||
|
Column {
|
||||||
|
Box(modifier = Modifier.fillMaxWidth()) {
|
||||||
OutlinedTextField(
|
OutlinedTextField(
|
||||||
value = uriToDisplay(s.localPath), onValueChange = {},
|
value = uriToDisplay(s.localPath), onValueChange = {},
|
||||||
label = { Text("Local folder") },
|
label = { Text("Local folder") },
|
||||||
leadingIcon = { Icon(Icons.Default.PhoneAndroid, null) },
|
leadingIcon = { Icon(Icons.Default.PhoneAndroid, null) },
|
||||||
trailingIcon = {
|
trailingIcon = { Icon(Icons.Default.FolderOpen, null) },
|
||||||
IconButton(onClick = { dirPicker.launch(null) }) {
|
|
||||||
Icon(Icons.Default.FolderOpen, "Browse")
|
|
||||||
}
|
|
||||||
},
|
|
||||||
readOnly = true, singleLine = true, modifier = Modifier.fillMaxWidth(),
|
readOnly = true, singleLine = true, modifier = Modifier.fillMaxWidth(),
|
||||||
placeholder = { Text("Tap to choose folder…") },
|
placeholder = { Text("Tap to choose folder…") },
|
||||||
)
|
)
|
||||||
|
Box(modifier = Modifier.matchParentSize().clickable { safLauncher.launch(null) })
|
||||||
|
}
|
||||||
|
TextButton(
|
||||||
|
onClick = { showLocalBrowser = true },
|
||||||
|
modifier = Modifier.align(Alignment.End),
|
||||||
|
contentPadding = PaddingValues(horizontal = 8.dp, vertical = 0.dp),
|
||||||
|
) {
|
||||||
|
Text("Browse manually", style = MaterialTheme.typography.labelSmall)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
// Remote folder
|
// Remote folder
|
||||||
OutlinedTextField(
|
OutlinedTextField(
|
||||||
|
|||||||
@@ -5,6 +5,7 @@ import androidx.lifecycle.SavedStateHandle
|
|||||||
import androidx.lifecycle.ViewModel
|
import androidx.lifecycle.ViewModel
|
||||||
import androidx.lifecycle.viewModelScope
|
import androidx.lifecycle.viewModelScope
|
||||||
import com.syncflow.data.db.CloudAccountDao
|
import com.syncflow.data.db.CloudAccountDao
|
||||||
|
import com.syncflow.data.db.SyncFileStateDao
|
||||||
import com.syncflow.data.db.SyncPairDao
|
import com.syncflow.data.db.SyncPairDao
|
||||||
import com.syncflow.data.db.entities.CloudAccountEntity
|
import com.syncflow.data.db.entities.CloudAccountEntity
|
||||||
import com.syncflow.data.db.entities.SyncPairEntity
|
import com.syncflow.data.db.entities.SyncPairEntity
|
||||||
@@ -58,6 +59,7 @@ data class AddPairUiState(
|
|||||||
@HiltViewModel
|
@HiltViewModel
|
||||||
class AddPairViewModel @Inject constructor(
|
class AddPairViewModel @Inject constructor(
|
||||||
private val syncPairDao: SyncPairDao,
|
private val syncPairDao: SyncPairDao,
|
||||||
|
private val fileStateDao: SyncFileStateDao,
|
||||||
private val accountDao: CloudAccountDao,
|
private val accountDao: CloudAccountDao,
|
||||||
@ApplicationContext private val context: Context,
|
@ApplicationContext private val context: Context,
|
||||||
savedState: SavedStateHandle,
|
savedState: SavedStateHandle,
|
||||||
@@ -148,7 +150,20 @@ class AddPairViewModel @Inject constructor(
|
|||||||
notifyOnComplete = s.notifyOnComplete, notifyOnError = s.notifyOnError,
|
notifyOnComplete = s.notifyOnComplete, notifyOnError = s.notifyOnError,
|
||||||
isEnabled = true, lastSyncAt = null, lastSyncResult = SyncStatus.IDLE, pendingConflicts = 0,
|
isEnabled = true, lastSyncAt = null, lastSyncResult = SyncStatus.IDLE, pendingConflicts = 0,
|
||||||
)
|
)
|
||||||
if (editPairId == null) syncPairDao.insert(entity) else syncPairDao.update(entity)
|
if (editPairId == null) {
|
||||||
|
syncPairDao.insert(entity)
|
||||||
|
} else {
|
||||||
|
val existing = syncPairDao.getById(editPairId)
|
||||||
|
syncPairDao.update(entity)
|
||||||
|
// If local or remote folder changed, old file-state records no longer
|
||||||
|
// correspond to any real path — wipe them so the next sync starts fresh
|
||||||
|
// instead of trying to delete/re-upload stale paths.
|
||||||
|
if (existing != null &&
|
||||||
|
(existing.localPath != entity.localPath || existing.remotePath != entity.remotePath)
|
||||||
|
) {
|
||||||
|
fileStateDao.deleteForPair(editPairId)
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
.onSuccess {
|
.onSuccess {
|
||||||
if (s.scheduleType == ScheduleType.ON_CHANGE) FileWatchService.start(context)
|
if (s.scheduleType == ScheduleType.ON_CHANGE) FileWatchService.start(context)
|
||||||
|
|||||||
@@ -0,0 +1,391 @@
|
|||||||
|
package com.syncflow.ui.browser
|
||||||
|
|
||||||
|
import androidx.compose.foundation.background
|
||||||
|
import androidx.compose.foundation.clickable
|
||||||
|
import androidx.compose.foundation.layout.*
|
||||||
|
import androidx.compose.foundation.lazy.LazyColumn
|
||||||
|
import androidx.compose.foundation.lazy.LazyRow
|
||||||
|
import androidx.compose.foundation.lazy.items
|
||||||
|
import androidx.compose.foundation.lazy.itemsIndexed
|
||||||
|
import androidx.compose.foundation.lazy.rememberLazyListState
|
||||||
|
import androidx.compose.foundation.shape.RoundedCornerShape
|
||||||
|
import androidx.compose.foundation.text.KeyboardActions
|
||||||
|
import androidx.compose.foundation.text.KeyboardOptions
|
||||||
|
import androidx.compose.material.icons.Icons
|
||||||
|
import androidx.compose.material.icons.automirrored.filled.ArrowBack
|
||||||
|
import androidx.compose.material.icons.filled.*
|
||||||
|
import androidx.compose.material3.*
|
||||||
|
import androidx.compose.runtime.*
|
||||||
|
import androidx.compose.ui.Alignment
|
||||||
|
import androidx.compose.ui.Modifier
|
||||||
|
import androidx.compose.ui.draw.clip
|
||||||
|
import androidx.compose.ui.focus.FocusRequester
|
||||||
|
import androidx.compose.ui.focus.focusRequester
|
||||||
|
import androidx.compose.ui.graphics.Color
|
||||||
|
import androidx.compose.ui.graphics.vector.ImageVector
|
||||||
|
import androidx.compose.ui.text.font.FontWeight
|
||||||
|
import androidx.compose.ui.text.input.ImeAction
|
||||||
|
import androidx.compose.ui.text.style.TextOverflow
|
||||||
|
import androidx.compose.ui.platform.LocalDensity
|
||||||
|
import androidx.compose.ui.platform.LocalView
|
||||||
|
import androidx.compose.ui.unit.Dp
|
||||||
|
import androidx.compose.ui.unit.dp
|
||||||
|
import androidx.compose.ui.window.Dialog
|
||||||
|
import androidx.compose.ui.window.DialogProperties
|
||||||
|
import androidx.core.view.ViewCompat
|
||||||
|
import androidx.core.view.WindowInsetsCompat
|
||||||
|
import android.content.Intent
|
||||||
|
import android.net.Uri
|
||||||
|
import android.os.Build
|
||||||
|
import android.os.Environment
|
||||||
|
import android.provider.Settings
|
||||||
|
import androidx.compose.ui.platform.LocalContext
|
||||||
|
import kotlinx.coroutines.Dispatchers
|
||||||
|
import kotlinx.coroutines.launch
|
||||||
|
import kotlinx.coroutines.withContext
|
||||||
|
import java.io.File
|
||||||
|
|
||||||
|
private data class LocalEntry(val file: File, val childCount: Int)
|
||||||
|
|
||||||
|
private val STORAGE_ROOT = File("/storage/emulated/0")
|
||||||
|
|
||||||
|
private data class Shortcut(val label: String, val icon: ImageVector, val path: String)
|
||||||
|
private val SHORTCUTS = listOf(
|
||||||
|
Shortcut("Camera", Icons.Default.PhotoCamera, "/storage/emulated/0/DCIM/Camera"),
|
||||||
|
Shortcut("Downloads", Icons.Default.Download, "/storage/emulated/0/Download"),
|
||||||
|
Shortcut("Documents", Icons.Default.Description, "/storage/emulated/0/Documents"),
|
||||||
|
Shortcut("Pictures", Icons.Default.Image, "/storage/emulated/0/Pictures"),
|
||||||
|
Shortcut("Music", Icons.Default.MusicNote, "/storage/emulated/0/Music"),
|
||||||
|
Shortcut("Videos", Icons.Default.Videocam, "/storage/emulated/0/Movies"),
|
||||||
|
)
|
||||||
|
|
||||||
|
@OptIn(ExperimentalMaterial3Api::class)
|
||||||
|
@Composable
|
||||||
|
fun LocalBrowserDialog(
|
||||||
|
initialPath: String = STORAGE_ROOT.absolutePath,
|
||||||
|
onSelect: (path: String) -> Unit,
|
||||||
|
onDismiss: () -> Unit,
|
||||||
|
) {
|
||||||
|
var currentPath by remember { mutableStateOf(File(initialPath.ifBlank { STORAGE_ROOT.absolutePath }).let { if (it.isDirectory) it else STORAGE_ROOT }) }
|
||||||
|
var pathStack by remember { mutableStateOf(listOf(currentPath)) }
|
||||||
|
var entries by remember { mutableStateOf<List<LocalEntry>>(emptyList()) }
|
||||||
|
var isLoading by remember { mutableStateOf(true) }
|
||||||
|
var searchQuery by remember { mutableStateOf("") }
|
||||||
|
var searchActive by remember { mutableStateOf(false) }
|
||||||
|
val breadcrumbState = rememberLazyListState()
|
||||||
|
val scope = rememberCoroutineScope()
|
||||||
|
|
||||||
|
fun loadDir(dir: File) {
|
||||||
|
isLoading = true
|
||||||
|
entries = emptyList()
|
||||||
|
scope.launch {
|
||||||
|
val result = withContext(Dispatchers.IO) {
|
||||||
|
dir.listFiles()
|
||||||
|
?.filter { it.isDirectory && !it.name.startsWith(".") }
|
||||||
|
?.sortedBy { it.name.lowercase() }
|
||||||
|
?.map { f -> LocalEntry(f, f.listFiles()?.count { it.isDirectory } ?: 0) }
|
||||||
|
?: emptyList()
|
||||||
|
}
|
||||||
|
entries = result
|
||||||
|
isLoading = false
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
fun navigateTo(dir: File) {
|
||||||
|
currentPath = dir
|
||||||
|
pathStack = pathStack + dir
|
||||||
|
searchQuery = ""
|
||||||
|
searchActive = false
|
||||||
|
loadDir(dir)
|
||||||
|
}
|
||||||
|
|
||||||
|
fun navigateUp(): Boolean {
|
||||||
|
if (pathStack.size <= 1) return false
|
||||||
|
val newStack = pathStack.dropLast(1)
|
||||||
|
pathStack = newStack
|
||||||
|
currentPath = newStack.last()
|
||||||
|
searchQuery = ""
|
||||||
|
searchActive = false
|
||||||
|
loadDir(currentPath)
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
|
||||||
|
fun navigateToBreadcrumb(dir: File) {
|
||||||
|
val idx = pathStack.indexOfLast { it.absolutePath == dir.absolutePath }
|
||||||
|
pathStack = if (idx >= 0) pathStack.take(idx + 1) else listOf(dir)
|
||||||
|
currentPath = dir
|
||||||
|
searchQuery = ""
|
||||||
|
searchActive = false
|
||||||
|
loadDir(dir)
|
||||||
|
}
|
||||||
|
|
||||||
|
LaunchedEffect(Unit) { loadDir(currentPath) }
|
||||||
|
|
||||||
|
// Build breadcrumb segments relative to storage root
|
||||||
|
val relParts = currentPath.absolutePath
|
||||||
|
.removePrefix(STORAGE_ROOT.absolutePath)
|
||||||
|
.trimStart('/')
|
||||||
|
.split('/')
|
||||||
|
.filter { it.isNotEmpty() }
|
||||||
|
|
||||||
|
// Auto-scroll breadcrumbs to end
|
||||||
|
LaunchedEffect(currentPath) {
|
||||||
|
scope.launch { breadcrumbState.animateScrollToItem(maxOf(0, relParts.size)) }
|
||||||
|
}
|
||||||
|
|
||||||
|
val filtered = if (searchQuery.isBlank()) entries
|
||||||
|
else entries.filter { it.file.name.contains(searchQuery, ignoreCase = true) }
|
||||||
|
|
||||||
|
val currentFolderName = currentPath.name.ifBlank { "Internal Storage" }
|
||||||
|
val context = LocalContext.current
|
||||||
|
val hasAllFilesAccess = remember {
|
||||||
|
Build.VERSION.SDK_INT < Build.VERSION_CODES.R || Environment.isExternalStorageManager()
|
||||||
|
}
|
||||||
|
|
||||||
|
Dialog(
|
||||||
|
onDismissRequest = onDismiss,
|
||||||
|
properties = DialogProperties(usePlatformDefaultWidth = false, decorFitsSystemWindows = false),
|
||||||
|
) {
|
||||||
|
val view = LocalView.current
|
||||||
|
val density = LocalDensity.current
|
||||||
|
var topInset by remember { mutableStateOf(0.dp) }
|
||||||
|
var bottomInset by remember { mutableStateOf(56.dp) }
|
||||||
|
DisposableEffect(view) {
|
||||||
|
ViewCompat.setOnApplyWindowInsetsListener(view) { _, insets ->
|
||||||
|
val bars = insets.getInsets(WindowInsetsCompat.Type.systemBars())
|
||||||
|
with(density) {
|
||||||
|
topInset = bars.top.toDp()
|
||||||
|
bottomInset = maxOf(bars.bottom.toDp(), 56.dp)
|
||||||
|
}
|
||||||
|
insets
|
||||||
|
}
|
||||||
|
ViewCompat.requestApplyInsets(view)
|
||||||
|
onDispose { ViewCompat.setOnApplyWindowInsetsListener(view, null) }
|
||||||
|
}
|
||||||
|
|
||||||
|
Surface(modifier = Modifier.fillMaxSize(), color = MaterialTheme.colorScheme.surface) {
|
||||||
|
Column(modifier = Modifier.fillMaxSize().padding(top = topInset)) {
|
||||||
|
|
||||||
|
// ── Top bar ──────────────────────────────────────────────────
|
||||||
|
TopAppBar(
|
||||||
|
navigationIcon = {
|
||||||
|
IconButton(onClick = { if (!navigateUp()) onDismiss() }) {
|
||||||
|
Icon(Icons.AutoMirrored.Filled.ArrowBack, null)
|
||||||
|
}
|
||||||
|
},
|
||||||
|
title = {
|
||||||
|
if (searchActive) {
|
||||||
|
val focusRequester = remember { FocusRequester() }
|
||||||
|
LaunchedEffect(Unit) { focusRequester.requestFocus() }
|
||||||
|
OutlinedTextField(
|
||||||
|
value = searchQuery,
|
||||||
|
onValueChange = { searchQuery = it },
|
||||||
|
modifier = Modifier.fillMaxWidth().focusRequester(focusRequester),
|
||||||
|
placeholder = { Text("Search folders…") },
|
||||||
|
singleLine = true,
|
||||||
|
keyboardOptions = KeyboardOptions(imeAction = ImeAction.Search),
|
||||||
|
keyboardActions = KeyboardActions(onSearch = {}),
|
||||||
|
colors = OutlinedTextFieldDefaults.colors(
|
||||||
|
focusedBorderColor = Color.Transparent,
|
||||||
|
unfocusedBorderColor = Color.Transparent,
|
||||||
|
),
|
||||||
|
)
|
||||||
|
} else {
|
||||||
|
Text("Choose Local Folder", style = MaterialTheme.typography.titleMedium)
|
||||||
|
}
|
||||||
|
},
|
||||||
|
actions = {
|
||||||
|
IconButton(onClick = { searchActive = !searchActive; if (!searchActive) searchQuery = "" }) {
|
||||||
|
Icon(if (searchActive) Icons.Default.Close else Icons.Default.Search, "Search")
|
||||||
|
}
|
||||||
|
},
|
||||||
|
colors = TopAppBarDefaults.topAppBarColors(containerColor = MaterialTheme.colorScheme.surface),
|
||||||
|
)
|
||||||
|
|
||||||
|
// ── All-files-access banner ──────────────────────────────────
|
||||||
|
if (!hasAllFilesAccess) {
|
||||||
|
Surface(color = MaterialTheme.colorScheme.errorContainer) {
|
||||||
|
Row(
|
||||||
|
modifier = Modifier.fillMaxWidth().padding(horizontal = 16.dp, vertical = 8.dp),
|
||||||
|
verticalAlignment = Alignment.CenterVertically,
|
||||||
|
horizontalArrangement = Arrangement.spacedBy(8.dp),
|
||||||
|
) {
|
||||||
|
Icon(Icons.Default.Warning, null, Modifier.size(18.dp), tint = MaterialTheme.colorScheme.onErrorContainer)
|
||||||
|
Text(
|
||||||
|
"Grant \"All files access\" to browse and sync all folders",
|
||||||
|
style = MaterialTheme.typography.bodySmall,
|
||||||
|
color = MaterialTheme.colorScheme.onErrorContainer,
|
||||||
|
modifier = Modifier.weight(1f),
|
||||||
|
)
|
||||||
|
TextButton(
|
||||||
|
onClick = {
|
||||||
|
val intent = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.R)
|
||||||
|
Intent(Settings.ACTION_MANAGE_APP_ALL_FILES_ACCESS_PERMISSION,
|
||||||
|
Uri.fromParts("package", context.packageName, null))
|
||||||
|
else Intent(Settings.ACTION_MANAGE_ALL_FILES_ACCESS_PERMISSION)
|
||||||
|
context.startActivity(intent)
|
||||||
|
},
|
||||||
|
contentPadding = PaddingValues(horizontal = 8.dp, vertical = 4.dp),
|
||||||
|
) { Text("Grant", style = MaterialTheme.typography.labelSmall) }
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// ── Breadcrumbs ──────────────────────────────────────────────
|
||||||
|
Surface(tonalElevation = 1.dp) {
|
||||||
|
LazyRow(
|
||||||
|
state = breadcrumbState,
|
||||||
|
modifier = Modifier.fillMaxWidth().padding(horizontal = 8.dp, vertical = 6.dp),
|
||||||
|
verticalAlignment = Alignment.CenterVertically,
|
||||||
|
horizontalArrangement = Arrangement.spacedBy(2.dp),
|
||||||
|
) {
|
||||||
|
item {
|
||||||
|
BreadcrumbChip(label = "📱 Storage", isLast = relParts.isEmpty(),
|
||||||
|
onClick = { navigateToBreadcrumb(STORAGE_ROOT) })
|
||||||
|
}
|
||||||
|
itemsIndexed(relParts) { idx, part ->
|
||||||
|
Icon(Icons.Default.ChevronRight, null, Modifier.size(14.dp), tint = MaterialTheme.colorScheme.onSurfaceVariant)
|
||||||
|
val partPath = STORAGE_ROOT.absolutePath + "/" + relParts.take(idx + 1).joinToString("/")
|
||||||
|
BreadcrumbChip(label = part, isLast = idx == relParts.lastIndex,
|
||||||
|
onClick = { navigateToBreadcrumb(File(partPath)) })
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
HorizontalDivider()
|
||||||
|
|
||||||
|
// ── Content ──────────────────────────────────────────────────
|
||||||
|
Box(modifier = Modifier.weight(1f)) {
|
||||||
|
when {
|
||||||
|
isLoading -> Box(Modifier.fillMaxSize(), contentAlignment = Alignment.Center) {
|
||||||
|
CircularProgressIndicator()
|
||||||
|
}
|
||||||
|
else -> LazyColumn(modifier = Modifier.fillMaxSize()) {
|
||||||
|
if (currentPath.absolutePath == STORAGE_ROOT.absolutePath && searchQuery.isBlank()) {
|
||||||
|
item {
|
||||||
|
Text("Quick access", style = MaterialTheme.typography.labelSmall,
|
||||||
|
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
|
modifier = Modifier.padding(start = 20.dp, top = 14.dp, bottom = 6.dp))
|
||||||
|
LazyRow(contentPadding = PaddingValues(horizontal = 16.dp),
|
||||||
|
horizontalArrangement = Arrangement.spacedBy(10.dp),
|
||||||
|
modifier = Modifier.padding(bottom = 8.dp)) {
|
||||||
|
items(SHORTCUTS.filter { File(it.path).isDirectory }) { sc ->
|
||||||
|
ShortcutChip(sc, onClick = { navigateTo(File(sc.path)) })
|
||||||
|
}
|
||||||
|
}
|
||||||
|
HorizontalDivider(modifier = Modifier.padding(vertical = 6.dp))
|
||||||
|
Text("All folders", style = MaterialTheme.typography.labelSmall,
|
||||||
|
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
|
modifier = Modifier.padding(start = 20.dp, top = 4.dp, bottom = 4.dp))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if (filtered.isEmpty()) {
|
||||||
|
item {
|
||||||
|
Box(Modifier.fillParentMaxSize(), contentAlignment = Alignment.Center) {
|
||||||
|
Column(horizontalAlignment = Alignment.CenterHorizontally) {
|
||||||
|
Icon(Icons.Default.FolderOpen, null, Modifier.size(56.dp),
|
||||||
|
tint = MaterialTheme.colorScheme.onSurfaceVariant.copy(alpha = 0.4f))
|
||||||
|
Spacer(Modifier.height(12.dp))
|
||||||
|
Text(if (searchQuery.isBlank()) "No subfolders" else "No results for \"$searchQuery\"",
|
||||||
|
style = MaterialTheme.typography.bodyLarge,
|
||||||
|
color = MaterialTheme.colorScheme.onSurfaceVariant)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
items(filtered, key = { it.file.absolutePath }) { entry ->
|
||||||
|
LocalFolderItem(entry = entry, onClick = { navigateTo(entry.file) })
|
||||||
|
}
|
||||||
|
}
|
||||||
|
item { Spacer(Modifier.height(8.dp)) }
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// ── Select button ────────────────────────────────────────────
|
||||||
|
Surface(tonalElevation = 4.dp, modifier = Modifier.fillMaxWidth()) {
|
||||||
|
Column {
|
||||||
|
Button(
|
||||||
|
onClick = { onSelect(currentPath.absolutePath) },
|
||||||
|
modifier = Modifier
|
||||||
|
.fillMaxWidth()
|
||||||
|
.padding(start = 16.dp, top = 12.dp, end = 16.dp, bottom = 12.dp)
|
||||||
|
.height(52.dp),
|
||||||
|
shape = RoundedCornerShape(14.dp),
|
||||||
|
) {
|
||||||
|
Icon(Icons.Default.CheckCircle, null, Modifier.size(20.dp))
|
||||||
|
Spacer(Modifier.width(8.dp))
|
||||||
|
Text("Select \"$currentFolderName\"",
|
||||||
|
style = MaterialTheme.typography.titleSmall, fontWeight = FontWeight.SemiBold)
|
||||||
|
}
|
||||||
|
Spacer(Modifier.height(bottomInset))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Composable
|
||||||
|
private fun BreadcrumbChip(label: String, isLast: Boolean, onClick: () -> Unit) {
|
||||||
|
if (isLast) {
|
||||||
|
Surface(shape = RoundedCornerShape(8.dp), color = MaterialTheme.colorScheme.primaryContainer) {
|
||||||
|
Text(label, modifier = Modifier.padding(horizontal = 10.dp, vertical = 4.dp),
|
||||||
|
style = MaterialTheme.typography.labelMedium, fontWeight = FontWeight.SemiBold,
|
||||||
|
color = MaterialTheme.colorScheme.onPrimaryContainer, maxLines = 1)
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
TextButton(onClick = onClick, contentPadding = PaddingValues(horizontal = 8.dp, vertical = 2.dp)) {
|
||||||
|
Text(label, style = MaterialTheme.typography.labelMedium, color = MaterialTheme.colorScheme.primary, maxLines = 1)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Composable
|
||||||
|
private fun ShortcutChip(sc: Shortcut, onClick: () -> Unit) {
|
||||||
|
Surface(
|
||||||
|
onClick = onClick,
|
||||||
|
shape = RoundedCornerShape(14.dp),
|
||||||
|
color = MaterialTheme.colorScheme.secondaryContainer,
|
||||||
|
modifier = Modifier.height(72.dp).width(80.dp),
|
||||||
|
) {
|
||||||
|
Column(
|
||||||
|
modifier = Modifier.fillMaxSize().padding(8.dp),
|
||||||
|
horizontalAlignment = Alignment.CenterHorizontally,
|
||||||
|
verticalArrangement = Arrangement.Center,
|
||||||
|
) {
|
||||||
|
Icon(sc.icon, null, Modifier.size(24.dp), tint = MaterialTheme.colorScheme.onSecondaryContainer)
|
||||||
|
Spacer(Modifier.height(4.dp))
|
||||||
|
Text(sc.label, style = MaterialTheme.typography.labelSmall, color = MaterialTheme.colorScheme.onSecondaryContainer, maxLines = 1)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Composable
|
||||||
|
private fun LocalFolderItem(entry: LocalEntry, onClick: () -> Unit) {
|
||||||
|
Row(
|
||||||
|
modifier = Modifier
|
||||||
|
.fillMaxWidth()
|
||||||
|
.clickable(onClick = onClick)
|
||||||
|
.padding(horizontal = 16.dp, vertical = 10.dp),
|
||||||
|
verticalAlignment = Alignment.CenterVertically,
|
||||||
|
horizontalArrangement = Arrangement.spacedBy(14.dp),
|
||||||
|
) {
|
||||||
|
Box(
|
||||||
|
modifier = Modifier
|
||||||
|
.size(46.dp)
|
||||||
|
.clip(RoundedCornerShape(12.dp))
|
||||||
|
.background(Color(0xFF1B5E20).copy(alpha = 0.12f)),
|
||||||
|
contentAlignment = Alignment.Center,
|
||||||
|
) {
|
||||||
|
Icon(Icons.Default.Folder, null, Modifier.size(26.dp), tint = Color(0xFF2E7D32))
|
||||||
|
}
|
||||||
|
Column(modifier = Modifier.weight(1f)) {
|
||||||
|
Text(entry.file.name, style = MaterialTheme.typography.bodyLarge, maxLines = 1, overflow = TextOverflow.Ellipsis, fontWeight = FontWeight.Medium)
|
||||||
|
if (entry.childCount > 0) {
|
||||||
|
Text("${entry.childCount} subfolder${if (entry.childCount == 1) "" else "s"}", style = MaterialTheme.typography.bodySmall, color = MaterialTheme.colorScheme.onSurfaceVariant)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
Icon(Icons.Default.ChevronRight, null, Modifier.size(20.dp), tint = MaterialTheme.colorScheme.onSurfaceVariant.copy(alpha = 0.5f))
|
||||||
|
}
|
||||||
|
HorizontalDivider(modifier = Modifier.padding(start = 76.dp), thickness = 0.5.dp)
|
||||||
|
}
|
||||||
@@ -1,22 +1,40 @@
|
|||||||
package com.syncflow.ui.browser
|
package com.syncflow.ui.browser
|
||||||
|
|
||||||
|
import androidx.compose.foundation.background
|
||||||
import androidx.compose.foundation.clickable
|
import androidx.compose.foundation.clickable
|
||||||
import androidx.compose.foundation.layout.*
|
import androidx.compose.foundation.layout.*
|
||||||
import androidx.compose.foundation.lazy.LazyColumn
|
import androidx.compose.foundation.lazy.LazyColumn
|
||||||
|
import androidx.compose.foundation.lazy.LazyRow
|
||||||
import androidx.compose.foundation.lazy.items
|
import androidx.compose.foundation.lazy.items
|
||||||
|
import androidx.compose.foundation.lazy.itemsIndexed
|
||||||
|
import androidx.compose.foundation.lazy.rememberLazyListState
|
||||||
|
import androidx.compose.foundation.shape.RoundedCornerShape
|
||||||
|
import androidx.compose.foundation.text.KeyboardActions
|
||||||
|
import androidx.compose.foundation.text.KeyboardOptions
|
||||||
import androidx.compose.material.icons.Icons
|
import androidx.compose.material.icons.Icons
|
||||||
|
import androidx.compose.material.icons.automirrored.filled.ArrowBack
|
||||||
import androidx.compose.material.icons.filled.*
|
import androidx.compose.material.icons.filled.*
|
||||||
import androidx.compose.material3.*
|
import androidx.compose.material3.*
|
||||||
import androidx.compose.runtime.*
|
import androidx.compose.runtime.*
|
||||||
import androidx.compose.ui.Alignment
|
import androidx.compose.ui.Alignment
|
||||||
import androidx.compose.ui.Modifier
|
import androidx.compose.ui.Modifier
|
||||||
|
import androidx.compose.ui.draw.clip
|
||||||
|
import androidx.compose.ui.focus.FocusRequester
|
||||||
|
import androidx.compose.ui.focus.focusRequester
|
||||||
|
import androidx.compose.ui.graphics.Color
|
||||||
import androidx.compose.ui.text.font.FontWeight
|
import androidx.compose.ui.text.font.FontWeight
|
||||||
|
import androidx.compose.ui.text.input.ImeAction
|
||||||
import androidx.compose.ui.text.style.TextOverflow
|
import androidx.compose.ui.text.style.TextOverflow
|
||||||
|
import androidx.compose.ui.platform.LocalDensity
|
||||||
|
import androidx.compose.ui.platform.LocalView
|
||||||
import androidx.compose.ui.unit.dp
|
import androidx.compose.ui.unit.dp
|
||||||
import androidx.compose.ui.window.Dialog
|
import androidx.compose.ui.window.Dialog
|
||||||
import androidx.compose.ui.window.DialogProperties
|
import androidx.compose.ui.window.DialogProperties
|
||||||
|
import androidx.core.view.ViewCompat
|
||||||
|
import androidx.core.view.WindowInsetsCompat
|
||||||
import androidx.hilt.navigation.compose.hiltViewModel
|
import androidx.hilt.navigation.compose.hiltViewModel
|
||||||
import com.syncflow.domain.model.RemoteFile
|
import com.syncflow.domain.model.RemoteFile
|
||||||
|
import kotlinx.coroutines.launch
|
||||||
|
|
||||||
@OptIn(ExperimentalMaterial3Api::class)
|
@OptIn(ExperimentalMaterial3Api::class)
|
||||||
@Composable
|
@Composable
|
||||||
@@ -30,79 +48,205 @@ fun RemoteBrowserDialog(
|
|||||||
LaunchedEffect(accountId, initialPath) { vm.init(accountId, initialPath) }
|
LaunchedEffect(accountId, initialPath) { vm.init(accountId, initialPath) }
|
||||||
|
|
||||||
val state by vm.state.collectAsState()
|
val state by vm.state.collectAsState()
|
||||||
|
var searchQuery by remember { mutableStateOf("") }
|
||||||
|
var searchActive by remember { mutableStateOf(false) }
|
||||||
|
var showNewFolderDialog by remember { mutableStateOf(false) }
|
||||||
|
val breadcrumbState = rememberLazyListState()
|
||||||
|
val scope = rememberCoroutineScope()
|
||||||
|
|
||||||
|
// Auto-scroll breadcrumbs to end when path changes
|
||||||
|
val segments = state.currentPath.trimEnd('/').split('/').filter { it.isNotEmpty() }
|
||||||
|
LaunchedEffect(state.currentPath) {
|
||||||
|
scope.launch { breadcrumbState.animateScrollToItem(maxOf(0, segments.size)) }
|
||||||
|
searchQuery = ""
|
||||||
|
searchActive = false
|
||||||
|
}
|
||||||
|
|
||||||
|
val filtered = if (searchQuery.isBlank()) state.entries
|
||||||
|
else state.entries.filter { it.name.contains(searchQuery, ignoreCase = true) }
|
||||||
|
|
||||||
|
val currentFolderName = state.currentPath.trimEnd('/').substringAfterLast('/').ifBlank { "Root" }
|
||||||
|
|
||||||
|
if (showNewFolderDialog) {
|
||||||
|
NewFolderDialog(
|
||||||
|
onConfirm = { name ->
|
||||||
|
vm.createFolder(name)
|
||||||
|
showNewFolderDialog = false
|
||||||
|
},
|
||||||
|
onDismiss = { showNewFolderDialog = false },
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
Dialog(
|
Dialog(
|
||||||
onDismissRequest = onDismiss,
|
onDismissRequest = onDismiss,
|
||||||
properties = DialogProperties(usePlatformDefaultWidth = false),
|
properties = DialogProperties(usePlatformDefaultWidth = false, decorFitsSystemWindows = false),
|
||||||
) {
|
) {
|
||||||
Surface(
|
val view = LocalView.current
|
||||||
modifier = Modifier
|
val density = LocalDensity.current
|
||||||
.fillMaxWidth(0.95f)
|
var topInset by remember { mutableStateOf(0.dp) }
|
||||||
.fillMaxHeight(0.85f),
|
var bottomInset by remember { mutableStateOf(56.dp) }
|
||||||
shape = MaterialTheme.shapes.extraLarge,
|
DisposableEffect(view) {
|
||||||
tonalElevation = 6.dp,
|
ViewCompat.setOnApplyWindowInsetsListener(view) { _, insets ->
|
||||||
) {
|
val bars = insets.getInsets(WindowInsetsCompat.Type.systemBars())
|
||||||
Column {
|
with(density) {
|
||||||
// Title bar
|
topInset = bars.top.toDp()
|
||||||
TopAppBar(
|
bottomInset = maxOf(bars.bottom.toDp(), 56.dp)
|
||||||
title = {
|
|
||||||
Column {
|
|
||||||
Text("Choose remote folder", style = MaterialTheme.typography.titleMedium)
|
|
||||||
Text(
|
|
||||||
state.currentPath,
|
|
||||||
style = MaterialTheme.typography.labelSmall,
|
|
||||||
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
|
||||||
maxLines = 1,
|
|
||||||
overflow = TextOverflow.Ellipsis,
|
|
||||||
)
|
|
||||||
}
|
}
|
||||||
},
|
insets
|
||||||
|
}
|
||||||
|
ViewCompat.requestApplyInsets(view)
|
||||||
|
onDispose { ViewCompat.setOnApplyWindowInsetsListener(view, null) }
|
||||||
|
}
|
||||||
|
|
||||||
|
Surface(modifier = Modifier.fillMaxSize(), color = MaterialTheme.colorScheme.surface) {
|
||||||
|
Column(modifier = Modifier.fillMaxSize().padding(top = topInset)) {
|
||||||
|
|
||||||
|
// ── Top bar ──────────────────────────────────────────────────
|
||||||
|
TopAppBar(
|
||||||
navigationIcon = {
|
navigationIcon = {
|
||||||
IconButton(onClick = { if (!vm.navigateUp()) onDismiss() }) {
|
IconButton(onClick = { if (!vm.navigateUp()) onDismiss() }) {
|
||||||
Icon(Icons.Default.ArrowBack, null)
|
Icon(Icons.AutoMirrored.Filled.ArrowBack, null)
|
||||||
|
}
|
||||||
|
},
|
||||||
|
title = {
|
||||||
|
if (searchActive) {
|
||||||
|
val focusRequester = remember { FocusRequester() }
|
||||||
|
LaunchedEffect(Unit) { focusRequester.requestFocus() }
|
||||||
|
OutlinedTextField(
|
||||||
|
value = searchQuery,
|
||||||
|
onValueChange = { searchQuery = it },
|
||||||
|
modifier = Modifier.fillMaxWidth().focusRequester(focusRequester),
|
||||||
|
placeholder = { Text("Search in folder…") },
|
||||||
|
singleLine = true,
|
||||||
|
keyboardOptions = KeyboardOptions(imeAction = ImeAction.Search),
|
||||||
|
keyboardActions = KeyboardActions(onSearch = {}),
|
||||||
|
colors = OutlinedTextFieldDefaults.colors(
|
||||||
|
focusedBorderColor = Color.Transparent,
|
||||||
|
unfocusedBorderColor = Color.Transparent,
|
||||||
|
),
|
||||||
|
)
|
||||||
|
} else {
|
||||||
|
Text("Choose Folder", style = MaterialTheme.typography.titleMedium)
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
actions = {
|
actions = {
|
||||||
// Select current folder
|
IconButton(onClick = { searchActive = !searchActive; if (!searchActive) searchQuery = "" }) {
|
||||||
TextButton(onClick = { onSelect(state.currentPath) }) {
|
Icon(if (searchActive) Icons.Default.Close else Icons.Default.Search, "Search")
|
||||||
Text("Select here")
|
}
|
||||||
|
IconButton(onClick = { showNewFolderDialog = true }) {
|
||||||
|
Icon(Icons.Default.CreateNewFolder, "New Folder")
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
colors = TopAppBarDefaults.topAppBarColors(containerColor = MaterialTheme.colorScheme.surfaceContainerHigh),
|
colors = TopAppBarDefaults.topAppBarColors(containerColor = MaterialTheme.colorScheme.surface),
|
||||||
)
|
)
|
||||||
|
|
||||||
|
// ── Breadcrumbs ──────────────────────────────────────────────
|
||||||
|
Surface(tonalElevation = 1.dp) {
|
||||||
|
LazyRow(
|
||||||
|
state = breadcrumbState,
|
||||||
|
modifier = Modifier.fillMaxWidth().padding(horizontal = 8.dp, vertical = 6.dp),
|
||||||
|
verticalAlignment = Alignment.CenterVertically,
|
||||||
|
horizontalArrangement = Arrangement.spacedBy(2.dp),
|
||||||
|
) {
|
||||||
|
item {
|
||||||
|
BreadcrumbChip(
|
||||||
|
label = "⌂",
|
||||||
|
isLast = segments.isEmpty(),
|
||||||
|
onClick = { vm.navigateToBreadcrumb("/") },
|
||||||
|
)
|
||||||
|
}
|
||||||
|
itemsIndexed(segments) { idx, seg ->
|
||||||
|
Icon(Icons.Default.ChevronRight, null, Modifier.size(14.dp), tint = MaterialTheme.colorScheme.onSurfaceVariant)
|
||||||
|
val segPath = "/" + segments.take(idx + 1).joinToString("/")
|
||||||
|
BreadcrumbChip(
|
||||||
|
label = seg,
|
||||||
|
isLast = idx == segments.lastIndex,
|
||||||
|
onClick = { vm.navigateToBreadcrumb(segPath) },
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
HorizontalDivider()
|
HorizontalDivider()
|
||||||
|
|
||||||
|
// ── Content ──────────────────────────────────────────────────
|
||||||
|
Box(modifier = Modifier.weight(1f)) {
|
||||||
when {
|
when {
|
||||||
state.isLoading -> Box(Modifier.fillMaxSize(), contentAlignment = Alignment.Center) {
|
state.isLoading -> Box(Modifier.fillMaxSize(), contentAlignment = Alignment.Center) {
|
||||||
CircularProgressIndicator()
|
CircularProgressIndicator()
|
||||||
}
|
}
|
||||||
state.error != null -> Box(Modifier.fillMaxSize().padding(24.dp), contentAlignment = Alignment.Center) {
|
|
||||||
Column(horizontalAlignment = Alignment.CenterHorizontally, verticalArrangement = Arrangement.spacedBy(8.dp)) {
|
state.error != null -> Column(
|
||||||
Icon(Icons.Default.ErrorOutline, null, Modifier.size(48.dp), tint = MaterialTheme.colorScheme.error)
|
modifier = Modifier.fillMaxSize().padding(32.dp),
|
||||||
Text(state.error!!, color = MaterialTheme.colorScheme.error)
|
horizontalAlignment = Alignment.CenterHorizontally,
|
||||||
FilledTonalButton(onClick = { vm.retry() }) { Text("Retry") }
|
verticalArrangement = Arrangement.Center,
|
||||||
|
) {
|
||||||
|
Icon(Icons.Default.CloudOff, null, Modifier.size(56.dp), tint = MaterialTheme.colorScheme.error)
|
||||||
|
Spacer(Modifier.height(12.dp))
|
||||||
|
Text(state.error!!, color = MaterialTheme.colorScheme.error, style = MaterialTheme.typography.bodyMedium)
|
||||||
|
Spacer(Modifier.height(16.dp))
|
||||||
|
FilledTonalButton(onClick = vm::retry) {
|
||||||
|
Icon(Icons.Default.Refresh, null, Modifier.size(18.dp))
|
||||||
|
Spacer(Modifier.width(6.dp))
|
||||||
|
Text("Retry")
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
state.entries.isEmpty() -> Box(Modifier.fillMaxSize(), contentAlignment = Alignment.Center) {
|
|
||||||
Column(horizontalAlignment = Alignment.CenterHorizontally, verticalArrangement = Arrangement.spacedBy(8.dp)) {
|
filtered.isEmpty() && searchQuery.isBlank() -> Column(
|
||||||
Icon(Icons.Default.FolderOff, null, Modifier.size(48.dp), tint = MaterialTheme.colorScheme.onSurfaceVariant)
|
modifier = Modifier.fillMaxSize().padding(32.dp),
|
||||||
Text("Empty folder", color = MaterialTheme.colorScheme.onSurfaceVariant)
|
horizontalAlignment = Alignment.CenterHorizontally,
|
||||||
TextButton(onClick = { onSelect(state.currentPath) }) { Text("Select this folder anyway") }
|
verticalArrangement = Arrangement.Center,
|
||||||
|
) {
|
||||||
|
Icon(Icons.Default.FolderOpen, null, Modifier.size(56.dp), tint = MaterialTheme.colorScheme.onSurfaceVariant.copy(alpha = 0.4f))
|
||||||
|
Spacer(Modifier.height(12.dp))
|
||||||
|
Text("This folder is empty", style = MaterialTheme.typography.bodyLarge, color = MaterialTheme.colorScheme.onSurfaceVariant)
|
||||||
|
Text("You can still select it", style = MaterialTheme.typography.bodySmall, color = MaterialTheme.colorScheme.onSurfaceVariant.copy(alpha = 0.6f))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
filtered.isEmpty() -> Column(
|
||||||
|
modifier = Modifier.fillMaxSize().padding(32.dp),
|
||||||
|
horizontalAlignment = Alignment.CenterHorizontally,
|
||||||
|
verticalArrangement = Arrangement.Center,
|
||||||
|
) {
|
||||||
|
Icon(Icons.Default.SearchOff, null, Modifier.size(56.dp), tint = MaterialTheme.colorScheme.onSurfaceVariant.copy(alpha = 0.4f))
|
||||||
|
Spacer(Modifier.height(12.dp))
|
||||||
|
Text("No results for \"$searchQuery\"", style = MaterialTheme.typography.bodyLarge, color = MaterialTheme.colorScheme.onSurfaceVariant)
|
||||||
}
|
}
|
||||||
|
|
||||||
else -> LazyColumn(modifier = Modifier.fillMaxSize()) {
|
else -> LazyColumn(modifier = Modifier.fillMaxSize()) {
|
||||||
items(state.entries, key = { it.path }) { entry ->
|
items(filtered, key = { it.path }) { entry ->
|
||||||
BrowserEntry(
|
FolderItem(
|
||||||
file = entry,
|
file = entry,
|
||||||
onClick = {
|
onClick = {
|
||||||
if (entry.isDirectory) vm.navigateTo(entry.path)
|
if (entry.isDirectory) vm.navigateTo(entry.path)
|
||||||
else onSelect(entry.path.substringBeforeLast('/').ifBlank { "/" })
|
else onSelect(entry.path.substringBeforeLast('/').ifBlank { "/" })
|
||||||
},
|
},
|
||||||
onSelectFolder = if (entry.isDirectory) ({ onSelect(entry.path) }) else null,
|
|
||||||
)
|
)
|
||||||
HorizontalDivider(modifier = Modifier.padding(start = 56.dp))
|
|
||||||
}
|
}
|
||||||
|
item { Spacer(Modifier.height(8.dp)) }
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// ── Select button ────────────────────────────────────────────
|
||||||
|
Surface(tonalElevation = 4.dp, modifier = Modifier.fillMaxWidth()) {
|
||||||
|
Column {
|
||||||
|
Button(
|
||||||
|
onClick = { onSelect(state.currentPath) },
|
||||||
|
modifier = Modifier
|
||||||
|
.fillMaxWidth()
|
||||||
|
.padding(start = 16.dp, top = 12.dp, end = 16.dp, bottom = 12.dp)
|
||||||
|
.height(52.dp),
|
||||||
|
shape = RoundedCornerShape(14.dp),
|
||||||
|
) {
|
||||||
|
Icon(Icons.Default.CheckCircle, null, Modifier.size(20.dp))
|
||||||
|
Spacer(Modifier.width(8.dp))
|
||||||
|
Text(
|
||||||
|
"Select \"$currentFolderName\"",
|
||||||
|
style = MaterialTheme.typography.titleSmall,
|
||||||
|
fontWeight = FontWeight.SemiBold,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
Spacer(Modifier.height(bottomInset))
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -111,39 +255,114 @@ fun RemoteBrowserDialog(
|
|||||||
}
|
}
|
||||||
|
|
||||||
@Composable
|
@Composable
|
||||||
private fun BrowserEntry(
|
private fun BreadcrumbChip(label: String, isLast: Boolean, onClick: () -> Unit) {
|
||||||
file: RemoteFile,
|
if (isLast) {
|
||||||
onClick: () -> Unit,
|
Surface(
|
||||||
onSelectFolder: (() -> Unit)?,
|
shape = RoundedCornerShape(8.dp),
|
||||||
|
color = MaterialTheme.colorScheme.primaryContainer,
|
||||||
) {
|
) {
|
||||||
|
Text(
|
||||||
|
label,
|
||||||
|
modifier = Modifier.padding(horizontal = 10.dp, vertical = 4.dp),
|
||||||
|
style = MaterialTheme.typography.labelMedium,
|
||||||
|
fontWeight = FontWeight.SemiBold,
|
||||||
|
color = MaterialTheme.colorScheme.onPrimaryContainer,
|
||||||
|
maxLines = 1,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
TextButton(
|
||||||
|
onClick = onClick,
|
||||||
|
contentPadding = PaddingValues(horizontal = 8.dp, vertical = 2.dp),
|
||||||
|
) {
|
||||||
|
Text(
|
||||||
|
label,
|
||||||
|
style = MaterialTheme.typography.labelMedium,
|
||||||
|
color = MaterialTheme.colorScheme.primary,
|
||||||
|
maxLines = 1,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Composable
|
||||||
|
private fun FolderItem(file: RemoteFile, onClick: () -> Unit) {
|
||||||
Row(
|
Row(
|
||||||
modifier = Modifier
|
modifier = Modifier
|
||||||
.fillMaxWidth()
|
.fillMaxWidth()
|
||||||
.clickable(onClick = onClick)
|
.clickable(onClick = onClick)
|
||||||
.padding(horizontal = 16.dp, vertical = 10.dp),
|
.padding(horizontal = 16.dp, vertical = 10.dp),
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
verticalAlignment = Alignment.CenterVertically,
|
||||||
|
horizontalArrangement = Arrangement.spacedBy(14.dp),
|
||||||
|
) {
|
||||||
|
// Colored icon badge
|
||||||
|
Box(
|
||||||
|
modifier = Modifier
|
||||||
|
.size(46.dp)
|
||||||
|
.clip(RoundedCornerShape(12.dp))
|
||||||
|
.background(
|
||||||
|
if (file.isDirectory) Color(0xFF1B5E20).copy(alpha = 0.12f)
|
||||||
|
else Color(0xFF0D47A1).copy(alpha = 0.10f)
|
||||||
|
),
|
||||||
|
contentAlignment = Alignment.Center,
|
||||||
) {
|
) {
|
||||||
Icon(
|
Icon(
|
||||||
imageVector = if (file.isDirectory) Icons.Default.Folder else Icons.Default.InsertDriveFile,
|
imageVector = if (file.isDirectory) Icons.Default.Folder else Icons.Default.InsertDriveFile,
|
||||||
contentDescription = null,
|
contentDescription = null,
|
||||||
modifier = Modifier.size(24.dp),
|
modifier = Modifier.size(26.dp),
|
||||||
tint = if (file.isDirectory) MaterialTheme.colorScheme.primary else MaterialTheme.colorScheme.onSurfaceVariant,
|
tint = if (file.isDirectory) Color(0xFF2E7D32) else Color(0xFF1565C0),
|
||||||
)
|
)
|
||||||
Spacer(Modifier.width(14.dp))
|
}
|
||||||
|
|
||||||
Column(modifier = Modifier.weight(1f)) {
|
Column(modifier = Modifier.weight(1f)) {
|
||||||
Text(file.name, style = MaterialTheme.typography.bodyMedium, maxLines = 1, overflow = TextOverflow.Ellipsis)
|
Text(
|
||||||
|
file.name,
|
||||||
|
style = MaterialTheme.typography.bodyLarge,
|
||||||
|
maxLines = 1,
|
||||||
|
overflow = TextOverflow.Ellipsis,
|
||||||
|
fontWeight = if (file.isDirectory) FontWeight.Medium else FontWeight.Normal,
|
||||||
|
)
|
||||||
if (!file.isDirectory) {
|
if (!file.isDirectory) {
|
||||||
Text(file.sizeBytes.formatBytes(), style = MaterialTheme.typography.labelSmall, color = MaterialTheme.colorScheme.onSurfaceVariant)
|
Text(
|
||||||
|
file.sizeBytes.formatBytes(),
|
||||||
|
style = MaterialTheme.typography.bodySmall,
|
||||||
|
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
|
)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if (onSelectFolder != null) {
|
|
||||||
IconButton(onClick = onSelectFolder, modifier = Modifier.size(36.dp)) {
|
if (file.isDirectory) {
|
||||||
Icon(Icons.Default.CheckCircleOutline, "Select this folder", Modifier.size(20.dp), tint = MaterialTheme.colorScheme.primary)
|
Icon(Icons.Default.ChevronRight, null, Modifier.size(20.dp), tint = MaterialTheme.colorScheme.onSurfaceVariant.copy(alpha = 0.5f))
|
||||||
}
|
|
||||||
} else {
|
|
||||||
Icon(Icons.Default.ChevronRight, null, Modifier.size(20.dp), tint = MaterialTheme.colorScheme.onSurfaceVariant)
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
HorizontalDivider(modifier = Modifier.padding(start = 76.dp), thickness = 0.5.dp)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Composable
|
||||||
|
private fun NewFolderDialog(onConfirm: (String) -> Unit, onDismiss: () -> Unit) {
|
||||||
|
var name by remember { mutableStateOf("") }
|
||||||
|
AlertDialog(
|
||||||
|
onDismissRequest = onDismiss,
|
||||||
|
icon = { Icon(Icons.Default.CreateNewFolder, null) },
|
||||||
|
title = { Text("New Folder") },
|
||||||
|
text = {
|
||||||
|
OutlinedTextField(
|
||||||
|
value = name,
|
||||||
|
onValueChange = { name = it },
|
||||||
|
label = { Text("Folder name") },
|
||||||
|
singleLine = true,
|
||||||
|
modifier = Modifier.fillMaxWidth(),
|
||||||
|
keyboardOptions = KeyboardOptions(imeAction = ImeAction.Done),
|
||||||
|
keyboardActions = KeyboardActions(onDone = { if (name.isNotBlank()) onConfirm(name.trim()) }),
|
||||||
|
)
|
||||||
|
},
|
||||||
|
confirmButton = {
|
||||||
|
TextButton(onClick = { if (name.isNotBlank()) onConfirm(name.trim()) }, enabled = name.isNotBlank()) {
|
||||||
|
Text("Create")
|
||||||
|
}
|
||||||
|
},
|
||||||
|
dismissButton = { TextButton(onClick = onDismiss) { Text("Cancel") } },
|
||||||
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
private fun Long.formatBytes(): String = when {
|
private fun Long.formatBytes(): String = when {
|
||||||
|
|||||||
@@ -57,6 +57,27 @@ class RemoteBrowserViewModel @Inject constructor(
|
|||||||
return true
|
return true
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fun navigateToBreadcrumb(path: String) {
|
||||||
|
val stack = _state.value.pathStack
|
||||||
|
val idx = stack.lastIndexOf(path)
|
||||||
|
val newStack = if (idx >= 0) stack.take(idx + 1) else listOf(path)
|
||||||
|
loadJob?.cancel()
|
||||||
|
_state.update { it.copy(currentPath = path, pathStack = newStack, isLoading = true, entries = emptyList(), error = null) }
|
||||||
|
loadJob = loadPath(_state.value.accountId, path)
|
||||||
|
}
|
||||||
|
|
||||||
|
fun createFolder(name: String) {
|
||||||
|
val s = _state.value
|
||||||
|
val newPath = if (s.currentPath.trimEnd('/') == "") "/$name" else "${s.currentPath.trimEnd('/')}/$name"
|
||||||
|
viewModelScope.launch {
|
||||||
|
val account = accountRepository.getAccount(s.accountId) ?: return@launch
|
||||||
|
val provider = runCatching { providerFactory.create(account) }.getOrElse { return@launch }
|
||||||
|
provider.createDirectory(newPath)
|
||||||
|
.onSuccess { retry() }
|
||||||
|
.onFailure { e -> _state.update { it.copy(error = "Could not create folder: ${e.message}") } }
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
fun retry() {
|
fun retry() {
|
||||||
val s = _state.value
|
val s = _state.value
|
||||||
if (s.accountId == -1L) return
|
if (s.accountId == -1L) return
|
||||||
|
|||||||
@@ -1,5 +1,6 @@
|
|||||||
package com.syncflow.ui.files
|
package com.syncflow.ui.files
|
||||||
|
|
||||||
|
import android.content.ClipData
|
||||||
import android.content.Intent
|
import android.content.Intent
|
||||||
import android.net.Uri
|
import android.net.Uri
|
||||||
import android.webkit.MimeTypeMap
|
import android.webkit.MimeTypeMap
|
||||||
@@ -57,15 +58,18 @@ fun FilesScreen(
|
|||||||
val uri = FileProvider.getUriForFile(
|
val uri = FileProvider.getUriForFile(
|
||||||
context, "${context.packageName}.fileprovider", action.file
|
context, "${context.packageName}.fileprovider", action.file
|
||||||
)
|
)
|
||||||
|
val mimeType = action.file.name.mimeType()
|
||||||
val intent = Intent(Intent.ACTION_VIEW).apply {
|
val intent = Intent(Intent.ACTION_VIEW).apply {
|
||||||
setDataAndType(uri, action.file.name.mimeType())
|
setDataAndType(uri, mimeType)
|
||||||
addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION or Intent.FLAG_ACTIVITY_NEW_TASK)
|
// ClipData is required so FLAG_GRANT_READ_URI_PERMISSION
|
||||||
}
|
// propagates to whichever app the system chooser picks.
|
||||||
context.startActivity(
|
clipData = ClipData.newRawUri("", uri)
|
||||||
Intent.createChooser(intent, "Open with").apply {
|
addFlags(
|
||||||
addFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
|
Intent.FLAG_GRANT_READ_URI_PERMISSION or
|
||||||
}
|
Intent.FLAG_ACTIVITY_NEW_TASK
|
||||||
)
|
)
|
||||||
|
}
|
||||||
|
context.startActivity(intent)
|
||||||
} catch (e: Exception) {
|
} catch (e: Exception) {
|
||||||
snackbarHostState.showSnackbar("Cannot open file: ${e.message}")
|
snackbarHostState.showSnackbar("Cannot open file: ${e.message}")
|
||||||
}
|
}
|
||||||
@@ -78,6 +82,7 @@ fun FilesScreen(
|
|||||||
val intent = Intent(Intent.ACTION_SEND).apply {
|
val intent = Intent(Intent.ACTION_SEND).apply {
|
||||||
type = action.file.name.mimeType()
|
type = action.file.name.mimeType()
|
||||||
putExtra(Intent.EXTRA_STREAM, uri)
|
putExtra(Intent.EXTRA_STREAM, uri)
|
||||||
|
clipData = ClipData.newRawUri("", uri)
|
||||||
addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION or Intent.FLAG_ACTIVITY_NEW_TASK)
|
addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION or Intent.FLAG_ACTIVITY_NEW_TASK)
|
||||||
}
|
}
|
||||||
context.startActivity(
|
context.startActivity(
|
||||||
|
|||||||
@@ -1,6 +1,7 @@
|
|||||||
package com.syncflow.ui.files
|
package com.syncflow.ui.files
|
||||||
|
|
||||||
import android.content.Context
|
import android.content.Context
|
||||||
|
import android.media.MediaScannerConnection
|
||||||
import androidx.lifecycle.ViewModel
|
import androidx.lifecycle.ViewModel
|
||||||
import androidx.lifecycle.viewModelScope
|
import androidx.lifecycle.viewModelScope
|
||||||
import com.syncflow.data.db.SyncFileStateDao
|
import com.syncflow.data.db.SyncFileStateDao
|
||||||
@@ -71,6 +72,8 @@ class FilesViewModel @Inject constructor(
|
|||||||
fun openFile(file: SyncFileStateEntity) {
|
fun openFile(file: SyncFileStateEntity) {
|
||||||
val resolved = resolveFile(file, emitErrorIfMissing = false)
|
val resolved = resolveFile(file, emitErrorIfMissing = false)
|
||||||
if (resolved != null) {
|
if (resolved != null) {
|
||||||
|
// Ensure MediaStore knows about this file so gallery apps can open it
|
||||||
|
MediaScannerConnection.scanFile(context, arrayOf(resolved.absolutePath), null, null)
|
||||||
viewModelScope.launch { _fileAction.emit(FileAction.Open(resolved)) }
|
viewModelScope.launch { _fileAction.emit(FileAction.Open(resolved)) }
|
||||||
} else {
|
} else {
|
||||||
downloadAndOpen(file)
|
downloadAndOpen(file)
|
||||||
@@ -192,6 +195,7 @@ class FilesViewModel @Inject constructor(
|
|||||||
cacheFile.outputStream().use { out ->
|
cacheFile.outputStream().use { out ->
|
||||||
provider.downloadFile("${pair.remotePath}/${file.relativePath}", out) { }.getOrThrow()
|
provider.downloadFile("${pair.remotePath}/${file.relativePath}", out) { }.getOrThrow()
|
||||||
}
|
}
|
||||||
|
MediaScannerConnection.scanFile(context, arrayOf(cacheFile.absolutePath), null, null)
|
||||||
cacheFile
|
cacheFile
|
||||||
} catch (e: Exception) {
|
} catch (e: Exception) {
|
||||||
Timber.e(e, "Download for preview failed: ${file.relativePath}")
|
Timber.e(e, "Download for preview failed: ${file.relativePath}")
|
||||||
|
|||||||
@@ -247,11 +247,11 @@ private fun EmptyState(modifier: Modifier = Modifier, onAdd: () -> Unit) {
|
|||||||
|
|
||||||
private val SyncStatus.accentColor: Color
|
private val SyncStatus.accentColor: Color
|
||||||
@Composable get() = when (this) {
|
@Composable get() = when (this) {
|
||||||
SyncStatus.SUCCESS -> MaterialTheme.colorScheme.primary
|
SyncStatus.SUCCESS -> Color(0xFF2E7D32) // green — done, healthy
|
||||||
SyncStatus.SYNCING -> MaterialTheme.colorScheme.secondary
|
SyncStatus.SYNCING -> Color(0xFF1565C0) // blue — in progress
|
||||||
SyncStatus.FAILED -> MaterialTheme.colorScheme.error
|
SyncStatus.FAILED -> Color(0xFFC62828) // red — error
|
||||||
SyncStatus.CONFLICT,
|
SyncStatus.PARTIAL -> Color(0xFFE65100) // orange — some files failed
|
||||||
SyncStatus.PARTIAL -> MaterialTheme.colorScheme.tertiary
|
SyncStatus.CONFLICT -> Color(0xFFF9A825) // amber — needs resolution
|
||||||
SyncStatus.IDLE -> MaterialTheme.colorScheme.outline
|
SyncStatus.IDLE -> MaterialTheme.colorScheme.outline
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -20,9 +20,13 @@ import com.syncflow.MainActivity
|
|||||||
import com.syncflow.R
|
import com.syncflow.R
|
||||||
import com.syncflow.data.db.SyncFileStateDao
|
import com.syncflow.data.db.SyncFileStateDao
|
||||||
import com.syncflow.data.db.SyncPairDao
|
import com.syncflow.data.db.SyncPairDao
|
||||||
|
import com.syncflow.data.db.entities.toDomain
|
||||||
import com.syncflow.domain.model.ScheduleType
|
import com.syncflow.domain.model.ScheduleType
|
||||||
|
import com.syncflow.domain.sync.LocalAccessor
|
||||||
import dagger.hilt.android.AndroidEntryPoint
|
import dagger.hilt.android.AndroidEntryPoint
|
||||||
import kotlinx.coroutines.*
|
import kotlinx.coroutines.*
|
||||||
|
import kotlinx.coroutines.sync.Mutex
|
||||||
|
import kotlinx.coroutines.sync.withLock
|
||||||
import timber.log.Timber
|
import timber.log.Timber
|
||||||
import java.io.File
|
import java.io.File
|
||||||
import javax.inject.Inject
|
import javax.inject.Inject
|
||||||
@@ -35,11 +39,18 @@ class FileWatchService : Service() {
|
|||||||
|
|
||||||
private val scope = CoroutineScope(Dispatchers.IO + SupervisorJob())
|
private val scope = CoroutineScope(Dispatchers.IO + SupervisorJob())
|
||||||
private val mainHandler = Handler(Looper.getMainLooper())
|
private val mainHandler = Handler(Looper.getMainLooper())
|
||||||
|
// Prevents concurrent refresh() calls from doubling watchers + catchup scans
|
||||||
|
private val refreshMutex = Mutex()
|
||||||
|
|
||||||
// Multiple FileObserver instances per pair: one per directory (recursive)
|
// Multiple FileObserver instances per pair: one per directory (recursive)
|
||||||
private val fileObservers = mutableMapOf<Long, MutableList<FileObserver>>()
|
private val fileObservers = mutableMapOf<Long, MutableList<FileObserver>>()
|
||||||
private val contentObservers = mutableMapOf<Long, ContentObserver>()
|
private val contentObservers = mutableMapOf<Long, ContentObserver>()
|
||||||
private val debounceJobs = mutableMapOf<Long, Job>()
|
private val debounceJobs = mutableMapOf<Long, Job>()
|
||||||
|
// Persistent monitors that watch WorkManager for ANY sync (manual, catchup, onchange)
|
||||||
|
// so the cooldown is set regardless of who triggered the sync.
|
||||||
|
private val syncMonitorJobs = mutableMapOf<Long, Job>()
|
||||||
|
// After a sync completes, suppress FileObserver events for this long.
|
||||||
|
private val syncCooldownUntil = mutableMapOf<Long, Long>()
|
||||||
|
|
||||||
companion object {
|
companion object {
|
||||||
const val CHANNEL_WATCH = "sync_watching"
|
const val CHANNEL_WATCH = "sync_watching"
|
||||||
@@ -78,7 +89,7 @@ class FileWatchService : Service() {
|
|||||||
|
|
||||||
override fun onBind(intent: Intent?): IBinder? = null
|
override fun onBind(intent: Intent?): IBinder? = null
|
||||||
|
|
||||||
private suspend fun refresh() {
|
private suspend fun refresh() = refreshMutex.withLock {
|
||||||
clearWatchers()
|
clearWatchers()
|
||||||
val pairs = syncPairDao.getEnabled().filter { it.scheduleType == ScheduleType.ON_CHANGE }
|
val pairs = syncPairDao.getEnabled().filter { it.scheduleType == ScheduleType.ON_CHANGE }
|
||||||
|
|
||||||
@@ -142,11 +153,41 @@ class FileWatchService : Service() {
|
|||||||
return
|
return
|
||||||
}
|
}
|
||||||
fileObservers[pairId] = mutableListOf()
|
fileObservers[pairId] = mutableListOf()
|
||||||
|
// Set startup cooldown BEFORE registering watchers so inotify events that fire
|
||||||
|
// immediately on registration don't trigger the debounce before catchupScan runs.
|
||||||
|
syncCooldownUntil[pairId] = System.currentTimeMillis() + 15_000
|
||||||
watchDirRecursive(dir, pairId, wifiOnly, chargingOnly)
|
watchDirRecursive(dir, pairId, wifiOnly, chargingOnly)
|
||||||
Timber.d("FileWatchService: watching pair $pairId at $path (${fileObservers[pairId]?.size} dirs)")
|
Timber.d("FileWatchService: watching pair $pairId at $path (${fileObservers[pairId]?.size} dirs)")
|
||||||
|
startSyncMonitor(pairId)
|
||||||
scope.launch { catchupScan(pairId, dir, wifiOnly, chargingOnly) }
|
scope.launch { catchupScan(pairId, dir, wifiOnly, chargingOnly) }
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Watches WorkManager for ANY sync tagged sync_$pairId (manual, catchup, onchange).
|
||||||
|
// Sets cooldown while running and for 60s after, so FileObserver events from our
|
||||||
|
// own file writes never trigger a re-sync regardless of what started the sync.
|
||||||
|
private fun startSyncMonitor(pairId: Long) {
|
||||||
|
syncMonitorJobs[pairId]?.cancel()
|
||||||
|
syncMonitorJobs[pairId] = scope.launch {
|
||||||
|
var wasSyncing = false
|
||||||
|
WorkManager.getInstance(applicationContext)
|
||||||
|
.getWorkInfosByTagFlow("sync_$pairId")
|
||||||
|
.collect { infos ->
|
||||||
|
val isSyncing = infos.any {
|
||||||
|
it.state == WorkInfo.State.RUNNING || it.state == WorkInfo.State.ENQUEUED
|
||||||
|
}
|
||||||
|
if (isSyncing) {
|
||||||
|
Timber.d("FileWatchService: sync active for pair $pairId — cooldown extended")
|
||||||
|
syncCooldownUntil[pairId] = System.currentTimeMillis() + 120_000
|
||||||
|
wasSyncing = true
|
||||||
|
} else if (wasSyncing) {
|
||||||
|
Timber.d("FileWatchService: sync finished for pair $pairId — 60s settle cooldown")
|
||||||
|
syncCooldownUntil[pairId] = System.currentTimeMillis() + 60_000
|
||||||
|
wasSyncing = false
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
private fun watchDirRecursive(dir: File, pairId: Long, wifiOnly: Boolean, chargingOnly: Boolean) {
|
private fun watchDirRecursive(dir: File, pairId: Long, wifiOnly: Boolean, chargingOnly: Boolean) {
|
||||||
if (!dir.isDirectory) return
|
if (!dir.isDirectory) return
|
||||||
val mask = FileObserver.CREATE or FileObserver.DELETE or FileObserver.MODIFY or
|
val mask = FileObserver.CREATE or FileObserver.DELETE or FileObserver.MODIFY or
|
||||||
@@ -185,51 +226,85 @@ class FileWatchService : Service() {
|
|||||||
val known = fileStateDao.getForPair(pairId).associateBy { it.relativePath }
|
val known = fileStateDao.getForPair(pairId).associateBy { it.relativePath }
|
||||||
if (known.isEmpty()) return // Never synced — first sync will be triggered manually
|
if (known.isEmpty()) return // Never synced — first sync will be triggered manually
|
||||||
|
|
||||||
val current = mutableMapOf<String, Long>()
|
val pairEntity = syncPairDao.getById(pairId) ?: return
|
||||||
dir.walk().filter { it.isFile }.forEach { f ->
|
val pair = pairEntity.toDomain()
|
||||||
current[f.relativeTo(dir).path.replace('\\', '/')] = f.lastModified()
|
// Use the same accessor + filters as SyncEngine so hidden/excluded/size-filtered files
|
||||||
}
|
// don't appear as "new" in the catchup scan and trigger a perpetual sync loop.
|
||||||
|
val accessor = if (pair.localPath.startsWith("content://"))
|
||||||
|
LocalAccessor.Saf(Uri.parse(pair.localPath), contentResolver)
|
||||||
|
else
|
||||||
|
LocalAccessor.JavaFile(dir)
|
||||||
|
val current = accessor.walkFiles(pair)
|
||||||
|
|
||||||
val hasNew = current.any { (rel, _) -> rel !in known }
|
val hasNew = current.any { (rel, _) -> rel !in known }
|
||||||
val hasModified = current.any { (rel, mtime) ->
|
val hasModified = current.any { (rel, info) ->
|
||||||
val s = known[rel]; s != null && s.localModifiedAt != null &&
|
val s = known[rel]; s != null && s.localModifiedAt != null &&
|
||||||
s.localModifiedAt.toEpochMilli() != mtime
|
s.localModifiedAt.epochSecond != info.lastModifiedMs / 1000
|
||||||
}
|
}
|
||||||
val hasDeleted = known.keys.any { rel -> rel !in current }
|
val hasDeleted = known.keys.any { rel -> rel !in current }
|
||||||
|
|
||||||
if (hasNew || hasModified || hasDeleted) {
|
if (hasNew || hasModified || hasDeleted) {
|
||||||
Timber.d("FileWatchService: catchup detected changes for pair $pairId, scheduling sync")
|
Timber.d("FileWatchService: catchup detected changes for pair $pairId, scheduling sync")
|
||||||
val pair = syncPairDao.getById(pairId) ?: return
|
// Cancel any debounce that started before our startup cooldown was set
|
||||||
|
debounceJobs[pairId]?.cancel()
|
||||||
|
debounceJobs.remove(pairId)
|
||||||
|
// Hold cooldown for duration of sync + 60s settle
|
||||||
|
syncCooldownUntil[pairId] = System.currentTimeMillis() + 120_000
|
||||||
|
val req = SyncWorker.buildOneTimeRequest(pairId, wifiOnly, chargingOnly)
|
||||||
WorkManager.getInstance(applicationContext)
|
WorkManager.getInstance(applicationContext)
|
||||||
.enqueueUniqueWork(
|
.enqueueUniqueWork("catchup_$pairId", ExistingWorkPolicy.KEEP, req)
|
||||||
"catchup_$pairId",
|
scope.launch {
|
||||||
ExistingWorkPolicy.KEEP,
|
try {
|
||||||
SyncWorker.buildOneTimeRequest(pairId, wifiOnly, chargingOnly),
|
WorkManager.getInstance(applicationContext)
|
||||||
)
|
.getWorkInfoByIdFlow(req.id)
|
||||||
|
.first { it?.state?.isFinished == true }
|
||||||
|
syncCooldownUntil[pairId] = System.currentTimeMillis() + 60_000
|
||||||
|
} catch (e: CancellationException) {
|
||||||
|
throw e
|
||||||
|
} catch (_: Exception) {
|
||||||
|
syncCooldownUntil[pairId] = System.currentTimeMillis() + 60_000
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
private fun onChangeDetected(pairId: Long, wifiOnly: Boolean, chargingOnly: Boolean) {
|
private fun onChangeDetected(pairId: Long, wifiOnly: Boolean, chargingOnly: Boolean) {
|
||||||
|
// Ignore events fired by our own sync writing files — prevents the feedback loop
|
||||||
|
// where downloaded/uploaded files trigger another sync indefinitely.
|
||||||
|
if (System.currentTimeMillis() < (syncCooldownUntil[pairId] ?: 0L)) {
|
||||||
|
Timber.d("FileWatchService: suppressing change event for pair $pairId (sync cooldown)")
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
debounceJobs[pairId]?.cancel()
|
debounceJobs[pairId]?.cancel()
|
||||||
debounceJobs[pairId] = scope.launch {
|
debounceJobs[pairId] = scope.launch {
|
||||||
delay(5_000)
|
delay(5_000)
|
||||||
|
// Re-check: catchupScan or another path may have already set a cooldown
|
||||||
|
// and handled this sync while we were waiting.
|
||||||
|
if (System.currentTimeMillis() < (syncCooldownUntil[pairId] ?: 0L)) {
|
||||||
|
Timber.d("FileWatchService: debounce fired but cooldown active for pair $pairId, skipping")
|
||||||
|
return@launch
|
||||||
|
}
|
||||||
val pair = syncPairDao.getById(pairId)
|
val pair = syncPairDao.getById(pairId)
|
||||||
if (pair == null || !pair.isEnabled) return@launch
|
if (pair == null || !pair.isEnabled) return@launch
|
||||||
Timber.d("FileWatchService: triggering sync for pair $pairId after debounce")
|
Timber.d("FileWatchService: triggering sync for pair $pairId after debounce")
|
||||||
|
|
||||||
|
// Block new triggers from this point until 60s after sync completes
|
||||||
|
syncCooldownUntil[pairId] = System.currentTimeMillis() + 120_000
|
||||||
|
|
||||||
val req = SyncWorker.buildOneTimeRequest(pairId, wifiOnly, chargingOnly, silent = true)
|
val req = SyncWorker.buildOneTimeRequest(pairId, wifiOnly, chargingOnly, silent = true)
|
||||||
WorkManager.getInstance(applicationContext)
|
WorkManager.getInstance(applicationContext)
|
||||||
.enqueueUniqueWork("onchange_$pairId", ExistingWorkPolicy.KEEP, req)
|
.enqueueUniqueWork("onchange_$pairId", ExistingWorkPolicy.KEEP, req)
|
||||||
|
|
||||||
// Update notification while sync is in progress
|
|
||||||
updateNotificationDynamic("Syncing: ${pair.name}…")
|
updateNotificationDynamic("Syncing: ${pair.name}…")
|
||||||
|
|
||||||
// Wait for completion and show result in the persistent notification
|
|
||||||
scope.launch {
|
scope.launch {
|
||||||
try {
|
try {
|
||||||
val info = WorkManager.getInstance(applicationContext)
|
val info = WorkManager.getInstance(applicationContext)
|
||||||
.getWorkInfoByIdFlow(req.id)
|
.getWorkInfoByIdFlow(req.id)
|
||||||
.first { it?.state?.isFinished == true }
|
.first { it?.state?.isFinished == true }
|
||||||
|
// Extend cooldown: 60s after sync finishes to let filesystem settle
|
||||||
|
syncCooldownUntil[pairId] = System.currentTimeMillis() + 60_000
|
||||||
val summary = info?.outputData?.getString(SyncWorker.KEY_RESULT_SUMMARY)
|
val summary = info?.outputData?.getString(SyncWorker.KEY_RESULT_SUMMARY)
|
||||||
val watchCount = fileObservers.keys.size + contentObservers.size
|
val watchCount = fileObservers.keys.size + contentObservers.size
|
||||||
val watching = "Watching $watchCount folder${if (watchCount != 1) "s" else ""}"
|
val watching = "Watching $watchCount folder${if (watchCount != 1) "s" else ""}"
|
||||||
@@ -239,8 +314,11 @@ class FileWatchService : Service() {
|
|||||||
updateNotificationDynamic("$watching")
|
updateNotificationDynamic("$watching")
|
||||||
}
|
}
|
||||||
delay(12_000)
|
delay(12_000)
|
||||||
updateNotificationDynamic(null) // revert to default watching text
|
updateNotificationDynamic(null)
|
||||||
|
} catch (e: CancellationException) {
|
||||||
|
throw e
|
||||||
} catch (_: Exception) {
|
} catch (_: Exception) {
|
||||||
|
syncCooldownUntil[pairId] = System.currentTimeMillis() + 60_000
|
||||||
updateNotificationDynamic(null)
|
updateNotificationDynamic(null)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -254,6 +332,9 @@ class FileWatchService : Service() {
|
|||||||
contentObservers.clear()
|
contentObservers.clear()
|
||||||
debounceJobs.values.forEach { it.cancel() }
|
debounceJobs.values.forEach { it.cancel() }
|
||||||
debounceJobs.clear()
|
debounceJobs.clear()
|
||||||
|
syncMonitorJobs.values.forEach { it.cancel() }
|
||||||
|
syncMonitorJobs.clear()
|
||||||
|
syncCooldownUntil.clear()
|
||||||
}
|
}
|
||||||
|
|
||||||
private fun ensureChannel() {
|
private fun ensureChannel() {
|
||||||
|
|||||||
|
After Width: | Height: | Size: 30 KiB |
|
After Width: | Height: | Size: 14 KiB |
|
After Width: | Height: | Size: 52 KiB |
|
After Width: | Height: | Size: 110 KiB |
|
After Width: | Height: | Size: 188 KiB |
@@ -1,36 +0,0 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
|
||||||
<vector xmlns:android="http://schemas.android.com/apk/res/android"
|
|
||||||
xmlns:aapt="http://schemas.android.com/aapt"
|
|
||||||
android:width="108dp"
|
|
||||||
android:height="108dp"
|
|
||||||
android:viewportWidth="108"
|
|
||||||
android:viewportHeight="108">
|
|
||||||
|
|
||||||
<!-- Deep blue-to-teal gradient background matching reference icon -->
|
|
||||||
<path
|
|
||||||
android:pathData="M0,0 H108 V108 H0 Z">
|
|
||||||
<aapt:attr name="android:fillColor">
|
|
||||||
<gradient
|
|
||||||
android:type="linear"
|
|
||||||
android:startX="0" android:startY="0"
|
|
||||||
android:endX="108" android:endY="108"
|
|
||||||
android:startColor="#1565C0"
|
|
||||||
android:endColor="#00897B"/>
|
|
||||||
</aapt:attr>
|
|
||||||
</path>
|
|
||||||
|
|
||||||
<!-- Subtle radial highlight in upper-right -->
|
|
||||||
<path
|
|
||||||
android:pathData="M108,0 A90,90 0 0,1 108,90 Z"
|
|
||||||
android:fillAlpha="0.18">
|
|
||||||
<aapt:attr name="android:fillColor">
|
|
||||||
<gradient
|
|
||||||
android:type="radial"
|
|
||||||
android:gradientRadius="80"
|
|
||||||
android:centerX="85" android:centerY="23"
|
|
||||||
android:startColor="#80DEEA"
|
|
||||||
android:endColor="#00000000"/>
|
|
||||||
</aapt:attr>
|
|
||||||
</path>
|
|
||||||
|
|
||||||
</vector>
|
|
||||||
@@ -1,70 +0,0 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
|
||||||
<vector xmlns:android="http://schemas.android.com/apk/res/android"
|
|
||||||
xmlns:aapt="http://schemas.android.com/aapt"
|
|
||||||
android:width="108dp"
|
|
||||||
android:height="108dp"
|
|
||||||
android:viewportWidth="108"
|
|
||||||
android:viewportHeight="108">
|
|
||||||
|
|
||||||
<!-- Cloud body: white, centred at (54,56). Composed of three arc bumps. -->
|
|
||||||
<path
|
|
||||||
android:pathData="
|
|
||||||
M 38,67
|
|
||||||
A 9,9 0 0,1 38,49
|
|
||||||
A 9,9 0 0,1 47,40.5
|
|
||||||
A 12,12 0 0,1 68,42
|
|
||||||
A 8,8 0 0,1 76,53
|
|
||||||
A 8,8 0 0,1 70,67
|
|
||||||
Z"
|
|
||||||
android:fillColor="#FFFFFF"/>
|
|
||||||
|
|
||||||
<!-- Cloud drop shadow -->
|
|
||||||
<path
|
|
||||||
android:pathData="
|
|
||||||
M 38,69
|
|
||||||
A 9,9 0 0,1 38,51
|
|
||||||
A 9,9 0 0,1 47,42.5
|
|
||||||
A 12,12 0 0,1 68,44
|
|
||||||
A 8,8 0 0,1 76,55
|
|
||||||
A 8,8 0 0,1 70,69
|
|
||||||
Z"
|
|
||||||
android:fillColor="#000000"
|
|
||||||
android:fillAlpha="0.10"/>
|
|
||||||
|
|
||||||
<!-- Sync arc 1: lower half CW, cyan to teal -->
|
|
||||||
<path
|
|
||||||
android:pathData="M 49.14,81.57 A 28,28 0 1,1 49.14,26.43"
|
|
||||||
android:fillColor="#00000000"
|
|
||||||
android:strokeWidth="5.5"
|
|
||||||
android:strokeLineCap="round">
|
|
||||||
<aapt:attr name="android:strokeColor">
|
|
||||||
<gradient android:type="linear"
|
|
||||||
android:startX="49.14" android:startY="81.57"
|
|
||||||
android:endX="49.14" android:endY="26.43"
|
|
||||||
android:startColor="#40C4FF"
|
|
||||||
android:endColor="#00BFA5"/>
|
|
||||||
</aapt:attr>
|
|
||||||
</path>
|
|
||||||
<!-- Arrowhead at end of arc 1 (near 260 deg) -->
|
|
||||||
<path android:pathData="M 42.5,30.5 L 49.14,26.43 L 46.0,34.5 Z"
|
|
||||||
android:fillColor="#00BFA5"/>
|
|
||||||
|
|
||||||
<!-- Sync arc 2: upper half CW, teal to cyan -->
|
|
||||||
<path
|
|
||||||
android:pathData="M 58.86,26.43 A 28,28 0 1,1 58.86,81.57"
|
|
||||||
android:fillColor="#00000000"
|
|
||||||
android:strokeWidth="5.5"
|
|
||||||
android:strokeLineCap="round">
|
|
||||||
<aapt:attr name="android:strokeColor">
|
|
||||||
<gradient android:type="linear"
|
|
||||||
android:startX="58.86" android:startY="26.43"
|
|
||||||
android:endX="58.86" android:endY="81.57"
|
|
||||||
android:startColor="#00BFA5"
|
|
||||||
android:endColor="#40C4FF"/>
|
|
||||||
</aapt:attr>
|
|
||||||
</path>
|
|
||||||
<!-- Arrowhead at end of arc 2 (near 80 deg) -->
|
|
||||||
<path android:pathData="M 65.5,77.5 L 58.86,81.57 L 62.0,73.5 Z"
|
|
||||||
android:fillColor="#40C4FF"/>
|
|
||||||
|
|
||||||
</vector>
|
|
||||||
@@ -1,5 +1,5 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
||||||
<background android:drawable="@drawable/ic_launcher_background"/>
|
<background android:drawable="@color/ic_launcher_background"/>
|
||||||
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
||||||
</adaptive-icon>
|
</adaptive-icon>
|
||||||
@@ -1,5 +1,5 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
||||||
<background android:drawable="@drawable/ic_launcher_background"/>
|
<background android:drawable="@color/ic_launcher_background"/>
|
||||||
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
||||||
</adaptive-icon>
|
</adaptive-icon>
|
||||||
|
After Width: | Height: | Size: 14 KiB |
@@ -1,5 +0,0 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
|
||||||
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
|
||||||
<background android:drawable="@drawable/ic_launcher_background"/>
|
|
||||||
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
|
||||||
</adaptive-icon>
|
|
||||||
|
After Width: | Height: | Size: 12 KiB |
@@ -1,5 +0,0 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
|
||||||
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
|
||||||
<background android:drawable="@drawable/ic_launcher_background"/>
|
|
||||||
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
|
||||||
</adaptive-icon>
|
|
||||||
|
After Width: | Height: | Size: 6.5 KiB |
|
After Width: | Height: | Size: 5.5 KiB |
@@ -1,5 +0,0 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
|
||||||
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
|
||||||
<background android:drawable="@drawable/ic_launcher_background"/>
|
|
||||||
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
|
||||||
</adaptive-icon>
|
|
||||||
|
After Width: | Height: | Size: 24 KiB |
@@ -1,5 +0,0 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
|
||||||
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
|
||||||
<background android:drawable="@drawable/ic_launcher_background"/>
|
|
||||||
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
|
||||||
</adaptive-icon>
|
|
||||||
|
After Width: | Height: | Size: 20 KiB |
|
After Width: | Height: | Size: 52 KiB |
@@ -1,5 +0,0 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
|
||||||
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
|
||||||
<background android:drawable="@drawable/ic_launcher_background"/>
|
|
||||||
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
|
||||||
</adaptive-icon>
|
|
||||||
|
After Width: | Height: | Size: 44 KiB |
@@ -1,5 +0,0 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
|
||||||
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
|
||||||
<background android:drawable="@drawable/ic_launcher_background"/>
|
|
||||||
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
|
||||||
</adaptive-icon>
|
|
||||||
|
After Width: | Height: | Size: 88 KiB |
@@ -1,5 +0,0 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
|
||||||
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
|
||||||
<background android:drawable="@drawable/ic_launcher_background"/>
|
|
||||||
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
|
||||||
</adaptive-icon>
|
|
||||||
|
After Width: | Height: | Size: 75 KiB |
@@ -1,5 +0,0 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
|
||||||
<adaptive-icon xmlns:android="http://schemas.android.com/apk/res/android">
|
|
||||||
<background android:drawable="@drawable/ic_launcher_background"/>
|
|
||||||
<foreground android:drawable="@drawable/ic_launcher_foreground"/>
|
|
||||||
</adaptive-icon>
|
|
||||||
@@ -1,4 +1,4 @@
|
|||||||
<?xml version="1.0" encoding="utf-8"?>
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
<resources>
|
<resources>
|
||||||
<color name="ic_launcher_background">#2196F3</color>
|
<color name="ic_launcher_background">#050E05</color>
|
||||||
</resources>
|
</resources>
|
||||||
|
|||||||
@@ -1,2 +1,2 @@
|
|||||||
VERSION_NAME=1.0.26
|
VERSION_NAME=1.0.56
|
||||||
VERSION_CODE=27
|
VERSION_CODE=57
|
||||||
|
|||||||