Removed Debouncing mechanism

.github/workflows/ci.yml

@@ -13,7 +13,7 @@ jobs:
     name: Automated tests
     strategy:
       matrix:
-        api-level: [ 21, 30, 33 ]
+        api-level: [ 24, 30, 33 ]
       fail-fast: false
     runs-on: macos-11
     steps:
@@ -79,16 +79,16 @@ jobs:
           path: screencap.png
 
       - name: create unit coverage
-        if: ${{ matrix.api-level==21 }}
+        if: ${{ matrix.api-level==24 }}
         run: ./gradlew testDebugUnitTest testCustomexampleDebugUnitTest
 
       - name: Upload coverage to Codecov
-        if: ${{ matrix.api-level==21 }}
+        if: ${{ matrix.api-level==24 }}
         uses: codecov/codecov-action@v3
 
       - name: Upload Coverage to GH-Actions
         uses: actions/upload-artifact@v3
-        if: ${{ matrix.api-level==21 }}
+        if: ${{ matrix.api-level==24 }}
         with:
           name: Tests Coverage Report
           path: |

.github/workflows/pull_request.yml

@@ -72,7 +72,7 @@ jobs:
             ${{ runner.os }}-gradle-
 
       - name: Static Analysis
-        run: ./gradlew detekt
+        run: ./gradlew detektDebug detektCustomExampleDebug
       - name: Upload Lint Reports
         uses: actions/upload-artifact@v3
         if: ${{ always() }}

.github/workflows/release.yml

@@ -1,13 +1,8 @@
 name: Publish App
 
 on:
-  push:
-    branches-ignore:
-      - '*'
-      - '*/**'
-    tags:
-      - '*'
-      - '*/**'
+  release:
+    types: [published]
 
 jobs:
   publish:
@@ -32,25 +27,24 @@ jobs:
           echo "$SSH_KEY" | base64 -d > ssh_key
           chmod 600 ssh_key
 
