Ensure that synchronization indicator in the files list is stopped if unexepected...

[pub/Android/ownCloud.git] / src / eu / alefzero / owncloud / ui / fragment / FileDetailFragment.java

diff --git a/src/eu/alefzero/owncloud/ui/fragment/FileDetailFragment.java b/src/eu/alefzero/owncloud/ui/fragment/FileDetailFragment.java
index d2f5d10..627c26a 100644 (file)
--- a/src/eu/alefzero/owncloud/ui/fragment/FileDetailFragment.java
+++ b/src/eu/alefzero/owncloud/ui/fragment/FileDetailFragment.java
@@ -19,12 +19,14 @@ package eu.alefzero.owncloud.ui.fragment;
 \r
 import android.accounts.Account;\r
 import android.accounts.AccountManager;\r
 \r
 import android.accounts.Account;\r
 import android.accounts.AccountManager;\r

+import android.content.ActivityNotFoundException;\r

 import android.content.BroadcastReceiver;\r
 import android.content.Context;\r
 import android.content.Intent;\r
 import android.content.IntentFilter;\r
 import android.graphics.Bitmap;\r
 import android.graphics.BitmapFactory;\r
 import android.content.BroadcastReceiver;\r
 import android.content.Context;\r
 import android.content.Intent;\r
 import android.content.IntentFilter;\r
 import android.graphics.Bitmap;\r
 import android.graphics.BitmapFactory;\r

+import android.graphics.BitmapFactory.Options;\r

 import android.net.Uri;\r
 import android.os.Bundle;\r
 import android.util.Log;\r
 import android.net.Uri;\r
 import android.os.Bundle;\r
 import android.util.Log;\r


@@ -32,6 +34,7 @@ import android.view.LayoutInflater;
 import android.view.View;\r
 import android.view.View.OnClickListener;\r
 import android.view.ViewGroup;\r
 import android.view.View;\r
 import android.view.View.OnClickListener;\r
 import android.view.ViewGroup;\r

+import android.webkit.MimeTypeMap;\r

 import android.widget.Button;\r
 import android.widget.ImageView;\r
 import android.widget.TextView;\r
 import android.widget.Button;\r
 import android.widget.ImageView;\r
 import android.widget.TextView;\r


@@ -40,10 +43,10 @@ import android.widget.Toast;
 import com.actionbarsherlock.app.SherlockFragment;\r
 \r
 import eu.alefzero.owncloud.DisplayUtils;\r
 import com.actionbarsherlock.app.SherlockFragment;\r
 \r
 import eu.alefzero.owncloud.DisplayUtils;\r

-import eu.alefzero.owncloud.FileDownloader;\r

 import eu.alefzero.owncloud.R;\r
 import eu.alefzero.owncloud.authenticator.AccountAuthenticator;\r
 import eu.alefzero.owncloud.datamodel.OCFile;\r
 import eu.alefzero.owncloud.R;\r
 import eu.alefzero.owncloud.authenticator.AccountAuthenticator;\r
 import eu.alefzero.owncloud.datamodel.OCFile;\r

+import eu.alefzero.owncloud.files.services.FileDownloader;\r

 import eu.alefzero.owncloud.utils.OwnCloudVersion;\r
 \r
 /**\r
 import eu.alefzero.owncloud.utils.OwnCloudVersion;\r
 \r
 /**\r


@@ -56,48 +59,80 @@ public class FileDetailFragment extends SherlockFragment implements
         OnClickListener {\r
 \r
     public static final String EXTRA_FILE = "FILE";\r
         OnClickListener {\r
 \r
     public static final String EXTRA_FILE = "FILE";\r

+    public static final String EXTRA_ACCOUNT = "ACCOUNT";\r

 \r
 \r

-    private DownloadFinishReceiver mDownloadFinishReceiver;\r
-    private Intent mIntent;\r

     private int mLayout;\r
     private View mView;\r
     private OCFile mFile;\r
     private int mLayout;\r
     private View mView;\r
     private OCFile mFile;\r

+    private Account mAccount;\r
+    \r
+    private DownloadFinishReceiver mDownloadFinishReceiver;\r
+\r

     private static final String TAG = "FileDetailFragment";\r
     private static final String TAG = "FileDetailFragment";\r

+    public static final String FTAG = "FileDetails"; \r

 \r
 \r

+    \r

     /**\r
     /**\r

-     * Default constructor - contains real layout\r
+     * Creates an empty details fragment.\r
+     * \r
+     * It's necessary to keep a public constructor without parameters; the system uses it when tries to reinstantiate a fragment automatically. \r

      */\r
      */\r

