Skip to content

Don't copy existing media files when updating media only #6946

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
grzesiek2010 merged 19 commits into from
Dec 2, 2025
Merged

Don't copy existing media files when updating media only #6946

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
f8996ca
Remove test replaced by FormDownloadActionTest
seadowg Oct 23, 2025
bf57e78
Remove unneeded open
seadowg Oct 23, 2025
b797772
Move ServerFormDetailsFetcher logic to use case
seadowg Oct 23, 2025
36560a9
Isolate authentication overriding to where its used
seadowg Oct 23, 2025
6ab2581
Convert ServerFormDetailsFetcher to interface
seadowg Oct 23, 2025
aaf058b
Only use interface for Java code
seadowg Oct 24, 2025
6e09b31
Separate out tests for ServerFormUseCases
seadowg Oct 24, 2025
98fd70d
Remove unused annotation
seadowg Oct 24, 2025
acb89bf
Add enum to replace booleans
seadowg Oct 24, 2025
f7de496
Convert boolean to proxies for type
seadowg Oct 24, 2025
a224e46
Remove isNotOnDevice
seadowg Oct 24, 2025
17cfc35
Replace isUpdated property with helper
seadowg Oct 24, 2025
ad16c3d
Change target for benchmark
seadowg Oct 27, 2025
d71bfcc
Optimize updating media
seadowg Oct 27, 2025
7643da9
Add clarifying comment
seadowg Oct 27, 2025
f9db6d5
Remove unneeded cache clear
seadowg Oct 27, 2025
73f7eb7
Remove unused variable
seadowg Nov 26, 2025
ee6de92
Fix comment
seadowg Nov 26, 2025
493711e
Update test names
seadowg Nov 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Replace isUpdated property with helper
  • Loading branch information
@seadowg
seadowg committed Nov 26, 2025
commit 17cfc35642a377c8df4f0abcccc9ee2209af0c7e
2 changes: 1 addition & 1 deletion collect_app/src/main/java/org/odk/collect/android/formmanagement/FormsDataService.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ class FormsDataService(
projectDependencies.formsRepository,
projectDependencies.formSource
)
val updatedForms = serverForms.filter { it.isUpdated }
val updatedForms = serverForms.filter { it.isUpdated() }
if (updatedForms.isNotEmpty()) {
if (projectDependencies.generalSettings.getBoolean(ProjectKeys.KEY_AUTOMATIC_UPDATE)) {
val results = ServerFormUseCases.downloadForms(
Expand Down
12 changes: 2 additions & 10 deletions collect_app/src/main/java/org/odk/collect/android/formmanagement/ServerFormDetails.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,16 +52,8 @@ data class ServerFormDetails @JvmOverloads constructor(
}
)

@Deprecated(
message = "Use type instead",
replaceWith = ReplaceWith("type")
)
val isUpdated: Boolean = when (type) {
Type.OnDevice -> false
Type.New -> false
Type.UpdatedVersion -> true
Type.UpdatedHash -> true
Type.UpdatedMedia -> true
fun isUpdated(): Boolean {
return type != Type.OnDevice && type != Type.New
}

companion object {
Expand Down