Fix some ForceCloses (real bad code, fixed with even worse code)

  this is a bad bad bad approach. i have taken
  a sledgehammer to the exceptions and made them
  shut up. if i have some time to retrospect back
  on this, i'd like to properly fix these errors.

Author: Arnav Gupta

App/src/main/java/in/ac/dtu/subtlenews/MainActivity.java

@@ -3,12 +3,10 @@
 import android.content.Intent;
 import android.content.SharedPreferences;
 import android.os.Bundle;
-import android.support.v4.app.Fragment;
 import android.support.v4.app.FragmentManager;
 import android.support.v4.widget.DrawerLayout;
 import android.support.v7.app.ActionBar;
 import android.support.v7.app.ActionBarActivity;
-import android.util.Log;
 import android.view.Menu;
 import android.view.MenuItem;
 import android.view.Window;
@@ -129,7 +127,7 @@ public boolean onOptionsItemSelected(MenuItem item) {
         switch (id){
             case R.id.action_refresh:
                 if(Utils.isNetworkConnected(MainActivity.this)){
-                    new UpdateNews(MainActivity.this).execute();
+                    new UpdateNews(MainActivity.this, false).execute();
                     MainFragment mainFragment = (MainFragment) getSupportFragmentManager().findFragmentById(R.id.container);
                     mainFragment.updateView();
                 } else {

App/src/main/java/in/ac/dtu/subtlenews/MainFragment.java

@@ -1,7 +1,6 @@
 package in.ac.dtu.subtlenews;
 
 import android.annotation.SuppressLint;
-import android.annotation.TargetApi;
 import android.app.Activity;
 import android.app.AlertDialog;
 import android.content.Context;
@@ -17,10 +16,8 @@
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
-import android.webkit.WebView;
 import android.widget.AdapterView;
 import android.widget.BaseAdapter;
-import android.widget.ImageView;
 import android.widget.ListView;
 import android.widget.TextView;
 import android.widget.Toast;
@@ -32,9 +29,7 @@
 import java.io.BufferedReader;
 import java.io.File;
 import java.io.FileReader;
-import java.lang.annotation.Target;
 import java.util.ArrayList;
-import java.util.List;
 
 /**
  * Created by omerjerk on 2/12/13.
@@ -119,7 +114,7 @@ protected ArrayList<JSONObject> doInBackground(Void... v) {
             } catch (Exception e){
                 e.printStackTrace();
                 if (Utils.isNetworkConnected(getActivity())) {
-                    new UpdateNews(getActivity()).execute();
+                    new UpdateNews(getActivity(), true).execute();
                 } else {
                     Toast.makeText(getActivity(), "Please turn on your internet connection!", Toast.LENGTH_SHORT).show();
                 }

App/src/main/java/in/ac/dtu/subtlenews/NewsAutoRefresh.java

@@ -6,7 +6,6 @@
 import android.content.Context;
 import android.content.Intent;
 import android.os.Bundle;
-import android.util.Log;
 
 import java.util.Calendar;
 
@@ -39,7 +38,7 @@ public NewsAutoRefresh(Context context, Bundle extras, int timeoutInSeconds){
     public void onReceive(Context context, Intent intent) {
 
         if(Utils.isNetworkConnected(context)){
-            new UpdateNews(context).execute();
+            new UpdateNews(context, true).execute();
         }
     }
 }

App/src/main/java/in/ac/dtu/subtlenews/UpdateNews.java

@@ -23,12 +23,16 @@
 public class UpdateNews extends AsyncTask <Void, Void, String> {
 
     private Context context;
+    private Boolean ranAutomatically;
 
-    public UpdateNews(Context context){
+    public UpdateNews(Context context, Boolean autorun){
         this.context = context;
+        ranAutomatically = autorun;
 
         Log.d("Update News", "Inside the constructor of UpdateNews class");
-        ((Activity)context).setProgressBarIndeterminateVisibility(true);
+         if (!autorun) {
+             ((Activity)context).setProgressBarIndeterminateVisibility(true);
+         }
     }
 
     @Override
@@ -49,7 +53,9 @@ protected String doInBackground(Void... v){
 
     protected void onPostExecute(String r) {
 
-        ((Activity)context).setProgressBarIndeterminateVisibility(false);
+        if (!ranAutomatically) {
+            ((Activity)context).setProgressBarIndeterminateVisibility(false);
+        }
 
         Log.d("[GET RESPONSE]", r);
         File cacheFile = new File(context.getFilesDir(), "data.json");

App/src/main/java/in/ac/dtu/subtlenews/Utils.java

@@ -13,12 +13,20 @@ public class Utils {
     public static String[] categoryMap = {"India", "World", "Entertainment" , "Technology", "Business","Science" , "Sports", "Health"};
 
     public static boolean isNetworkConnected(Context context) {
-        ConnectivityManager cm = (ConnectivityManager) context.getSystemService(Context.CONNECTIVITY_SERVICE);
-        NetworkInfo ni = cm.getActiveNetworkInfo();
-        if (ni == null) {
-            // There are no active networks.
+
+        try {
+            ConnectivityManager cm = (ConnectivityManager) context.getSystemService(Context.CONNECTIVITY_SERVICE);
+            NetworkInfo ni = cm.getActiveNetworkInfo();
+            if (ni == null) {
+                // There are no active networks.
+                return false;
+            } else {
+                return true;
+            }
+        } catch (Exception e) {
+            e.printStackTrace();
             return false;
-        } else
-            return true;
+        }
+
     }
 }