-    public FileDetailFragment(){\r
-        mLayout = R.layout.file_details_fragment;\r
+    public FileDetailFragment() {\r
+        mFile = null;\r
+        mAccount = null;\r
+        mLayout = R.layout.file_details_empty;\r

     }\r
     \r
     }\r
     \r

+    \r

     /**\r
     /**\r

-     * Creates a dummy layout. For use if the user never has\r
-     * tapped on a file before\r
+     * Creates a details fragment.\r

      * \r
      * \r

-     * @param useEmptyView If true, use empty layout\r
+     * When 'fileToDetail' or 'ocAccount' are null, creates a dummy layout (to use when a file wasn't tapped before).\r
+     * \r
+     * @param fileToDetail      An {@link OCFile} to show in the fragment\r
+     * @param ocAccount         An ownCloud account; needed to start downloads\r

      */\r
      */\r

-    public FileDetailFragment(boolean useEmptyView){\r
-        if(useEmptyView){\r
-            mLayout = R.layout.file_details_empty;\r
-        } else {\r
+    public FileDetailFragment(OCFile fileToDetail, Account ocAccount){\r
+        mFile = fileToDetail;\r
+        mAccount = ocAccount;\r
+        mLayout = R.layout.file_details_empty;\r
+        \r
+        if(fileToDetail != null && ocAccount != null) {\r

             mLayout = R.layout.file_details_fragment;\r
         }\r
     }\r
     \r
             mLayout = R.layout.file_details_fragment;\r
         }\r
     }\r
     \r

-    /**\r
-     * Use this when creating the fragment and display\r
-     * a file at the same time\r
-     * \r
-     * @param showDetailsIntent The Intent with the required parameters\r
-     * @see FileDetailFragment#updateFileDetails(Intent)\r
-     */\r
-    public FileDetailFragment(Intent showDetailsIntent) {\r
-        mIntent = showDetailsIntent;\r
-        mLayout = R.layout.file_details_fragment;\r
+    \r
+    @Override\r
+    public View onCreateView(LayoutInflater inflater, ViewGroup container,\r
+            Bundle savedInstanceState) {\r
+        super.onCreateView(inflater, container, savedInstanceState);\r
+        \r
+        if (savedInstanceState != null) {\r
+            mFile = savedInstanceState.getParcelable(FileDetailFragment.EXTRA_FILE);\r
+            mAccount = savedInstanceState.getParcelable(FileDetailFragment.EXTRA_ACCOUNT);\r
+        }\r
+        \r
+        View view = null;\r
+        view = inflater.inflate(mLayout, container, false);\r
+        mView = view;\r
+        \r
+        updateFileDetails();\r
+        return view;\r

     }\r
     }\r

+    \r

 \r
     @Override\r
 \r
     @Override\r

+    public void onSaveInstanceState(Bundle outState) {\r
+        Log.i(getClass().toString(), "onSaveInstanceState() start");\r
+        super.onSaveInstanceState(outState);\r
+        outState.putParcelable(FileDetailFragment.EXTRA_FILE, mFile);\r
+        outState.putParcelable(FileDetailFragment.EXTRA_ACCOUNT, mAccount);\r
+        Log.i(getClass().toString(), "onSaveInstanceState() end");\r
+    }\r
+\r
+    \r
+    @Override\r

     public void onResume() {\r
         super.onResume();\r
         mDownloadFinishReceiver = new DownloadFinishReceiver();\r
     public void onResume() {\r
         super.onResume();\r
         mDownloadFinishReceiver = new DownloadFinishReceiver();\r


@@ -114,23 +149,6 @@ public class FileDetailFragment extends SherlockFragment implements
     }\r
 \r
     @Override\r
     }\r
 \r
     @Override\r

