Compare commits
	
		
			66 Commits
		
	
	
		
			v123020491
			...
			v123102841
		
	
	| Author | SHA1 | Date | |
|---|---|---|---|
|  | 68aedb7641 | ||
|  | 754d526b49 | ||
|  | c458871569 | ||
| 056825aa0c | |||
| 16b19fc5ce | |||
| 4ad4a23ed8 | |||
| d8c215eacc | |||
| 2b446ab22b | |||
| a029d8a7dc | |||
| 4482234e1a | |||
| b5de30f561 | |||
| 70ad5f322c | |||
| d167092c83 | |||
| c4f4bafe85 | |||
| ed06b22a77 | |||
|  | 172362b533 | ||
|  | ad72cb6f56 | ||
|  | 9057ee0052 | ||
|  | 50d0b44315 | ||
|  | 21b08ed384 | ||
|  | 993c4d2ee9 | ||
|  | 57a9d51027 | ||
|  | 673f0edb8b | ||
|  | 7f96798f13 | ||
|  | 6e5704a45b | ||
|  | 495591159f | ||
|  | 718fe7c5ee | ||
|  | ecd23213f9 | ||
|  | e6baed8cb4 | ||
|  | c87abec0b9 | ||
|  | 0aba41d8bf | ||
|  | 2a2d1047b4 | ||
|  | 66ef1ccf32 | ||
|  | 677ede5bc7 | ||
|  | 996a7ed22c | ||
|  | 85208c4e5a | ||
|  | 5cfec50cba | ||
| 76ad71e1dc | |||
| 0277fb507c | |||
| 8d7d3174aa | |||
|  | 00eb3333fe | ||
|  | 629ca01d99 | ||
|  | c2d8681ce8 | ||
|  | 08f79cb148 | ||
| e21906e70d | |||
|  | 9d2cc32bc9 | ||
|  | d9d057c8dc | ||
|  | 1f3fa0c4a6 | ||
|  | dea3def385 | ||
|  | f72ef2f5d4 | ||
|  | f28cb759df | ||
|  | b9d69c3e64 | ||
|  | c2a1c9eaac | ||
|  | bf37209a15 | ||
|  | 2c558fe6fd | ||
|  | ad88011454 | ||
|  | 559c17bc1d | ||
|  | ab9c46f0eb | ||
|  | aa799d2ca8 | ||
|  | 177c978474 | ||
|  | 39b9991413 | ||
|  | b303f110f1 | ||
|  | f851941a6a | ||
|  | a313552976 | ||
|  | 6ac97ed3fe | ||
|  | d583b937b7 | 
							
								
								
									
										93
									
								
								.drone.yml
									
									
									
									
									
								
							
							
						
						
									
										93
									
								
								.drone.yml
									
									
									
									
									
								
							| @@ -3,35 +3,38 @@ type: docker | ||||
| name: test | ||||
|  | ||||
| steps: | ||||
|   - name: Lint | ||||
|     failure: ignore | ||||
|     image: mingc/android-build-box:latest | ||||
|     commands: | ||||
|       - echo "---------------------------------------------------------" | ||||
|       - echo "Install linters..." | ||||
|       - curl -sSLO https://github.com/pinterest/ktlint/releases/download/1.0.0/ktlint && chmod a+x ktlint && mv ktlint /usr/local/bin/ | ||||
|       - curl -sSLO https://github.com/detekt/detekt/releases/download/v1.23.1/detekt-cli-1.23.1.zip && unzip detekt-cli-1.23.1.zip | ||||
|       - echo "---------------------------------------------------------" | ||||
|       - echo "Linting..." | ||||
|       - ktlint || true | ||||
|       - echo "---------------------------------------------------------" | ||||
|       - echo "Detecting..." | ||||
|       - ./detekt-cli-1.23.1/bin/detekt-cli --all-rules || true | ||||
|       - echo "---------------------------------------------------------" | ||||
|     command_timeout: 1m | ||||
|   - name: BuildAndTest | ||||
|     image: mingc/android-build-box:latest | ||||
|     commands: | ||||
|       - echo "---------------------------------------------------------" | ||||
|       - echo "Configure gradle..." | ||||
|       - mkdir -p ~/.gradle && echo "org.gradle.daemon=false\nignoreGitVersion=true\npushCache=false\nsystemProp.org.gradle.internal.http.connectionTimeout=180000\nsystemProp.org.gradle.internal.http.socketTimeout=180000" >> ~/.gradle/gradle.properties | ||||
|       - mkdir -p ~/.gradle && echo "org.gradle.daemon=false\nignoreGitVersion=true\nsystemProp.org.gradle.internal.http.connectionTimeout=180000\nsystemProp.org.gradle.internal.http.socketTimeout=180000" >> ~/.gradle/gradle.properties | ||||
|       - echo "---------------------------------------------------------" | ||||
|       - echo "Building..." | ||||
|       - ./gradlew build -x test | ||||
|       - echo "Configure java..." | ||||
|       - . ~/.bash_profile | ||||
|       - jenv global 17.0 | ||||
|       - java --version | ||||
|       - date | ||||
|       - echo "---------------------------------------------------------" | ||||
|       - echo "Testing..." | ||||
|       - echo "Building and testing..." | ||||
|       - ./gradlew build | ||||
|       - echo "---------------------------------------------------------" | ||||
|       - ./gradlew koverMergedXmlReport | ||||
|     environment: | ||||
|       TZ: Europe/Paris | ||||
|       SONAR_HOST_URL: | ||||
|         from_secret: sonarScannerHostUrl | ||||
|       SONAR_LOGIN: | ||||
|         from_secret: sonarScannerLogin | ||||
|   - name: Analyse | ||||
|     image: kytay/sonar-node-plugin | ||||
|     settings: | ||||
|       sonar_host: | ||||
|         from_secret: sonarScannerHostUrl | ||||
|       sonar_token: | ||||
|         from_secret: sonarScannerLogin | ||||
|       use_node_version: 16.18.1 | ||||
|       sonar_debug: true | ||||
|       sonar_project_settings: ./sonar-project.properties | ||||
| trigger: | ||||
|   event: | ||||
|     - push | ||||
| @@ -48,21 +51,26 @@ steps: | ||||
|     commands: | ||||
|       - apt-get update && apt-get install -y git | ||||
|       - git fetch --tags -p | ||||
|       - PREV=$(git describe --tags --abbrev=0) | ||||
|       - ./build.sh --publish --from-ci | ||||
|       - VER=$(git describe --tags --abbrev=0) | ||||
|       - CHANGELOG=$(git log $VER..HEAD --pretty="- %s") | ||||
|       - CHANGELOG=$(git log $PREV..HEAD --pretty="- %s") | ||||
|       - echo "**$VER**\n\n$CHANGELOG\n\n--------------------------------------------------------------------\n\n$(cat CHANGELOG.md)" > CHANGELOG.md | ||||
|       - git add CHANGELOG.md | ||||
|       - git commit -m "Changelog for $VER [CI SKIP]" | ||||
|       - ./build.sh --publish --from-ci | ||||
|       - git remote add pushing https://$GITEA_USR:$GITEA_PASS@gitea.amine-louveau.fr/Louvorg/ReaderForSelfoss-multiplatform.git | ||||
|       - git push pushing master | ||||
|       - git push pushing --tags | ||||
|     environment: | ||||
|       TZ: Europe/Paris | ||||
|       GITEA_USR: | ||||
|         from_secret: giteaUsr | ||||
|       GITEA_PASS: | ||||
|         from_secret: giteaPass | ||||
|  | ||||
|   - name: git-push | ||||
|     image: appleboy/drone-git-push | ||||
|     settings: | ||||
|       branch: master | ||||
|       remote: | ||||
|         from_secret: remoteUrl | ||||
|       followtags: true | ||||
|       ssh_key: | ||||
|         from_secret: privateKey | ||||
|       skip_verify: true | ||||
|  | ||||
|   - name: scpFiles | ||||
|     image: appleboy/drone-scp | ||||
| @@ -106,10 +114,10 @@ steps: | ||||
|       - git fetch --tags | ||||
|       - echo "---------------------------------------------------------" | ||||
|       - echo "Configure gradle..." | ||||
|       - mkdir -p ~/.gradle && echo "org.gradle.daemon=false\nignoreGitVersion=false\npushCache=false\nsystemProp.org.gradle.internal.http.connectionTimeout=180000\nsystemProp.org.gradle.internal.http.socketTimeout=180000" >> ~/.gradle/gradle.properties | ||||
|       - mkdir -p ~/.gradle && echo "org.gradle.daemon=false\nignoreGitVersion=false\nsystemProp.org.gradle.internal.http.connectionTimeout=180000\nsystemProp.org.gradle.internal.http.socketTimeout=180000" >> ~/.gradle/gradle.properties | ||||
|       - echo "---------------------------------------------------------" | ||||
|       - echo "Generate APK" | ||||
|       - ./gradlew :androidApp:assembleGithubConfigRelease  -P pushCache=false | ||||
|       - ./gradlew :androidApp:assembleGithubConfigRelease | ||||
|       - echo "---------------------------------------------------------" | ||||
|       - echo "Get Key" | ||||
|       - wget https://amine-louveau.fr/key | ||||
| @@ -136,6 +144,27 @@ steps: | ||||
|         from_secret: giteaAPI | ||||
|       base_url: https://gitea.amine-louveau.fr | ||||
|       files: signed.apk | ||||
|  | ||||
|   - name: notify | ||||
|     image: drillster/drone-email | ||||
|     failure: ignore | ||||
|     settings: | ||||
|       host: | ||||
|         from_secret: smtpHOST | ||||
|       port: | ||||
|         from_secret: smtpPORT | ||||
|       username: | ||||
|         from_secret: smtpUSERNAME | ||||
|       password: | ||||
|         from_secret: smtpPASSWORD | ||||
|       from: | ||||
|         from_secret: smtpFROM | ||||
|       subject: Mapping file | ||||
|       recipients: | ||||
|         from_secret: smtpTO | ||||
|       recipients_only: true | ||||
|       skip_verify: true | ||||
|       attachment: androidApp/build/outputs/mapping/githubConfigRelease/mapping.txt | ||||
| trigger: | ||||
|   event: | ||||
|     - tag | ||||
							
								
								
									
										125
									
								
								CHANGELOG.md
									
									
									
									
									
								
							
							
						
						
									
										125
									
								
								CHANGELOG.md
									
									
									
									
									
								
							| @@ -1,3 +1,128 @@ | ||||
