fix refreshing bug

Author: tawazz

.idea/modules.xml

@@ -1,6 +1,5 @@
 package net.tawazz.spendee;
 
-import android.app.ProgressDialog;
 import android.content.Intent;
 import android.graphics.Color;
 import android.os.Bundle;
@@ -344,6 +343,8 @@ public void run() {
 
     public void loadData() {
 
+        final ViewsFragment fragment = (ViewsFragment) fragmentList.get(viewPager.getCurrentItem());
+        fragment.setRefreshing(true);
         String url;
 
         if (dates != null) {
@@ -419,16 +420,19 @@ public void onResponse(JSONObject response) {
                     }
 
                     updateViews(currentPosition);
+                    fragment.setRefreshing(false);
 
                 } catch (JSONException e) {
                     updateViews(currentPosition);
+                    fragment.setRefreshing(false);
                 }
 
 
             }
         }, new Response.ErrorListener() {
             @Override
             public void onErrorResponse(VolleyError error) {
+                fragment.setRefreshing(false);
                 updateViews(currentPosition);
             }
         });

app/src/main/java/net/tawazz/spendee/MainActivity.java

@@ -2,6 +2,7 @@
 
 
 import android.os.Bundle;
+import android.os.Handler;
 import android.support.v4.widget.SwipeRefreshLayout;
 import android.support.v7.widget.LinearLayoutManager;
 import android.support.v7.widget.RecyclerView;
@@ -46,9 +47,16 @@ public View onCreateView(LayoutInflater inflater, ViewGroup container,
 
         if (onCreateViewCallback != null) {
             this.onCreateViewCallback.onFragmentCreateView();
-            refreshLayout.setRefreshing(true);
-            this.onCreateViewCallback.onRefresh();
-            refreshLayout.setRefreshing(true);
+            Handler handler = new Handler();
+            handler.post(new Runnable() {
+                @Override
+                public void run() {
+                    refreshLayout.setRefreshing(true);
+                    onCreateViewCallback.onRefresh();
+                    refreshLayout.setRefreshing(false);
+                }
+            });
+
         }
 
         return view;
@@ -89,10 +97,18 @@ public void setExpenses(ArrayList<ExpData> expenses) {
 
     }
 
-    public void refresh(){
-        refreshLayout.setRefreshing(true);
-        onCreateViewCallback.onRefresh();
-        refreshLayout.setRefreshing(false);
+    @Override
+    public void setRefreshing(final boolean isRefreshing) {
+        Handler handler = new Handler();
+
+        handler.post(new Runnable() {
+            @Override
+            public void run() {
+                refreshLayout.setRefreshing(isRefreshing);
+            }
+        });
+
     }
 
+
 }

app/src/main/java/net/tawazz/spendee/fragments/ExpFragment.java

@@ -2,15 +2,14 @@
 
 
 import android.os.Bundle;
+import android.os.Handler;
 import android.support.v4.app.Fragment;
 import android.support.v4.widget.SwipeRefreshLayout;
 import android.support.v7.widget.LinearLayoutManager;
 import android.support.v7.widget.RecyclerView;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
-import android.view.animation.BounceInterpolator;
-import android.view.animation.OvershootInterpolator;
 import android.widget.TextView;
 
 import net.tawazz.spendee.AppData.IncData;
@@ -90,4 +89,17 @@ public void setIncomes(ArrayList<IncData> incomes) {
             listView.setLayoutManager(new LinearLayoutManager(this.getContext()));
         }
     }
+
+    @Override
+    public void setRefreshing(final boolean isRefreshing) {
+        Handler handler = new Handler();
+
+        handler.post(new Runnable() {
+            @Override
+            public void run() {
+                refreshLayout.setRefreshing(isRefreshing);
+            }
+        });
+
+    }
 }

app/src/main/java/net/tawazz/spendee/fragments/IncFragment.java

@@ -25,4 +25,8 @@ public interface onCreateViewListener {
 
         void onRefresh();
     }
+
+    public  void setRefreshing(final boolean isRefreshing){
+
+    }
 }