Merge branch 'develop' into refactor_remote_operation_to_create_folder
authorDavid A. Velasco <dvelasco@solidgear.es>
Mon, 18 Nov 2013 13:10:16 +0000 (14:10 +0100)
committerDavid A. Velasco <dvelasco@solidgear.es>
Mon, 18 Nov 2013 13:10:16 +0000 (14:10 +0100)
1  2 
AndroidManifest.xml
res/values/strings.xml
src/com/owncloud/android/files/OwnCloudFileObserver.java
src/com/owncloud/android/files/services/FileObserverService.java

diff --combined AndroidManifest.xml
@@@ -18,8 -18,8 +18,8 @@@
    along with this program.  If not, see <http://www.gnu.org/licenses/>.
   -->
  <manifest package="com.owncloud.android"
-     android:versionCode="104006"
-     android:versionName="1.4.6" xmlns:android="http://schemas.android.com/apk/res/android">
+     android:versionCode="105000"
+     android:versionName="1.5.0" xmlns:android="http://schemas.android.com/apk/res/android">
  
      <uses-permission android:name="android.permission.GET_ACCOUNTS" />
      <uses-permission android:name="android.permission.USE_CREDENTIALS" />
@@@ -63,7 -63,7 +63,7 @@@
                <activity android:name=".ui.activity.InstantUploadActivity">
          </activity>
          <activity android:name=".ui.activity.FailedUploadActivity" android:theme="@android:style/Theme.Dialog" android:excludeFromRecents="true"/>
 -        <activity android:name=".Uploader" >
 +        <activity android:name=".ui.activity.Uploader" >
              <intent-filter>
                  <action android:name="android.intent.action.SEND" >
                  </action>
diff --combined res/values/strings.xml
      <string name="sync_file_fail_msg">Remote file could not be checked</string>
      <string name="sync_file_nothing_to_do_msg">File contents already synchronized</string>
      <string name="create_dir_fail_msg">Directory could not be created</string>
 +    <string name="filename_forbidden_characters">Forbidden characters: / \\ &lt; &gt; : " | ? *</string>
      <string name="wait_a_moment">Wait a moment</string>
      <string name="filedisplay_unexpected_bad_get_content">"Unexpected problem ; please select the file from a different app"</string>
      <string name="filedisplay_no_file_selected">No file was selected</string>
      <string name="preview_image_error_unknown_format">This image can not be shown</string>
      
      <string name="error__upload__local_file_not_copied">%1$s could not be copied to %2$s local directory</string>
-     <string name="actionbar_failed_instant_upload">Failed InstantUpload"</string>
+     <string name="actionbar_failed_instant_upload">Failed InstantUpload</string>
      <string name="failed_upload_headline_text">Failed instant uploads</string>
      <string name="failed_upload_headline_hint">Summary of all failed instant uploads</string>
      <string name="failed_upload_all_cb">select all</string>
@@@ -20,16 -20,14 +20,14 @@@ package com.owncloud.android.files
  
  import java.io.File;
  
 -import com.owncloud.android.Log_OC;
  import com.owncloud.android.datamodel.FileDataStorageManager;
  import com.owncloud.android.datamodel.OCFile;
 -import com.owncloud.android.operations.RemoteOperationResult;
 +import com.owncloud.android.oc_framework.operations.RemoteOperationResult;
  import com.owncloud.android.operations.SynchronizeFileOperation;
 -import com.owncloud.android.operations.RemoteOperationResult.ResultCode;
 +import com.owncloud.android.oc_framework.operations.RemoteOperationResult.ResultCode;
  import com.owncloud.android.ui.activity.ConflictsResolveActivity;
 +import com.owncloud.android.utils.Log_OC;
  
  import android.accounts.Account;
  import android.content.Context;
  import android.content.Intent;