| **v123061811** | ||||
|  | ||||
| - feat: Added confirmation dialog for disconnect item menu. | ||||
| - Changelog for v123061651 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123061651** | ||||
|  | ||||
| - i18n: Translation update. | ||||
| - i18n: Translation update. | ||||
| - i18n: Translation update. | ||||
| - fix: avoid trying to open invalid image urls. | ||||
| - Changelog for v123051471 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123051471** | ||||
|  | ||||
| - fix: images could be null. | ||||
| - fix: Check if color is not empty before parsing it. | ||||
| - chore: Removed unused log. | ||||
| - Changelog for v123051331 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123051331** | ||||
|  | ||||
| - fix: illegal input. | ||||
| - Changelog for v123051321 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123051321** | ||||
|  | ||||
| - debug: Debug null context. | ||||
| - Changelog for v123051301 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123051301** | ||||
|  | ||||
| - feat: Basic auth from url. Fixes #142 (#143) | ||||
| - debug: Debug index out of bound exception. | ||||
| - Changelog for v123051211 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123051211** | ||||
|  | ||||
| - fix: Sometimes url isn't even defined. | ||||
| - Changelog for v123041021 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123041021** | ||||
|  | ||||
| - fix: 'Enable Core Library Desugaring to support older Android versions' (#138) from davidoskky/ReaderForSelfoss-multiplatform:desugaring into master | ||||
| - Enable Core Library Desugaring to support older Android versions | ||||
| - Changelog for v123030851 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123030851** | ||||
|  | ||||
| - chore: replace textDrawable library (#136) | ||||
| - refactor: Remove slow login check. Closes #135. | ||||
| - ci: send the mapping file after a release. | ||||
| - Changelog for v123030751 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123030751** | ||||
|  | ||||
| - debug: added a lot to pinpoint the url issue. | ||||
| - feat: Use /sources/stats in the home (#133) | ||||
| - Changelog for v123030681 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123030681** | ||||
|  | ||||
| - fix: Unread and starred can be null. | ||||
| - Fixed version number issue. | ||||
| - Changelog for v123030621 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123030621** | ||||
|  | ||||
| - fix: url required issue. | ||||
| - fix: Canvas reused issue. | ||||
| - Changelog for v123020572 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123020572** | ||||
|  | ||||
| - fix: requirecontext issues ? | ||||
| - debug: activity not found exception. | ||||
| - Changelog for v123020571 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123020571** | ||||
|  | ||||
| - chore: remove errors logging. | ||||
| - fix: quickfix for url param not provided for some sources. | ||||
| - Update 'CHANGELOG.md' | ||||
| - Changelog for v123020523 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123020523** | ||||
|  | ||||
| - fix: Git changelog. | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123020491** | ||||
|  | ||||
| - fix: Fixed acra bug reporting. | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123010301** | ||||
|  | ||||
| - Chore: acra config. | ||||
|   | ||||
| @@ -8,15 +8,15 @@ plugins { | ||||
|     kotlin("android") | ||||
|     kotlin("kapt") | ||||
|     id("com.mikepenz.aboutlibraries.plugin") | ||||
|     id("org.jetbrains.kotlinx.kover") version "0.6.1" | ||||
|     id("org.jetbrains.kotlinx.kover") | ||||
| } | ||||
|  | ||||
| fun Project.execWithOutput(cmd: String, ignore: Boolean = false): String { | ||||
|     var result: String = ByteArrayOutputStream().use { outputStream -> | ||||
|     val result: String = ByteArrayOutputStream().use { outputStream -> | ||||
|         project.exec { | ||||
|             commandLine = cmd.split(" ") | ||||
|             standardOutput = outputStream | ||||
|             isIgnoreExitValue = ignore ?: false | ||||
|             isIgnoreExitValue = ignore | ||||
|         } | ||||
|         outputStream.toString() | ||||
|     } | ||||
| @@ -24,11 +24,10 @@ fun Project.execWithOutput(cmd: String, ignore: Boolean = false): String { | ||||
| } | ||||
|  | ||||
| fun gitVersion(): String { | ||||
|     var process = "" | ||||
|     val maybeTagOfCurrentCommit = execWithOutput("git -C ../ describe --contains HEAD", true) | ||||
|     process = if (maybeTagOfCurrentCommit.isEmpty()) { | ||||
|     val process = if (maybeTagOfCurrentCommit.isEmpty()) { | ||||
|         println("No tag on current commit. Will take the latest one.") | ||||
|         execWithOutput("git -C ../ for-each-ref refs/tags --sort=-authordate --format='%(refname:short)' --count=1") | ||||
|         execWithOutput("git -C ../ for-each-ref refs/tags --sort=-refname --format='%(refname:short)' --count=1") | ||||
|     } else { | ||||
|         println("Tag found on current commit") | ||||
|         execWithOutput("git -C ../ describe --contains HEAD") | ||||
| @@ -56,24 +55,24 @@ fun versionNameFromGit(): String { | ||||
|  | ||||
| android { | ||||
|     compileOptions { | ||||
|         isCoreLibraryDesugaringEnabled = true | ||||
|         // Flag to enable support for the new language APIs | ||||
|         sourceCompatibility = JavaVersion.VERSION_11 | ||||
|         targetCompatibility = JavaVersion.VERSION_11 | ||||
|         sourceCompatibility = JavaVersion.VERSION_17 | ||||
|         targetCompatibility = JavaVersion.VERSION_17 | ||||
|     } | ||||
|  | ||||
|     // For Kotlin projects | ||||
|     kotlinOptions { | ||||
|         jvmTarget = "11" | ||||
|         jvmTarget = "17" | ||||
|     } | ||||
|     compileSdk = 33 | ||||
|     buildToolsVersion = "33.0.0" | ||||
|     compileSdk = 34 | ||||
|     buildFeatures { | ||||
|         viewBinding = true | ||||
|     } | ||||
|     defaultConfig { | ||||
|         applicationId = "bou.amine.apps.readerforselfossv2.android" | ||||
|         minSdk = 21 | ||||
|         targetSdk = 33 | ||||
|         minSdk = 25 | ||||
|         targetSdk = 34 | ||||
|         versionCode = versionCodeFromGit() | ||||
|         versionName = versionNameFromGit() | ||||
|  | ||||
| @@ -86,7 +85,7 @@ android { | ||||
|         // tests | ||||
|         testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner" | ||||
|     } | ||||
|     packagingOptions { | ||||
|     packaging { | ||||
|         resources { | ||||
|             excludes += "/META-INF/{AL2.0,LGPL2.1}" | ||||
|         } | ||||
| @@ -112,26 +111,28 @@ android { | ||||
| } | ||||
|  | ||||
| dependencies { | ||||
|     implementation(project(":shared")) | ||||
|     implementation("com.google.android.material:material:1.5.0") | ||||
|     implementation("androidx.appcompat:appcompat:1.4.1") | ||||
|     implementation("org.jetbrains.kotlinx:kotlinx-coroutines-android:1.6.0") | ||||
|     coreLibraryDesugaring("com.android.tools:desugar_jdk_libs:2.0.3") | ||||
|  | ||||
|     implementation("androidx.preference:preference-ktx:1.1.1") | ||||
|     implementation(project(":shared")) | ||||
|     implementation("com.google.android.material:material:1.9.0") | ||||
|     implementation("androidx.appcompat:appcompat:1.6.1") | ||||
|     implementation("org.jetbrains.kotlinx:kotlinx-coroutines-android:1.7.1") | ||||
|  | ||||
|     implementation("androidx.preference:preference-ktx:1.2.1") | ||||
|  | ||||
|     implementation(fileTree(mapOf("include" to listOf("*.jar"), "dir" to "libs"))) | ||||
|  | ||||
|     // Android Support | ||||
|     implementation("androidx.appcompat:appcompat:1.4.1") | ||||
|     implementation("com.google.android.material:material:1.5.0") | ||||
|     implementation("androidx.recyclerview:recyclerview:1.3.0-alpha01") | ||||
|     implementation("androidx.appcompat:appcompat:1.6.1") | ||||
|     implementation("com.google.android.material:material:1.9.0") | ||||
|     implementation("androidx.recyclerview:recyclerview:1.3.1") | ||||
|     implementation("androidx.legacy:legacy-support-v4:1.0.0") | ||||
|     implementation("androidx.vectordrawable:vectordrawable:1.2.0-alpha02") | ||||
|     implementation("androidx.vectordrawable:vectordrawable:1.2.0-beta01") | ||||
|     implementation("androidx.cardview:cardview:1.0.0") | ||||
|     implementation("androidx.annotation:annotation:1.3.0") | ||||
|     implementation("androidx.work:work-runtime-ktx:2.7.1") | ||||
|     implementation("androidx.annotation:annotation:1.7.0") | ||||
|     implementation("androidx.work:work-runtime-ktx:2.8.1") | ||||
|     implementation("androidx.constraintlayout:constraintlayout:2.1.4") | ||||
|     implementation("org.jsoup:jsoup:1.14.3") | ||||
|     implementation("org.jsoup:jsoup:1.15.4") | ||||
|  | ||||
|     //multidex | ||||
|     implementation("androidx.multidex:multidex:2.0.1") | ||||
| @@ -142,18 +143,17 @@ dependencies { | ||||
|  | ||||
|     // Material-ish things | ||||
|     implementation("com.ashokvarma.android:bottom-navigation-bar:2.2.0") | ||||
|     implementation("com.amulyakhare:com.amulyakhare.textdrawable:1.0.1") | ||||
|  | ||||
|     // glide | ||||
|     kapt("com.github.bumptech.glide:compiler:4.14.2") | ||||
|     implementation("com.github.bumptech.glide:okhttp3-integration:4.14.2") | ||||
|     kapt("com.github.bumptech.glide:compiler:4.15.0") | ||||
|     implementation("com.github.bumptech.glide:okhttp3-integration:4.15.0") | ||||
|  | ||||
|     // Themes | ||||
|     implementation("com.github.rubensousa:floatingtoolbar:1.5.1") | ||||
|  | ||||
|     // Pager | ||||
|     implementation("me.relex:circleindicator:2.1.6") | ||||
|     implementation("androidx.viewpager2:viewpager2:1.1.0-beta01") | ||||
|     implementation("androidx.viewpager2:viewpager2:1.1.0-beta02") | ||||
|  | ||||
|     //Dependency Injection | ||||
|     implementation("org.kodein.di:kodein-di:7.14.0") | ||||
| @@ -169,12 +169,12 @@ dependencies { | ||||
|     //PhotoView | ||||
|     implementation("com.github.chrisbanes:PhotoView:2.3.0") | ||||
|  | ||||
|     implementation("androidx.core:core-ktx:1.8.0") | ||||
|     implementation("androidx.core:core-ktx:1.12.0") | ||||
|  | ||||
|     implementation("androidx.lifecycle:lifecycle-extensions:2.2.0") | ||||
|  | ||||
|     // Network information | ||||
|      implementation("com.github.ln-12:multiplatform-connectivity-status:1.3.0") | ||||
|     implementation("com.github.ln-12:multiplatform-connectivity-status:1.3.0") | ||||
|  | ||||
|     // SQLDELIGHT | ||||
|     implementation("com.squareup.sqldelight:android-driver:1.5.4") | ||||
| @@ -182,7 +182,7 @@ dependencies { | ||||
|     //test | ||||
|     testImplementation("junit:junit:4.13.2") | ||||
|     testImplementation("io.mockk:mockk:1.12.0") | ||||
|     testImplementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.6.0") | ||||
|     testImplementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.7.1") | ||||
|     implementation("org.jetbrains.kotlinx:kotlinx-datetime:0.4.0") | ||||
|  | ||||
|     implementation("ch.acra:acra-http:$acraVersion") | ||||
|   | ||||
							
								
								
									
										1
									
								
								androidApp/proguard-rules.pro
									
									
									
									
										vendored
									
									
								
							
							
						
						
									
										1
									
								
								androidApp/proguard-rules.pro
									
									
									
									
										vendored
									
									
								
							| @@ -55,6 +55,7 @@ | ||||
| # maybe remove later ? | ||||
| -keep class * extends androidx.fragment.app.Fragment | ||||
|  | ||||
| -dontwarn org.slf4j.impl.StaticLoggerBinder | ||||
|  | ||||
| # Keep `Companion` object fields of serializable classes. | ||||
| # This avoids serializer lookup through `getDeclaredClasses` as done for named companion objects. | ||||
|   | ||||
| @@ -599,12 +599,14 @@ class HomeActivity : AppCompatActivity(), SearchView.OnQueryTextListener, DIAwar | ||||
|                 return true | ||||
|             } | ||||
|             R.id.action_disconnect -> { | ||||
|                 runBlocking { | ||||
|                     repository.logout() | ||||
|                 needsConfirmation(R.string.confirm_disconnect_title, R.string.confirm_disconnect_description) { | ||||
|                     runBlocking { | ||||
|                         repository.logout() | ||||
|                     } | ||||
|                     val intent = Intent(this, LoginActivity::class.java) | ||||
|                     this.startActivity(intent) | ||||
|                     finish() | ||||
|                 } | ||||
|                 val intent = Intent(this, LoginActivity::class.java) | ||||
|                 this.startActivity(intent) | ||||
|                 finish() | ||||
|                 return true | ||||
|             } | ||||
|             R.id.action_settings -> { | ||||
|   | ||||
| @@ -57,29 +57,7 @@ class LoginActivity : AppCompatActivity(), DIAware { | ||||
|  | ||||
|         if (appSettingsService.getBaseUrl().isNotEmpty()) { | ||||
|             showProgress(true) | ||||
|             // This should be reverted when "old" users connected with a non-selfoss rss | ||||
|             // are handled. Revert to "simple" way. | ||||
|             CoroutineScope(Dispatchers.Main).launch { | ||||
|                 try { | ||||
|                     val (errorFetching, displaySelfossOnly) = repository.shouldBeSelfossInstance() | ||||
|                     if (!errorFetching && !displaySelfossOnly) { | ||||
|                         goToMain() | ||||
|                     } else { | ||||
|                         showProgress(false) | ||||
|                         if (displaySelfossOnly) { | ||||
|                             Toast.makeText( | ||||
|                                 applicationContext, | ||||
|                                 R.string.application_selfoss_only, | ||||
|                                 Toast.LENGTH_LONG | ||||
|                             ).show() | ||||
|                         } | ||||
|                         repository.logout() | ||||
|                     } | ||||
|                 } catch (e: Throwable) { | ||||
|                     repository.logout() | ||||
|                     showProgress(false) | ||||
|                 } | ||||
|             } | ||||
|             goToMain() | ||||
|         } | ||||
|  | ||||
|         handleActions() | ||||
| @@ -161,9 +139,12 @@ class LoginActivity : AppCompatActivity(), DIAware { | ||||
|  | ||||
|         showProgress(true) | ||||
|  | ||||
|         appSettingsService.updateSelfSigned(binding.selfSigned.isChecked) | ||||
|  | ||||
|         repository.refreshLoginInformation(url, login, password) | ||||
|  | ||||
|         CoroutineScope(Dispatchers.Main).launch { | ||||
|             repository.updateApiInformation() | ||||
|             val result = repository.login() | ||||
|             if (result) { | ||||
|                 val (errorFetching, displaySelfossOnly) = repository.shouldBeSelfossInstance() | ||||
|   | ||||
| @@ -34,6 +34,7 @@ import org.kodein.di.* | ||||
| class MyApp : MultiDexApplication(), DIAware { | ||||
|  | ||||
|     override val di by DI.lazy { | ||||
|         bind<AppSettingsService>() with singleton { AppSettingsService(ACRA.isACRASenderServiceProcess()) } | ||||
|         import(networkModule) | ||||
|         bind<DriverFactory>() with singleton { DriverFactory(applicationContext) } | ||||
|         bind<ReaderForSelfossDB>() with singleton { ReaderForSelfossDB(driverFactory.createDriver()) } | ||||
|   | ||||
| @@ -71,7 +71,11 @@ class ReaderActivity : AppCompatActivity(), DIAware { | ||||
|             finish() | ||||
|         } | ||||
|  | ||||
|         readItem(allItems[currentItem]) | ||||
|         try { | ||||
|             readItem(allItems[currentItem]) | ||||
|         } catch (e: IndexOutOfBoundsException) { | ||||
|             finish() | ||||
|         } | ||||
|  | ||||
|         binding.pager.adapter = ScreenSlidePagerAdapter(this) | ||||
|         binding.pager.setCurrentItem(currentItem, false) | ||||
|   | ||||
| @@ -49,13 +49,13 @@ class SourcesActivity : AppCompatActivity(), DIAware { | ||||
|         super.onResume() | ||||
|         val mLayoutManager = LinearLayoutManager(this) | ||||
|  | ||||
|         var items: ArrayList<SelfossModel.Source> | ||||
|         var items: ArrayList<SelfossModel.SourceDetail> | ||||
|  | ||||
|         binding.recyclerView.setHasFixedSize(true) | ||||
|         binding.recyclerView.layoutManager = mLayoutManager | ||||
|  | ||||
|         CoroutineScope(Dispatchers.Main).launch { | ||||
|             val response = repository.getSources() | ||||
|             val response = repository.getSourcesDetails() | ||||
|             if (response.isNotEmpty()) { | ||||
|                 items = response | ||||
|                 val mAdapter = SourcesListAdapter( | ||||
|   | ||||
| @@ -24,7 +24,7 @@ import org.kodein.di.instance | ||||
|  | ||||
| class UpsertSourceActivity : AppCompatActivity(), DIAware { | ||||
|  | ||||
|     private var existingSource: SelfossModel.Source? = null | ||||
|     private var existingSource: SelfossModel.SourceDetail? = null | ||||
|     private var mSpoutsValue: String? = null | ||||
|  | ||||
|     private lateinit var binding: ActivityUpsertSourceBinding | ||||
| @@ -68,7 +68,7 @@ class UpsertSourceActivity : AppCompatActivity(), DIAware { | ||||
|  | ||||
|     private fun initFields(items: Map<String, SelfossModel.Spout>) { | ||||
|         binding.nameInput.setText(existingSource!!.title) | ||||
|         binding.tags.setText(existingSource!!.tags.joinToString(", ")) | ||||
|         binding.tags.setText(existingSource!!.tags?.joinToString(", ")) | ||||
|         binding.sourceUri.setText(existingSource!!.params?.url) | ||||
|         binding.spoutsSpinner.setSelection(items.keys.indexOf(existingSource!!.spout)) | ||||
|         binding.progress.visibility = View.GONE | ||||
|   | ||||
| @@ -9,10 +9,9 @@ import android.widget.ImageView.ScaleType | ||||
| import androidx.recyclerview.widget.RecyclerView | ||||
| import bou.amine.apps.readerforselfossv2.android.R | ||||
| import bou.amine.apps.readerforselfossv2.android.databinding.CardItemBinding | ||||
| import bou.amine.apps.readerforselfossv2.android.model.toTextDrawableString | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.LinkOnTouchListener | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.glide.bitmapCenterCrop | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.glide.circularBitmapDrawable | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.glide.circularDrawable | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.openInBrowserAsNewTask | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.openItemUrl | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.shareLink | ||||
| @@ -22,8 +21,6 @@ import bou.amine.apps.readerforselfossv2.service.AppSettingsService | ||||
| import bou.amine.apps.readerforselfossv2.utils.getHtmlDecoded | ||||
| import bou.amine.apps.readerforselfossv2.utils.getIcon | ||||
| import bou.amine.apps.readerforselfossv2.utils.getThumbnail | ||||
| import com.amulyakhare.textdrawable.TextDrawable | ||||
| import com.amulyakhare.textdrawable.util.ColorGenerator | ||||
| import com.bumptech.glide.Glide | ||||
| import kotlinx.coroutines.CoroutineScope | ||||
| import kotlinx.coroutines.Dispatchers | ||||
| @@ -38,7 +35,6 @@ class ItemCardAdapter( | ||||
|     override val updateItems: (ArrayList<SelfossModel.Item>) -> Unit | ||||
| ) : ItemsAdapter<ItemCardAdapter.ViewHolder>() { | ||||
|     private val c: Context = app.baseContext | ||||
|     private val generator: ColorGenerator = ColorGenerator.MATERIAL | ||||
|     private val imageMaxHeight: Int = | ||||
|         c.resources.getDimension(R.dimen.card_image_max_height).toInt() | ||||
|  | ||||
| @@ -83,16 +79,9 @@ class ItemCardAdapter( | ||||
|             } | ||||
|  | ||||
|             if (itm.getIcon(repository.baseUrl).isEmpty()) { | ||||
|                 val color = generator.getColor(itm.title.getHtmlDecoded()) | ||||
|  | ||||
|                 val drawable = | ||||
|                     TextDrawable | ||||
|                         .builder() | ||||
|                         .round() | ||||
|                         .build(itm.title.getHtmlDecoded().toTextDrawableString(), color) | ||||
|                 binding.sourceImage.setImageDrawable(drawable) | ||||
|                 binding.sourceImage.setBackgroundAndText(itm.sourcetitle.getHtmlDecoded()) | ||||
|             } else { | ||||
|                 c.circularBitmapDrawable(itm.getIcon(repository.baseUrl), binding.sourceImage) | ||||
|                 c.circularDrawable(itm.getIcon(repository.baseUrl), binding.sourceImage) | ||||
|             } | ||||
|         } | ||||
|     } | ||||
|   | ||||
| @@ -7,10 +7,8 @@ import android.view.ViewGroup | ||||
| import androidx.recyclerview.widget.RecyclerView | ||||
| import bou.amine.apps.readerforselfossv2.android.R | ||||
| import bou.amine.apps.readerforselfossv2.android.databinding.ListItemBinding | ||||
| import bou.amine.apps.readerforselfossv2.android.model.toTextDrawableString | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.LinkOnTouchListener | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.glide.bitmapCenterCrop | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.glide.circularBitmapDrawable | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.glide.circularDrawable | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.openItemUrl | ||||
| import bou.amine.apps.readerforselfossv2.model.SelfossModel | ||||
| import bou.amine.apps.readerforselfossv2.repository.Repository | ||||
| @@ -18,8 +16,6 @@ import bou.amine.apps.readerforselfossv2.service.AppSettingsService | ||||
| import bou.amine.apps.readerforselfossv2.utils.getHtmlDecoded | ||||
| import bou.amine.apps.readerforselfossv2.utils.getIcon | ||||
| import bou.amine.apps.readerforselfossv2.utils.getThumbnail | ||||
| import com.amulyakhare.textdrawable.TextDrawable | ||||
| import com.amulyakhare.textdrawable.util.ColorGenerator | ||||
| import org.kodein.di.DI | ||||
| import org.kodein.di.android.closestDI | ||||
| import org.kodein.di.instance | ||||
| @@ -29,7 +25,6 @@ class ItemListAdapter( | ||||
|     override var items: ArrayList<SelfossModel.Item>, | ||||
|     override val updateItems: (ArrayList<SelfossModel.Item>) -> Unit | ||||
| ) : ItemsAdapter<ItemListAdapter.ViewHolder>() { | ||||
|     private val generator: ColorGenerator = ColorGenerator.MATERIAL | ||||
|     private val c: Context = app.baseContext | ||||
|  | ||||
|     override val di: DI by closestDI(app) | ||||
| @@ -56,20 +51,12 @@ class ItemListAdapter( | ||||
|             if (itm.getThumbnail(repository.baseUrl).isEmpty()) { | ||||
|  | ||||
|                 if (itm.getIcon(repository.baseUrl).isEmpty()) { | ||||
|                     val color = generator.getColor(itm.title.getHtmlDecoded()) | ||||
|  | ||||
|                     val drawable = | ||||
|                             TextDrawable | ||||
|                                     .builder() | ||||
|                                     .round() | ||||
|                                     .build(itm.title.getHtmlDecoded().toTextDrawableString(), color) | ||||
|  | ||||
|                     binding.itemImage.setImageDrawable(drawable) | ||||
|                     binding.itemImage.setBackgroundAndText(itm.sourcetitle.getHtmlDecoded()) | ||||
|                 } else { | ||||
|                     c.circularBitmapDrawable(itm.getIcon(repository.baseUrl), binding.itemImage) | ||||
|                     c.circularDrawable(itm.getIcon(repository.baseUrl), binding.itemImage) | ||||
|                 } | ||||
|             } else { | ||||
|                 c.bitmapCenterCrop(itm.getThumbnail(repository.baseUrl), binding.itemImage) | ||||
|                 c.circularDrawable(itm.getThumbnail(repository.baseUrl), binding.itemImage) | ||||
|             } | ||||
|         } | ||||
|     } | ||||
|   | ||||
| @@ -10,17 +10,14 @@ import android.widget.Button | ||||
| import android.widget.Toast | ||||
| import androidx.constraintlayout.widget.ConstraintLayout | ||||
| import androidx.recyclerview.widget.RecyclerView | ||||
| import bou.amine.apps.readerforselfossv2.android.UpsertSourceActivity | ||||
| import bou.amine.apps.readerforselfossv2.android.R | ||||
| import bou.amine.apps.readerforselfossv2.android.UpsertSourceActivity | ||||
| import bou.amine.apps.readerforselfossv2.android.databinding.SourceListItemBinding | ||||
| import bou.amine.apps.readerforselfossv2.android.model.toTextDrawableString | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.glide.circularBitmapDrawable | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.glide.circularDrawable | ||||
| import bou.amine.apps.readerforselfossv2.model.SelfossModel | ||||
| import bou.amine.apps.readerforselfossv2.repository.Repository | ||||
| import bou.amine.apps.readerforselfossv2.utils.getHtmlDecoded | ||||
| import bou.amine.apps.readerforselfossv2.utils.getIcon | ||||
| import com.amulyakhare.textdrawable.TextDrawable | ||||
| import com.amulyakhare.textdrawable.util.ColorGenerator | ||||
| import kotlinx.coroutines.CoroutineScope | ||||
| import kotlinx.coroutines.Dispatchers | ||||
| import kotlinx.coroutines.launch | ||||
| @@ -31,10 +28,9 @@ import org.kodein.di.instance | ||||
|  | ||||
| class SourcesListAdapter( | ||||
|     private val app: Activity, | ||||
|     private val items: ArrayList<SelfossModel.Source> | ||||
|     private val items: ArrayList<SelfossModel.SourceDetail> | ||||
| ) : RecyclerView.Adapter<SourcesListAdapter.ViewHolder>(), DIAware { | ||||
|     private val c: Context = app.baseContext | ||||
|     private val generator: ColorGenerator = ColorGenerator.MATERIAL | ||||
|     private lateinit var binding: SourceListItemBinding | ||||
|  | ||||
|     override val di: DI by closestDI(app) | ||||
| @@ -49,19 +45,12 @@ class SourcesListAdapter( | ||||
|         val itm = items[position] | ||||
|  | ||||
|         if (itm.getIcon(repository.baseUrl).isEmpty()) { | ||||
|             val color = generator.getColor(itm.title.getHtmlDecoded()) | ||||
|  | ||||
|             val drawable = | ||||
|                 TextDrawable | ||||
|                     .builder() | ||||
|                     .round() | ||||
|                     .build(itm.title.getHtmlDecoded().toTextDrawableString(), color) | ||||
|             binding.itemImage.setImageDrawable(drawable) | ||||
|             binding.itemImage.setBackgroundAndText(itm.title.getHtmlDecoded()) | ||||
|         } else { | ||||
|             c.circularBitmapDrawable(itm.getIcon(repository.baseUrl), binding.itemImage) | ||||
|             c.circularDrawable(itm.getIcon(repository.baseUrl), binding.itemImage) | ||||
|         } | ||||
|  | ||||
|         if (itm.error.isNotBlank()) { | ||||
|         if (!itm.error.isNullOrBlank()) { | ||||
|             binding.errorText.visibility = View.VISIBLE | ||||
|             binding.errorText.text = itm.error | ||||
|         } else { | ||||
|   | ||||
| @@ -1,5 +1,6 @@ | ||||
| package bou.amine.apps.readerforselfossv2.android.fragments | ||||
|  | ||||
| import android.content.ActivityNotFoundException | ||||
| import android.content.Intent | ||||
| import android.content.res.ColorStateList | ||||
| import android.content.res.TypedArray | ||||
| @@ -26,11 +27,11 @@ import bou.amine.apps.readerforselfossv2.android.model.toModel | ||||
| import bou.amine.apps.readerforselfossv2.android.model.toParcelable | ||||
| import bou.amine.apps.readerforselfossv2.android.sendSilentlyWithAcraWithName | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.glide.getBitmapInputStream | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.isUrlValid | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.openInBrowserAsNewTask | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.shareLink | ||||
| import bou.amine.apps.readerforselfossv2.model.MercuryModel | ||||
| import bou.amine.apps.readerforselfossv2.model.SelfossModel | ||||
| import bou.amine.apps.readerforselfossv2.model.StatusAndData | ||||
| import bou.amine.apps.readerforselfossv2.repository.Repository | ||||
| import bou.amine.apps.readerforselfossv2.rest.MercuryApi | ||||
| import bou.amine.apps.readerforselfossv2.service.AppSettingsService | ||||
| @@ -150,17 +151,19 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|  | ||||
|         } catch (e: InflateException) { | ||||
|             e.sendSilentlyWithAcraWithName("webview not available") | ||||
|             AlertDialog.Builder(requireContext()) | ||||
|                 .setMessage(requireContext().getString(R.string.webview_dialog_issue_message)) | ||||
|                 .setTitle(requireContext().getString(R.string.webview_dialog_issue_title)) | ||||
|                 .setPositiveButton( | ||||
|                     android.R.string.ok | ||||
|                 ) { _, _ -> | ||||
|                     appSettingsService.disableArticleViewer() | ||||
|                     requireActivity().finish() | ||||
|                 } | ||||
|                 .create() | ||||
|                 .show() | ||||
|             if (context != null) { | ||||
|                 AlertDialog.Builder(requireContext()) | ||||
|                     .setMessage(requireContext().getString(R.string.webview_dialog_issue_message)) | ||||
|                     .setTitle(requireContext().getString(R.string.webview_dialog_issue_title)) | ||||
|                     .setPositiveButton( | ||||
|                         android.R.string.ok | ||||
|                     ) { _, _ -> | ||||
|                         appSettingsService.disableArticleViewer() | ||||
|                         requireActivity().finish() | ||||
|                     } | ||||
|                     .create() | ||||
|                     .show() | ||||
|             } | ||||
|         } | ||||
|  | ||||
|         return binding.root | ||||
| @@ -257,21 +260,8 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|         CoroutineScope(Dispatchers.Main).launch { | ||||
|             try { | ||||
|                 val response = mercuryApi.query(url) | ||||
|                 if (response.success && response.data != null && !response.data?.content.isNullOrEmpty()) { | ||||
|                     binding.titleView.text = response.data!!.title.orEmpty() | ||||
|                     if (typeface != null) { | ||||
|                         binding.titleView.typeface = typeface | ||||
|                     } | ||||
|                     URL(response.data!!.url) | ||||
|                     url = response.data!!.url | ||||
|  | ||||
|                     contentText = response.data!!.content.orEmpty() | ||||
|                     htmlToWebview() | ||||
|  | ||||
|                     handleLeadImage(response) | ||||
|  | ||||
|                     binding.nestedScrollView.scrollTo(0, 0) | ||||
|                     binding.progressBar.visibility = View.GONE | ||||
|                 if (response.success && response.data != null) { | ||||
|                     handleMercuryData(response.data!!) | ||||
|                 } else { | ||||
|                     openInBrowserAfterFailing() | ||||
|                 } | ||||
| @@ -284,14 +274,35 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|         } | ||||
|     } | ||||
|  | ||||
|     private fun handleLeadImage(response: StatusAndData<MercuryModel.ParsedContent>) { | ||||
|         if (!response.data?.lead_image_url.isNullOrEmpty() && context != null) { | ||||
|     private fun handleMercuryData(data: MercuryModel.ParsedContent) { | ||||
|         if (data.error == true || data.failed == true) { | ||||
|             openInBrowserAfterFailing() | ||||
|         } else { | ||||
|             binding.titleView.text = data.title.orEmpty() | ||||
|             if (typeface != null) { | ||||
|                 binding.titleView.typeface = typeface | ||||
|             } | ||||
|             URL(data.url) | ||||
|             url = data.url!! | ||||
|  | ||||
|             contentText = data.content.orEmpty() | ||||
|             htmlToWebview() | ||||
|  | ||||
|             handleLeadImage(data?.lead_image_url) | ||||
|  | ||||
|             binding.nestedScrollView.scrollTo(0, 0) | ||||
|             binding.progressBar.visibility = View.GONE | ||||
|         } | ||||
|     } | ||||
|  | ||||
|     private fun handleLeadImage(lead_image_url: String?) { | ||||
|         if (!lead_image_url.isNullOrEmpty() && context != null) { | ||||
|             binding.imageView.visibility = View.VISIBLE | ||||
|             Glide | ||||
|                 .with(requireContext()) | ||||
|                 .asBitmap() | ||||
|                 .load( | ||||
|                     response.data!!.lead_image_url.orEmpty() | ||||
|                     lead_image_url | ||||
|                 ) | ||||
|                 .apply(RequestOptions.fitCenterTransform()) | ||||
|                 .into(binding.imageView) | ||||
| @@ -304,10 +315,16 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|         binding.webcontent.webViewClient = object : WebViewClient() { | ||||
|             @Deprecated("Deprecated in Java") | ||||
|             override fun shouldOverrideUrlLoading(view: WebView?, url: String): Boolean { | ||||
|                 if (binding.webcontent.hitTestResult.type != WebView.HitTestResult.SRC_IMAGE_ANCHOR_TYPE) { | ||||
|                     requireContext().startActivity(Intent(Intent.ACTION_VIEW, Uri.parse(url))) | ||||
|                 return if (context != null && url.isUrlValid() && binding.webcontent.hitTestResult.type != WebView.HitTestResult.SRC_IMAGE_ANCHOR_TYPE) { | ||||
|                     try { | ||||
|                         requireContext().startActivity(Intent(Intent.ACTION_VIEW, Uri.parse(url))) | ||||
|                     } catch (e: ActivityNotFoundException) { | ||||
|                         e.sendSilentlyWithAcraWithName("activityNotFound > $url") | ||||
|                     } | ||||
|                     true | ||||
|                 } else { | ||||
|                     false | ||||
|                 } | ||||
|                 return true | ||||
|             } | ||||
|  | ||||
|             @Deprecated("Deprecated in Java") | ||||
| @@ -325,7 +342,7 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|                             getBitmapInputStream(image, Bitmap.CompressFormat.JPEG) | ||||
|                         ) | ||||
|                     } catch (e: ExecutionException) { | ||||
|                         e.sendSilentlyWithAcraWithName("shouldInterceptRequest > jpeg > $url") | ||||
|                         // Do nothing | ||||
|                     } | ||||
|                 } else if (url.lowercase(Locale.US).contains(".png")) { | ||||
|                     try { | ||||
| @@ -337,7 +354,7 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|                             getBitmapInputStream(image, Bitmap.CompressFormat.PNG) | ||||
|                         ) | ||||
|                     } catch (e: ExecutionException) { | ||||
|                         e.sendSilentlyWithAcraWithName("shouldInterceptRequest > png > $url") | ||||
|                         // Do nothing | ||||
|                     } | ||||
|                 } else if (url.lowercase(Locale.US).contains(".webp")) { | ||||
|                     try { | ||||
| @@ -349,7 +366,7 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|                             getBitmapInputStream(image, Bitmap.CompressFormat.WEBP) | ||||
|                         ) | ||||
|                     } catch (e: ExecutionException) { | ||||
|                         e.sendSilentlyWithAcraWithName("shouldInterceptRequest > webp > $url") | ||||
|                         // Do nothing | ||||
|                     } | ||||
|                 } | ||||
|  | ||||
| @@ -359,74 +376,74 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|     } | ||||
|  | ||||
|     private fun htmlToWebview() { | ||||
|  | ||||
|         val attrs: IntArray = intArrayOf(android.R.attr.fontFamily) | ||||
|         val a: TypedArray = requireContext().obtainStyledAttributes(resId, attrs) | ||||
|         if (context != null) { | ||||
|             val attrs: IntArray = intArrayOf(android.R.attr.fontFamily) | ||||
|             val a: TypedArray = requireContext().obtainStyledAttributes(resId, attrs) | ||||
|  | ||||
|  | ||||
|         binding.webcontent.settings.standardFontFamily = a.getString(0) | ||||
|         binding.webcontent.visibility = View.VISIBLE | ||||
|             binding.webcontent.settings.standardFontFamily = a.getString(0) | ||||
|             binding.webcontent.visibility = View.VISIBLE | ||||
|  | ||||
|         val colorOnSurface = TypedValue() | ||||
|         requireContext().theme.resolveAttribute(R.attr.colorOnSurface, colorOnSurface, true) | ||||
|             val colorOnSurface = TypedValue() | ||||
|             requireContext().theme.resolveAttribute(R.attr.colorOnSurface, colorOnSurface, true) | ||||
|  | ||||
|         val colorSurface = TypedValue() | ||||
|         requireContext().theme.resolveAttribute(R.attr.colorSurface, colorSurface, true) | ||||
|             val colorSurface = TypedValue() | ||||
|             requireContext().theme.resolveAttribute(R.attr.colorSurface, colorSurface, true) | ||||
|  | ||||
|         binding.webcontent.settings.useWideViewPort = true | ||||
|         binding.webcontent.settings.loadWithOverviewMode = true | ||||
|         binding.webcontent.settings.javaScriptEnabled = false | ||||
|             binding.webcontent.settings.useWideViewPort = true | ||||
|             binding.webcontent.settings.loadWithOverviewMode = true | ||||
|             binding.webcontent.settings.javaScriptEnabled = false | ||||
|  | ||||
|         handleImageLoading() | ||||
|             handleImageLoading() | ||||
|  | ||||
|         val gestureDetector = | ||||
|             GestureDetector(activity, object : GestureDetector.SimpleOnGestureListener() { | ||||
|                 override fun onSingleTapUp(e: MotionEvent): Boolean { | ||||
|                     return performClick() | ||||
|                 } | ||||
|             }) | ||||
|             val gestureDetector = | ||||
|                 GestureDetector(activity, object : GestureDetector.SimpleOnGestureListener() { | ||||
|                     override fun onSingleTapUp(e: MotionEvent): Boolean { | ||||
|                         return performClick() | ||||
|                     } | ||||
|                 }) | ||||
|  | ||||
|         binding.webcontent.setOnTouchListener { _, event -> gestureDetector.onTouchEvent(event) } | ||||
|             binding.webcontent.setOnTouchListener { _, event -> gestureDetector.onTouchEvent(event) } | ||||
|  | ||||
|         binding.webcontent.settings.layoutAlgorithm = | ||||
|             WebSettings.LayoutAlgorithm.TEXT_AUTOSIZING | ||||
|             binding.webcontent.settings.layoutAlgorithm = | ||||
|                 WebSettings.LayoutAlgorithm.TEXT_AUTOSIZING | ||||
|  | ||||
|         var baseUrl: String? = null | ||||
|             var baseUrl: String? = null | ||||
|  | ||||
|         try { | ||||
|             val itemUrl = URL(url) | ||||
|             baseUrl = itemUrl.protocol + "://" + itemUrl.host | ||||
|         } catch (e: MalformedURLException) { | ||||
|             e.sendSilentlyWithAcraWithName("htmlToWebview > item url") | ||||
|         } | ||||
|             try { | ||||
|                 val itemUrl = URL(url) | ||||
|                 baseUrl = itemUrl.protocol + "://" + itemUrl.host | ||||
|             } catch (e: MalformedURLException) { | ||||
|                 e.sendSilentlyWithAcraWithName("htmlToWebview > $url") | ||||
|             } | ||||
|  | ||||
|         val fontName = when (font) { | ||||
|             getString(R.string.open_sans_font_id) -> "Open Sans" | ||||
|             getString(R.string.roboto_font_id) -> "Roboto" | ||||
|             getString(R.string.source_code_pro_font_id) -> "Source Code Pro" | ||||
|             else -> "" | ||||
|         } | ||||
|             val fontName = when (font) { | ||||
|                 getString(R.string.open_sans_font_id) -> "Open Sans" | ||||
|                 getString(R.string.roboto_font_id) -> "Roboto" | ||||
|                 getString(R.string.source_code_pro_font_id) -> "Source Code Pro" | ||||
|                 else -> "" | ||||
|             } | ||||
|  | ||||
|         val fontLinkAndStyle = if (font.isNotEmpty()) { | ||||
|             """<link href="https://fonts.googleapis.com/css?family=${ | ||||
|                 fontName.replace( | ||||
|                     " ", | ||||
|                     "+" | ||||
|                 ) | ||||
|             }" rel="stylesheet"> | ||||
|             val fontLinkAndStyle = if (font.isNotEmpty()) { | ||||
|                 """<link href="https://fonts.googleapis.com/css?family=${ | ||||
|                     fontName.replace( | ||||
|                         " ", | ||||
|                         "+" | ||||
|                     ) | ||||
|                 }" rel="stylesheet"> | ||||
|                 |<style> | ||||
|                 |   * { | ||||
|                 |       font-family: '$fontName'; | ||||
|                 |   } | ||||
|                 |</style> | ||||
|             """.trimMargin() | ||||
|         } else { | ||||
|             "" | ||||
|         } | ||||
|             } else { | ||||
|                 "" | ||||
|             } | ||||
|  | ||||
|         binding.webcontent.loadDataWithBaseURL( | ||||
|             baseUrl, | ||||
|             """<html> | ||||
|             binding.webcontent.loadDataWithBaseURL( | ||||
|                 baseUrl, | ||||
|                 """<html> | ||||
|                 |<head> | ||||
|                 |   <meta name="viewport" content="width=device-width, initial-scale=1"> | ||||
|                 |   <style> | ||||
| @@ -438,11 +455,11 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|                 |      } | ||||
|                 |      a { | ||||
|                 |        color: ${ | ||||
|                 String.format( | ||||
|                     "#%06X", | ||||
|                     0xFFFFFF and resources.getColor(R.color.colorAccent) | ||||
|                 ) | ||||
|             } !important; | ||||
|                     String.format( | ||||
|                         "#%06X", | ||||
|                         0xFFFFFF and resources.getColor(R.color.colorAccent) | ||||
|                     ) | ||||
|                 } !important; | ||||
|                 |      } | ||||
|                 |      *:not(a) { | ||||
|                 |        color: ${String.format("#%06X", 0xFFFFFF and colorOnSurface.data)}; | ||||
| @@ -454,25 +471,25 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|                 |        overflow:hidden; | ||||
|                 |        line-height: 1.5em; | ||||
|                 |        background-color: ${ | ||||
|                 String.format( | ||||
|                     "#%06X", | ||||
|                     0xFFFFFF and colorSurface.data | ||||
|                 ) | ||||
|             }; | ||||
|                     String.format( | ||||
|                         "#%06X", | ||||
|                         0xFFFFFF and colorSurface.data | ||||
|                     ) | ||||
|                 }; | ||||
|                 |      } | ||||
|                 |      body, html { | ||||
|                 |        background-color: ${ | ||||
|                 String.format( | ||||
|                     "#%06X", | ||||
|                     0xFFFFFF and colorSurface.data | ||||
|                 ) | ||||
|             } !important; | ||||
|                     String.format( | ||||
|                         "#%06X", | ||||
|                         0xFFFFFF and colorSurface.data | ||||
|                     ) | ||||
|                 } !important; | ||||
|                 |        border-color: ${ | ||||
|                 String.format( | ||||
|                     "#%06X", | ||||
|                     0xFFFFFF and colorSurface.data | ||||
|                 ) | ||||
|             }  !important; | ||||
|                     String.format( | ||||
|                         "#%06X", | ||||
|                         0xFFFFFF and colorSurface.data | ||||
|                     ) | ||||
|                 }  !important; | ||||
|                 |        padding: 0 !important; | ||||
|                 |        margin: 0 !important; | ||||
|                 |      } | ||||
| @@ -483,11 +500,11 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|                 |        white-space: pre-wrap; | ||||
|                 |        width:100%; | ||||
|                 |        background-color: ${ | ||||
|                 String.format( | ||||
|                     "#%06X", | ||||
|                     0xFFFFFF and colorSurface.data | ||||
|                 ) | ||||
|             }; | ||||
|                     String.format( | ||||
|                         "#%06X", | ||||
|                         0xFFFFFF and colorSurface.data | ||||
|                     ) | ||||
|                 }; | ||||
|                 |      } | ||||
|                 |   </style> | ||||
|                 |   $fontLinkAndStyle | ||||
| @@ -495,10 +512,11 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|                 |<body> | ||||
|                 |   $contentText | ||||
|                 |</body>""".trimMargin(), | ||||
|             "text/html", | ||||
|             "utf-8", | ||||
|             null | ||||
|         ) | ||||
|                 "text/html", | ||||
|                 "utf-8", | ||||
|                 null | ||||
|             ) | ||||
|         } | ||||
|     } | ||||
|  | ||||
|     fun scrollDown() { | ||||
| @@ -513,7 +531,12 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|  | ||||
|     private fun openInBrowserAfterFailing() { | ||||
|         binding.progressBar.visibility = View.GONE | ||||
|         requireActivity().openInBrowserAsNewTask(this@ArticleFragment.item) | ||||
|         if (context != null) { | ||||
|             requireContext().openInBrowserAsNewTask(this@ArticleFragment.item) | ||||
|         } else { | ||||
|             Exception("openInBrowserAfterFailing context is null").sendSilentlyWithAcraWithName("openInBrowserAfterFailing > $context") | ||||
|  | ||||
|         } | ||||
|     } | ||||
|  | ||||
|     companion object { | ||||
| @@ -531,8 +554,8 @@ class ArticleFragment : Fragment(), DIAware { | ||||
|     } | ||||
|  | ||||
|     fun performClick(): Boolean { | ||||
|         if (binding.webcontent.hitTestResult.type == WebView.HitTestResult.IMAGE_TYPE || | ||||
|             binding.webcontent.hitTestResult.type == WebView.HitTestResult.SRC_IMAGE_ANCHOR_TYPE | ||||
|         if (allImages != null && (binding.webcontent.hitTestResult.type == WebView.HitTestResult.IMAGE_TYPE || | ||||
|             binding.webcontent.hitTestResult.type == WebView.HitTestResult.SRC_IMAGE_ANCHOR_TYPE) | ||||
|         ) { | ||||
|  | ||||
|             val position: Int = allImages.indexOf(binding.webcontent.hitTestResult.extra) | ||||
|   | ||||
| @@ -20,10 +20,8 @@ import bou.amine.apps.readerforselfossv2.repository.Repository | ||||
| import bou.amine.apps.readerforselfossv2.utils.getHtmlDecoded | ||||
| import bou.amine.apps.readerforselfossv2.utils.getIcon | ||||
| import com.bumptech.glide.Glide | ||||
| import com.bumptech.glide.load.DataSource | ||||
| import com.bumptech.glide.load.engine.GlideException | ||||
| import com.bumptech.glide.request.RequestListener | ||||
| import com.bumptech.glide.request.target.Target | ||||
| import com.bumptech.glide.request.target.ViewTarget | ||||
| import com.bumptech.glide.request.transition.Transition | ||||
| import com.google.android.material.bottomsheet.BottomSheetDialogFragment | ||||
| import com.google.android.material.chip.Chip | ||||
| import kotlinx.coroutines.CoroutineScope | ||||
| @@ -84,37 +82,25 @@ class FilterSheetFragment : BottomSheetDialogFragment(), DIAware { | ||||
|     ) { | ||||
|         val sourceGroup = binding.sourcesGroup | ||||
|  | ||||
|         repository.getSources().forEach { source -> | ||||
|         repository.getSourcesDetailsOrStats().forEachIndexed { _, source -> | ||||
|             val c = Chip(context) | ||||
|             c.ellipsize = TextUtils.TruncateAt.END | ||||
|  | ||||
|             Glide.with(context) | ||||
|                 .load(source.getIcon(repository.baseUrl)) | ||||
|                 .listener(object : RequestListener<Drawable?> { | ||||
|                     override fun onLoadFailed( | ||||
|                         e: GlideException?, | ||||
|                         model: Any?, | ||||
|                         target: Target<Drawable?>?, | ||||
|                         isFirstResource: Boolean | ||||
|                     ): Boolean { | ||||
|                         return false | ||||
|                     } | ||||
|  | ||||
|                 .into(object : ViewTarget<Chip?, Drawable?>(c) { | ||||
|                     override fun onResourceReady( | ||||
|                         resource: Drawable?, | ||||
|                         model: Any?, | ||||
|                         target: Target<Drawable?>?, | ||||
|                         dataSource: DataSource?, | ||||
|                         isFirstResource: Boolean | ||||
|                     ): Boolean { | ||||
|                         resource: Drawable, | ||||
|                         transition: Transition<in Drawable?>? | ||||
|                     ) { | ||||
|                         try { | ||||
|                             c.chipIcon = resource | ||||
|                         } catch (e: Exception) { | ||||
|                             e.sendSilentlyWithAcraWithName("sources > onResourceReady") | ||||
|                         } | ||||
|                         return false | ||||
|                     } | ||||
|                 }).preload() | ||||
|  | ||||
|                 }) | ||||
|  | ||||
|             c.text = source.title.getHtmlDecoded() | ||||
|  | ||||
| @@ -141,9 +127,9 @@ class FilterSheetFragment : BottomSheetDialogFragment(), DIAware { | ||||
|                 selectedChip = c | ||||
|             } | ||||
|  | ||||
|             c.isEnabled = source.error.isBlank() | ||||
|             c.isEnabled = source.error.isNullOrBlank() | ||||
|  | ||||
|             if (source.error.isNotBlank() && Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) { | ||||
|             if (!source.error.isNullOrBlank() && Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) { | ||||
|                 c.tooltipText = source.error | ||||
|             } | ||||
|  | ||||
| @@ -158,26 +144,28 @@ class FilterSheetFragment : BottomSheetDialogFragment(), DIAware { | ||||
|  | ||||
|         val tags = repository.getTags() | ||||
|  | ||||
|         tags.forEach { tag -> | ||||
|         tags.forEachIndexed { _, tag -> | ||||
|             val c = Chip(context) | ||||
|             c.ellipsize = TextUtils.TruncateAt.END | ||||
|             c.text = tag.tag | ||||
|  | ||||
|             try { | ||||
|                 val gd = GradientDrawable() | ||||
|                 val gdColor = try { | ||||
|                     Color.parseColor(tag.color) | ||||
|                 } catch (e: IllegalArgumentException) { | ||||
|                     e.sendSilentlyWithAcraWithName("color issue " + tag.color) | ||||
|                     resources.getColor(R.color.colorPrimary) | ||||
|             if (tag.color.isNotEmpty()) { | ||||
|                 try { | ||||
|                     val gd = GradientDrawable() | ||||
|                     val gdColor = try { | ||||
|                         Color.parseColor(tag.color) | ||||
|                     } catch (e: IllegalArgumentException) { | ||||
|                         e.sendSilentlyWithAcraWithName("color issue " + tag.color) | ||||
|                         resources.getColor(R.color.colorPrimary) | ||||
|                     } | ||||
|                     gd.setColor(gdColor) | ||||
|                     gd.shape = GradientDrawable.RECTANGLE | ||||
|                     gd.setSize(30, 30) | ||||
|                     gd.cornerRadius = 30F | ||||
|                     c.chipIcon = gd | ||||
|                 } catch (e: Exception) { | ||||
|                     e.sendSilentlyWithAcraWithName("tags > GradientDrawable") | ||||
|                 } | ||||
|                 gd.setColor(gdColor) | ||||
|                 gd.shape = GradientDrawable.RECTANGLE | ||||
|                 gd.setSize(30, 30) | ||||
|                 gd.cornerRadius = 30F | ||||
|                 c.chipIcon = gd | ||||
|             } catch (e: Exception) { | ||||
|                 e.sendSilentlyWithAcraWithName("tags > GradientDrawable") | ||||
|             } | ||||
|  | ||||
|             c.setOnCloseIconClickListener { | ||||
|   | ||||
| @@ -78,7 +78,7 @@ class SettingsActivity : AppCompatActivity(), | ||||
|         val args = pref.extras | ||||
|         val fragment = supportFragmentManager.fragmentFactory.instantiate( | ||||
|                 classLoader, | ||||
|                 pref.fragment | ||||
|             pref.fragment.toString() | ||||
|         ).apply { | ||||
|             arguments = args | ||||
|             setTargetFragment(caller, 0) | ||||
|   | ||||
| @@ -0,0 +1,62 @@ | ||||
| package bou.amine.apps.readerforselfossv2.android.utils | ||||
|  | ||||
| import android.content.Context | ||||
| import android.graphics.drawable.GradientDrawable | ||||
| import android.util.AttributeSet | ||||
| import android.view.LayoutInflater | ||||
| import android.view.View | ||||
| import android.widget.RelativeLayout | ||||
| import android.widget.TextView | ||||
| import bou.amine.apps.readerforselfossv2.android.R | ||||
| import bou.amine.apps.readerforselfossv2.android.model.toTextDrawableString | ||||
| import com.google.android.material.imageview.ShapeableImageView | ||||
| import kotlin.math.abs | ||||
|  | ||||
| class CircleImageView @JvmOverloads constructor( | ||||
|     context: Context, | ||||
|     attrs: AttributeSet? = null, | ||||
|     defStyleAttr: Int = 0 | ||||
| ) : RelativeLayout(context, attrs, defStyleAttr) { | ||||
|     val view: View | ||||
|     val imageView: ShapeableImageView | ||||
|     val textView: TextView | ||||
|  | ||||
|     private val colorScheme = listOf( | ||||
|     -0x1a8c8d, | ||||
|     -0xf9d6e, | ||||
|     -0x459738, | ||||
|     -0x6a8a33, | ||||
|     -0x867935, | ||||
|     -0x9b4a0a, | ||||
|     -0xb03c09, | ||||
|     -0xb22f1f, | ||||
|     -0xb24954, | ||||
|     -0x7e387c, | ||||
|     -0x512a7f, | ||||
|     -0x759b, | ||||
|     -0x2b1ea9, | ||||
|     -0x2ab1, | ||||
|     -0x48b3, | ||||
|     -0x5e7781, | ||||
|     -0x6f5b52 | ||||
|     ) | ||||
|  | ||||
|     init { | ||||
|         view = LayoutInflater.from(context).inflate(R.layout.circle_image_view, this, true) | ||||
|         imageView = view.findViewById(R.id.circleImage) | ||||
|         textView = view.findViewById(R.id.circleText) | ||||
|     } | ||||
|  | ||||
|     fun setBackgroundAndText(text: String) { | ||||
|         val circleDrawable = GradientDrawable() | ||||
|         val color = colorFromIdentifier(text) | ||||
|         circleDrawable.setColor(color) | ||||
|         imageView.setImageDrawable(circleDrawable) | ||||
|  | ||||
|         textView.text = text.toTextDrawableString() | ||||
|     } | ||||
|  | ||||
|     private fun colorFromIdentifier(key: String): Int { | ||||
|         return colorScheme[abs(key.hashCode()) % colorScheme.size] | ||||
|     } | ||||
| } | ||||
| @@ -3,10 +3,9 @@ package bou.amine.apps.readerforselfossv2.android.utils.glide | ||||
| import android.content.Context | ||||
| import android.graphics.Bitmap | ||||
| import android.widget.ImageView | ||||
| import androidx.core.graphics.drawable.RoundedBitmapDrawableFactory | ||||
| import bou.amine.apps.readerforselfossv2.android.utils.CircleImageView | ||||
| import com.bumptech.glide.Glide | ||||
| import com.bumptech.glide.request.RequestOptions | ||||
| import com.bumptech.glide.request.target.BitmapImageViewTarget | ||||
| import java.io.ByteArrayInputStream | ||||
| import java.io.ByteArrayOutputStream | ||||
| import java.io.InputStream | ||||
| @@ -18,21 +17,13 @@ fun Context.bitmapCenterCrop(url: String, iv: ImageView) = | ||||
|         .apply(RequestOptions.centerCropTransform()) | ||||
|         .into(iv) | ||||
|  | ||||
| fun Context.circularBitmapDrawable(url: String, iv: ImageView) = | ||||
| fun Context.circularDrawable(url: String, view: CircleImageView) { | ||||
|     view.textView.text ="" | ||||
|  | ||||
|     Glide.with(this) | ||||
|         .asBitmap() | ||||
|         .load(url) | ||||
|         .apply(RequestOptions.centerCropTransform()) | ||||
|         .into(object : BitmapImageViewTarget(iv) { | ||||
|             override fun setResource(resource: Bitmap?) { | ||||
|                 val circularBitmapDrawable = RoundedBitmapDrawableFactory.create( | ||||
|                     resources, | ||||
|                     resource | ||||
|                 ) | ||||
|                 circularBitmapDrawable.isCircular = true | ||||
|                 iv.setImageDrawable(circularBitmapDrawable) | ||||
|             } | ||||
|         }) | ||||
|         .into(view.imageView) | ||||
| } | ||||
|  | ||||
| fun getBitmapInputStream(bitmap:Bitmap,compressFormat: Bitmap.CompressFormat): InputStream { | ||||
|     val byteArrayOutputStream = ByteArrayOutputStream() | ||||
|   | ||||
| @@ -51,6 +51,13 @@ | ||||
|                 android:maxLines="1" | ||||
|                 android:minHeight="48dp" /> | ||||
|  | ||||
|             <com.google.android.material.switchmaterial.SwitchMaterial | ||||
|                 android:id="@+id/selfSigned" | ||||
|                 android:layout_width="match_parent" | ||||
|                 android:layout_height="wrap_content" | ||||
|                 android:text="@string/disable_ssl" | ||||
|                 android:textAlignment="viewStart" /> | ||||
|  | ||||
|             <com.google.android.material.switchmaterial.SwitchMaterial | ||||
|                 android:id="@+id/withLogin" | ||||
|                 android:layout_width="match_parent" | ||||
|   | ||||
| @@ -39,7 +39,7 @@ | ||||
|             app:layout_constraintStart_toStartOf="parent" | ||||
|             app:layout_constraintTop_toBottomOf="@+id/itemImage"> | ||||
|  | ||||
|             <ImageView | ||||
|             <bou.amine.apps.readerforselfossv2.android.utils.CircleImageView | ||||
|                 android:id="@+id/sourceImage" | ||||
|                 android:layout_width="40dp" | ||||
|                 android:layout_height="40dp" | ||||
|   | ||||
							
								
								
									
										26
									
								
								androidApp/src/main/res/layout/circle_image_view.xml
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										26
									
								
								androidApp/src/main/res/layout/circle_image_view.xml
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,26 @@ | ||||
| <?xml version="1.0" encoding="utf-8"?> | ||||
| <RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android" | ||||
|     xmlns:app="http://schemas.android.com/apk/res-auto" | ||||
|     android:layout_width="wrap_content" | ||||
|     android:layout_height="wrap_content"> | ||||
|  | ||||
|     <com.google.android.material.imageview.ShapeableImageView | ||||
|         android:id="@+id/circleImage" | ||||
|         android:layout_width="match_parent" | ||||
|         android:layout_height="match_parent" | ||||
|         android:scaleType="centerCrop" | ||||
|         app:shapeAppearanceOverlay="@style/circleImageView" | ||||
|         app:srcCompat="@drawable/background_splash" /> | ||||
|  | ||||
|     <TextView | ||||
|         android:id="@+id/circleText" | ||||
|         android:layout_width="match_parent" | ||||
|         android:layout_height="match_parent" | ||||
|         android:ellipsize="none" | ||||
|         android:gravity="center" | ||||
|         android:singleLine="true" | ||||
|         android:textColor="@color/white" | ||||
|         android:textIsSelectable="false" | ||||
|         android:textSize="20sp" | ||||
|         android:typeface="normal" /> | ||||
| </RelativeLayout> | ||||
| @@ -5,7 +5,7 @@ | ||||
|     android:layout_width="match_parent" | ||||
|     android:layout_height="wrap_content"> | ||||
|  | ||||
|     <ImageView | ||||
|     <bou.amine.apps.readerforselfossv2.android.utils.CircleImageView | ||||
|         android:id="@+id/itemImage" | ||||
|         android:layout_width="46dp" | ||||
|         android:layout_height="46dp" | ||||
|   | ||||
| @@ -25,7 +25,7 @@ | ||||
|         app:layout_constraintTop_toTopOf="parent" | ||||
|         app:layout_constraintVertical_bias="0.0" /> | ||||
|  | ||||
|     <ImageView | ||||
|     <bou.amine.apps.readerforselfossv2.android.utils.CircleImageView | ||||
|         android:id="@+id/itemImage" | ||||
|         android:layout_width="36dp" | ||||
|         android:layout_height="36dp" | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -33,8 +33,8 @@ | ||||
|     <string name="addStringNoUrl">"Accede pra engadir fontes."</string> | ||||
|     <string name="cant_get_sources">"Non se pode obter a lista de fontes."</string> | ||||
|     <string name="cant_create_source">"Non se pode crear unha fonte."</string> | ||||
|     <string name="cant_get_spouts_no_network">"Can't get spouts list because of a network issue."</string> | ||||
|     <string name="cant_get_spouts">"Can't get spouts list. There may ben an api issue."</string> | ||||
|     <string name="cant_get_spouts_no_network">"Non se pode obter a lista de spouts por mor dun erro de rede."</string> | ||||
|     <string name="cant_get_spouts">"Non se pode obter a lista de spoits. Pode que haxa algún problema coa api."</string> | ||||
|     <string name="form_not_complete">"O formulario non está completo"</string> | ||||
|     <string name="pref_header_links">"Ligazóns"</string> | ||||
|     <string name="issue_tracker_link">"Rastrexador de Incidencias"</string> | ||||
| @@ -116,16 +116,19 @@ | ||||
|     <string name="reader_static_bar_on">A barra inferior mostrarase sempre</string> | ||||
|     <string name="reader_static_bar_off">A barra inferior pode mostrarse a través do botón flotante</string> | ||||
|     <string name="remove_source">Eliminar fonte</string> | ||||
|     <string name="pref_theme_title">Light/Dark mode</string> | ||||
|     <string name="mode_dark">Dark mode</string> | ||||
|     <string name="mode_system">Follow the system setting</string> | ||||
|     <string name="mode_light">Light mode</string> | ||||
|     <string name="gdpr_dialog_title">The app does not share any personal data about you.</string> | ||||
|     <string name="gdpr_dialog_message"><![CDATA[Crash reports sending is now enabled. It can be disabled from the settings page. Keep in mind that crash reports are essential for the app development.]]></string> | ||||
|     <string name="crash_toast_text">A crash occured. Sending the details to the developper.</string> | ||||
|     <string name="pref_switch_disable_acra">"Disable automatic bug reporting. "</string> | ||||
|     <string name="menu_home_filter">Filters</string> | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="pref_theme_title">Modo Claro/Escuro</string> | ||||
|     <string name="mode_dark">Modo escuro</string> | ||||
|     <string name="mode_system">Seguir axustes do sistema</string> | ||||
|     <string name="mode_light">Modo claro</string> | ||||
|     <string name="gdpr_dialog_title">A aplicación non comparte ningún dato persoal seu.</string> | ||||
|     <string name="gdpr_dialog_message"><![CDATA[O envío de informes de erros está habilitado. Pode deshabilitarse dende a páxina de axustes. Ten en conta que os informes de erros son esenciais para o desenvolvemento da aplicación.]]></string> | ||||
|     <string name="crash_toast_text">Ocurriu un erro. Enviando os detalles o desenvolvedor.</string> | ||||
|     <string name="pref_switch_disable_acra">"Deshabilitar o reporte automático de erros. "</string> | ||||
|     <string name="menu_home_filter">Filtros</string> | ||||
|     <string name="application_selfoss_only">Esta aplicación só funciona cunha instancia de Selfoss, e con ningún outro filtro RSS.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -90,7 +90,7 @@ | ||||
|     <string name="pref_switch_items_caching">Save items for offline use</string> | ||||
|     <string name="pref_switch_update_sources">Check for new sources and tags</string> | ||||
|     <string name="pref_switch_update_sources_summary">Disable this if your server is receiving excessive amounts of database queries.</string> | ||||
|     <string name="network_connectivity_lost">"Network connection lost"</string> | ||||
|     <string name="network_connectivity_lost">"Koneksi jaringan hilang"</string> | ||||
|     <string name="network_connectivity_retrieved">"Network connection is now available"</string> | ||||
|     <string name="pref_switch_periodic_refresh">Sync articles</string> | ||||
|     <string name="pref_switch_periodic_refresh_off">Articles will not be synced in the background</string> | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -1,11 +0,0 @@ | ||||
| <?xml version="1.0" encoding="utf-8"?> | ||||
| <resources> | ||||
|     <string name="gdpr_dialog_title">The app does not share any personal data about you.</string> | ||||
|     <string name="gdpr_dialog_message"><![CDATA[Crash reports sending is now enabled. It can be disabled from the settings page. Keep in mind that crash reports are essential for the app development.]]></string> | ||||
|     <string name="crash_toast_text">A crash occured. Sending the details to the developper.</string> | ||||
|     <string name="pref_switch_disable_acra">"Disable automatic bug reporting. "</string> | ||||
|     <string name="menu_home_filter">Filters</string> | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
| </resources> | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -125,7 +125,10 @@ | ||||
|     <string name="crash_toast_text">发生崩溃。请将细节发送给开发人员。</string> | ||||
|     <string name="pref_switch_disable_acra">"禁用自动错误报告 "</string> | ||||
|     <string name="menu_home_filter">筛选器</string> | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="application_selfoss_only">此应用只适用于 Selfoss 实例,不适用于其他 RSS 。</string> | ||||
|     <string name="menu_home_sources">源</string> | ||||
|     <string name="update_source">更新源</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -128,4 +128,7 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -6,6 +6,7 @@ | ||||
|     <string name="error_invalid_password">"Password not long enough"</string> | ||||
|     <string name="error_field_required">"Field required"</string> | ||||
|     <string name="prompt_url">"Url"</string> | ||||
|     <string name="disable_ssl">"Disable SSL"</string> | ||||
|     <string name="withLoginSwitch">"Login required ?"</string> | ||||
|     <string name="login_url_problem">"Oops. You may need to add a \"/\" at the end of the url."</string> | ||||
|     <string name="prompt_login">"Username"</string> | ||||
| @@ -131,4 +132,6 @@ | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -32,4 +32,10 @@ | ||||
|         <item name="android:colorBackgroundCacheHint">@null</item> | ||||
|         <item name="android:windowIsTranslucent">true</item> | ||||
|     </style> | ||||
|  | ||||
|     <style name="circleImageView" parent=""> | ||||
|         <item name="cornerFamily">rounded</item> | ||||
|         <item name="cornerSize">50%</item> | ||||
|     </style> | ||||
|  | ||||
| </resources> | ||||
|   | ||||
| @@ -9,38 +9,49 @@ import org.junit.Test | ||||
|  | ||||
| class DatesTest { | ||||
|  | ||||
|     private val v3Date = "2013-04-07T13:43:00+01:00" | ||||
|     private val v4Date = "2013-04-07 13:43:00" | ||||
|     private val bug1Date = "2022-12-24T17:00:08+00" | ||||
|     private val newVersionDateVariant =  "2022-12-24T17:00:08+00" | ||||
|     private val newVersionDate =         "2013-04-07T13:43:00+01:00" | ||||
|     private val oldVersionDate =         "2013-05-07 13:46:00" | ||||
|     private val oldVersionDateVariant =  "2021-03-21 10:32:00.000000" | ||||
|  | ||||
|     @Test | ||||
|     fun v3_date_should_be_parsed() { | ||||
|         val date = DateUtils.parseDate(v3Date) | ||||
|         val expected = | ||||
|             LocalDateTime(2013, 4, 7, 14, 43, 0, 0).toInstant(TimeZone.currentSystemDefault()) | ||||
|                 .toEpochMilliseconds() | ||||
|  | ||||
|         assertEquals(date, expected) | ||||
|     } | ||||
|  | ||||
|     @Test | ||||
|     fun v4_date_should_be_parsed() { | ||||
|         val date = DateUtils.parseDate(v4Date) | ||||
|     fun new_version_date_should_be_parsed() { | ||||
|         val date = DateUtils.parseDate(newVersionDate) | ||||
|         val expected = | ||||
|             LocalDateTime(2013, 4, 7, 13, 43, 0, 0).toInstant(TimeZone.currentSystemDefault()) | ||||
|                 .toEpochMilliseconds() | ||||
|  | ||||
|         assertEquals(date, expected) | ||||
|         assertEquals(expected, date) | ||||
|     } | ||||
|  | ||||
|     @Test | ||||
|     fun bug1_date_should_be_parsed() { | ||||
|         val date = DateUtils.parseDate(bug1Date) | ||||
|     fun old_version_date_should_be_parsed() { | ||||
|         val date = DateUtils.parseDate(oldVersionDate) | ||||
|         val expected = | ||||
|             LocalDateTime(2022, 12, 24, 18, 0, 8, 0).toInstant(TimeZone.currentSystemDefault()) | ||||
|             LocalDateTime(2013, 5, 7, 13, 46, 0, 0).toInstant(TimeZone.currentSystemDefault()) | ||||
|                 .toEpochMilliseconds() | ||||
|  | ||||
|         assertEquals(date, expected) | ||||
|         assertEquals(expected, date) | ||||
|     } | ||||
|  | ||||
|     @Test | ||||
|     fun old_version_variant_date_should_be_parsed() { | ||||
|         val date = DateUtils.parseDate(oldVersionDateVariant) | ||||
|         val expected = | ||||
|             LocalDateTime(2021, 3, 21, 10, 32, 0, 0).toInstant(TimeZone.currentSystemDefault()) | ||||
|                 .toEpochMilliseconds() | ||||
|  | ||||
|         assertEquals(expected, date) | ||||
|     } | ||||
|  | ||||
|     @Test | ||||
|     fun new_version_variant_date_should_be_parsed() { | ||||
|         val date = DateUtils.parseDate(newVersionDateVariant) | ||||
|         val expected = | ||||
|             LocalDateTime(2022, 12, 24, 17, 0, 8, 0).toInstant(TimeZone.currentSystemDefault()) | ||||
|                 .toEpochMilliseconds() | ||||
|  | ||||
|         assertEquals(expected, date) | ||||
|     } | ||||
|  | ||||
| } | ||||
|   | ||||
| @@ -300,9 +300,10 @@ class RepositoryTest { | ||||
|         every { appSettingsService.isItemCachingEnabled() } returns true | ||||
|  | ||||
|         initializeRepository(MutableStateFlow(false)) | ||||
|         repository.setSourceFilter(SelfossModel.Source( | ||||
|         repository.setSourceFilter(SelfossModel.SourceDetail( | ||||
|             1, | ||||
|             "Test", | ||||
|             null, | ||||
|             listOf("tags"), | ||||
|             SPOUT, | ||||
|             "", | ||||
| @@ -609,30 +610,32 @@ class RepositoryTest { | ||||
|     fun get_sources() { | ||||
|         val (sources, sourcesDB) = prepareSources() | ||||
|         initializeRepository() | ||||
|         var testSources: List<SelfossModel.Source>? | ||||
|         var testSources: List<SelfossModel.Source> | ||||
|         runBlocking { | ||||
|             testSources = repository.getSources() | ||||
|             testSources = repository.getSourcesDetails() | ||||
|         } | ||||
|  | ||||
|         assertSame(sources, testSources) | ||||
|         assertEquals(sources, testSources) | ||||
|         assertNotEquals(sourcesDB.map { it.toView() }, testSources) | ||||
|         coVerify(exactly = 1) { api.sources() } | ||||
|         coVerify(exactly = 1) { api.sourcesDetailed() } | ||||
|     } | ||||
|  | ||||
|     private fun prepareSources(): Pair<ArrayList<SelfossModel.Source>, List<SOURCE>> { | ||||
|     private fun prepareSources(): Pair<ArrayList<SelfossModel.SourceDetail>, List<SOURCE>> { | ||||
|         val sources = arrayListOf( | ||||
|             SelfossModel.Source( | ||||
|             SelfossModel.SourceDetail( | ||||
|                 1, | ||||
|                 "First source", | ||||
|                 null, | ||||
|                 listOf("Test", "second"), | ||||
|                 SPOUT, | ||||
|                 "", | ||||
|                 IMAGE_URL_2, | ||||
|                 SelfossModel.SourceParams("url") | ||||
|             ), | ||||
|             SelfossModel.Source( | ||||
|             SelfossModel.SourceDetail( | ||||
|                 2, | ||||
|                 "Second source", | ||||
|                 null, | ||||
|                 listOf("second"), | ||||
|                 SPOUT, | ||||
|                 "", | ||||
| @@ -661,7 +664,7 @@ class RepositoryTest { | ||||
|             ) | ||||
|         ) | ||||
|  | ||||
|         coEvery { api.sources() } returns StatusAndData(success = true, data = sources) | ||||
|         coEvery { api.sourcesDetailed() } returns StatusAndData(success = true, data = sources) | ||||
|         every { db.sourcesQueries.sources().executeAsList() } returns sourcesDB | ||||
|         return Pair(sources, sourcesDB) | ||||
|     } | ||||
| @@ -675,13 +678,13 @@ class RepositoryTest { | ||||
|         initializeRepository() | ||||
|         var testSources: List<SelfossModel.Source>? | ||||
|         runBlocking { | ||||
|             testSources = repository.getSources() | ||||
|             testSources = repository.getSourcesDetails() | ||||
|             // Sources will be fetched from the database on the second call, thus testSources != sources | ||||
|             testSources = repository.getSources() | ||||
|             testSources = repository.getSourcesDetails() | ||||
|         } | ||||
|  | ||||
|         coVerify(exactly = 1) { api.sources() } | ||||
|         assertNotSame(sources, testSources) | ||||
|         coVerify(exactly = 1) { api.sourcesDetailed() } | ||||
|         assertNotEquals(sources, testSources) | ||||
|         assertEquals(sourcesDB.map { it.toView() }, testSources) | ||||
|         verify(atLeast = 1) { db.sourcesQueries.sources().executeAsList() } | ||||
|     } | ||||
| @@ -693,13 +696,13 @@ class RepositoryTest { | ||||
|         every { appSettingsService.isUpdateSourcesEnabled() } returns true | ||||
|         every { appSettingsService.isItemCachingEnabled() } returns false | ||||
|         initializeRepository() | ||||
|         var testSources: List<SelfossModel.Source>? | ||||
|         var testSources: List<SelfossModel.Source> | ||||
|         runBlocking { | ||||
|             testSources = repository.getSources() | ||||
|             testSources = repository.getSourcesDetails() | ||||
|         } | ||||
|  | ||||
|         assertSame(sources, testSources) | ||||
|         coVerify(exactly = 1) { api.sources() } | ||||
|         assertEquals(sources, testSources) | ||||
|         coVerify(exactly = 1) { api.sourcesDetailed() } | ||||
|         verify(exactly = 0) { db.sourcesQueries } | ||||
|     } | ||||
|  | ||||
| @@ -710,13 +713,13 @@ class RepositoryTest { | ||||
|         every { appSettingsService.isUpdateSourcesEnabled() } returns false | ||||
|         every { appSettingsService.isItemCachingEnabled() } returns false | ||||
|         initializeRepository() | ||||
|         var testSources: List<SelfossModel.Source>? | ||||
|         var testSources: List<SelfossModel.Source> | ||||
|         runBlocking { | ||||
|             testSources = repository.getSources() | ||||
|             testSources = repository.getSourcesDetails() | ||||
|         } | ||||
|  | ||||
|         assertSame(sources, testSources) | ||||
|         coVerify(exactly = 1) { api.sources() } | ||||
|         assertEquals(sources, testSources) | ||||
|         coVerify(exactly = 1) { api.sourcesDetailed() } | ||||
|         verify(atLeast = 1) { db.sourcesQueries } | ||||
|     } | ||||
|  | ||||
| @@ -724,13 +727,13 @@ class RepositoryTest { | ||||
|     fun get_sources_without_connection() { | ||||
|         val (_, sourcesDB) = prepareSources() | ||||
|         initializeRepository(MutableStateFlow(false)) | ||||
|         var testSources: List<SelfossModel.Source>? | ||||
|         var testSources: List<SelfossModel.Source> | ||||
|         runBlocking { | ||||
|             testSources = repository.getSources() | ||||
|             testSources = repository.getSourcesDetails() | ||||
|         } | ||||
|  | ||||
|         assertEquals(sourcesDB.map { it.toView() }, testSources) | ||||
|         coVerify(exactly = 0) { api.sources() } | ||||
|         coVerify(exactly = 0) { api.sourcesDetailed() } | ||||
|         verify(atLeast = 1) { db.sourcesQueries.sources().executeAsList() } | ||||
|     } | ||||
|  | ||||
| @@ -741,13 +744,13 @@ class RepositoryTest { | ||||
|         every { appSettingsService.isItemCachingEnabled() } returns false | ||||
|         every { appSettingsService.isUpdateSourcesEnabled() } returns true | ||||
|         initializeRepository(MutableStateFlow(false)) | ||||
|         var testSources: List<SelfossModel.Source>? | ||||
|         var testSources: List<SelfossModel.Source> | ||||
|         runBlocking { | ||||
|             testSources = repository.getSources() | ||||
|             testSources = repository.getSourcesDetails() | ||||
|         } | ||||
|  | ||||
|         assertEquals(emptyList<SelfossModel.Source>(), testSources) | ||||
|         coVerify(exactly = 0) { api.sources() } | ||||
|         coVerify(exactly = 0) { api.sourcesDetailed() } | ||||
|         verify(exactly = 0) { db.sourcesQueries.sources().executeAsList() } | ||||
|     } | ||||
|  | ||||
| @@ -758,13 +761,13 @@ class RepositoryTest { | ||||
|         every { appSettingsService.isItemCachingEnabled() } returns true | ||||
|         every { appSettingsService.isUpdateSourcesEnabled() } returns false | ||||
|         initializeRepository(MutableStateFlow(false)) | ||||
|         var testSources: List<SelfossModel.Source>? | ||||
|         var testSources: List<SelfossModel.Source> | ||||
|         runBlocking { | ||||
|             testSources = repository.getSources() | ||||
|             testSources = repository.getSourcesDetails() | ||||
|         } | ||||
|  | ||||
|         assertEquals(sourcesDB.map { it.toView() }, testSources) | ||||
|         coVerify(exactly = 0) { api.sources() } | ||||
|         coVerify(exactly = 0) { api.sourcesDetailed() } | ||||
|         verify(atLeast = 1) { db.sourcesQueries.sources().executeAsList() } | ||||
|     } | ||||
|  | ||||
| @@ -775,13 +778,13 @@ class RepositoryTest { | ||||
|         every { appSettingsService.isItemCachingEnabled() } returns false | ||||
|         every { appSettingsService.isUpdateSourcesEnabled() } returns false | ||||
|         initializeRepository(MutableStateFlow(false)) | ||||
|         var testSources: List<SelfossModel.Source>? | ||||
|         var testSources: List<SelfossModel.Source> | ||||
|         runBlocking { | ||||
|             testSources = repository.getSources() | ||||
|             testSources = repository.getSourcesDetails() | ||||
|         } | ||||
|  | ||||
|         assertEquals(sourcesDB.map { it.toView() }, testSources) | ||||
|         coVerify(exactly = 0) { api.sources() } | ||||
|         coVerify(exactly = 0) { api.sourcesDetailed() } | ||||
|         verify(atLeast = 1) { db.sourcesQueries.sources().executeAsList() } | ||||
|     } | ||||
|  | ||||
| @@ -1102,9 +1105,10 @@ class RepositoryTest { | ||||
|     private fun prepareSearch() { | ||||
|         repository.setTagFilter(SelfossModel.Tag("Tag", "read", 0)) | ||||
|         repository.setSourceFilter( | ||||
|             SelfossModel.Source( | ||||
|             SelfossModel.SourceDetail( | ||||
|                 1, | ||||
|                 "First source", | ||||
|                 5, | ||||
|                 listOf("Test", "second"), | ||||
|                 SPOUT, | ||||
|                 "", | ||||
|   | ||||
| @@ -1,18 +1,18 @@ | ||||
| buildscript { | ||||
|     dependencies { | ||||
|         // SqlDelight | ||||
|         classpath("com.squareup.sqldelight:gradle-plugin:1.5.4") | ||||
|         classpath("com.squareup.sqldelight:gradle-plugin:1.5.5") | ||||
|     } | ||||
| } | ||||
|  | ||||
| plugins { | ||||
|     //trick: for the same plugin versions in all sub-modules | ||||
|     id("com.android.application").version("7.4.0").apply(false) | ||||
|     id("com.android.library").version("7.4.0").apply(false) | ||||
|     kotlin("android").version("1.7.20").apply(false) | ||||
|     kotlin("multiplatform").version("1.7.20").apply(false) | ||||
|     id("com.android.application").version("8.1.2").apply(false) | ||||
|     id("com.android.library").version("8.1.2").apply(false) | ||||
|     id("org.jetbrains.kotlin.android").version("1.9.10").apply(false) | ||||
|     kotlin("multiplatform").version("1.9.10").apply(false) | ||||
|     id("com.mikepenz.aboutlibraries.plugin").version("10.5.1").apply(false) | ||||
|     id("org.jetbrains.kotlinx.kover") version "0.6.1" | ||||
|     id("org.jetbrains.kotlinx.kover").version("0.6.1").apply(true) | ||||
| } | ||||
|  | ||||
| allprojects { | ||||
| @@ -20,7 +20,6 @@ allprojects { | ||||
|         // maven { url = uri("https://nexus.amine-louveau.fr/repository/maven-public/")} | ||||
|         google() | ||||
|         mavenCentral() | ||||
|         jcenter() | ||||
|         maven { url = uri("https://www.jitpack.io") } | ||||
|     } | ||||
| } | ||||
|   | ||||
| @@ -13,24 +13,16 @@ | ||||
| #Tue Mar 22 16:50:00 CET 2022 | ||||
| #Gradle | ||||
| org.gradle.jvmargs=-Xmx2048M -Dfile.encoding=UTF-8 -Dkotlin.daemon.jvm.options\="-Xmx2048M" | ||||
|  | ||||
| #Kotlin | ||||
| kotlin.code.style=official | ||||
|  | ||||
| #Android | ||||
| android.useAndroidX=true | ||||
| kotlin.native.enableDependencyPropagation=false | ||||
| #android.nonTransitiveRClass=true | ||||
| android.enableJetifier=true | ||||
|  | ||||
|  | ||||
| android.nonTransitiveRClass=false | ||||
| #MPP | ||||
| kotlin.mpp.enableCInteropCommonization=true | ||||
| kotlin.mpp.enableGranularSourceSetsMetadata=true | ||||
|  | ||||
|  | ||||
| org.gradle.parallel=true | ||||
| org.gradle.caching=true | ||||
| ignoreGitVersion=false | ||||
| kotlin.native.cacheKind.iosX64=none | ||||
| pushCache=true | ||||
|   | ||||
							
								
								
									
										6
									
								
								gradle/wrapper/gradle-wrapper.properties
									
									
									
									
										vendored
									
									
								
							
							
						
						
									
										6
									
								
								gradle/wrapper/gradle-wrapper.properties
									
									
									
									
										vendored
									
									
								
							| @@ -1,6 +1,6 @@ | ||||
| #Mon Jan 23 20:47:46 CET 2023 | ||||
| #Thu Jul 13 11:41:19 CEST 2023 | ||||
| distributionBase=GRADLE_USER_HOME | ||||
| distributionUrl=https\://services.gradle.org/distributions/gradle-7.5-bin.zip | ||||
| distributionPath=wrapper/dists | ||||
| zipStorePath=wrapper/dists | ||||
| distributionUrl=https\://services.gradle.org/distributions/gradle-8.1.1-bin.zip | ||||
| zipStoreBase=GRADLE_USER_HOME | ||||
| zipStorePath=wrapper/dists | ||||
|   | ||||
| @@ -1,5 +1,3 @@ | ||||
| val pushCache: String by settings | ||||
|  | ||||
| pluginManagement { | ||||
|     repositories { | ||||
|         // maven { url = uri("https://nexus.amine-louveau.fr/repository/maven-public/")} | ||||
| @@ -17,17 +15,6 @@ dependencyResolutionManagement { | ||||
|     } | ||||
| } | ||||
|  | ||||
|  | ||||
| buildCache { | ||||
|     remote<HttpBuildCache> { | ||||
|         url = uri("http://18.0.0.7:3071/cache/") | ||||
|         isAllowInsecureProtocol = true | ||||
|         isAllowUntrustedServer = true | ||||
|         isUseExpectContinue = true | ||||
|         isPush = (pushCache == "true") | ||||
|     } | ||||
| } | ||||
|  | ||||
| rootProject.name = "ReaderForSelfossV2" | ||||
| include(":androidApp") | ||||
| include(":shared") | ||||
| @@ -1,3 +1,5 @@ | ||||
| val ktorVersion = "2.3.2" | ||||
|  | ||||
| object SqlDelight { | ||||
|     const val runtime = "com.squareup.sqldelight:runtime:1.5.4" | ||||
|     const val android = "com.squareup.sqldelight:android-driver:1.5.4" | ||||
| @@ -9,12 +11,12 @@ plugins { | ||||
|     kotlin("multiplatform") | ||||
|     id("com.android.library") | ||||
|     id("com.squareup.sqldelight") | ||||
|     kotlin("plugin.serialization") version "1.4.10" | ||||
|     id("org.jetbrains.kotlinx.kover") version "0.6.1" | ||||
|     kotlin("plugin.serialization") version "1.9.0" | ||||
|     id("org.jetbrains.kotlinx.kover") | ||||
| } | ||||
|  | ||||
| kotlin { | ||||
|     android() | ||||
|     androidTarget() | ||||
|  | ||||
|     listOf( | ||||
|         iosX64(), | ||||
| @@ -29,16 +31,18 @@ kotlin { | ||||
|     sourceSets { | ||||
|         val commonMain by getting { | ||||
|             dependencies { | ||||
|                 implementation("io.ktor:ktor-client-core:2.1.1") | ||||
|                 implementation("io.ktor:ktor-client-content-negotiation:2.1.1") | ||||
|                 implementation("io.ktor:ktor-serialization-kotlinx-json:2.1.1") | ||||
|                 implementation("io.ktor:ktor-client-logging:2.1.1") | ||||
|                 implementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.6.0") | ||||
|                 implementation("io.ktor:ktor-client-auth:2.1.1") | ||||
|                 implementation("org.jsoup:jsoup:1.14.3") | ||||
|                 implementation("io.ktor:ktor-client-core:$ktorVersion") | ||||
|                 implementation("io.ktor:ktor-client-content-negotiation:$ktorVersion") | ||||
|                 implementation("io.ktor:ktor-serialization-kotlinx-json:$ktorVersion") | ||||
|                 implementation("io.ktor:ktor-client-logging:$ktorVersion") | ||||
|                 implementation("io.ktor:ktor-client-auth:$ktorVersion") | ||||
|                 implementation("io.ktor:ktor-client-cio:$ktorVersion") | ||||
|                 implementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.7.1") | ||||
|  | ||||
|                 implementation("org.jsoup:jsoup:1.15.4") | ||||
|  | ||||
|                 //Dependency Injection | ||||
|                 implementation("org.kodein.di:kodein-di:7.12.0") | ||||
|                 implementation("org.kodein.di:kodein-di:7.14.0") | ||||
|  | ||||
|                 //Settings | ||||
|                 implementation("com.russhwolf:multiplatform-settings-no-arg:1.0.0-RC") | ||||
| @@ -58,14 +62,15 @@ kotlin { | ||||
|         } | ||||
|         val androidMain by getting { | ||||
|             dependencies { | ||||
|                 implementation("io.ktor:ktor-client-okhttp:2.1.1") | ||||
|                 implementation("com.squareup.okhttp3:okhttp:4.11.0") | ||||
|                 implementation("io.ktor:ktor-client-okhttp:2.2.4") | ||||
|                 implementation("org.jetbrains.kotlinx:kotlinx-datetime:0.4.0") | ||||
|  | ||||
|                 // Sql | ||||
|                 implementation(SqlDelight.android) | ||||
|             } | ||||
|         } | ||||
|         val androidTest by getting { | ||||
|         val androidUnitTest by getting { | ||||
|             dependencies { | ||||
|                 implementation(kotlin("test-junit")) | ||||
|                 implementation("junit:junit:4.13.2") | ||||
| @@ -98,15 +103,14 @@ kotlin { | ||||
| } | ||||
|  | ||||
| android { | ||||
|     compileSdk = 32 | ||||
|     compileSdk = 34 | ||||
|     sourceSets["main"].manifest.srcFile("src/androidMain/AndroidManifest.xml") | ||||
|     defaultConfig { | ||||
|         minSdk = 21 | ||||
|         targetSdk = 32 | ||||
|         minSdk = 25 | ||||
|     } | ||||
|     compileOptions { | ||||
|         sourceCompatibility = JavaVersion.VERSION_1_8 | ||||
|         targetCompatibility = JavaVersion.VERSION_1_8 | ||||
|         sourceCompatibility = JavaVersion.VERSION_17 | ||||
|         targetCompatibility = JavaVersion.VERSION_17 | ||||
|     } | ||||
|     namespace = "bou.amine.apps.readerforselfossv2" | ||||
| } | ||||
|   | ||||
| @@ -0,0 +1,17 @@ | ||||
| package bou.amine.apps.readerforselfossv2.rest | ||||
|  | ||||
| import io.ktor.client.engine.cio.CIOEngineConfig | ||||
| import java.security.cert.X509Certificate | ||||
| import javax.net.ssl.X509TrustManager | ||||
|  | ||||
| class NaiveTrustManager : X509TrustManager { | ||||
|     override fun checkClientTrusted(chain: Array<out X509Certificate>?, authType: String?) {} | ||||
|  | ||||
|     override fun checkServerTrusted(chain: Array<out X509Certificate>?, authType: String?) {} | ||||
|  | ||||
|     override fun getAcceptedIssuers(): Array<out X509Certificate> = arrayOf() | ||||
| } | ||||
|  | ||||
| actual fun setupInsecureHTTPEngine(config: CIOEngineConfig) { | ||||
|     config.https.trustManager = NaiveTrustManager() | ||||
| } | ||||
| @@ -6,16 +6,25 @@ import kotlinx.datetime.* | ||||
|  | ||||
| actual class DateUtils { | ||||
|     actual companion object { | ||||
|  | ||||
|         // Possible formats are | ||||
|         // yyyy-mm-dd hh:mm:ss format | ||||
|         private val oldVersionFormat = "\\d{4}-\\d{2}-\\d{2} \\d{2}:\\d{2}:\\d{2}(.()\\d*)?".toRegex() | ||||
|         // yyyy-MM-dd'T'HH:mm:ss[.SSS]XXX (RFC3339) | ||||
|         private val newVersionFormat = "\\d{4}-\\d{2}-\\d{2}T\\d{2}:\\d{2}:\\d{2}\\+\\d{2}(:\\d{2})?".toRegex() | ||||
|  | ||||
|         // We may need to consider moving the formatting to platform specific code, even if the tests are doubled | ||||
|         // For now, we handle this in a hacky way, because kotlin only accepts iso formats | ||||
|         actual fun parseDate(dateString: String): Long { | ||||
|             return try { | ||||
|                 Instant.parse(dateString).toEpochMilliseconds() | ||||
|             } catch (e: Exception) { | ||||
|                 var str = dateString.replace(" ", "T") | ||||
|                 if (str.matches("\\d{4}-\\d{2}-\\d{2}T\\d{2}:\\d{2}:\\d{2}\\+\\d{2}".toRegex())) { | ||||
|                     str = str.split("+")[0] | ||||
|                 } | ||||
|                 LocalDateTime.parse(str).toInstant(TimeZone.currentSystemDefault()).toEpochMilliseconds() | ||||
|             var isoDateString: String = if (dateString.matches(oldVersionFormat)) { | ||||
|                 dateString.replace(" ", "T") | ||||
|             } else if (dateString.matches(newVersionFormat)) { | ||||
|                 dateString.split("+")[0] | ||||
|             } else { | ||||
|                 throw Exception("Unrecognized format for $dateString") | ||||
|             } | ||||
|  | ||||
|             return LocalDateTime.parse(isoDateString).toInstant(TimeZone.currentSystemDefault()).toEpochMilliseconds() | ||||
|         } | ||||
|  | ||||
|         actual fun parseRelativeDate(dateString: String): String { | ||||
|   | ||||
| @@ -9,7 +9,6 @@ import org.kodein.di.instance | ||||
| import org.kodein.di.singleton | ||||
|  | ||||
| val networkModule by DI.Module { | ||||
|     bind<AppSettingsService>() with singleton { AppSettingsService() } | ||||
|     bind<SelfossApi>() with singleton { SelfossApi(instance()) } | ||||
|     bind<MercuryApi>() with singleton { MercuryApi() } | ||||
| } | ||||
| @@ -6,9 +6,12 @@ class MercuryModel { | ||||
|  | ||||
|     @Serializable | ||||
|     class ParsedContent( | ||||
|         val title: String?, | ||||
|         val content: String?, | ||||
|         val lead_image_url: String?, // NOSONAR | ||||
|         val url: String | ||||
|         val title: String? = null, | ||||
|         val content: String? = null, | ||||
|         val lead_image_url: String? = null, // NOSONAR | ||||
|         val url: String? = null, | ||||
|         val error: Boolean? = null, | ||||
|         val message: String? = null, | ||||
|         val failed: Boolean? = null | ||||
|     ) | ||||
| } | ||||
|   | ||||
| @@ -24,8 +24,8 @@ class SelfossModel { | ||||
|     @Serializable | ||||
|     class Stats( | ||||
|         val total: Int, | ||||
|         val unread: Int, | ||||
|         val starred: Int | ||||
|         val unread: Int? = null, | ||||
|         val starred: Int? = null | ||||
|     ) | ||||
|  | ||||
|     @Serializable | ||||
| @@ -36,9 +36,9 @@ class SelfossModel { | ||||
|  | ||||
|     @Serializable | ||||
|     data class ApiInformation( | ||||
|         val version: String?, | ||||
|         val apiversion: String?, | ||||
|         val configuration: ApiConfiguration? | ||||
|         val version: String? = null, | ||||
|         val apiversion: String? = null, | ||||
|         val configuration: ApiConfiguration? = null | ||||
|     ) { | ||||
|         fun getApiMajorVersion(): Int { | ||||
|             var versionNumber = 0 | ||||
| @@ -54,29 +54,48 @@ class SelfossModel { | ||||
|     @Serializable | ||||
|     data class ApiConfiguration( | ||||
|         @Serializable(with = BooleanSerializer::class) | ||||
|         val publicMode: Boolean?, | ||||
|         val publicMode: Boolean? = null, | ||||
|         @Serializable(with = BooleanSerializer::class) | ||||
|         val authEnabled: Boolean? | ||||
|         val authEnabled: Boolean? = null | ||||
|     ) { | ||||
|         fun isAuthEnabled() = authEnabled ?: true | ||||
|  | ||||
|         fun isPublicModeEnabled() = publicMode ?: false | ||||
|     } | ||||
|  | ||||
|     interface Source { | ||||
|         val id: Int | ||||
|         var title: String | ||||
|         var unread: Int? | ||||
|         var error: String? | ||||
|         var icon: String? | ||||
|     } | ||||
|  | ||||
|     @Serializable | ||||
|     data class Source( | ||||
|         val id: Int, | ||||
|         val title: String, | ||||
|     data class SourceStats( | ||||
|         override val id: Int, | ||||
|         override var title: String, | ||||
|         override var unread: Int? = null, | ||||
|         override var error: String? = null, | ||||
|         override var icon: String? = null | ||||
|         ) : Source | ||||
|  | ||||
|     @Serializable | ||||
|     data class SourceDetail( | ||||
|         override val id: Int, | ||||
|         override var title: String, | ||||
|         override var unread: Int? = null, | ||||
|         @Serializable(with = TagsListSerializer::class) | ||||
|         val tags: List<String>, | ||||
|         val spout: String, | ||||
|         val error: String, | ||||
|         val icon: String?, | ||||
|         val params: SourceParams? | ||||
|     ) | ||||
|         var tags: List<String>? = null, | ||||
|         var spout: String? = null, | ||||
|         override var error: String? = null, | ||||
|         override var icon: String? = null, | ||||
|         var params: SourceParams? = null | ||||
|     ) : Source | ||||
|  | ||||
|     @Serializable | ||||
|     data class SourceParams( | ||||
|         val url: String | ||||
|         val url: String? = null | ||||
|     ) | ||||
|     @Serializable | ||||
|     data class Item( | ||||
| @@ -88,13 +107,13 @@ class SelfossModel { | ||||
|         var unread: Boolean, | ||||
|         @Serializable(with = BooleanSerializer::class) | ||||
|         var starred: Boolean, | ||||
|         val thumbnail: String?, | ||||
|         val icon: String?, | ||||
|         val thumbnail: String? = null, | ||||
|         val icon: String? = null, | ||||
|         val link: String, | ||||
|         val sourcetitle: String, | ||||
|         @Serializable(with = TagsListSerializer::class) | ||||
|         val tags: List<String>, | ||||
|         val author: String? | ||||
|         val author: String? = null | ||||
|     ) { | ||||
|         fun getLinkDecoded(): String { | ||||
|             var stringUrl: String | ||||
|   | ||||
| @@ -44,11 +44,11 @@ class Repository( | ||||
|     private val _badgeStarred = MutableStateFlow(0) | ||||
|     val badgeStarred = _badgeStarred.asStateFlow() | ||||
|  | ||||
|     private var fetchedSources = false | ||||
|     private var fetchedTags = false | ||||
|     private var fetchedSources = false | ||||
|  | ||||
|     private var _readerItems = ArrayList<SelfossModel.Item>() | ||||
|     private var _selectedSource: SelfossModel.Source? = null | ||||
|     private var _selectedSource: SelfossModel.SourceDetail? = null | ||||
|  | ||||
|     suspend fun getNewerItems(): ArrayList<SelfossModel.Item> { | ||||
|         var fetchedItems: StatusAndData<List<SelfossModel.Item>> = StatusAndData.error() | ||||
| @@ -132,9 +132,9 @@ class Repository( | ||||
|         if (isNetworkAvailable()) { | ||||
|             val response = api.stats() | ||||
|             if (response.success && response.data != null) { | ||||
|                 _badgeUnread.value = response.data.unread | ||||
|                 _badgeUnread.value = response.data.unread ?: 0 | ||||
|                 _badgeAll.value = response.data.total | ||||
|                 _badgeStarred.value = response.data.starred | ||||
|                 _badgeStarred.value = response.data.starred ?: 0 | ||||
|                 success = true | ||||
|             } | ||||
|         } else if (appSettingsService.isItemCachingEnabled()) { | ||||
| @@ -180,23 +180,46 @@ class Repository( | ||||
|         } | ||||
|     } | ||||
|  | ||||
|     suspend fun getSources(): ArrayList<SelfossModel.Source> { | ||||
|     suspend fun getSourcesDetailsOrStats(): ArrayList<SelfossModel.Source> { | ||||
|         var sources = ArrayList<SelfossModel.Source>() | ||||
|         val isDatabaseEnabled = | ||||
|             appSettingsService.isItemCachingEnabled() || !appSettingsService.isUpdateSourcesEnabled() | ||||
|         return if (isNetworkAvailable() && !fetchedSources) { | ||||
|             val apiSources = api.sources() | ||||
|             if (apiSources.success && apiSources.data != null && isDatabaseEnabled) { | ||||
|                 resetDBSourcesWithData(apiSources.data) | ||||
|                 if (!appSettingsService.isUpdateSourcesEnabled()) { | ||||
|         val shouldFetch = if (!appSettingsService.isUpdateSourcesEnabled()) !fetchedSources else true | ||||
|         if (shouldFetch && isNetworkAvailable()) { | ||||
|             if (appSettingsService.getPublicAccess()) { | ||||
|                 val apiSources = api.sourcesStats() | ||||
|                 if (apiSources.success && apiSources.data != null) { | ||||
|                     fetchedSources = true | ||||
|                     sources = apiSources.data as ArrayList<SelfossModel.Source> | ||||
|                 } | ||||
|             } else { | ||||
|                 sources = getSourcesDetails() as ArrayList<SelfossModel.Source> | ||||
|             } | ||||
|         } else if (isDatabaseEnabled) { | ||||
|             sources = getDBSources().map { it.toView() } as ArrayList<SelfossModel.Source> | ||||
|         } | ||||
|  | ||||
|         return sources | ||||
|     } | ||||
|  | ||||
|     suspend fun getSourcesDetails(): ArrayList<SelfossModel.SourceDetail> { | ||||
|         var sources = ArrayList<SelfossModel.SourceDetail>() | ||||
|         val isDatabaseEnabled = | ||||
|             appSettingsService.isItemCachingEnabled() || !appSettingsService.isUpdateSourcesEnabled() | ||||
|         val shouldFetch = if (!appSettingsService.isUpdateSourcesEnabled()) !fetchedSources else true | ||||
|         if (shouldFetch && isNetworkAvailable()) { | ||||
|             val apiSources = api.sourcesDetailed() | ||||
|             if (apiSources.success && apiSources.data != null) { | ||||
|                 fetchedSources = true | ||||
|                 sources = apiSources.data | ||||
|                 if (isDatabaseEnabled) { | ||||
|                     resetDBSourcesWithData(sources) | ||||
|                 } | ||||
|             } | ||||
|             apiSources.data ?: ArrayList() | ||||
|         } else if (isDatabaseEnabled) { | ||||
|             ArrayList(getDBSources().map { it.toView() }) | ||||
|         } else { | ||||
|             ArrayList() | ||||
|             sources = getDBSources().map { it.toView() } as ArrayList<SelfossModel.SourceDetail> | ||||
|         } | ||||
|         return sources | ||||
|     } | ||||
|  | ||||
|     suspend fun markAsRead(item: SelfossModel.Item): Boolean { | ||||
| @@ -482,7 +505,7 @@ class Repository( | ||||
|         } | ||||
|     } | ||||
|  | ||||
|     private fun resetDBSourcesWithData(sources: List<SelfossModel.Source>) { | ||||
|     private fun resetDBSourcesWithData(sources: List<SelfossModel.SourceDetail>) { | ||||
|         db.sourcesQueries.deleteAllSources() | ||||
|  | ||||
|         db.sourcesQueries.transaction { | ||||
| @@ -592,7 +615,7 @@ class Repository( | ||||
|         ReaderForSelfossDB.Schema.migrate(driverFactory.createDriver(), 0, 1) | ||||
|     } | ||||
|  | ||||
|     fun setSelectedSource(source: SelfossModel.Source) { | ||||
|     fun setSelectedSource(source: SelfossModel.SourceDetail) { | ||||
|         _selectedSource = source | ||||
|     } | ||||
|  | ||||
| @@ -600,7 +623,7 @@ class Repository( | ||||
|         _selectedSource = null | ||||
|     } | ||||
|  | ||||
|     fun getSelectedSource(): SelfossModel.Source? { | ||||
|     fun getSelectedSource(): SelfossModel.SourceDetail? { | ||||
|         return _selectedSource | ||||
|     } | ||||
| } | ||||
| @@ -5,33 +5,53 @@ import bou.amine.apps.readerforselfossv2.model.StatusAndData | ||||
| import bou.amine.apps.readerforselfossv2.model.SuccessResponse | ||||
| import bou.amine.apps.readerforselfossv2.service.AppSettingsService | ||||
| import io.github.aakira.napier.Napier | ||||
| import io.ktor.client.* | ||||
| import io.ktor.client.plugins.* | ||||
| import io.ktor.client.plugins.cache.* | ||||
| import io.ktor.client.plugins.contentnegotiation.* | ||||
| import io.ktor.client.plugins.cookies.* | ||||
| import io.ktor.client.plugins.logging.* | ||||
| import io.ktor.client.request.* | ||||
| import io.ktor.client.statement.* | ||||
| import io.ktor.http.* | ||||
| import io.ktor.serialization.kotlinx.json.* | ||||
| import io.ktor.client.HttpClient | ||||
| import io.ktor.client.engine.cio.CIO | ||||
| import io.ktor.client.engine.cio.CIOEngineConfig | ||||
| import io.ktor.client.plugins.HttpRequestRetry | ||||
| import io.ktor.client.plugins.HttpTimeout | ||||
| import io.ktor.client.plugins.auth.providers.BasicAuthCredentials | ||||
| import io.ktor.client.plugins.cache.HttpCache | ||||
| import io.ktor.client.plugins.contentnegotiation.ContentNegotiation | ||||
| import io.ktor.client.plugins.cookies.HttpCookies | ||||
| import io.ktor.client.plugins.logging.LogLevel | ||||
| import io.ktor.client.plugins.logging.Logger | ||||
| import io.ktor.client.plugins.logging.Logging | ||||
| import io.ktor.client.request.get | ||||
| import io.ktor.client.request.headers | ||||
| import io.ktor.client.request.parameter | ||||
| import io.ktor.client.statement.HttpResponse | ||||
| import io.ktor.http.HttpHeaders | ||||
| import io.ktor.http.HttpStatusCode | ||||
| import io.ktor.http.Parameters | ||||
| import io.ktor.serialization.kotlinx.json.json | ||||
| import io.ktor.util.encodeBase64 | ||||
| import io.ktor.utils.io.charsets.Charsets | ||||
| import io.ktor.utils.io.core.toByteArray | ||||
| import kotlinx.coroutines.CoroutineScope | ||||
| import kotlinx.coroutines.Dispatchers | ||||
| import kotlinx.coroutines.launch | ||||
| import kotlinx.serialization.json.Json | ||||
|  | ||||
| expect fun setupInsecureHTTPEngine(config: CIOEngineConfig) | ||||
|  | ||||
| class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|  | ||||
|     var client = createHttpClient() | ||||
|  | ||||
|     private fun createHttpClient(): HttpClient { | ||||
|         val client = HttpClient { | ||||
|     fun createHttpClient() = | ||||
|         HttpClient(CIO) { | ||||
|             if (appSettingsService.getSelfSigned()) { | ||||
|                 engine { | ||||
|                     setupInsecureHTTPEngine(this) | ||||
|                 } | ||||
|             } | ||||
|             install(ContentNegotiation) { | ||||
|                 install(HttpCache) | ||||
|                 json(Json { | ||||
|                     prettyPrint = true | ||||
|                     isLenient = true | ||||
|                     ignoreUnknownKeys = true | ||||
|                     explicitNulls = false | ||||
|                 }) | ||||
|             } | ||||
|             install(Logging) { | ||||
| @@ -55,7 +75,7 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                     Napier.i("Will modify", tag = "HttpSend") | ||||
|                     CoroutineScope(Dispatchers.Main).launch { | ||||
|                         Napier.i("Will login", tag = "HttpSend") | ||||
|                         this@SelfossApi.login() | ||||
|                         login() | ||||
|                         Napier.i("Did login", tag = "HttpSend") | ||||
|                     } | ||||
|                 } | ||||
| @@ -63,9 +83,6 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|             expectSuccess = false | ||||
|         } | ||||
|  | ||||
|         return client | ||||
|     } | ||||
|  | ||||
|     fun url(path: String) = | ||||
|         "${appSettingsService.getBaseUrl()}$path" | ||||
|  | ||||
| @@ -74,6 +91,13 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|         client = createHttpClient() | ||||
|     } | ||||
|  | ||||
|     fun constructBasicAuthValue(credentials: BasicAuthCredentials): String { | ||||
|         val authString = "${credentials.username}:${credentials.password}" | ||||
|         val authBuf = authString.toByteArray(Charsets.UTF_8).encodeBase64() | ||||
|  | ||||
|         return "Basic $authBuf" | ||||
|     } | ||||
|  | ||||
|     // Api version was introduces after the POST login, so when there is a version, it should be available | ||||
|     private fun shouldHavePostLogin() = appSettingsService.getApiVersion() != -1 | ||||
|     private fun hasLoginInfo() = | ||||
| @@ -96,11 +120,23 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|     private suspend fun getLogin() = maybeResponse(client.tryToGet(url("/login")) { | ||||
|         parameter("username", appSettingsService.getUserName()) | ||||
|         parameter("password", appSettingsService.getPassword()) | ||||
|         if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|             headers { | ||||
|                 append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                 ) | ||||
|             } | ||||
|         } | ||||
|     }) | ||||
|  | ||||
|     private suspend fun postLogin() = maybeResponse(client.tryToPost(url("/login")) { | ||||
|         parameter("username", appSettingsService.getUserName()) | ||||
|         parameter("password", appSettingsService.getPassword()) | ||||
|         if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|             headers { | ||||
|                 append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                 ) | ||||
|             } | ||||
|         } | ||||
|     }) | ||||
|  | ||||
|     private fun shouldHaveNewLogout() = | ||||
| @@ -114,9 +150,23 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|         } | ||||
|  | ||||
|     private suspend fun maybeLogoutIfAvailable() = | ||||
|         responseOrSuccessIf404(client.tryToGet(url("/logout"))) | ||||
|         responseOrSuccessIf404(client.tryToGet(url("/logout")) { | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     private suspend fun doLogout() = maybeResponse(client.tryToDelete(url("/api/session/current"))) | ||||
|     private suspend fun doLogout() = maybeResponse(client.tryToDelete(url("/api/session/current")) { | ||||
|         if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|             headers { | ||||
|                 append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                 ) | ||||
|             } | ||||
|         } | ||||
|     }) | ||||
|  | ||||
|     suspend fun getItems( | ||||
|         type: String, | ||||
| @@ -139,6 +189,12 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|             parameter("updatedsince", updatedSince) | ||||
|             parameter("items", items ?: appSettingsService.getItemsNumber()) | ||||
|             parameter("offset", offset) | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun getItemsWithoutCatch(): StatusAndData<List<SelfossModel.Item>> = | ||||
| @@ -149,6 +205,12 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|             } | ||||
|             parameter("type", "all") | ||||
|             parameter("items", 1) | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun stats(): StatusAndData<SelfossModel.Stats> = | ||||
| @@ -157,6 +219,12 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                 parameter("username", appSettingsService.getUserName()) | ||||
|                 parameter("password", appSettingsService.getPassword()) | ||||
|             } | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun tags(): StatusAndData<List<SelfossModel.Tag>> = | ||||
| @@ -165,6 +233,12 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                 parameter("username", appSettingsService.getUserName()) | ||||
|                 parameter("password", appSettingsService.getPassword()) | ||||
|             } | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun update(): StatusAndData<String> = | ||||
| @@ -173,6 +247,12 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                 parameter("username", appSettingsService.getUserName()) | ||||
|                 parameter("password", appSettingsService.getPassword()) | ||||
|             } | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun spouts(): StatusAndData<Map<String, SelfossModel.Spout>> = | ||||
| @@ -181,18 +261,51 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                 parameter("username", appSettingsService.getUserName()) | ||||
|                 parameter("password", appSettingsService.getPassword()) | ||||
|             } | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun sources(): StatusAndData<ArrayList<SelfossModel.Source>> = | ||||
|     suspend fun sourcesStats(): StatusAndData<ArrayList<SelfossModel.SourceStats>> = | ||||
|         bodyOrFailure(client.tryToGet(url("/sources/stats")) { | ||||
|             if (!shouldHavePostLogin()) { | ||||
|                 parameter("username", appSettingsService.getUserName()) | ||||
|                 parameter("password", appSettingsService.getPassword()) | ||||
|             } | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun sourcesDetailed(): StatusAndData<ArrayList<SelfossModel.SourceDetail>> = | ||||
|         bodyOrFailure(client.tryToGet(url("/sources/list")) { | ||||
|             if (!shouldHavePostLogin()) { | ||||
|                 parameter("username", appSettingsService.getUserName()) | ||||
|                 parameter("password", appSettingsService.getPassword()) | ||||
|             } | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun apiInformation(): StatusAndData<SelfossModel.ApiInformation> = | ||||
|         bodyOrFailure(client.tryToGet(url("/api/about"))) | ||||
|         bodyOrFailure(client.tryToGet(url("/api/about")) { | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun markAsRead(id: String): SuccessResponse = | ||||
|         maybeResponse(client.tryToPost(url("/mark/$id")) { | ||||
| @@ -200,6 +313,12 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                 parameter("username", appSettingsService.getUserName()) | ||||
|                 parameter("password", appSettingsService.getPassword()) | ||||
|             } | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun unmarkAsRead(id: String): SuccessResponse = | ||||
| @@ -208,6 +327,12 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                 parameter("username", appSettingsService.getUserName()) | ||||
|                 parameter("password", appSettingsService.getPassword()) | ||||
|             } | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun starr(id: String): SuccessResponse = | ||||
| @@ -216,6 +341,12 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                 parameter("username", appSettingsService.getUserName()) | ||||
|                 parameter("password", appSettingsService.getPassword()) | ||||
|             } | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun unstarr(id: String): SuccessResponse = | ||||
| @@ -224,6 +355,12 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                 parameter("username", appSettingsService.getUserName()) | ||||
|                 parameter("password", appSettingsService.getPassword()) | ||||
|             } | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
|  | ||||
|     suspend fun markAllAsRead(ids: List<String>): SuccessResponse = | ||||
| @@ -235,6 +372,14 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                     append("password", appSettingsService.getPassword()) | ||||
|                 } | ||||
|                 ids.map { append("ids[]", it) } | ||||
|             }, | ||||
|             block = { | ||||
|                 if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                     headers { | ||||
|                         append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                         ) | ||||
|                     } | ||||
|                 } | ||||
|             } | ||||
|         )) | ||||
|  | ||||
| @@ -270,6 +415,14 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                 append("url", url) | ||||
|                 append("spout", spout) | ||||
|                 append(tagsParamName, tags) | ||||
|             }, | ||||
|             block = { | ||||
|                 if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                     headers { | ||||
|                         append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                         ) | ||||
|                     } | ||||
|                 } | ||||
|             } | ||||
|         ) | ||||
|  | ||||
| @@ -307,6 +460,14 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                 append("url", url) | ||||
|                 append("spout", spout) | ||||
|                 append(tagsParamName, tags) | ||||
|             }, | ||||
|             block = { | ||||
|                 if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                     headers { | ||||
|                         append(HttpHeaders.Authorization, constructBasicAuthValue(BasicAuthCredentials(username = appSettingsService.getBasicUserName(), password = appSettingsService.getBasicPassword())) | ||||
|                         ) | ||||
|                     } | ||||
|                 } | ||||
|             } | ||||
|         ) | ||||
|  | ||||
| @@ -316,5 +477,18 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|                 parameter("username", appSettingsService.getUserName()) | ||||
|                 parameter("password", appSettingsService.getPassword()) | ||||
|             } | ||||
|             if (appSettingsService.getBasicUserName().isNotEmpty() && appSettingsService.getBasicPassword().isNotEmpty()) { | ||||
|                 headers { | ||||
|                     append( | ||||
|                         HttpHeaders.Authorization, | ||||
|                         constructBasicAuthValue( | ||||
|                             BasicAuthCredentials( | ||||
|                                 username = appSettingsService.getBasicUserName(), | ||||
|                                 password = appSettingsService.getBasicPassword() | ||||
|                             ) | ||||
|                         ) | ||||
|                     ) | ||||
|                 } | ||||
|             } | ||||
|         }) | ||||
| } | ||||
| @@ -0,0 +1,70 @@ | ||||
| package bou.amine.apps.readerforselfossv2.service | ||||
|  | ||||
| import com.russhwolf.settings.Settings | ||||
|  | ||||
| // This will be used in ACRA process. For now, it does nothing. | ||||
| // This is to fix ACRA not sending reports anymore. | ||||
| // See https://www.acra.ch/docs/Troubleshooting-Guide#applicationoncreate | ||||
| class ACRASettings : Settings { | ||||
|     override val keys: Set<String> = emptySet() | ||||
|     override val size: Int = 0 | ||||
|  | ||||
|     override fun clear() { | ||||
|         // Nothing | ||||
|     } | ||||
|  | ||||
|     override fun getBoolean(key: String, defaultValue: Boolean): Boolean = false | ||||
|  | ||||
|     override fun getBooleanOrNull(key: String): Boolean? = null | ||||
|  | ||||
|     override fun getDouble(key: String, defaultValue: Double): Double = 0.0 | ||||
|  | ||||
|     override fun getDoubleOrNull(key: String): Double? = null | ||||
|  | ||||
|     override fun getFloat(key: String, defaultValue: Float): Float = 0.0F | ||||
|  | ||||
|     override fun getFloatOrNull(key: String): Float? = null | ||||
|  | ||||
|     override fun getInt(key: String, defaultValue: Int): Int = 0 | ||||
|  | ||||
|     override fun getIntOrNull(key: String): Int? = null | ||||
|  | ||||
|     override fun getLong(key: String, defaultValue: Long): Long = 0 | ||||
|  | ||||
|     override fun getLongOrNull(key: String): Long? = null | ||||
|  | ||||
|     override fun getString(key: String, defaultValue: String): String = "0" | ||||
|  | ||||
|     override fun getStringOrNull(key: String): String? = null | ||||
|  | ||||
|     override fun hasKey(key: String): Boolean = false | ||||
|  | ||||
|     override fun putBoolean(key: String, value: Boolean) { | ||||
|         // Nothing | ||||
|     } | ||||
|  | ||||
|     override fun putDouble(key: String, value: Double) { | ||||
|         // Nothing | ||||
|     } | ||||
|  | ||||
|     override fun putFloat(key: String, value: Float) { | ||||
|         // Nothing | ||||
|     } | ||||
|  | ||||
|     override fun putInt(key: String, value: Int) { | ||||
|         // Nothing | ||||
|     } | ||||
|  | ||||
|     override fun putLong(key: String, value: Long) { | ||||
|         // Nothing | ||||
|     } | ||||
|  | ||||
|     override fun putString(key: String, value: String) { | ||||
|         // Nothing | ||||
|     } | ||||
|  | ||||
|     override fun remove(key: String) { | ||||
|         // Nothing | ||||
|     } | ||||
|  | ||||
| } | ||||
| @@ -2,15 +2,18 @@ package bou.amine.apps.readerforselfossv2.service | ||||
|  | ||||
| import com.russhwolf.settings.Settings | ||||
|  | ||||
| class AppSettingsService { | ||||
|     val settings: Settings = Settings() | ||||
| class AppSettingsService(acraSenderServiceProcess: Boolean = false) { | ||||
|     val settings: Settings = if (acraSenderServiceProcess) { ACRASettings() } else { Settings() } | ||||
|  | ||||
|     // Api related | ||||
|     private var _apiVersion: Int = -1 | ||||
|     private var _publicAccess: Boolean? = null | ||||
|     private var _selfSigned: Boolean? = null | ||||
|     private var _baseUrl: String = "" | ||||
|     private var _userName: String = "" | ||||
|     private var _basicUserName: String = "" | ||||
|     private var _password: String = "" | ||||
|     private var _basicPassword: String = "" | ||||
|  | ||||
|     // User settings related | ||||
|     private var _itemsCaching: Boolean? = null | ||||
| @@ -75,6 +78,22 @@ class AppSettingsService { | ||||
|         _publicAccess = settings.getBoolean(API_PUBLIC_ACCESS, false) | ||||
|     } | ||||
|  | ||||
|     fun getSelfSigned(): Boolean { | ||||
|         if (_selfSigned == null) { | ||||
|             refreshSelfSigned() | ||||
|         } | ||||
|         return _selfSigned!! | ||||
|     } | ||||
|  | ||||
|     fun updateSelfSigned(selfSigned: Boolean) { | ||||
|         settings.putBoolean(API_SELF_SIGNED, selfSigned) | ||||
|         refreshSelfSigned() | ||||
|     } | ||||
|  | ||||
|     private fun refreshSelfSigned() { | ||||
|         _selfSigned = settings.getBoolean(API_SELF_SIGNED, false) | ||||
|     } | ||||
|  | ||||
|     fun getBaseUrl(): String { | ||||
|         if (_baseUrl.isEmpty()) { | ||||
|             refreshBaseUrl() | ||||
| @@ -96,6 +115,20 @@ class AppSettingsService { | ||||
|         return _password | ||||
|     } | ||||
|  | ||||
|     fun getBasicUserName(): String { | ||||
|         if (_basicUserName.isEmpty()) { | ||||
|             refreshBasicUsername() | ||||
|         } | ||||
|         return _basicUserName | ||||
|     } | ||||
|  | ||||
|     fun getBasicPassword(): String { | ||||
|         if (_basicPassword.isEmpty()) { | ||||
|             refreshBasicPassword() | ||||
|         } | ||||
|         return _basicPassword | ||||
|     } | ||||
|  | ||||
|     fun getItemsNumber(): Int { | ||||
|         if (_itemsNumber == null) { | ||||
|             refreshItemsNumber() | ||||
| @@ -149,6 +182,14 @@ class AppSettingsService { | ||||
|         _password = settings.getString(PASSWORD, "") | ||||
|     } | ||||
|  | ||||
|     private fun refreshBasicUsername() { | ||||
|         _basicUserName = settings.getString(BASIC_LOGIN, "") | ||||
|     } | ||||
|  | ||||
|     private fun refreshBasicPassword() { | ||||
|         _basicPassword = settings.getString(BASIC_PASSWORD, "") | ||||
|     } | ||||
|  | ||||
|     private fun refreshArticleViewerEnabled() { | ||||
|         _articleViewer = settings.getBoolean(PREFER_ARTICLE_VIEWER, true) | ||||
|     } | ||||
| @@ -354,9 +395,12 @@ class AppSettingsService { | ||||
|     fun refreshApiSettings() { | ||||
|         refreshPassword() | ||||
|         refreshUsername() | ||||
|         refreshBasicUsername() | ||||
|         refreshBasicPassword() | ||||
|         refreshBaseUrl() | ||||
|         refreshApiVersion() | ||||
|         refreshPublicAccess() | ||||
|         refreshSelfSigned() | ||||
|     } | ||||
|  | ||||
|     fun refreshUserSettings() { | ||||
| @@ -387,7 +431,17 @@ class AppSettingsService { | ||||
|         login: String, | ||||
|         password: String | ||||
|     ) { | ||||
|         settings.putString(BASE_URL, url) | ||||
|         val regex = """\/\/(\D+):(\D+)@""".toRegex() | ||||
|         val matchResult = regex.find(url) | ||||
|         if (matchResult != null) { | ||||
|             val (basicLogin, basicPassword) = matchResult.destructured | ||||
|             settings.putString(BASIC_LOGIN, basicLogin) | ||||
|             settings.putString(BASIC_PASSWORD, basicPassword) | ||||
|             val urlWithoutBasicAuth = url.replace(regex, "//") | ||||
|             settings.putString(BASE_URL, urlWithoutBasicAuth) | ||||
|         } else { | ||||
|             settings.putString(BASE_URL, url) | ||||
|         } | ||||
|         settings.putString(LOGIN, login) | ||||
|         settings.putString(PASSWORD, password) | ||||
|         refreshApiSettings() | ||||
| @@ -397,6 +451,8 @@ class AppSettingsService { | ||||
|         settings.remove(BASE_URL) | ||||
|         settings.remove(LOGIN) | ||||
|         settings.remove(PASSWORD) | ||||
|         settings.remove(BASIC_LOGIN) | ||||
|         settings.remove(BASIC_PASSWORD) | ||||
|         refreshApiSettings() | ||||
|     } | ||||
|  | ||||
| @@ -430,6 +486,8 @@ class AppSettingsService { | ||||
|  | ||||
|         const val API_PUBLIC_ACCESS = "apiPublicAccess" | ||||
|  | ||||
|         const val API_SELF_SIGNED = "apiSelfSigned" | ||||
|  | ||||
|         const val API_ITEMS_NUMBER = "prefer_api_items_number" | ||||
|  | ||||
|         const val API_TIMEOUT = "api_timeout" | ||||
| @@ -440,6 +498,10 @@ class AppSettingsService { | ||||
|  | ||||
|         const val PASSWORD = "password" | ||||
|  | ||||
|         const val BASIC_LOGIN = "basic_login" | ||||
|  | ||||
|         const val BASIC_PASSWORD = "basic_password" | ||||
|  | ||||
|         const val PREFER_ARTICLE_VIEWER = "prefer_article_viewer" | ||||
|  | ||||
|         const val CARD_VIEW_ACTIVE = "card_view_active" | ||||
| @@ -470,7 +532,6 @@ class AppSettingsService { | ||||
|  | ||||
|         const val PERIODIC_REFRESH_MINUTES = "periodic_refresh_minutes" | ||||
|  | ||||
|  | ||||
|         const val INFINITE_LOADING = "infinite_loading" | ||||
|  | ||||
|         const val ITEMS_CACHING = "items_caching" | ||||
|   | ||||
| @@ -12,24 +12,25 @@ fun TAG.toView(): SelfossModel.Tag = | ||||
|         this.unread.toInt() | ||||
|     ) | ||||
|  | ||||
| fun SOURCE.toView(): SelfossModel.Source = | ||||
|     SelfossModel.Source( | ||||
| fun SOURCE.toView(): SelfossModel.SourceDetail = | ||||
|     SelfossModel.SourceDetail( | ||||
|         this.id.toInt(), | ||||
|         this.title, | ||||
|         this.tags.split(","), | ||||
|         null, | ||||
|         this.tags?.split(","), | ||||
|         this.spout, | ||||
|         this.error, | ||||
|         this.icon, | ||||
|         if (this.url != null) SelfossModel.SourceParams(this.url) else null | ||||
|     ) | ||||
|  | ||||
| fun SelfossModel.Source.toEntity(): SOURCE = | ||||
| fun SelfossModel.SourceDetail.toEntity(): SOURCE = | ||||
|     SOURCE( | ||||
|         this.id.toString(), | ||||
|         this.title.getHtmlDecoded(), | ||||
|         this.tags.joinToString(","), | ||||
|         this.spout, | ||||
|         this.error, | ||||
|         this.tags?.joinToString(",").orEmpty(), | ||||
|         this.spout.orEmpty(), | ||||
|         this.error.orEmpty(), | ||||
|         this.icon.orEmpty(), | ||||
|         this.params?.url | ||||
|     ) | ||||
|   | ||||
| @@ -0,0 +1,6 @@ | ||||
| package bou.amine.apps.readerforselfossv2.rest | ||||
|  | ||||
| import io.ktor.client.engine.cio.CIOEngineConfig | ||||
|  | ||||
| actual fun setupInsecureHTTPEngine(config: CIOEngineConfig) { | ||||
| } | ||||
| @@ -0,0 +1,6 @@ | ||||
| package bou.amine.apps.readerforselfossv2.rest | ||||
|  | ||||
| import io.ktor.client.engine.cio.CIOEngineConfig | ||||
|  | ||||
| actual fun setupInsecureHTTPEngine(config: CIOEngineConfig) { | ||||
| } | ||||
		Reference in New Issue
	
	Block a user