Android Downloading ZIM list require confirmation on network

res/layout/activity_library.xml

@@ -14,7 +14,22 @@
       android:layout_width="match_parent"
       android:layout_height="match_parent"
       android:id="@+id/library_list" />
-
+  <TextView
+    android:layout_width="wrap_content"
+    android:layout_height="wrap_content"
+    android:visibility="gone"
+    android:layout_marginTop="200dp"
+    android:text="@string/get_library_over_network"
+    android:layout_centerHorizontal="true"
+    android:id="@+id/network_permission_text"/>
+  <Button
+    android:layout_width="wrap_content"
+    android:layout_height="wrap_content"
+    android:layout_marginTop="230dp"
+    android:id="@+id/network_permission_button"
+    android:text="@string/proceed"
+    android:layout_centerHorizontal="true"
+    android:visibility="gone"/>
 
     <RelativeLayout
         android:layout_width="wrap_content"

res/values/strings.xml

@@ -114,4 +114,6 @@
   <string name="zim_nopic">No Pictures</string>
   <string name="open_partial_zim">This file could be incomplete. Do you wish to attempt to open it?</string>
   <string name="no_network_connection">No network connection</string>
+  <string name="get_library_over_network">Use network to download ZIM list. (Approximately 6MB)</string>
+  <string name="proceed">Proceed</string>
 </resources>

src/org/kiwix/kiwixmobile/LibraryFragment.java

@@ -28,6 +28,7 @@ import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
 import android.widget.AdapterView;
+import android.widget.Button;
 import android.widget.LinearLayout;
 import android.widget.ListView;
 import android.widget.ProgressBar;
@@ -67,6 +68,8 @@ public class LibraryFragment extends Fragment implements AdapterView.OnItemClick
   @BindView(R.id.progressBar) ProgressBar progressBar;
   @BindView(R.id.progressbar_message) TextView progressBarMessage;
   @BindView(R.id.progressbar_layout) RelativeLayout progressBarLayout;
+  @BindView(R.id.network_permission_text) TextView permissionText;
+  @BindView(R.id.network_permission_button) Button permissionButton;
 
 
   private KiwixService kiwixService;
@@ -102,15 +105,30 @@ public class LibraryFragment extends Fragment implements AdapterView.OnItemClick
         // setContentView(R.layout.activity_layout);
       ButterKnife.bind(this, llLayout);
 
-      progressBar.setVisibility(View.VISIBLE);
-      progressBarMessage.setVisibility(View.VISIBLE);
-      progressBarLayout.setVisibility(View.VISIBLE);
-
       kiwixService = ((KiwixApplication) super.getActivity().getApplication()).getKiwixService();
       conMan = (ConnectivityManager) super.getActivity().getSystemService(Context.CONNECTIVITY_SERVICE);
       NetworkInfo network = conMan.getActiveNetworkInfo();
       bookDao = new BookDao(new KiwixDatabase(super.getActivity()));
       if (network != null && network.isConnected()) {
+        if (isWiFi()) {
+          getLibraryData();
+        } else {
+          displayNetworkConfirmation();
+        }
+      } else {
+        noNetworkConnection();
+      }
+        // The FragmentActivity doesn't contain the layout directly so we must use our instance of     LinearLayout :
+        //llLayout.findViewById(R.id.someGuiElement);
+        // Instead of :
+        // findViewById(R.id.someGuiElement);
+        return llLayout; // We must return the loaded Layout
+    }
+
+  public void getLibraryData(){
+    progressBar.setVisibility(View.VISIBLE);
+    progressBarMessage.setVisibility(View.VISIBLE);
+    progressBarLayout.setVisibility(View.VISIBLE);
     kiwixService.getLibrary()
         .observeOn(AndroidSchedulers.mainThread())
         .subscribe(library -> {
@@ -128,14 +146,19 @@ public class LibraryFragment extends Fragment implements AdapterView.OnItemClick
     libraryList.setOnItemClickListener(this);
 
     active = true;
-      } else {
-        noNetworkConnection();
   }
-        // The FragmentActivity doesn't contain the layout directly so we must use our instance of     LinearLayout :
+
-        //llLayout.findViewById(R.id.someGuiElement);
+  public void displayNetworkConfirmation(){
-        // Instead of :
+    permissionText.setVisibility(View.VISIBLE);
-        // findViewById(R.id.someGuiElement);
+    permissionButton.setVisibility(View.VISIBLE);
-        return llLayout; // We must return the loaded Layout
+    permissionButton.setOnClickListener(new View.OnClickListener() {
+      @Override
+      public void onClick(View view) {
+        getLibraryData();
+        permissionButton.setVisibility(View.GONE);
+        permissionText.setVisibility(View.GONE);
+      }
+    });
   }
 
   public void noNetworkConnection() {
@@ -167,19 +190,28 @@ public class LibraryFragment extends Fragment implements AdapterView.OnItemClick
       return;
     }
     bookDao.saveBook(books.get(position));
-    if (Build.VERSION.SDK_INT >= 23) {
+    if (isWiFi()){
-      NetworkInfo network = conMan.getActiveNetworkInfo();
-      if (network.getType() != ConnectivityManager.TYPE_WIFI){
       mobileDownloadDialog(position);
     } else {
       downloadFile(position, books.get(position));
     }
+
+  }
+
+  public boolean isWiFi(){
+    if (Build.VERSION.SDK_INT >= 23) {
+      NetworkInfo network = conMan.getActiveNetworkInfo();
+      if (network.getType() != ConnectivityManager.TYPE_WIFI){
+        return false;
+      } else {
+        return true;
+      }
     } else {
       NetworkInfo wifi = conMan.getNetworkInfo(ConnectivityManager.TYPE_WIFI);
       if (!wifi.isConnected()){
-        mobileDownloadDialog(position);
+        return false;
       } else {
-        downloadFile(position, books.get(position));
+        return true;
       }
     }
   }