From e5ac3bf5d4be7f6cb765a4de21299d39ef9c71e0 Mon Sep 17 00:00:00 2001 From: HissPirat Date: Thu, 27 Aug 2020 11:19:21 +0200 Subject: [PATCH] #2341 fixed zim host memory leaks --- .../org/kiwix/kiwixmobile/webserver/ZimHostFragment.kt | 5 +++-- .../kiwix/kiwixmobile/core/settings/CorePrefsFragment.java | 7 +++---- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/app/src/main/java/org/kiwix/kiwixmobile/webserver/ZimHostFragment.kt b/app/src/main/java/org/kiwix/kiwixmobile/webserver/ZimHostFragment.kt index 58ec7c730..3efa14f21 100644 --- a/app/src/main/java/org/kiwix/kiwixmobile/webserver/ZimHostFragment.kt +++ b/app/src/main/java/org/kiwix/kiwixmobile/webserver/ZimHostFragment.kt @@ -40,11 +40,11 @@ import org.kiwix.kiwixmobile.core.BuildConfig import org.kiwix.kiwixmobile.core.base.BaseActivity import org.kiwix.kiwixmobile.core.base.BaseFragment import org.kiwix.kiwixmobile.core.extensions.toast -import org.kiwix.kiwixmobile.core.utils.dialog.AlertDialogShower import org.kiwix.kiwixmobile.core.utils.ConnectivityReporter -import org.kiwix.kiwixmobile.core.utils.dialog.KiwixDialog import org.kiwix.kiwixmobile.core.utils.ServerUtils import org.kiwix.kiwixmobile.core.utils.SharedPreferenceUtil +import org.kiwix.kiwixmobile.core.utils.dialog.AlertDialogShower +import org.kiwix.kiwixmobile.core.utils.dialog.KiwixDialog import org.kiwix.kiwixmobile.core.zim_manager.fileselect_view.SelectionMode import org.kiwix.kiwixmobile.core.zim_manager.fileselect_view.adapter.BookOnDiskDelegate import org.kiwix.kiwixmobile.core.zim_manager.fileselect_view.adapter.BooksOnDiskAdapter @@ -234,6 +234,7 @@ class ZimHostFragment : BaseFragment(), ZimHostCallbacks, ZimHostContract.View { override fun onDestroyView() { super.onDestroyView() recyclerViewZimHost.adapter = null + hotspotService?.registerCallBack(null) presenter.detachView() } diff --git a/core/src/main/java/org/kiwix/kiwixmobile/core/settings/CorePrefsFragment.java b/core/src/main/java/org/kiwix/kiwixmobile/core/settings/CorePrefsFragment.java index 4182e159a..915486534 100644 --- a/core/src/main/java/org/kiwix/kiwixmobile/core/settings/CorePrefsFragment.java +++ b/core/src/main/java/org/kiwix/kiwixmobile/core/settings/CorePrefsFragment.java @@ -26,7 +26,6 @@ import android.content.pm.PackageManager; import android.os.Bundle; import android.view.LayoutInflater; import android.webkit.WebView; -import androidx.appcompat.app.AppCompatActivity; import androidx.core.content.ContextCompat; import androidx.preference.EditTextPreference; import androidx.preference.ListPreference; @@ -46,10 +45,10 @@ import org.kiwix.kiwixmobile.core.CoreApp; import org.kiwix.kiwixmobile.core.NightModeConfig; import org.kiwix.kiwixmobile.core.R; import org.kiwix.kiwixmobile.core.main.AddNoteDialog; -import org.kiwix.kiwixmobile.core.utils.dialog.DialogShower; -import org.kiwix.kiwixmobile.core.utils.dialog.KiwixDialog; import org.kiwix.kiwixmobile.core.utils.LanguageUtils; import org.kiwix.kiwixmobile.core.utils.SharedPreferenceUtil; +import org.kiwix.kiwixmobile.core.utils.dialog.DialogShower; +import org.kiwix.kiwixmobile.core.utils.dialog.KiwixDialog; import static org.kiwix.kiwixmobile.core.utils.ConstantsKt.RESULT_RESTART; import static org.kiwix.kiwixmobile.core.utils.SharedPreferenceUtil.PREF_NIGHT_MODE; @@ -267,7 +266,7 @@ public abstract class CorePrefsFragment extends PreferenceFragmentCompat impleme public void openFolderSelect() { StorageSelectDialog dialogFragment = new StorageSelectDialog(); dialogFragment.setOnSelectAction(this::onStorageDeviceSelected); - dialogFragment.show(((AppCompatActivity) getActivity()).getSupportFragmentManager(), + dialogFragment.show(getActivity().getSupportFragmentManager(), getResources().getString(R.string.pref_storage)); }