Skip to content

Commit c195e72

Browse files
alperozturk96AndyScherzinger
alperozturk96
authored and
AndyScherzinger
committed
add: nextcloud network exception check
Signed-off-by: alperozturk <alper_ozturk@proton.me>
1 parent cf00357 commit c195e72

7 files changed

Lines changed: 62 additions & 16 deletions

File tree

‎app/build.gradle‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,12 @@ dependencies {
120120

121121
// Nextcloud SSO
122122
implementation 'com.github.nextcloud.android-common:ui:0.27.2'
123-
implementation 'com.github.nextcloud:Android-SingleSignOn:1.3.2'
124-
implementation "com.github.stefan-niedermann.android-commons:shared-preferences:$androidCommonsVersion"
123+
implementation("com.github.nextcloud:Android-SingleSignOn:$singleSignOnVersion") {
124+
version {
125+
strictly(singleSignOnVersion)
126+
}
127+
}
128+
implementation "com.github.stefan-niedermann.android-commons:shared-preferences:$androidCommonsVersion"
125129
implementation "com.github.stefan-niedermann.android-commons:reactive-livedata:$androidCommonsVersion"
126130
implementation "com.github.stefan-niedermann.android-commons:util:$androidCommonsVersion"
127131
implementation "com.github.stefan-niedermann.nextcloud-commons:sso-glide:$commonsVersion"

‎app/src/main/java/it/niedermann/owncloud/notes/NotesApplication.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ private void registerFilesAppType() {
6161
return;
6262
}
6363

64-
FilesAppTypeRegistry.getInstance().init(new FilesAppType(packageId, accountType, FilesAppType.Type.PROD));
64+
FilesAppTypeRegistry.getInstance().init(new FilesAppType(packageId, accountType, FilesAppType.Stage.PROD));
6565
}
6666

