Load Gist in background task when missing from store

kevinsawicki · kevinsawicki · commit 6685e4da0de4 · 2012-07-10T09:07:34.000-07:00
The pager will only be configured to displays the Gist's
files once the Gist has been refreshed.
diff --git a/app/src/main/java/com/github/mobile/ui/gist/GistFilesViewActivity.java b/app/src/main/java/com/github/mobile/ui/gist/GistFilesViewActivity.java
@@ -22,15 +22,20 @@
 import android.content.Intent;
 import android.os.Bundle;
 import android.support.v4.view.ViewPager;
+import android.widget.ProgressBar;
 
 import com.actionbarsherlock.app.ActionBar;
 import com.actionbarsherlock.view.MenuItem;
 import com.github.mobile.Intents.Builder;
 import com.github.mobile.R.id;
 import com.github.mobile.R.layout;
 import com.github.mobile.R.string;
+import com.github.mobile.core.gist.FullGist;
 import com.github.mobile.core.gist.GistStore;
+import com.github.mobile.core.gist.RefreshGistTask;
 import com.github.mobile.util.AvatarLoader;
+import com.github.mobile.util.HttpImageGetter;
+import com.github.mobile.util.ViewUtils;
 import com.github.rtyley.android.sherlock.roboguice.activity.RoboSherlockFragmentActivity;
 import com.google.inject.Inject;
 import com.viewpagerindicator.TitlePageIndicator;
@@ -39,6 +44,7 @@
 import org.eclipse.egit.github.core.User;
 
 import roboguice.inject.InjectExtra;
+import roboguice.inject.InjectView;
 
 /**
  * Activity to page through the content of all the files in a Gist
@@ -63,6 +69,15 @@ public static Intent createIntent(Gist gist, int position) {
     @InjectExtra(EXTRA_POSITION)
     private int initialPosition;
 
+    @InjectView(id.vp_pages)
+    private ViewPager pager;
+
+    @InjectView(id.pb_loading)
+    private ProgressBar loadingBar;
+
+    @InjectView(id.tpi_header)
+    private TitlePageIndicator indicator;
+
     private Gist gist;
 
     @Inject
@@ -71,39 +86,74 @@ public static Intent createIntent(Gist gist, int position) {
     @Inject
     private AvatarLoader avatars;
 
+    @Inject
+    private HttpImageGetter imageGetter;
+
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
 
         setContentView(layout.pager_with_title);
 
+        if (initialPosition < 0)
+            initialPosition = 0;
+
+        getSupportActionBar().setTitle(getString(string.gist_title) + gistId);
+
         gist = store.getGist(gistId);
+        if (gist != null)
+            configurePager();
+        else {
+            ViewUtils.setGone(loadingBar, false);
+            ViewUtils.setGone(pager, true);
+            ViewUtils.setGone(indicator, true);
+            new RefreshGistTask(this, gistId, imageGetter) {
+
+                @Override
+                protected void onSuccess(FullGist gist) throws Exception {
+                    super.onSuccess(gist);
+
+                    GistFilesViewActivity.this.gist = gist.getGist();
+                    configurePager();
+                }
+
+            }.execute();
+        }
+    }
 
+    private void configurePager() {
         ActionBar actionBar = getSupportActionBar();
         actionBar.setDisplayHomeAsUpEnabled(true);
-        actionBar.setTitle(getString(string.gist_title) + gistId);
         User author = gist.getUser();
         if (author != null) {
             actionBar.setSubtitle(author.getLogin());
             avatars.bind(actionBar, author);
         } else
             actionBar.setSubtitle(string.anonymous);
 
-        ViewPager pager = (ViewPager) findViewById(id.vp_pages);
-        pager.setAdapter(new GistFilesPagerAdapter(getSupportFragmentManager(),
-                gist));
-        ((TitlePageIndicator) findViewById(id.tpi_header)).setViewPager(pager);
+        ViewUtils.setGone(loadingBar, true);
+        ViewUtils.setGone(pager, false);
+        ViewUtils.setGone(indicator, false);
+
+        GistFilesPagerAdapter pagerAdapter = new GistFilesPagerAdapter(
+                getSupportFragmentManager(), gist);
+        pager.setAdapter(pagerAdapter);
+        indicator.setViewPager(pager);
 
-        pager.setCurrentItem(initialPosition);
+        if (initialPosition < pagerAdapter.getCount())
+            pager.setCurrentItem(initialPosition);
     }
 
     @Override
     public boolean onOptionsItemSelected(MenuItem item) {
         switch (item.getItemId()) {
         case android.R.id.home:
-            Intent intent = GistsViewActivity.createIntent(gist);
-            intent.addFlags(FLAG_ACTIVITY_CLEAR_TOP | FLAG_ACTIVITY_SINGLE_TOP);
-            startActivity(intent);
+            if (gist != null) {
+                Intent intent = GistsViewActivity.createIntent(gist);
+                intent.addFlags(FLAG_ACTIVITY_CLEAR_TOP
+                        | FLAG_ACTIVITY_SINGLE_TOP);
+                startActivity(intent);
+            }
             return true;
         default:
             return super.onOptionsItemSelected(item);