-    public View onCreateView(LayoutInflater inflater, ViewGroup container,\r
-            Bundle savedInstanceState) {\r
-        View view = null;\r
-        view = inflater.inflate(mLayout, container, false);\r
-        mView = view;\r
-        if(mLayout == R.layout.file_details_fragment){\r
-            // Phones will launch an activity with this intent\r
-            if(mIntent == null){\r
-                mIntent = getActivity().getIntent();\r
-            }\r
-            updateFileDetails();\r
-        }\r
-        \r
-        return view;\r
-    }\r
-\r
-    @Override\r

     public View getView() {\r
         return super.getView() == null ? mView : super.getView();\r
     }\r
     public View getView() {\r
         return super.getView() == null ? mView : super.getView();\r
     }\r


@@ -139,9 +157,11 @@ public class FileDetailFragment extends SherlockFragment implements
     public void onClick(View v) {\r
         Toast.makeText(getActivity(), "Downloading", Toast.LENGTH_LONG).show();\r
         Intent i = new Intent(getActivity(), FileDownloader.class);\r
     public void onClick(View v) {\r
         Toast.makeText(getActivity(), "Downloading", Toast.LENGTH_LONG).show();\r
         Intent i = new Intent(getActivity(), FileDownloader.class);\r

-        i.putExtra(FileDownloader.EXTRA_ACCOUNT,\r
-                mIntent.getParcelableExtra(FileDownloader.EXTRA_ACCOUNT));\r
-        i.putExtra(FileDownloader.EXTRA_FILE_PATH, mFile.getRemotePath());\r
+        i.putExtra(FileDownloader.EXTRA_ACCOUNT, mAccount);\r
+        i.putExtra(FileDownloader.EXTRA_REMOTE_PATH, mFile.getRemotePath());\r
+        i.putExtra(FileDownloader.EXTRA_FILE_PATH, mFile.getURLDecodedRemotePath());\r
+        i.putExtra(FileDownloader.EXTRA_FILE_SIZE, mFile.getFileLength());\r
+        v.setEnabled(false);\r

         getActivity().startService(i);\r
     }\r
 \r
         getActivity().startService(i);\r
     }\r
 \r


@@ -156,27 +176,22 @@ public class FileDetailFragment extends SherlockFragment implements
     /**\r
      * Use this method to signal this Activity that it shall update its view.\r
      * \r
     /**\r
      * Use this method to signal this Activity that it shall update its view.\r
      * \r

-     * @param intent The {@link Intent} that contains extra information about\r
-     *            this file The intent needs to have these extras:\r
-     *            <p>\r
-     * \r
-     *            {@link FileDetailFragment#EXTRA_FILE}: An {@link OCFile}\r
-     *            {@link FileDownloader#EXTRA_ACCOUNT}: The Account that file\r
-     *            belongs to (required for downloading)\r
+     * @param file : An {@link OCFile}\r

      */\r
      */\r

-    public void updateFileDetails(Intent intent) {\r
-        mIntent = intent;\r
+    public void updateFileDetails(OCFile file, Account ocAccount) {\r
+        mFile = file;\r
+        mAccount = ocAccount;\r

         updateFileDetails();\r
     }\r
         updateFileDetails();\r
     }\r

+    \r

 \r
     /**\r
      * Updates the view with all relevant details about that file.\r
      */\r
 \r
     /**\r
      * Updates the view with all relevant details about that file.\r
      */\r

