Fixed bug: show the selected image in the gallery, instead of always the first in...
[pub/Android/ownCloud.git] / src / com / owncloud / android / ui / activity / FileDetailActivity.java
index 13e61a9..6994c09 100644 (file)
@@ -17,8 +17,6 @@
  */\r
 package com.owncloud.android.ui.activity;\r
 \r
-import java.lang.ref.WeakReference;\r
-\r
 import android.accounts.Account;\r
 import android.app.Dialog;\r
 import android.app.ProgressDialog;\r
@@ -32,7 +30,6 @@ import android.os.IBinder;
 import android.support.v4.app.Fragment;\r
 import android.support.v4.app.FragmentTransaction;\r
 import android.util.Log;\r
-import android.widget.ProgressBar;\r
 \r
 import com.actionbarsherlock.app.ActionBar;\r
 import com.actionbarsherlock.app.SherlockFragmentActivity;\r
@@ -47,8 +44,6 @@ import com.owncloud.android.ui.fragment.FilePreviewFragment;
 \r
 import com.owncloud.android.R;\r
 \r
-import eu.alefzero.webdav.OnDatatransferProgressListener;\r
-\r
 /**\r
  * This activity displays the details of a file like its name, its size and so\r
  * on.\r
@@ -66,21 +61,25 @@ public class FileDetailActivity extends SherlockFragmentActivity implements File
     public static final int MODE_DETAILS = 0;\r
     public static final int MODE_PREVIEW = 1;\r
 \r
-    private static final String KEY_WAITING_TO_PREVIEW = "WAITING_TO_PREVIEW";\r
+    public static final String KEY_WAITING_TO_PREVIEW = "WAITING_TO_PREVIEW";\r
     \r
     private boolean mConfigurationChangedToLandscape = false;\r
     private FileDownloaderBinder mDownloaderBinder = null;\r
     private ServiceConnection mDownloadConnection, mUploadConnection = null;\r
     private FileUploaderBinder mUploaderBinder = null;\r
     private boolean mWaitingToPreview;\r
-\r
-    public ProgressListener mProgressListener;\r
+    \r
+    private OCFile mFile;\r
+    private Account mAccount;\r
     \r
 \r
     @Override\r
     protected void onCreate(Bundle savedInstanceState) {\r
         super.onCreate(savedInstanceState);\r
 \r
+        mFile = getIntent().getParcelableExtra(FileDetailFragment.EXTRA_FILE);\r
+        mAccount = getIntent().getParcelableExtra(FileDetailFragment.EXTRA_ACCOUNT);\r
+        \r
         // check if configuration changed to large-land ; for a tablet being changed from portrait to landscape when in FileDetailActivity \r
         Configuration conf = getResources().getConfiguration();\r
         mConfigurationChangedToLandscape = (conf.orientation == Configuration.ORIENTATION_LANDSCAPE && \r
@@ -88,11 +87,6 @@ public class FileDetailActivity extends SherlockFragmentActivity implements File
                                            );\r
 \r
         if (!mConfigurationChangedToLandscape) {\r
-            mDownloadConnection = new DetailsServiceConnection();\r
-            bindService(new Intent(this, FileDownloader.class), mDownloadConnection, Context.BIND_AUTO_CREATE);\r
-            mUploadConnection = new DetailsServiceConnection();\r
-            bindService(new Intent(this, FileUploader.class), mUploadConnection, Context.BIND_AUTO_CREATE);\r
-            \r
             setContentView(R.layout.file_activity_details);\r
         \r
             ActionBar actionBar = getSupportActionBar();\r
@@ -105,6 +99,12 @@ public class FileDetailActivity extends SherlockFragmentActivity implements File
                 mWaitingToPreview = savedInstanceState.getBoolean(KEY_WAITING_TO_PREVIEW);\r
             }\r
             \r
+            mDownloadConnection = new DetailsServiceConnection();\r
+            bindService(new Intent(this, FileDownloader.class), mDownloadConnection, Context.BIND_AUTO_CREATE);\r
+            mUploadConnection = new DetailsServiceConnection();\r
+            bindService(new Intent(this, FileUploader.class), mUploadConnection, Context.BIND_AUTO_CREATE);\r
+            \r
+            \r
         }  else {\r
             backToDisplayActivity();   // the 'back' won't be effective until this.onStart() and this.onResume() are completed;\r
         }\r
@@ -117,22 +117,20 @@ public class FileDetailActivity extends SherlockFragmentActivity implements File
      * the requested {@link Intent}.\r
      */\r
     private void createChildFragment() {\r
-        OCFile file = getIntent().getParcelableExtra(FileDetailFragment.EXTRA_FILE);\r
-        Account account = getIntent().getParcelableExtra(FileDetailFragment.EXTRA_ACCOUNT);\r
         int mode = getIntent().getIntExtra(EXTRA_MODE, MODE_PREVIEW); \r
         \r
         Fragment newFragment = null;\r
-        if (FilePreviewFragment.canBePreviewed(file) && mode == MODE_PREVIEW) {\r
-            if (file.isDown()) {\r
-                newFragment = new FilePreviewFragment(file, account);\r
+        if (FilePreviewFragment.canBePreviewed(mFile) && mode == MODE_PREVIEW) {\r
+            if (mFile.isDown()) {\r
+                newFragment = new FilePreviewFragment(mFile, mAccount);\r
             \r
             } else {\r
-                newFragment = new FileDetailFragment(file, account);\r
+                newFragment = new FileDetailFragment(mFile, mAccount);\r
                 mWaitingToPreview = true;\r
             }\r
             \r
         } else {\r
-            newFragment = new FileDetailFragment(file, account);\r
+            newFragment = new FileDetailFragment(mFile, mAccount);\r
         }\r
         FragmentTransaction ft = getSupportFragmentManager().beginTransaction();\r
         ft.replace(R.id.fragment, newFragment, FileDetailFragment.FTAG);\r
@@ -142,6 +140,7 @@ public class FileDetailActivity extends SherlockFragmentActivity implements File
 \r
     @Override\r
     protected void onSaveInstanceState(Bundle outState) {\r
+        super.onSaveInstanceState(outState);\r
         outState.putBoolean(KEY_WAITING_TO_PREVIEW, mWaitingToPreview);\r
     }\r
 \r
@@ -151,37 +150,26 @@ public class FileDetailActivity extends SherlockFragmentActivity implements File
 \r
         @Override\r
         public void onServiceConnected(ComponentName component, IBinder service) {\r
-            Fragment fragment = getSupportFragmentManager().findFragmentByTag(FileDetailFragment.FTAG);\r
-            FileDetailFragment detailsFragment = (fragment instanceof FileDetailFragment) ? (FileDetailFragment) fragment : null;\r
                 \r
             if (component.equals(new ComponentName(FileDetailActivity.this, FileDownloader.class))) {\r
                 Log.d(TAG, "Download service connected");\r
                 mDownloaderBinder = (FileDownloaderBinder) service;\r
-                if (detailsFragment != null) {\r
-                    mProgressListener = new ProgressListener(detailsFragment.getProgressBar());\r
-                    mDownloaderBinder.addDatatransferProgressListener(\r
-                            mProgressListener, \r
-                            (Account) getIntent().getParcelableExtra(FileDetailFragment.EXTRA_ACCOUNT), \r
-                            (OCFile) getIntent().getParcelableExtra(FileDetailFragment.EXTRA_FILE)\r
-                            );\r
+                if (mWaitingToPreview) {\r
+                    requestForDownload();\r
                 }\r
+                    \r
             } else if (component.equals(new ComponentName(FileDetailActivity.this, FileUploader.class))) {\r
                 Log.d(TAG, "Upload service connected");\r
                 mUploaderBinder = (FileUploaderBinder) service;\r
-                if (detailsFragment != null) {\r
-                    mProgressListener = new ProgressListener(detailsFragment.getProgressBar());\r
-                    mUploaderBinder.addDatatransferProgressListener(\r
-                            mProgressListener, \r
-                            (Account) getIntent().getParcelableExtra(FileDetailFragment.EXTRA_ACCOUNT), \r
-                            (OCFile) getIntent().getParcelableExtra(FileDetailFragment.EXTRA_FILE)\r
-                            );\r
-                }\r
             } else {\r
                 return;\r
             }\r
             \r
+            Fragment fragment = getSupportFragmentManager().findFragmentByTag(FileDetailFragment.FTAG);\r
+            FileDetailFragment detailsFragment = (fragment instanceof FileDetailFragment) ? (FileDetailFragment) fragment : null;\r
             if (detailsFragment != null) {\r
-                detailsFragment.updateFileDetails(false);   // let the fragment gets the mDownloadBinder through getDownloadBinder() (see FileDetailFragment#updateFileDetais())\r
+                detailsFragment.listenForTransferProgress();\r
+                detailsFragment.updateFileDetails(mWaitingToPreview);   // let the fragment gets the mDownloadBinder through getDownloadBinder() (see FileDetailFragment#updateFileDetais())\r
             }\r
         }\r
 \r
@@ -198,39 +186,6 @@ public class FileDetailActivity extends SherlockFragmentActivity implements File
     };    \r
     \r
     \r
-    /**\r
-     * Helper class responsible for updating the progress bar shown for file uploading or downloading  \r
-     * \r
-     * @author David A. Velasco\r
-     */\r
-    private class ProgressListener implements OnDatatransferProgressListener {\r
-        int mLastPercent = 0;\r
-        WeakReference<ProgressBar> mProgressBar = null;\r
-        \r
-        ProgressListener(ProgressBar progressBar) {\r
-            mProgressBar = new WeakReference<ProgressBar>(progressBar);\r
-        }\r
-        \r
-        @Override\r
-        public void onTransferProgress(long progressRate) {\r
-            // old method, nothing here\r
-        };\r
-\r
-        @Override\r
-        public void onTransferProgress(long progressRate, long totalTransferredSoFar, long totalToTransfer, String filename) {\r
-            int percent = (int)(100.0*((double)totalTransferredSoFar)/((double)totalToTransfer));\r
-            if (percent != mLastPercent) {\r
-                ProgressBar pb = mProgressBar.get();\r
-                if (pb != null) {\r
-                    pb.setProgress(percent);\r
-                }\r
-            }\r
-            mLastPercent = percent;\r
-        }\r
-\r
-    };\r
-    \r
-\r
     @Override\r
     public void onDestroy() {\r
         super.onDestroy();\r
@@ -279,8 +234,8 @@ public class FileDetailActivity extends SherlockFragmentActivity implements File
     private void backToDisplayActivity() {\r
         Intent intent = new Intent(this, FileDisplayActivity.class);\r
         intent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP);\r
-        intent.putExtra(FileDetailFragment.EXTRA_FILE, getIntent().getParcelableExtra(FileDetailFragment.EXTRA_FILE));\r
-        intent.putExtra(FileDetailFragment.EXTRA_ACCOUNT, getIntent().getParcelableExtra(FileDetailFragment.EXTRA_ACCOUNT));\r
+        intent.putExtra(FileDetailFragment.EXTRA_FILE, mFile);\r
+        intent.putExtra(FileDetailFragment.EXTRA_ACCOUNT, mAccount);\r
         startActivity(intent);\r
         finish();\r
     }\r
@@ -333,8 +288,30 @@ public class FileDetailActivity extends SherlockFragmentActivity implements File
     @Override\r
     public void showFragmentWithDetails(OCFile file) {\r
         FragmentTransaction transaction = getSupportFragmentManager().beginTransaction();\r
-        transaction.replace(R.id.fragment, new FileDetailFragment(file, (Account) getIntent().getParcelableExtra(FileDetailFragment.EXTRA_ACCOUNT)), FileDetailFragment.FTAG); \r
+        transaction.replace(R.id.fragment, new FileDetailFragment(file, mAccount), FileDetailFragment.FTAG); \r
         transaction.commit();\r
     }\r
+\r
+    \r
+    private void requestForDownload() {\r
+        if (!mDownloaderBinder.isDownloading(mAccount, mFile)) {\r
+            Intent i = new Intent(this, FileDownloader.class);\r
+            i.putExtra(FileDownloader.EXTRA_ACCOUNT, mAccount);\r
+            i.putExtra(FileDownloader.EXTRA_FILE, mFile);\r
+            startService(i);\r
+        }\r
+    }\r
+\r
+    @Override\r
+    public void notifySuccessfulDownload(OCFile file, Intent intent, boolean success) {\r
+        if (success) {\r
+            if (mWaitingToPreview) {\r
+                FragmentTransaction transaction = getSupportFragmentManager().beginTransaction();\r
+                transaction.replace(R.id.fragment, new FilePreviewFragment(file, mAccount), FileDetailFragment.FTAG); \r
+                transaction.commit();\r
+                mWaitingToPreview = false;\r
+            }\r
+        }\r
+    }\r
     \r
 }\r