Small improvements to the refresh of the files list and memory handling while synchro...

[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 3e3082c..627c26a 100644 (file)
--- a/src/eu/alefzero/owncloud/ui/fragment/FileDetailFragment.java
+++ b/src/eu/alefzero/owncloud/ui/fragment/FileDetailFragment.java
@@ -17,29 +17,37 @@
  */\r
 package eu.alefzero.owncloud.ui.fragment;\r
 \r
  */\r
 package eu.alefzero.owncloud.ui.fragment;\r
 \r

-import android.app.FragmentTransaction;\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.os.Bundle;\r

+import android.util.Log;\r

 import android.view.LayoutInflater;\r
 import android.view.View;\r
 import android.view.View.OnClickListener;\r
 import android.view.ViewGroup;\r
 import android.view.LayoutInflater;\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.Toast;\r
 import android.widget.ImageView;\r
 import android.widget.TextView;\r
 import android.widget.Toast;\r

-import android.widget.VideoView;\r

 \r
 import com.actionbarsherlock.app.SherlockFragment;\r
 \r
 import eu.alefzero.owncloud.DisplayUtils;\r
 \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.R;\r

+import eu.alefzero.owncloud.authenticator.AccountAuthenticator;\r

 import eu.alefzero.owncloud.datamodel.OCFile;\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
  * This Fragment is used to display the details about a file.\r
 \r
 /**\r
  * This Fragment is used to display the details about a file.\r


@@ -50,37 +58,80 @@ import eu.alefzero.owncloud.datamodel.OCFile;
 public class FileDetailFragment extends SherlockFragment implements\r
         OnClickListener {\r
 \r
 public class FileDetailFragment extends SherlockFragment implements\r
         OnClickListener {\r
 \r

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

 \r
 \r

-    private Intent mIntent;\r
+    private int mLayout;\r

     private View mView;\r
     private View mView;\r

-    private DownloadFinishReceiver mDownloadFinishReceiver;\r

     private OCFile mFile;\r
     private OCFile mFile;\r

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

 \r
 \r

-    private int mLayout;\r
-    private boolean mEmptyLayout;\r
+    private static final String TAG = "FileDetailFragment";\r
+    public static final String FTAG = "FileDetails"; \r

 \r
 \r

+    \r

     /**\r
     /**\r

-     * Default constructor. When inflated by android -> display empty 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
     public FileDetailFragment() {\r
      */\r
     public FileDetailFragment() {\r

+        mFile = null;\r
+        mAccount = null;\r

         mLayout = R.layout.file_details_empty;\r
         mLayout = R.layout.file_details_empty;\r

-        mEmptyLayout = true;\r

     }\r
     }\r

-\r
+    \r
+    \r

     /**\r
     /**\r

-     * Custom construtor. Use with a {@link FragmentTransaction}. The intent has\r
-     * to contain {@link FileDetailFragment#FILE} with an OCFile and also\r
-     * {@link FileDownloader#EXTRA_ACCOUNT} with the account.\r
+     * Creates a details fragment.\r

      * \r
      * \r

-     * @param nonEmptyFragment True, to enable file detail rendering\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(Intent intent) {\r
-        mLayout = R.layout.file_details_fragment;\r
-        mIntent = intent;\r
-        mEmptyLayout = false;\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
+    \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
+    @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
 \r

+    \r

     @Override\r
     public void onResume() {\r
         super.onResume();\r
     @Override\r
     public void onResume() {\r
         super.onResume();\r


@@ -97,129 +148,244 @@ public class FileDetailFragment extends SherlockFragment implements
         mDownloadFinishReceiver = null;\r
     }\r
 \r
         mDownloadFinishReceiver = null;\r
     }\r
 \r

+    @Override\r
+    public View getView() {\r
+        return super.getView() == null ? mView : super.getView();\r
+    }\r
+\r
+    @Override\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, 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
+    /**\r
+     * Can be used to get the file that is currently being displayed.\r
+     * @return The file on the screen.\r
+     */\r
+    public OCFile getDisplayedFile(){\r
+        return mFile;\r
+    }\r
+    \r

     /**\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#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

-    private void updateFileDetails() {\r
-        mFile = mIntent.getParcelableExtra(FILE);\r
+    /**\r
+     * Updates the view with all relevant details about that file.\r
+     */\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
                     .getMimetype()));\r
             setFilesize(mFile.getFileLength());\r
             // set file details\r
             setFilename(mFile.getFileName());\r
             setFiletype(DisplayUtils.convertMIMEtoPrettyPrint(mFile\r
                     .getMimetype()));\r
             setFilesize(mFile.getFileLength());\r

-\r
-            // set file preview if available and possible\r
-            VideoView videoView = (VideoView) mView\r
-                    .findViewById(R.id.videoView1);\r
-            videoView.setVisibility(View.INVISIBLE);\r
-            if (mFile.getStoragePath() == null) {\r
-                ImageView imageView = (ImageView) getView().findViewById(\r
-                        R.id.imageView2);\r
-                imageView.setImageResource(R.drawable.download);\r
-                imageView.setOnClickListener(this);\r
-            } else {\r
-                if (mFile.getMimetype().startsWith("image/")) {\r
-                    ImageView imageView = (ImageView) mView\r
-                            .findViewById(R.id.imageView2);\r
-                    Bitmap bmp = BitmapFactory.decodeFile(mFile.getStoragePath());\r
-                    imageView.setImageBitmap(bmp);\r
-                } else if (mFile.getMimetype().startsWith("video/")) {\r
-                    videoView.setVisibility(View.VISIBLE);\r
-                    videoView.setVideoPath(mFile.getStoragePath());\r
-                    videoView.start();\r
-                }\r
+            if(ocVersionSupportsTimeCreated()){\r
+                setTimeCreated(mFile.getCreationTimestamp());\r

             }\r
             }\r

