Skip to content

Commit 57df3dd

Browse files
Fixed lint
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
1 parent 971ae9a commit 57df3dd

3 files changed

Lines changed: 28 additions & 28 deletions

File tree

app/src/main/java/com/nextcloud/client/jobs/offlineOperations/repository/OfflineOperationsRepository.kt

Lines changed: 19 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -81,29 +81,27 @@ class OfflineOperationsRepository(private val fileDataStorageManager: FileDataSt
8181
.forEach { dao.update(it) }
8282
}
8383

84-
private fun updateOperationPath(
85-
newPath: String,
86-
nextOperation: OfflineOperationEntity
87-
): OfflineOperationEntity? = if (newPath != nextOperation.path) {
88-
nextOperation.apply {
89-
type = when (type) {
90-
is OfflineOperationType.CreateFile ->
91-
(type as OfflineOperationType.CreateFile).copy(
92-
remotePath = newPath
93-
)
94-
95-
is OfflineOperationType.CreateFolder ->
96-
(type as OfflineOperationType.CreateFolder).copy(
97-
path = newPath
98-
)
99-
100-
else -> type
84+
private fun updateOperationPath(newPath: String, nextOperation: OfflineOperationEntity): OfflineOperationEntity? =
85+
if (newPath != nextOperation.path) {
86+
nextOperation.apply {
87+
type = when (type) {
88+
is OfflineOperationType.CreateFile ->
89+
(type as OfflineOperationType.CreateFile).copy(
90+
remotePath = newPath
91+
)
92+
93+
is OfflineOperationType.CreateFolder ->
94+
(type as OfflineOperationType.CreateFolder).copy(
95+
path = newPath
96+
)
97+
98+
else -> type
99+
}
100+
path = newPath
101101
}
102-
path = newPath
102+
} else {
103+
null
103104
}
104-
} else {
105-
null
106-
}
107105

108106
override fun updateOperationForKeepBoth(operation: OfflineOperationEntity, newPath: String) {
109107
updateOperationPath(newPath, operation)

app/src/main/java/com/owncloud/android/ui/dialog/conflict/ConflictResolveDialogFactory.kt

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,14 +24,18 @@ import java.io.File
2424

2525
object ConflictResolveDialogFactory {
2626

27-
fun forOffline(context: Context, leftFile: OfflineOperationEntity, rightFile: OCFile, user: User?):
28-
ConflictsResolveDialog {
29-
27+
fun forOffline(
28+
context: Context,
29+
leftFile: OfflineOperationEntity,
30+
rightFile: OCFile,
31+
user: User?
32+
): ConflictsResolveDialog {
3033
val localFile =
31-
if (leftFile.type is OfflineOperationType.CreateFile)
34+
if (leftFile.type is OfflineOperationType.CreateFile) {
3235
(leftFile.type as OfflineOperationType.CreateFile).localPath.toFile()
33-
else
36+
} else {
3437
null
38+
}
3539

3640
val data = ConflictDialogData(
3741
headline = context.getString(R.string.choose_which_file),

app/src/main/res/values/strings.xml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1267,8 +1267,6 @@
12671267
<string name="sync_not_enough_space_dialog_action_free_space">Free up space</string>
12681268
<string name="sync_not_enough_space_dialog_placeholder">%1$s is %2$s, but there is only %3$s available on device.</string>
12691269
<string name="sync_not_enough_space_dialog_title">Not enough space</string>
1270-
<string name="conflict_folder_headline">Folder conflict</string>
1271-
<string name="conflict_message_description_for_folder">If you select both versions, the local folder will have a number appended to its name.</string>
12721270
<string name="conflict_message_description">If you select both versions, the local file will have a number appended to its name.</string>
12731271
<string name="conflict_local_file">Local file</string>
12741272
<string name="conflict_server_file">Server file</string>

0 commit comments

Comments
 (0)