-    private void updateFileDetails() {\r
-        mFile = mIntent.getParcelableExtra(EXTRA_FILE);\r
-        Button downloadButton = (Button) getView().findViewById(R.id.fdDownloadBtn);\r
+    public void updateFileDetails() {\r

 \r
 \r

-        if (mFile != null) {\r
+        if (mFile != null && mLayout == R.layout.file_details_fragment) {\r
+            Button downloadButton = (Button) getView().findViewById(R.id.fdDownloadBtn);\r

             // set file details\r
             setFilename(mFile.getFileName());\r
             setFiletype(DisplayUtils.convertMIMEtoPrettyPrint(mFile\r
             // set file details\r
             setFilename(mFile.getFileName());\r
             setFiletype(DisplayUtils.convertMIMEtoPrettyPrint(mFile\r


@@ -190,23 +205,86 @@ public class FileDetailFragment extends SherlockFragment implements
             \r
             // Update preview\r
             if (mFile.getStoragePath() != null) {\r
             \r
             // Update preview\r
             if (mFile.getStoragePath() != null) {\r

+                ImageView preview = (ImageView) getView().findViewById(R.id.fdPreview);\r

                 try {\r
                     if (mFile.getMimetype().startsWith("image/")) {\r
                 try {\r
                     if (mFile.getMimetype().startsWith("image/")) {\r

-                        ImageView preview = (ImageView) getView().findViewById(\r
-                                R.id.fdPreview);\r
-                        Bitmap bmp = BitmapFactory.decodeFile(mFile.getStoragePath());\r
+                        BitmapFactory.Options options = new Options();\r
+                        options.inScaled = true;\r
+                        options.inPurgeable = true;\r
+                        if (android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.GINGERBREAD_MR1) {\r
+                            options.inPreferQualityOverSpeed = false;\r
+                        }\r
+                        if (android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.HONEYCOMB) {\r
+                            options.inMutable = false;\r
+                        }\r
+\r
+                        Bitmap bmp = BitmapFactory.decodeFile(mFile.getStoragePath(), options);\r
+\r
+                        int width = options.outWidth;\r
+                        int height = options.outHeight;\r
+                        int scale = 1;\r
+                        if (width >= 2048 || height >= 2048) {\r
+                            scale = (int) (Math.ceil(Math.max(height, width)/2048.));\r
+                            options.inSampleSize = scale;\r
+                            bmp.recycle();\r
+\r
+                            bmp = BitmapFactory.decodeFile(mFile.getStoragePath(), options);\r
+                        }\r

                         preview.setImageBitmap(bmp);\r
                     }\r
                 } catch (OutOfMemoryError e) {\r
                         preview.setImageBitmap(bmp);\r
                     }\r
                 } catch (OutOfMemoryError e) {\r

+                    preview.setVisibility(View.INVISIBLE);\r

                     Log.e(TAG, "Out of memory occured for file with size " + mFile.getFileLength());\r
                     Log.e(TAG, "Out of memory occured for file with size " + mFile.getFileLength());\r

+                    \r
+                } catch (NoSuchFieldError e) {\r
+                    preview.setVisibility(View.INVISIBLE);\r
+                    Log.e(TAG, "Error from access to unexisting field despite protection " + mFile.getFileLength());\r
+                    \r
+                } catch (Throwable t) {\r
+                    preview.setVisibility(View.INVISIBLE);\r
+                    Log.e(TAG, "Unexpected error while creating image preview " + mFile.getFileLength(), t);\r

                 }\r
                 downloadButton.setText(R.string.filedetails_open);\r
                 downloadButton.setOnClickListener(new OnClickListener() {\r
                     @Override\r
                     public void onClick(View v) {\r
                 }\r
                 downloadButton.setText(R.string.filedetails_open);\r
                 downloadButton.setOnClickListener(new OnClickListener() {\r
                     @Override\r
                     public void onClick(View v) {\r

-                        Intent i = new Intent(Intent.ACTION_VIEW);\r
-                        i.setDataAndType(Uri.parse("file://"+mFile.getStoragePath()), mFile.getMimetype());\r
-                        startActivity(i);\r
+                        String storagePath = mFile.getStoragePath();\r
+                        try {\r
+                            Intent i = new Intent(Intent.ACTION_VIEW);\r
+                            i.setDataAndType(Uri.parse("file://"+ storagePath), mFile.getMimetype());\r
+                            i.setFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION | Intent.FLAG_GRANT_WRITE_URI_PERMISSION);\r
+                            startActivity(i);\r
+                            \r
+                        } catch (Throwable t) {\r
+                            Log.e(TAG, "Fail when trying to open with the mimeType provided from the ownCloud server: " + mFile.getMimetype());\r
+                            boolean toastIt = true; \r
+                            String mimeType = "";\r
+                            try {\r
+                                Intent i = new Intent(Intent.ACTION_VIEW);\r
+                                mimeType = MimeTypeMap.getSingleton().getMimeTypeFromExtension(storagePath.substring(storagePath.lastIndexOf('.') + 1));\r
+                                if (mimeType != mFile.getMimetype()) {\r
+                                    i.setDataAndType(Uri.parse("file://"+mFile.getStoragePath()), mimeType);\r
+                                    i.setFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION | Intent.FLAG_GRANT_WRITE_URI_PERMISSION);\r
+                                    startActivity(i);\r
+                                    toastIt = false;\r
+                                }\r
+                                \r
+                            } catch (IndexOutOfBoundsException e) {\r
+                                Log.e(TAG, "Trying to find out MIME type of a file without extension: " + storagePath);\r
+                                \r
+                            } catch (ActivityNotFoundException e) {\r
+                                Log.e(TAG, "No activity found to handle: " + storagePath + " with MIME type " + mimeType + " obtained from extension");\r
+                                \r
+                            } catch (Throwable th) {\r
+                                Log.e(TAG, "Unexpected problem when opening: " + storagePath, th);\r
+                                \r
+                            } finally {\r
+                                if (toastIt) {\r
+                                    Toast.makeText(getActivity(), "There is no application to handle file " + mFile.getFileName(), Toast.LENGTH_SHORT).show();\r
+                                }\r
+                            }\r
+                            \r
+                        }\r

                     }\r
                 });\r
             } else {\r
                     }\r
                 });\r
             } else {\r


@@ -272,24 +350,21 @@ public class FileDetailFragment extends SherlockFragment implements
     }\r
     \r
     /**\r
     }\r
     \r
     /**\r

-     * In ownCloud 3.0.3 and 4.0.0 there is a bug that SabreDAV does not return\r
+     * In ownCloud 3.X.X and 4.X.X there is a bug that SabreDAV does not return\r

      * the time that the file was created. There is a chance that this will\r
      * be fixed in future versions. Use this method to check if this version of\r
      * ownCloud has this fix.\r
      * the time that the file was created. There is a chance that this will\r
      * be fixed in future versions. Use this method to check if this version of\r
      * ownCloud has this fix.\r

-     * @return True, if ownCloud the ownCloud version is > 3.0.4 and 4.0.1\r
+     * @return True, if ownCloud the ownCloud version is supporting creation time\r

      */\r
     private boolean ocVersionSupportsTimeCreated(){\r
      */\r
     private boolean ocVersionSupportsTimeCreated(){\r

-        if(mIntent != null){\r
-            Account ocAccount = mIntent.getParcelableExtra(FileDownloader.EXTRA_ACCOUNT);\r
-            if(ocAccount != null){\r
-                AccountManager accManager = (AccountManager) getActivity().getSystemService(Context.ACCOUNT_SERVICE);\r
-                OwnCloudVersion ocVersion = new OwnCloudVersion(accManager\r
-                        .getUserData(ocAccount, AccountAuthenticator.KEY_OC_VERSION));\r
-                if(ocVersion.compareTo(new OwnCloudVersion(0x030004)) >= 0 || ocVersion.compareTo(new OwnCloudVersion(0x040001)) >= 0){\r
-                    return true;\r
-                }\r
+        /*if(mAccount != null){\r
+            AccountManager accManager = (AccountManager) getActivity().getSystemService(Context.ACCOUNT_SERVICE);\r
+            OwnCloudVersion ocVersion = new OwnCloudVersion(accManager\r
+                    .getUserData(mAccount, AccountAuthenticator.KEY_OC_VERSION));\r
+            if(ocVersion.compareTo(new OwnCloudVersion(0x030000)) < 0) {\r
+                return true;\r

             }\r
             }\r

-        }\r
+        }*/\r

         return false;\r
     }\r
 \r
         return false;\r
     }\r
 \r


@@ -300,9 +375,17 @@ public class FileDetailFragment extends SherlockFragment implements
     private class DownloadFinishReceiver extends BroadcastReceiver {\r
         @Override\r
         public void onReceive(Context context, Intent intent) {\r
     private class DownloadFinishReceiver extends BroadcastReceiver {\r
         @Override\r
         public void onReceive(Context context, Intent intent) {\r

-            updateFileDetails();\r
+            getView().findViewById(R.id.fdDownloadBtn).setEnabled(true);\r
+            if (intent.getAction().equals(FileDownloader.BAD_DOWNLOAD_MESSAGE)) {\r
+                Toast.makeText(context, R.string.downloader_download_failed , Toast.LENGTH_SHORT).show();\r
+                \r
+            } else if (intent.getAction().equals(FileDownloader.DOWNLOAD_FINISH_MESSAGE)) {\r
+                mFile.setStoragePath(intent.getStringExtra(FileDownloader.EXTRA_FILE_PATH));\r
+                updateFileDetails();\r
+            }\r

         }\r
         }\r

-\r
+        \r

     }\r
     }\r

+    \r

 \r
 }\r
 \r
 }\r