-        }\r
-    }\r
+           \r
+            setTimeModified(mFile.getModificationTimestamp());\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
+                        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
 \r

-    @Override\r
-    public View onCreateView(LayoutInflater inflater, ViewGroup container,\r
-            Bundle savedInstanceState) {\r
-        View view = null;\r
+                        Bitmap bmp = BitmapFactory.decodeFile(mFile.getStoragePath(), options);\r

 \r
 \r

-        view = inflater.inflate(mLayout, container, false);\r
-        mIntent = getActivity().getIntent();\r
-        mView = view;\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
 \r

-        // make sure we are not using the empty layout\r
-        if (mEmptyLayout == false) {\r
-            updateFileDetails();\r
+                            bmp = BitmapFactory.decodeFile(mFile.getStoragePath(), options);\r
+                        }\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
+                    \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
+                        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
+                // Make download button effective\r
+                downloadButton.setOnClickListener(this);\r
+            }\r

         }\r
         }\r

-\r
-        return view;\r

     }\r
     }\r

-\r
-    @Override\r
-    public View getView() {\r
-        return mView == null ? super.getView() : mView;\r
-    };\r
-\r
+    \r
+    /**\r
+     * Updates the filename in view\r
+     * @param filename to set\r
+     */\r

     private void setFilename(String filename) {\r
     private void setFilename(String filename) {\r

-        TextView tv = (TextView) getView().findViewById(R.id.textView1);\r
+        TextView tv = (TextView) getView().findViewById(R.id.fdFilename);\r

         if (tv != null)\r
             tv.setText(filename);\r
     }\r
 \r
         if (tv != null)\r
             tv.setText(filename);\r
     }\r
 \r

+    /**\r
+     * Updates the MIME type in view\r
+     * @param mimetype to set\r
+     */\r

     private void setFiletype(String mimetype) {\r
     private void setFiletype(String mimetype) {\r

-        TextView tv = (TextView) getView().findViewById(R.id.textView2);\r
+        TextView tv = (TextView) getView().findViewById(R.id.fdType);\r

         if (tv != null)\r
             tv.setText(mimetype);\r
     }\r
 \r
         if (tv != null)\r
             tv.setText(mimetype);\r
     }\r
 \r

+    /**\r
+     * Updates the file size in view\r
+     * @param filesize in bytes to set\r
+     */\r

     private void setFilesize(long filesize) {\r
     private void setFilesize(long filesize) {\r

-        TextView tv = (TextView) getView().findViewById(R.id.textView3);\r
+        TextView tv = (TextView) getView().findViewById(R.id.fdSize);\r

         if (tv != null)\r
         if (tv != null)\r

-            tv.setText(DisplayUtils.bitsToHumanReadable(filesize));\r
+            tv.setText(DisplayUtils.bytesToHumanReadable(filesize));\r

     }\r
     }\r

-\r
+    \r

     /**\r
     /**\r

-     * Use this to check if the correct layout is loaded. When android\r
-     * instanciates this class using the default constructor, the layout will be\r
-     * empty.\r
-     * \r
-     * Once a user touches a file for the first time, you must instanciate a new\r
-     * Fragment with the new FileDetailFragment(true) to inflate the actual\r
-     * details\r
-     * \r
-     * @return If the layout is empty, this method will return true, otherwise\r
-     *         false\r
+     * Updates the time that the file was created in view\r
+     * @param milliseconds Unix time to set\r

      */\r
      */\r

-    public boolean isEmptyLayout() {\r
-        return mEmptyLayout;\r
+    private void setTimeCreated(long milliseconds){\r
+        TextView tv = (TextView) getView().findViewById(R.id.fdCreated);\r
+        TextView tvLabel = (TextView) getView().findViewById(R.id.fdCreatedLabel);\r
+        if(tv != null){\r
+            tv.setText(DisplayUtils.unixTimeToHumanReadable(milliseconds));\r
+            tv.setVisibility(View.VISIBLE);\r
+            tvLabel.setVisibility(View.VISIBLE);\r
+        }\r

     }\r
     }\r

-\r
-    @Override\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,\r
-                mIntent.getStringExtra(FileDownloader.EXTRA_FILE_PATH));\r
-        getActivity().startService(i);\r
+    \r
+    /**\r
+     * Updates the time that the file was last modified\r
+     * @param milliseconds Unix time to set\r
+     */\r
+    private void setTimeModified(long milliseconds){\r
+        TextView tv = (TextView) getView().findViewById(R.id.fdModified);\r
+        if(tv != null){\r
+            tv.setText(DisplayUtils.unixTimeToHumanReadable(milliseconds));\r
+        }\r
+    }\r
+    \r
+    /**\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
+     * @return True, if ownCloud the ownCloud version is supporting creation time\r
+     */\r
+    private boolean ocVersionSupportsTimeCreated(){\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
+        return false;\r

     }\r
 \r
     }\r
 \r

+    /**\r
+     * Once the file download has finished -> update view\r
+     * @author Bartek Przybylski\r
+     */\r

     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