Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Auto upload uploads existing files causing duplicates #1141

Closed
rugk opened this issue Jun 26, 2017 · 15 comments
Closed

Auto upload uploads existing files causing duplicates #1141

rugk opened this issue Jun 26, 2017 · 15 comments

Comments

@rugk
Copy link

rugk commented Jun 26, 2017

Actual behaviour

On every phone restart the auto uploader uploads all files in the directory (just the camera dir chosen).
I can see how the file name in the notification changes from "IMG123456.jpg" to "IMG123456 (2).jpg" in the "upload finished" notification. So I can clearly see it duplicates these photos.

Expected behaviour

First check if the file exists on the server (& has already be uploaded) and only then upload it.

Steps to reproduce

  1. Upload some files already (e.g. with a previous app version).
  2. Upgrade NextCloud app (and/or) enable the upload-feature
  3. See how uploads all your photos and you have to cancel each single item to stop the process.

Environment data

Android version: 7.1

Device model: LineageOS (14.1)

Stock or customized system: see above

Nextcloud app version: 1.4.2

Nextcloud server version: 12

@marvinwankersteen
Copy link

marvinwankersteen commented Jun 29, 2017

I have the same problem with the activated beta in Google Play Store.

After the most phone restarts, the app starts to re-upload all Images. Sometimes not. I have to deactivate and activate the app. If I just deactivate the auto-upload, the images furthermore uploading.

If the app was deactivated, I have to re-enter the address and credentials.

Android version: 6.0.0 and 6.0.1
Stock or customized system: Both
Nextcloud app version: 1.4.3
Nextcloud server version: 12.0.0

@ShapeShifter499
Copy link

ShapeShifter499 commented Jul 2, 2017

I have this issue too, every reboot Nextcloud tries to auto upload some photos but it also ends up uploading duplicates.

My system
Moto X Pure
LineageOS 14.1 06-26-2017 Nightly
Nextcloud server version 12.0.0
Nextcloud android client version 1.4.2

@eloo
Copy link

eloo commented Jul 3, 2017

yep, facing this issue too. Should be nice if this could be fixed in time

@rcmcronny
Copy link

yes, i am facing this issue to

@ingmarcosp
Copy link

Me too..
Huawei p8 Android 6.0
Lasted Android app un Google playStore
Nextcloud 12.0. 0

@kst87
Copy link

kst87 commented Jul 18, 2017

Me too.
LG G4 original (Vodafone) Android 6.0
App Nextcloud 1.4.3
Nextcloud 12.0.0

@mario
Copy link
Contributor

mario commented Jul 19, 2017

Thanks for all your reports, but this will be fixed in 1.5 - we're aware of the issues.

@mario
Copy link
Contributor

mario commented Jul 26, 2017

Dear users,

thank you for your patience. After lots of work, we’re super excited to be able to give you a test version with new auto upload implementation. Keep in mind we will only accept bugs related to Auto Upload in this APK, as other things might be broken.

https://cloud.nextcloud.com/s/ftZraTruRQjTe0R

This is a dev APK so it will not overwrite your existing play store/f-droid APK unless you’re already running dev version. Also, you might need to remove your existing dev version before installing this one.

Explanation of the current functionality:

  • added support for custom folders
  • separated image and video folders
  • checks for new files happens every 15-20 minutes
  • automatic retry every 15-20 minutes or when you force close/open the app
  • automatic pause once network is changed (untested!)
  • significantly less battery usage

All and any testing is more than appreciated, especially with large and ever-going files like Videos. (like, try to record a 30 min video).

Let me know if you’ve got any questions.

Cheers,
Mario

@eloo
Copy link

eloo commented Jul 26, 2017

@mario
Thanks for your work.
Sounds really promising! I will give it a try in time.

@eloo
Copy link

eloo commented Jul 27, 2017

are these changes in the dev-20170727 version?

@mario
Copy link
Contributor

mario commented Jul 27, 2017

No, they are only in my APK. (There's already a new version, same URL) because we can't build this automatically yet until Google fixes their toolchain.

@eloo
Copy link

eloo commented Jul 29, 2017

So i've installed your apk and so far it looks promising!
Does seem to try reuploading already uploaded pictures and battery drain seems to be reduced, too.
Will try it for some more days.
But so far thanks!!!

@mario
Copy link
Contributor

mario commented Jul 29, 2017 via email

@eloo
Copy link

eloo commented Aug 7, 2017

@mario
So far i didn't notice any issue with your version.
No duplicates were created and all pictures are uploaded.
Battery consumption is very low
Nice work!

Will your changes be included in the next release 1.5?

@mario
Copy link
Contributor

mario commented Aug 7, 2017

@eloo thanks. Closing this issue then since this will be in 1.5 - yes! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants