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

Autoupload of images does not work when SD card is selected as the download destination of photos by camera app #496

Closed
elpraga opened this issue Jan 2, 2017 · 23 comments
Labels
bug needs info Waiting for info from user(s). Issues with this label will auto-stale.

Comments

@elpraga
Copy link

elpraga commented Jan 2, 2017

Actual behaviour

Select SD card as your storage in your camera app.
Photos don't get auto uploaded.

Expected behaviour

Photos should get auto uploaded.

Steps to reproduce

  1. Select SD card as your storage in your camera app.
  2. Select automatic upload in the NC android app
  3. Take a photo with your camera app.

Environment data

Android version:
6.0
Device model:
Huawei Y6II
Stock or customized system:

Nextcloud app version:
1.4.0
Nextcloud server version:
NC 11.0.0

Logs

Web server error log

Insert your webserver log here

Nextcloud log (data/nextcloud.log)

Insert your Nextcloud log here
@elpraga
Copy link
Author

elpraga commented Jan 2, 2017

Side note I forgot to mention: Autoupload works as expected on my device with a different camera app which has internal storage selected to save photos.

@AndyScherzinger
Copy link
Member

Hi @elpraga
Can you get a log cat from the phone / Nextcloud client since the question is if choosing SD card leads to the camera app not sending a new picture broadcast (which our app is listening to).

@tobiasKaminsky
Copy link
Member

@elpraga you mean "InstantUpload" as you are using Android 6?
"AutoUpload" is our new feature for Android 7.

@elpraga
Copy link
Author

elpraga commented Jan 4, 2017

@tobiasKaminsky yes, I do indeed. Sorry for the mismatch. @AndyScherzinger how can I get what you ask me for from the 1.4.0? I apologize for my ignorance...

Happy new year to both of you!

@AndyScherzinger
Copy link
Member

Hi @elpraga,

you probably can't since the stable doesn't have logging enabled, just the beta has :( Else you would have to connect your device via USB + adb to Android Studio (dev environment) and there check the devices logging in the log cat console within Android studio. My guess is that the stock camera, when writing to an external-sd-card storage won't send a broadcast, but I am not sure.

@OlivierS1
Copy link

Same behavior with Nextcloud app 1.4.1 RC1.
Nextcloud data can be stored eiher on internal memory or external SD card, Autoupload will not take effect if Camera app is using external SD card as the primary storage media.
Restarting the phone and selectign Internal memory as storage media for the camera app will allow Autoupload to take place.

@tobiasKaminsky
Copy link
Member

@OlivierS1 which device model do you have?

@OlivierS1
Copy link

@tobiasKaminsky:

Android version:
6.0
Device model:
LG-G3 LG-D852

Stock or customized system:
Stock

@elpraga
Copy link
Author

elpraga commented Jan 6, 2017

It seems to be working now on my phone. I'm not sure what the issue was exactly, but my phone might have been killing NextCloud app in the background.

In any case, I'm not able to reproduce the error. It's working now.

I'll post updates if I have new info.

I'll leave this issue open for @OlivierS1

@mario
Copy link
Contributor

mario commented Feb 13, 2017

@elpraga
Copy link
Author

elpraga commented Feb 13, 2017

@mario Although I did not use the link you provided, I believe that I am using the same version, because I updated to the latest version using the app. Beta20170213 still creates the tmp file in the main storage and downloads fail. It is not working yet.

@mario
Copy link
Contributor

mario commented Feb 13, 2017

@elpraga thanks. Gonna investigate right away mate :)

@elpraga
Copy link
Author

elpraga commented Feb 13, 2017

Thanks for the swift answer and your help pal! 😉

@mario
Copy link
Contributor

mario commented Feb 13, 2017

I just tried this and it works :-/ Will be hard to debug...

@mario
Copy link
Contributor

mario commented Feb 13, 2017

Can you please clarify "downloads fail"? What downloads? Do you mean uploads?

@elpraga
Copy link
Author

elpraga commented Feb 13, 2017

Wow.
Stupid question, do you have SDcard selected as the storage?

And where is the tmp file created? On the SD card or in the main storage?

How can I help to debug?

@mario
Copy link
Contributor

mario commented Feb 13, 2017

Hehe, yes I do have SD card selected as the storage. Can you screenshot your storage settings? I'd have to check where the tmp file is created.

@elpraga
Copy link
Author

elpraga commented Feb 13, 2017

No. I mean downloads. Uploads work fine for me, even automatic photo uploads.

The problem is when I try to download a file. The app creates a tmp file in the main storage (the official app doesn't, it creates the tmp file on the SD card). The file gets downloaded to the tmp folder correctly, but the app fails to copy it to its final destination, and an error stating download failed appears.

Did this help @mario ?

@mario
Copy link
Contributor

mario commented Feb 13, 2017

Ok, so this is a wholeeee separate issue :D Can you open a new issue, and copy/paste the error logs if you have them? :)

@elpraga
Copy link
Author

elpraga commented Feb 13, 2017

I'm so sorry! My bad!

I have already opened an issue for it a month ago: Beta20161224: files don't get downloaded properly when SD card storage is selected, and you said you'd have a look at it.

As you mentioned my name, I thought were referring to that issue.

I'm really sorry 👼

@mario
Copy link
Contributor

mario commented Feb 13, 2017

No worries, it's just complicated to track everything :D

@tobiasKaminsky tobiasKaminsky added bug needs info Waiting for info from user(s). Issues with this label will auto-stale. labels Jun 27, 2017
@tobiasKaminsky
Copy link
Member

@elpraga is it wroking now for you?

@elpraga
Copy link
Author

elpraga commented Jun 27, 2017

Yes. It's working now. I have totally forgotten about this issue.

Closing.

@elpraga elpraga closed this as completed Jun 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug needs info Waiting for info from user(s). Issues with this label will auto-stale.
Projects
None yet
Development

No branches or pull requests

5 participants