Skip to content

Commit 773e576

Browse files
alperozturk96alperozturk96
committed
add: nextcloud network exception check
Signed-off-by: alperozturk <[email protected]>
1 parent 822a645 commit 773e576

File tree

7 files changed

+62
-16
lines changed

7 files changed

+62
-16
lines changed

‎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
@@ -231,6 +231,7 @@
231231
<trusted-key id="E7DC75FC24FB3C8DFE8086AD3D5839A2262CBBFB" group="org.jetbrains.kotlinx"/>
232232
<trusted-key id="E85AED155021AF8A6C6B7A4A7C7D8456294423BA" group="org.objenesis"/>
233233
<trusted-key id="E9CC3CD1AE59E851E4DB3FA350FFD7487D34B5B9">
234+
<trusting group="io.reactivex.rxjava3" name="rxjava" version="3.1.11"/>
234235
<trusting group="io.reactivex.rxjava3" name="rxjava" version="3.1.8"/>
235236
<trusting group="io.reactivex.rxjava3" name="rxjava" version="3.1.9"/>
236237
</trusted-key>
@@ -9228,6 +9229,32 @@
92289229
</sha256>
92299230
</artifact>
92309231
</component>
9232+
<component group="com.github.nextcloud" name="Android-SingleSignOn" version="1d31061d17">
9233+
<artifact name="Android-SingleSignOn-1d31061d17.aar">
9234+
<sha256 value="e35b753b2142d329cc800314d5d31abf9a9ece72c28c5f9c830ed57176cee0b4" origin="Generated by Gradle" reason="Artifact is not signed"/>
9235+
</artifact>
9236+
<artifact name="Android-SingleSignOn-1d31061d17.module">
9237+
<sha256 value="a41c20f05f198d96fb8d0c8d33abec1b207e36650949c15de3fab45e0dcd6b2a" origin="Generated by Gradle" reason="Artifact is not signed"/>
9238+
</artifact>
9239+
</component>
9240+
<component group="com.github.nextcloud" name="Android-SingleSignOn" version="1d31061d17e1bd88b6501fa436a753ada4273230">
9241+
<artifact name="Android-SingleSignOn-1d31061d17e1bd88b6501fa436a753ada4273230.module">
9242+
<sha256 value="7567b6eb9817a4785a900eef12dfb0231611b030db2b17721c6fc62785f6d77c" origin="Generated by Gradle" reason="Artifact is not signed"/>
9243+
</artifact>
9244+
</component>
9245+
<component group="com.github.nextcloud" name="Android-SingleSignOn" version="ab41fe72d4">
9246+
<artifact name="Android-SingleSignOn-ab41fe72d4.aar">
9247+
<sha256 value="60cf4e8db291c411be42d725edb16a8ee8eae28ede740475a3bbbcae660b7621" origin="Generated by Gradle" reason="Artifact is not signed"/>
9248+
</artifact>
9249+
<artifact name="Android-SingleSignOn-ab41fe72d4.module">
9250+
<sha256 value="ce809fc39073f82ac59c9a5cc38134045857ec5169acc1f24d09e2cc3421234e" origin="Generated by Gradle" reason="Artifact is not signed"/>
9251+
</artifact>
9252+
</component>
9253+
<component group="com.github.nextcloud" name="Android-SingleSignOn" version="c8e05ec00f086e8fc464a6bc112a4b7e7cd38c4a">
9254+
<artifact name="Android-SingleSignOn-c8e05ec00f086e8fc464a6bc112a4b7e7cd38c4a.module">
9255+
<sha256 value="4d59876c3a6313d225cf0f847fc54e04eddb6e6eaf66d92587544752c02e2185" origin="Generated by Gradle" reason="Artifact is not signed"/>
9256+
</artifact>
9257+
</component>
92319258
<component group="com.github.nextcloud" name="android-common" version="0.14.0">
92329259
<artifact name="android-common-0.14.0.jar">
92339260
<sha256 value="12473a2140bdf23575f487ddce434ba8cdfc5bc28f99ddf7f6b6418c67597632" origin="Generated by Gradle" reason="Artifact is not signed"/>
@@ -11915,6 +11942,11 @@
1191511942
<sha256 value="b25bece50483de9a3e1dfd25011c2d85330216ef8a81b86f6abebdbdf9b0f127" origin="Generated by Gradle"/>
1191611943
</artifact>
1191711944
</component>
11945+
<component group="io.reactivex.rxjava3" name="rxjava" version="3.1.11">
11946+
<artifact name="rxjava-3.1.11.module">
11947+
<pgp value="E9CC3CD1AE59E851E4DB3FA350FFD7487D34B5B9"/>
11948+
</artifact>
11949+
</component>
1191811950
<component group="io.reactivex.rxjava3" name="rxjava" version="3.1.9">
1191911951
<artifact name="rxjava-3.1.9.jar">
1192011952
<sha256 value="88cba16c7f0fe78a8cc9b8c46469c8596cf166037ce22c2adfc4440cd6312e20" origin="Generated by Gradle"/>

0 commit comments

Comments
 (0)