-      - name: Set tag variable
-        run: echo "TAG=$(echo ${GITHUB_REF:10})" >> $GITHUB_ENV
-
       - name: Publish APK to download.kiwix.org
         env:
           KEY_ALIAS: ${{ secrets.KEY_ALIAS }}
           KEY_PASSWORD: ${{ secrets.KEY_PASSWORD }}
           KEY_STORE_PASSWORD: ${{ secrets.KEY_STORE_PASSWORD }}
           UNIVERSAL_RELEASE_APK: app/build/outputs/apk/release/*universal*.apk
+          ARCHIVE_NAME: kiwix-${{ github.event.release.tag_name }}.apk
         run: |
           ./gradlew assembleRelease
-          cp $UNIVERSAL_RELEASE_APK kiwix-${TAG}.apk
-          scp -P 30022 -vrp -i ssh_key -o StrictHostKeyChecking=no kiwix-${TAG}.apk ci@master.download.kiwix.org:/data/download/release/kiwix-android/
+          cp ${UNIVERSAL_RELEASE_APK} ${ARCHIVE_NAME}
+          scp -P 30022 -vrp -i ssh_key -o StrictHostKeyChecking=no "$ARCHIVE_NAME" ci@master.download.kiwix.org:/data/download/release/kiwix-android/
 
-      # This is necessary for F-Droid
-      - name: Publish "versionInfo" to download.kiwix.org
-        run: |
-          ./gradlew generateVersionCodeAndName
-          scp -P 30022 -vrp -i ssh_key -o StrictHostKeyChecking=no VERSION_INFO ci@master.download.kiwix.org:/data/download/release/kiwix-android/
+#      This is temporary, once we will publish 3.7.0 then we will uncommented this code.
+#      # This is necessary for F-Droid
+#      - name: Publish "versionInfo" to download.kiwix.org
+#        run: |
+#          ./gradlew generateVersionCodeAndName
+#          scp -P 30022 -vrp -i ssh_key -o StrictHostKeyChecking=no VERSION_INFO ci@master.download.kiwix.org:/data/download/release/kiwix-android/
 
       - name: Publish to GitHub
         uses: ncipollo/release-action@v1

CHANGELOG

@@ -1,5 +1,68 @@
-3.7.0
+3.9.0
+New Features:
+* Added a start-up message to inform users about restrictions on sideloading ZIM files in the Google Store app. (/~https://github.com/kiwix/kiwix-android/pull/3583)
+* Introduced the "Disable title" feature for custom apps. (/~https://github.com/kiwix/kiwix-android/pull/3566)
+* Introduced the "Disable external links" feature for custom apps. (/~https://github.com/kiwix/kiwix-android/pull/3595)
+* Added the show/hide "About the app" option in the navigation for custom apps. (/~https://github.com/kiwix/kiwix-android/pull/3597)
+* "Support Kiwix", "About the app" are configurable for custom apps. (/~https://github.com/kiwix/kiwix-android/pull/3602)
+
+Bug Fixes:
+* Application crash occurring when deleting letters in a search. (/~https://github.com/kiwix/kiwix-android/pull/3592)
+* Enhanced the permission-asking scenario for the Kiwix app. (/~https://github.com/kiwix/kiwix-android/pull/3593)
+* Zimit archives were not displaying correctly. (/~https://github.com/kiwix/kiwix-android/pull/3580)
+* Addressed the issue of not being able to find ZIM files on the SD card that are copied to the app-specific directory in the Play Store variant. (/~https://github.com/kiwix/kiwix-android/pull/3582)
+* Removed the "Help" menu from the sidebar for custom apps. (/~https://github.com/kiwix/kiwix-android/pull/3594)
+* Updated the "Support Kiwix" icon in sidebar. (/~https://github.com/kiwix/kiwix-android/pull/3588)
+
+3.8.1
+Bug Fixes:
+* Application crash occurring when users rapidly typed during a search for an entry. (/~https://github.com/kiwix/kiwix-android/pull/3558)
+* Play Store deep linking issue on Android 12 and above. (/~https://github.com/kiwix/kiwix-android/pull/3550)
+
+3.8.0
+New Features:
+* Download pause/resume feature. (/~https://github.com/kiwix/kiwix-android/pull/3459)
+* Added support to read ZIM files from USB-OTG stick. (/~https://github.com/kiwix/kiwix-android/pull/3471)
+* Endless suggestion list while searching articles. (/~https://github.com/kiwix/kiwix-android/pull/3217)
+* Refactored LocalFileTransfer UI to educate users on how to use this feature. (/~https://github.com/kiwix/kiwix-android/pull/3497)
 
+Bug Fixes:
+* All books notes switch is always enabled. (/~https://github.com/kiwix/kiwix-android/pull/3449)
+* Download remained pending indefinitely if we canceled it from the notification. (/~https://github.com/kiwix/kiwix-android/pull/3451)
+* Enhanced permission asking scenario for custom apps. (/~https://github.com/kiwix/kiwix-android/pull/3439)
+* If the storage is already configured, it was again showing while downloading. (/~https://github.com/kiwix/kiwix-android/pull/3464)
+* The keyboard repeatedly disappears and reappears while searching articles. (/~https://github.com/kiwix/kiwix-android/pull/3469)
+* Bookmark toggle is not ON for saved bookmarks. (/~https://github.com/kiwix/kiwix-android/pull/3473)
+* Article opening after search page result leads to a blank page. (/~https://github.com/kiwix/kiwix-android/pull/3217)
+* Random article function not working. (/~https://github.com/kiwix/kiwix-android/pull/3217)
+* 500 Error happens on the server. (/~https://github.com/kiwix/kiwix-android/pull/3217)
+* Unable to open the support link with the default browser. (/~https://github.com/kiwix/kiwix-android/pull/3504)
+* Unable to upload APK on the play store. (/~https://github.com/kiwix/kiwix-android/pull/3506)
+* Hosted Books don't update on Application. (/~https://github.com/kiwix/kiwix-android/pull/3218)
+* Cannot open ZIM files from phone storage via the file picker in Android 13. (/~https://github.com/kiwix/kiwix-android/pull/3518)
+* The redundancy of the 'All-Books-Mode' option for Bookmarks, Notes, and History in custom apps. (/~https://github.com/kiwix/kiwix-android/pull/3525)
+* The same Note saves multiple times while we are updating existing notes. (/~https://github.com/kiwix/kiwix-android/issues/3519)
+* Reduced mass-storage consumption with Android custom apps with embedded ZIM. (/~https://github.com/kiwix/kiwix-android/pull/3516)
+* Hitting back in the custom apps exits it. (/~https://github.com/kiwix/kiwix-android/pull/3512)
+* Multiple memory leaks
+* Updated README file to show file picker restriction. (/~https://github.com/kiwix/kiwix-android/pull/3414)
++ More
+
+Compilation/CI/CD:
+* Adapted libkiwix `1.0.0`. (/~https://github.com/kiwix/kiwix-android/pull/3217)
+* Deprecated code check in CI. (/~https://github.com/kiwix/kiwix-android/pull/3440)
+* Added/improved test cases.
+
+3.7.1
+* FIX: Useless permission request at start for custom apps
+* FIX: Few code deprecations
+* FIX: Broken back button in custom apps
+* FIX: Broken not saving notes in custom apps
+* FIX: Application crashing while adding notes
+* FIX: Partly broken fullscreen feature
+* FIX: Invisible suggestion box in custom apps
+
+3.7.0
 New Features:
 * Support for Android 13.
 * Scrollbar indicator in the download screen

README.md

@@ -11,7 +11,31 @@
 Kiwix is an offline reader for Web content. One of its main purposes
 is to make Wikipedia available offline. This is done by reading the
 content of a file in the ZIM format, a highly compressed open format
-with additional metadata. This is the version for Android.
+with additional metadata.
+
+This is the version for Android, with support versions ranging from 7
+to 13 (like [mentioned
+here](/~https://github.com/kiwix/kiwix-android/blob/develop/buildSrc/src/main/kotlin/Config.kt)).
+
+**Important Note**: Starting from Android 11, the ZIM file picker
+feature has been restricted in the Play Store variant due to Play
+Store policies.  This means that users running Android 11 and above
+will not be able to load ZIM files from internal/external storage
+directly within the app if they have downloaded Kiwix from the Google
+Play Store.  This restriction is in place to comply with the Play
+Store policies.  The Play Store variant of Kiwix does not require the
+`MANAGE_EXTERNAL_STORAGE` permission anymore, which is necessary to
+scan storage and access ZIM files at arbitrary locations.  Therefore,
+the storage scanning & file picking functionalities are not available
+in this variant anymore.  To use the full version of Kiwix and benefit
+of the ZIM file picker feature, you can download it directly from the
+[official
+repository](https://download.kiwix.org/release/kiwix-android/) or use
+[F-Droid](https://fdroid.kiwix.org). We understand that this
+restriction may cause inconvenience, but it is necessary to comply
+with the Play Store policies and ensure a smooth user experience.  We
+recommend using the official version of the app available on our
+website to access the complete set of features.
 
 Kiwix Android is written in [Kotlin](https://kotlinlang.org/)
 
@@ -39,13 +63,14 @@ If you are interested in our custom apps they have their own repo [kiwix-android
 
 ## Libraries Used
 
+- [Libkiwix](/~https://github.com/kiwix/java-libkiwix) - Kotlin/Java binding for the core Kiwix library
 - [Dagger 2](/~https://github.com/google/dagger) - A fast dependency injector for Android and Java
 - [Retrofit](https://square.github.io/retrofit/) - Retrofit turns your REST API into a Java interface
 - [OkHttp](/~https://github.com/square/okhttp) - An HTTP+SPDY client for Android and Java applications
 - [Butterknife](https://jakewharton.github.io/butterknife/) - View "injection" library for Android
 - [Mockito](/~https://github.com/mockito/mockito) - Most popular Mocking framework for unit tests written in Java
 - [RxJava](/~https://github.com/ReactiveX/RxJava) - Reactive Extensions for the JVM – a library for composing asynchronous and event-based programs using observable sequences for the Java VM.
-- [ObjectBox](/~https://github.com/objectbox/objectbox-java) - Reactive NoSQL Databse
+- [ObjectBox](/~https://github.com/objectbox/objectbox-java) - Reactive NoSQL Database
 - [MockK](/~https://github.com/mockk/mockk) - Kotlin mocking library that allows mocking of final classes by default.
 - [JUnit5](/~https://github.com/junit-team/junit5/) - The next generation of JUnit
 - [AssertJ](/~https://github.com/joel-costigliola/assertj-core) - Fluent assertions for test code

app/build.gradle.kts

@@ -8,13 +8,7 @@ plugins.apply(KiwixConfigurationPlugin::class)
 
 apply(from = rootProject.file("jacoco.gradle"))
 
-ext {
-  set("versionMajor", 3)
-  set("versionMinor", 7)
-  set("versionPatch", 0)
-}
-
-fun generateVersionName() = "${ext["versionMajor"]}.${ext["versionMinor"]}.${ext["versionPatch"]}"
+fun generateVersionName() = "${Config.versionMajor}.${Config.versionMinor}.${Config.versionPatch}"
 
 /*
 * max version code: 21-0-0-00-00-00
@@ -30,9 +24,9 @@ fun generateVersionName() = "${ext["versionMajor"]}.${ext["versionMinor"]}.${ext
 
 fun generateVersionCode() =
   20 * 10000 +
-    ext["versionMajor"] as Int * 10000 +
-    ext["versionMinor"] as Int * 100 +
-    ext["versionPatch"] as Int
+    Config.versionMajor * 10000 +
+    Config.versionMinor * 100 +
+    Config.versionPatch
 
 val apkPrefix get() = System.getenv("TAG") ?: "kiwix"
 
@@ -73,7 +67,7 @@ android {
     }
     create("nightly") {
       initWith(getByName("debug"))
-      setMatchingFallbacks("debug")
+      matchingFallbacks += "debug"
     }
   }
   bundle {
@@ -93,7 +87,7 @@ android {
 play {
   enabled.set(true)
   serviceAccountCredentials.set(file("../playstore.json"))
-  track.set("alpha")
+  track.set("internal")
   releaseStatus.set(com.github.triplet.gradle.androidpublisher.ReleaseStatus.DRAFT)
   resolutionStrategy.set(com.github.triplet.gradle.androidpublisher.ResolutionStrategy.FAIL)
 }

app/detekt_baseline.xml

@@ -11,7 +11,8 @@
     <ID>MagicNumber:ShareFiles.kt$ShareFiles$24</ID>
     <ID>MagicNumber:ZimManageViewModel.kt$ZimManageViewModel$5</ID>
     <ID>MagicNumber:ZimManageViewModel.kt$ZimManageViewModel$500</ID>
-    <ID>NestedBlockDepth:LocalLibraryFragment.kt$LocalLibraryFragment$private fun checkPermissions()</ID>
+    <ID>MagicNumber:LocalFileTransferFragment.kt$LocalFileTransferFragment$500</ID>
+    <ID>NestedBlockDepth:LocalLibraryFragment.kt$LocalLibraryFragment$private fun checkManageExternalStoragePermission()</ID>
     <ID>NestedBlockDepth:PeerGroupHandshake.kt$PeerGroupHandshake$private fun readHandshakeAndExchangeMetaData(): InetAddress?</ID>
     <ID>NestedBlockDepth:ReceiverHandShake.kt$ReceiverHandShake$override fun exchangeFileTransferMetadata(inputStream: InputStream, outputStream: OutputStream)</ID>
     <ID>PackageNaming:AvailableSpaceCalculator.kt$package
@@ -25,10 +26,6 @@
       org.kiwix.kiwixmobile.zimManager.fileselectView</ID>
     <ID>PackageNaming:FileSystemChecker.kt$package org.kiwix.kiwixmobile.zimManager</ID>
     <ID>PackageNaming:FileWritingFileSystemChecker.kt$package org.kiwix.kiwixmobile.zimManager</ID>
-    <ID>PackageNaming:HotspotNotificationManager.kt$package org.kiwix.kiwixmobile.webserver.wifi_hotspot</ID>
-    <ID>PackageNaming:HotspotStateReceiver.kt$package org.kiwix.kiwixmobile.webserver.wifi_hotspot</ID>
-    <ID>PackageNaming:IpAddressCallbacks.kt$package org.kiwix.kiwixmobile.webserver.wifi_hotspot</ID>
-    <ID>PackageNaming:HotspotService.kt$package org.kiwix.kiwixmobile.webserver.wifi_hotspot</ID>
     <ID>PackageNaming:LibraryAdapter.kt$package
       org.kiwix.kiwixmobile.zimManager.libraryView.adapter</ID>
     <ID>PackageNaming:LibraryDelegate.kt$package

app/proguard-rules.pro

@@ -27,8 +27,11 @@
 
 -keepclassmembers class org.kiwix.videowebview.VideoEnabledWebView$JavascriptInterface { public *; }
 
-#keep everything in kiwixlib
--keep class org.kiwix.kiwixlib.** { *; }
+#keep everything in libkiwix
+-keep class org.kiwix.libkiwix.** { *; }
+
+#keep everything in libzim
+-keep class org.kiwix.libzim.** { *; }
 
 ## SimpleXml
 

app/src/androidTest/java/org/kiwix/kiwixmobile/BaseActivityTest.kt

@@ -20,7 +20,7 @@ package org.kiwix.kiwixmobile
 
 import android.Manifest.permission
 import android.content.Context
-import androidx.test.ext.junit.rules.ActivityScenarioRule
+import androidx.test.core.app.ActivityScenario
 import androidx.test.ext.junit.runners.AndroidJUnit4
 import androidx.test.platform.app.InstrumentationRegistry.getInstrumentation
 import androidx.test.rule.GrantPermissionRule
@@ -32,16 +32,16 @@ import org.kiwix.kiwixmobile.main.KiwixMainActivity
 
 @RunWith(AndroidJUnit4::class)
 abstract class BaseActivityTest {
-  @get:Rule
-  open var activityScenarioRule = ActivityScenarioRule(KiwixMainActivity::class.java)
+  open lateinit var activityScenario: ActivityScenario<KiwixMainActivity>
 
-  @get:Rule
-  var readPermissionRule: GrantPermissionRule =
-    GrantPermissionRule.grant(permission.READ_EXTERNAL_STORAGE)
+  private val permissions = arrayOf(
+    permission.READ_EXTERNAL_STORAGE,
+    permission.WRITE_EXTERNAL_STORAGE
+  )
 
   @get:Rule
-  var writePermissionRule: GrantPermissionRule =
-    GrantPermissionRule.grant(permission.WRITE_EXTERNAL_STORAGE)
+  var permissionRules: GrantPermissionRule =
+    GrantPermissionRule.grant(*permissions)
 
   val context: Context by lazy {
     getInstrumentation().targetContext.applicationContext

app/src/androidTest/java/org/kiwix/kiwixmobile/NetworkTest.kt

@@ -19,14 +19,14 @@ package org.kiwix.kiwixmobile
 
 import android.Manifest
 import android.util.Log
-import androidx.test.InstrumentationRegistry
+import androidx.test.core.app.ActivityScenario
 import androidx.test.espresso.Espresso
 import androidx.test.espresso.IdlingPolicies
 import androidx.test.espresso.IdlingRegistry
 import androidx.test.espresso.action.ViewActions
 import androidx.test.espresso.matcher.ViewMatchers
 import androidx.test.ext.junit.runners.AndroidJUnit4
-import androidx.test.rule.ActivityTestRule
+import androidx.test.platform.app.InstrumentationRegistry
 import androidx.test.rule.GrantPermissionRule
 import com.adevinta.android.barista.interaction.BaristaClickInteractions.clickOn
 import com.adevinta.android.barista.interaction.BaristaDialogInteractions
@@ -57,21 +57,15 @@ class NetworkTest {
   // @Inject
   // MockWebServer mockWebServer
 
-  @Rule
-  @JvmField
-  var mActivityTestRule = ActivityTestRule(
-    KiwixMainActivity::class.java, false, false
+  private val permissions = arrayOf(
+    Manifest.permission.READ_EXTERNAL_STORAGE,
+    Manifest.permission.WRITE_EXTERNAL_STORAGE
   )
 
   @Rule
   @JvmField
-  var readPermissionRule: GrantPermissionRule =
-    GrantPermissionRule.grant(Manifest.permission.READ_EXTERNAL_STORAGE)
-
-  @Rule
-  @JvmField
-  var writePermissionRule: GrantPermissionRule =
-    GrantPermissionRule.grant(Manifest.permission.WRITE_EXTERNAL_STORAGE)
+  var permissionRules: GrantPermissionRule =
+    GrantPermissionRule.grant(*permissions)
 
   @Before fun setUp() {
     val component = DaggerTestComponent.builder().context(
@@ -99,7 +93,7 @@ class NetworkTest {
 
   @Test @Ignore("Broken in 2.5") // TODO Fix in 3.0
   fun networkTest() {
-    mActivityTestRule.launchActivity(null)
+    ActivityScenario.launch(KiwixMainActivity::class.java)
     BaristaSleepInteractions.sleep(TestUtils.TEST_PAUSE_MS.toLong())
     clickMenu(TestUtils.getResourceString(R.string.library))
     TestUtils.allowStoragePermissionsIfNeeded()

app/src/androidTest/java/org/kiwix/kiwixmobile/core/di/modules/TestJNIModule.kt

@@ -20,7 +20,7 @@ package org.kiwix.kiwixmobile.core.di.modules
 import dagger.Module
 import dagger.Provides
 import io.mockk.mockk
-import org.kiwix.kiwixlib.JNIKiwix
+import org.kiwix.libkiwix.JNIKiwix
 import javax.inject.Singleton
 
 /**