@@@ -37,45 -35,57 +35,57 @@@ import android.os.FileObserver
  
  public class OwnCloudFileObserver extends FileObserver {
  
-     public static int CHANGES_ONLY = CLOSE_WRITE;
+     private static int MASK = (FileObserver.MODIFY | FileObserver.CLOSE_WRITE);
      
      private static String TAG = OwnCloudFileObserver.class.getSimpleName();
      
      private String mPath;
      private int mMask;
      private Account mOCAccount;
-     //private OCFile mFile;
      private Context mContext;
+     private boolean mModified;
  
      
-     public OwnCloudFileObserver(String path, Account account, Context context, int mask) {
-         super(path, mask);
+     public OwnCloudFileObserver(String path, Account account, Context context) {
+         super(path, MASK);
          if (path == null)
              throw new IllegalArgumentException("NULL path argument received"); 
-         /*if (file == null)
-             throw new IllegalArgumentException("NULL file argument received");*/ 
          if (account == null)
              throw new IllegalArgumentException("NULL account argument received"); 
          if (context == null)
              throw new IllegalArgumentException("NULL context argument received");
-         /*if (!path.equals(file.getStoragePath()) && !path.equals(FileStorageUtils.getDefaultSavePathFor(account.name, file)))
-             throw new IllegalArgumentException("File argument is not linked to the local file set in path argument"); */
          mPath = path;
-         //mFile = file;
          mOCAccount = account;
          mContext = context; 
-         mMask = mask;
+         mModified = false;
      }
      
+     
      @Override
      public void onEvent(int event, String path) {
          Log_OC.d(TAG, "Got file modified with event " + event + " and path " + mPath + ((path != null) ? File.separator + path : ""));
-         if ((event & mMask) == 0) {
+         if ((event & MASK) == 0) {
              Log_OC.wtf(TAG, "Incorrect event " + event + " sent for file " + mPath + ((path != null) ? File.separator + path : "") +
                           " with registered for " + mMask + " and original path " +
                           mPath);
-             return;
-         }
+         } else {
+             if ((event & FileObserver.MODIFY) != 0) {
+                 // file changed
+                 mModified = true;
+             }
+             // not sure if it's possible, but let's assume that both kind of events can be received at the same time
+             if ((event & FileObserver.CLOSE_WRITE) != 0) {
+                 // file closed
+                 if (mModified) {
+                     mModified = false;
+                     startSyncOperation();
+                 }
+             }
+         }  
+     }
+     
+     private void startSyncOperation() {
          FileDataStorageManager storageManager = new FileDataStorageManager(mOCAccount, mContext.getContentResolver());
          OCFile file = storageManager.getFileByLocalPath(mPath);     // a fresh object is needed; many things could have occurred to the file since it was registered to observe
                                                                      // again, assuming that local files are linked to a remote file AT MOST, SOMETHING TO BE DONE; 
@@@ -22,13 -22,13 +22,13 @@@ import java.io.File
  import java.util.HashMap;
  import java.util.Map;
  
 -import com.owncloud.android.Log_OC;
  import com.owncloud.android.datamodel.FileDataStorageManager;
  import com.owncloud.android.datamodel.OCFile;
  import com.owncloud.android.db.ProviderMeta.ProviderTableMeta;
  import com.owncloud.android.files.OwnCloudFileObserver;
  import com.owncloud.android.operations.SynchronizeFileOperation;
  import com.owncloud.android.utils.FileStorageUtils;
 +import com.owncloud.android.utils.Log_OC;
  
  
  import android.accounts.Account;
@@@ -162,8 -162,7 +162,7 @@@ public class FileObserverService extend
              OwnCloudFileObserver observer =
                      new OwnCloudFileObserver(   path, 
                                                  account, 
-                                                 getApplicationContext(), 
-                                                 OwnCloudFileObserver.CHANGES_ONLY);
+                                                 getApplicationContext());
              mObserversMap.put(path, observer);
              if (new File(path).exists()) {
                  observer.startWatching();
              /// the local file was never registered to observe before
              observer = new OwnCloudFileObserver(    localPath, 
                                                      account, 
-                                                     getApplicationContext(), 
-                                                     OwnCloudFileObserver.CHANGES_ONLY);
+                                                     getApplicationContext());
              mObserversMap.put(localPath, observer);
              Log_OC.d(TAG, "Observer added for path " + localPath);