6767
public static BrandingUtil brandingUtil() {

‎app/src/main/java/it/niedermann/owncloud/notes/main/menu/MenuAdapter.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -87,8 +87,8 @@ private static Intent generateTrashbinIntent(@NonNull Context context, @NonNull
8787
// https://github.com/nextcloud/android/pull/8405#issuecomment-852966877
8888
final int minVersionCode = 30170090;
8989
try {
90-
Optional<FilesAppType> prod = FilesAppTypeRegistry.getInstance().getTypes().stream().filter(t -> t.type == FilesAppType.Type.PROD).findFirst();
91-
Optional<FilesAppType> dev = FilesAppTypeRegistry.getInstance().getTypes().stream().filter(t -> t.type == FilesAppType.Type.DEV).findFirst();
90+
Optional<FilesAppType> prod = FilesAppTypeRegistry.getInstance().getTypes().stream().filter(t -> t.stage() == FilesAppType.Stage.PROD).findFirst();
91+
Optional<FilesAppType> dev = FilesAppTypeRegistry.getInstance().getTypes().stream().filter(t -> t.stage() == FilesAppType.Stage.DEV).findFirst();
9292
if (prod.isPresent() && VersionCheckHelper.getNextcloudFilesVersionCode(context, prod.get()) > minVersionCode) {
9393
return generateTrashbinAppIntent(context, account, prod.get());
9494
} else if (dev.isPresent() && VersionCheckHelper.getNextcloudFilesVersionCode(context, dev.get()) > minVersionCode) {
@@ -105,7 +105,7 @@ private static Intent generateTrashbinIntent(@NonNull Context context, @NonNull
105105

106106
private static Intent generateTrashbinAppIntent(@NonNull Context context, @NonNull Account account, FilesAppType type) throws PackageManager.NameNotFoundException {
107107
final var packageManager = context.getPackageManager();
108-
final String packageName = type.packageId;
108+
final String packageName = type.packageId();
109109
final var intent = new Intent();
110110
intent.setClassName(packageName, "com.owncloud.android.ui.trashbin.TrashbinActivity");
111111
if (packageManager.resolveActivity(intent, 0) != null) {

‎app/src/main/java/it/niedermann/owncloud/notes/persistence/NotesServerSyncTask.java‎

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -6,18 +6,25 @@
66
*/
77
package it.niedermann.owncloud.notes.persistence;
88

9+
import static java.net.HttpURLConnection.HTTP_NOT_FOUND;
10+
import static java.net.HttpURLConnection.HTTP_NOT_MODIFIED;
11+
import static java.net.HttpURLConnection.HTTP_UNAVAILABLE;
12+
import static it.niedermann.owncloud.notes.shared.model.DBStatus.LOCAL_DELETED;
13+
import static it.niedermann.owncloud.notes.shared.util.NoteUtil.generateNoteExcerpt;
14+
915
import android.content.Context;
1016
import android.util.Log;
1117

1218
import androidx.annotation.NonNull;
1319

1420
import com.nextcloud.android.sso.AccountImporter;
15-
import com.nextcloud.android.sso.api.ParsedResponse;
1621
import com.nextcloud.android.sso.exceptions.NextcloudApiNotRespondingException;
1722
import com.nextcloud.android.sso.exceptions.NextcloudFilesAppAccountNotFoundException;
1823
import com.nextcloud.android.sso.exceptions.NextcloudHttpRequestFailedException;
24+
import com.nextcloud.android.sso.exceptions.NextcloudNetworkException;
1925
import com.nextcloud.android.sso.exceptions.TokenMismatchException;
2026
import com.nextcloud.android.sso.model.SingleSignOnAccount;
27+
import com.owncloud.android.lib.common.utils.Log_OC;
2128

2229
import java.util.ArrayList;
2330
import java.util.Calendar;
@@ -26,7 +33,6 @@
2633
import java.util.HashSet;
2734
import java.util.List;
2835
import java.util.Map;
29-
import java.util.Set;
3036

3137
import it.niedermann.owncloud.notes.BuildConfig;
3238
import it.niedermann.owncloud.notes.persistence.entity.Account;
@@ -36,13 +42,6 @@
3642
import it.niedermann.owncloud.notes.shared.model.ISyncCallback;
3743
import it.niedermann.owncloud.notes.shared.model.SyncResultStatus;
3844
import it.niedermann.owncloud.notes.shared.util.ApiVersionUtil;
39-
import retrofit2.Response;
40-
41-
import static it.niedermann.owncloud.notes.shared.model.DBStatus.LOCAL_DELETED;
42-
import static it.niedermann.owncloud.notes.shared.util.NoteUtil.generateNoteExcerpt;
43-
import static java.net.HttpURLConnection.HTTP_NOT_FOUND;
44-
import static java.net.HttpURLConnection.HTTP_NOT_MODIFIED;
45-
import static java.net.HttpURLConnection.HTTP_UNAVAILABLE;
4645

4746

4847
/**
@@ -221,6 +220,11 @@ private boolean pullRemoteChanges() {
221220
}
222221
localAccount.setModified(accountFromDatabase.getModified());
223222
localAccount.setETag(accountFromDatabase.getETag());
223+
final var modified = localAccount.getModified();
224+
if (modified == null) {
225+
Log_OC.e(TAG, "modified is null cannot fetch notes");
226+
return false;
227+
}
224228

225229
final var fetchResponse = notesAPI.getNotes(localAccount.getModified(), localAccount.getETag()).blockingSingle();
226230
final var remoteNotes = fetchResponse.getResponse();
@@ -289,7 +293,11 @@ private boolean pullRemoteChanges() {
289293
} else if (cause.getClass() == NextcloudApiNotRespondingException.class || cause instanceof NextcloudApiNotRespondingException) {
290294
apiProvider.invalidateAPICache(ssoAccount);
291295
}
296+
} else if (t.getClass() == NextcloudNetworkException.class) {
297+
Log.w(TAG, "Network connectivity issue during sync");
298+
return true;
292299
}
300+
293301
exceptions.add(t);
294302
return false;
295303
}

‎app/src/main/java/it/niedermann/owncloud/notes/persistence/sync/NotesAPI.java‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
import com.nextcloud.android.sso.api.EmptyResponse;
1616
import com.nextcloud.android.sso.api.NextcloudAPI;
1717
import com.nextcloud.android.sso.api.ParsedResponse;
18+
import com.nextcloud.android.sso.exceptions.NextcloudNetworkException;
1819

1920
import java.util.Calendar;
2021
import java.util.HashMap;
@@ -68,7 +69,7 @@ public NotesAPI(@NonNull NextcloudAPI nextcloudAPI, @Nullable ApiVersion preferr
6869
}
6970
}
7071

71-
public Observable<ParsedResponse<List<Note>>> getNotes(@NonNull Calendar lastModified, String lastETag) {
72+
public Observable<ParsedResponse<List<Note>>> getNotes(@NonNull Calendar lastModified, String lastETag) throws NextcloudNetworkException {
7273
if (ApiVersion.API_VERSION_1_0.equals(usedApiVersion)) {
7374
return notesAPI_1_0.getNotes(lastModified.getTimeInMillis() / 1_000, lastETag);
7475
} else if (ApiVersion.API_VERSION_0_2.equals(usedApiVersion)) {

‎build.gradle‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ buildscript {
1616
kotlinVersion = '2.2.10'
1717
commonsVersion = '2.3.7'
1818
androidCommonsVersion = '1.1.0'
19+
singleSignOnVersion = "1d31061d17"
1920
}
2021
repositories {
2122
mavenCentral()

‎gradle/verification-metadata.xml‎

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -233,6 +233,7 @@
233233
<trusted-key id="E7DC75FC24FB3C8DFE8086AD3D5839A2262CBBFB" group="org.jetbrains.kotlinx"/>
234234
<trusted-key id="E85AED155021AF8A6C6B7A4A7C7D8456294423BA" group="org.objenesis"/>
235235
<trusted-key id="E9CC3CD1AE59E851E4DB3FA350FFD7487D34B5B9">
236+
<trusting group="io.reactivex.rxjava3" name="rxjava" version="3.1.11"/>
236237
<trusting group="io.reactivex.rxjava3" name="rxjava" version="3.1.8"/>
237238
<trusting group="io.reactivex.rxjava3" name="rxjava" version="3.1.9"/>
238239
</trusted-key>
@@ -9235,6 +9236,32 @@
92359236
</sha256>
92369237
</artifact>
92379238
</component>
9239+
<component group="com.github.nextcloud" name="Android-SingleSignOn" version="1d31061d17">
9240+
<artifact name="Android-SingleSignOn-1d31061d17.aar">
9241+
<sha256 value="e35b753b2142d329cc800314d5d31abf9a9ece72c28c5f9c830ed57176cee0b4" origin="Generated by Gradle" reason="Artifact is not signed"/>
9242+
</artifact>
9243+
<artifact name="Android-SingleSignOn-1d31061d17.module">
9244+
<sha256 value="a41c20f05f198d96fb8d0c8d33abec1b207e36650949c15de3fab45e0dcd6b2a" origin="Generated by Gradle" reason="Artifact is not signed"/>
9245+
</artifact>
9246+
</component>
9247+
<component group="com.github.nextcloud" name="Android-SingleSignOn" version="1d31061d17e1bd88b6501fa436a753ada4273230">
9248+
<artifact name="Android-SingleSignOn-1d31061d17e1bd88b6501fa436a753ada4273230.module">
9249+
<sha256 value="7567b6eb9817a4785a900eef12dfb0231611b030db2b17721c6fc62785f6d77c" origin="Generated by Gradle" reason="Artifact is not signed"/>
9250+
</artifact>
9251+
</component>
9252+
<component group="com.github.nextcloud" name="Android-SingleSignOn" version="ab41fe72d4">
9253+
<artifact name="Android-SingleSignOn-ab41fe72d4.aar">
9254+
<sha256 value="60cf4e8db291c411be42d725edb16a8ee8eae28ede740475a3bbbcae660b7621" origin="Generated by Gradle" reason="Artifact is not signed"/>
9255+
</artifact>
9256+
<artifact name="Android-SingleSignOn-ab41fe72d4.module">
9257+
<sha256 value="ce809fc39073f82ac59c9a5cc38134045857ec5169acc1f24d09e2cc3421234e" origin="Generated by Gradle" reason="Artifact is not signed"/>
9258+
</artifact>
9259+
</component>
9260+
<component group="com.github.nextcloud" name="Android-SingleSignOn" version="c8e05ec00f086e8fc464a6bc112a4b7e7cd38c4a">
9261+
<artifact name="Android-SingleSignOn-c8e05ec00f086e8fc464a6bc112a4b7e7cd38c4a.module">
9262+
<sha256 value="4d59876c3a6313d225cf0f847fc54e04eddb6e6eaf66d92587544752c02e2185" origin="Generated by Gradle" reason="Artifact is not signed"/>
9263+
</artifact>
9264+
</component>
92389265
<component group="com.github.nextcloud" name="android-common" version="0.14.0">
92399266
<artifact name="android-common-0.14.0.jar">
92409267
<sha256 value="12473a2140bdf23575f487ddce434ba8cdfc5bc28f99ddf7f6b6418c67597632" origin="Generated by Gradle" reason="Artifact is not signed"/>
@@ -11922,6 +11949,11 @@
1192211949
<sha256 value="b25bece50483de9a3e1dfd25011c2d85330216ef8a81b86f6abebdbdf9b0f127" origin="Generated by Gradle"/>
1192311950
</artifact>
1192411951
</component>
11952+
<component group="io.reactivex.rxjava3" name="rxjava" version="3.1.11">
11953+
<artifact name="rxjava-3.1.11.module">
11954+
<pgp value="E9CC3CD1AE59E851E4DB3FA350FFD7487D34B5B9"/>
11955+
</artifact>
11956+
</component>
1192511957
<component group="io.reactivex.rxjava3" name="rxjava" version="3.1.9">
1192611958
<artifact name="rxjava-3.1.9.jar">
1192711959
<sha256 value="88cba16c7f0fe78a8cc9b8c46469c8596cf166037ce22c2adfc4440cd6312e20" origin="Generated by Gradle"/>

0 commit comments

Comments
 (0)