瀏覽代碼

checks and debug output

Luke Owncloud 10 年之前
父節點
當前提交
2ad3ef2456

+ 3 - 3
src/com/owncloud/android/ui/activity/UploadListActivity.java

@@ -158,13 +158,13 @@ public class UploadListActivity extends FileActivity implements UploadListFragme
                 if(mUploaderBinder == null)
                 {
                     mUploaderBinder = (FileUploaderBinder) service;
-                    Log_OC.e(TAG, "UploadListActivity connected to Upload service. component: " + component + " service: "
+                    Log_OC.d(TAG, "UploadListActivity connected to Upload service. component: " + component + " service: "
                             + service);
                 } else {
-                    Log_OC.e(TAG, "mUploaderBinder already set. mUploaderBinder: " + mUploaderBinder + " service:" + service);
+                    Log_OC.d(TAG, "mUploaderBinder already set. mUploaderBinder: " + mUploaderBinder + " service:" + service);
                 }
             } else {
-                Log_OC.e(TAG, "UploadListActivity not connected to Upload service. component: " + component
+                Log_OC.d(TAG, "UploadListActivity not connected to Upload service. component: " + component
                         + " service: " + service);
                 return;
             }            

+ 15 - 8
src/com/owncloud/android/ui/adapter/ExpandableUploadListAdapter.java

@@ -173,13 +173,17 @@ public class ExpandableUploadListAdapter extends BaseExpandableListAdapter imple
                 progressBar.setProgress(0);
                 progressBar.setVisibility(View.VISIBLE);
                 mProgressListener = new ProgressListener(progressBar);
-                if (parentFileActivity.getFileUploaderBinder() != null) {
+                if(parentFileActivity.getFileUploaderBinder() != null) {
                     mCurrentUpload = parentFileActivity.getFileUploaderBinder().getCurrentUploadOperation();
-                    mCurrentUpload.addDatatransferProgressListener(mProgressListener);
+                    if(mCurrentUpload != null) {
+                        mCurrentUpload.addDatatransferProgressListener(mProgressListener);
+                        Log_OC.d(TAG, "added progress listener for current upload: " + mCurrentUpload);
+                    } else {
+                        Log_OC.w(TAG, "getFileUploaderBinder().getCurrentUploadOperation() return null. That is odd.");
+                    }
                 } else {
-                    Log_OC.e(
-                            TAG,
-                            "UploadBinder == null. It should have been created on creating parentFileActivity which inherits from FileActivity. Fix that!");
+                    Log_OC.e(TAG, "UploadBinder == null. It should have been created on creating parentFileActivity"
+                            + " which inherits from FileActivity. Fix that!");
                 }
                 break;
             case UPLOAD_FAILED_GIVE_UP:
@@ -224,11 +228,14 @@ public class ExpandableUploadListAdapter extends BaseExpandableListAdapter imple
             if(uploadObject.getUploadStatus() != UploadStatus.UPLOAD_IN_PROGRESS) {
                 ProgressBar progressBar = (ProgressBar) view.findViewById(R.id.upload_progress_bar);
                 progressBar.setVisibility(View.GONE);
-                if(mCurrentUpload != null){
-                    mCurrentUpload.removeDatatransferProgressListener(mProgressListener);
+                if (parentFileActivity.getFileUploaderBinder() != null && mProgressListener != null
+                        && mCurrentUpload != null) {
+                    OCFile currentOcFile = mCurrentUpload.getFile();
+                    parentFileActivity.getFileUploaderBinder().removeDatatransferProgressListener(mProgressListener,
+                            uploadObject.getAccount(mActivity), currentOcFile);
                     mProgressListener = null;
                     mCurrentUpload = null;
-                }
+                }            
             }
             statusView.setText(status);