nextcloud / android

📱 Nextcloud Android app
https://play.google.com/store/apps/details?id=com.nextcloud.client
GNU General Public License v2.0
4.25k stars 1.76k forks source link

android app crashes when trying to sort a directory of photos to upload #336

Closed koreywithak closed 7 years ago

koreywithak commented 8 years ago

Actual behaviour

When trying to sort a folder of photos on my android device while in the upload screen of the nextcloud app, the app crashes.

Expected behaviour

It should sort the folder without crashing

Steps to reproduce

  1. Click upload in app
  2. Find the DCIM folder on phone to get to camera photos
  3. Click sort in the app

    Environment data

Android version: 6.0.1

Device model: Nexus 6

Stock or customized system: stock

Nextcloud app version: 1.3.1

Nextcloud server version: 10.0.1

Don't have time to upload logs right now. Hopefully I can add later.

Logs

Web server error log

Insert your webserver log here

Nextcloud log (data/nextcloud.log)

Insert your Nextcloud log here
AndyScherzinger commented 8 years ago

Hi @koreywithak thank you for the report. No logs needed since I can reproduce this crash.

koreywithak commented 8 years ago

Okay great. Glad to help.

AndyScherzinger commented 7 years ago

corresponding PR #346

AndyScherzinger commented 7 years ago

Hi @koreywithak, the fix has been merged and will be shipped with the next release! :rocket:

koreywithak commented 7 years ago

Awesome. Pretty fast and organized resolution. Thanks!

On Nov 10, 2016 3:13 PM, "Andy Scherzinger" notifications@github.com wrote:

Hi @koreywithak https://github.com/koreywithak, the fix has been merged and will be shipped with the next release! 🚀

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nextcloud/android/issues/336#issuecomment-259795039, or mute the thread https://github.com/notifications/unsubscribe-auth/AKfyv8RDFE9SAqnkoFOI0trKT7z193opks5q83regaJpZM4KXnBk .

AndyScherzinger commented 7 years ago

Thanks! I consider 27 days until merge quite long but that is due to the upcoming feature release and Hacktoberfest.