android icon indicating copy to clipboard operation
android copied to clipboard

folder view broken when sharing to empty folders >= 3.21.0 (3.20.3 is fine)

Open kolAflash opened this issue 2 years ago • 1 comments

⚠️ Before posting ⚠️

  • [X] This is a bug, not a question or an enhancement.
  • [X] I've searched for similar issues and didn't find a duplicate.
  • [X] I've written a clear and descriptive title for this issue, not just "Bug" or "Crash".
  • [X] I agree to follow Nextcloud's Code of Conduct.

Steps to reproduce

  1. Open Nextcloud Android app.
  2. Create an empty folder test in your root folder.
  3. Open another app which can share files. E.g. open https://en.wikipedia.org/wiki/Jupiter in Firefox.
  4. Long press the photo of Jupiter and choose Share image.
  5. Choose Nextcloud and walk to the root folder.
  6. Click the empty folder test.

Expected behaviour

Show the content of the empty folder test.

Actual behaviour

Now the top of the screen will say test but you will still see the contents of your root folder. You don't have to start from the root folder. Just create an empty folder anywhere and you can reproduce the bug.

screenshot

Screenshot_20220911-140826_Nextcloud

Android version

Android 13

Device brand and model

Google Pixel 4a

Stock or custom OS?

Stock

Nextcloud android app version

3.21.0

Nextcloud server version

24.0.5

Using a reverse proxy?

No

Android logs

No response

Server error logs

No response

Additional information

REGRESSION

Bug happens with PIxel 4a (running stock Android-13) and with Fairphone 3 Plus (running /e/OS 1.0 - Android-10).

kolAflash avatar Sep 11 '22 12:09 kolAflash

If you first start sharing and create a new folder within the sharing screen (folder picker), you get the same issue (folder name is shown in the header, but the parent folder content is still shown).

XueSheng-GIT avatar Sep 12 '22 10:09 XueSheng-GIT

Still present in Nextcloud 3.21.2. My device is a Samsung A9 running Android 10.

uusijani avatar Oct 01 '22 15:10 uusijani

Confirmed for 3.21.2

svenb1234 avatar Oct 11 '22 06:10 svenb1234

No need to bump the issue: it's approved and still open, we haven't fixed it yet.

AlvaroBrey avatar Oct 11 '22 12:10 AlvaroBrey

I fixed it now. Will create a PR soon.

abdusalamApps avatar Nov 04 '22 16:11 abdusalamApps

PR https://github.com/nextcloud/android/pull/10989

abdusalamApps avatar Nov 07 '22 13:11 abdusalamApps