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.

Sign up for GitHub

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

Jump to bottom

Java error when delete a photo from media #12409

Closed
4 tasks done
raffaelevall opened this issue Jan 24, 2024 · 1 comment · Fixed by #12792
Closed
4 tasks done

Java error when delete a photo from media #12409

raffaelevall opened this issue Jan 24, 2024 · 1 comment · Fixed by #12792
Assignees
@alperozturk96
Labels
bug feature: media tab Media tab performance: ISE/IAE 😕 Illegal State Exception / Illegal Argument Exception Version/3027

Comments

@raffaelevall
Copy link

⚠️ Before posting ⚠️

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

Steps to reproduce

  1. Open android App
  2. open Media
  3. select a photo and delete it

Expected behaviour

Delete the image

Actual behaviour

show error, but delete the image:

Cause of error

Exception in thread "main" java.lang.IllegalStateException: Fragment PreviewMediaFragment{1221852} (dd0cef02-50a4-4bd2-96d5-156ca81d0928) not attached to a context.
    at androidx.fragment.app.Fragment.requireContext(Fragment.java:972)
    at com.owncloud.android.ui.preview.PreviewMediaFragment.lambda$onStart$1(PreviewMediaFragment.java:369)
    at com.owncloud.android.ui.preview.PreviewMediaFragment.$r8$lambda$AkxMJlf2KgePzqOIh5h57BVdLW8(Unknown Source:0)
    at com.owncloud.android.ui.preview.PreviewMediaFragment$$ExternalSyntheticLambda0.run(Unknown Source:2)
    at android.os.Handler.handleCallback(Handler.java:938)
    at android.os.Handler.dispatchMessage(Handler.java:99)
    at android.os.Looper.loopOnce(Looper.java:210)
    at android.os.Looper.loop(Looper.java:299)
    at android.app.ActivityThread.main(ActivityThread.java:8292)
    at java.lang.reflect.Method.invoke(Native Method)
    at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:556)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1045)

App information

  • ID: com.nextcloud.client
  • Version: 30270090
  • Build flavor: gplay

Device information

  • Brand: Xiaomi
  • Device: gauguin
  • Model: M2007J17G
  • Id: SKQ1.211006.001
  • Product: gauguin_eea

Firmware

  • SDK: 31
  • Release: 12
  • Incremental: V14.0.2.0.SJSEUXM

Android version

12SKQ1.21100 6.001

Device brand and model

Brand: Xiaomi' Model: M2007J17G`

Stock or custom OS?

Stock

Nextcloud android app version

30270090

Nextcloud server version

Nextcloudpi 27.1.3

Using a reverse proxy?

No

Android logs

No response

Server error logs

No response

Additional information

No response
@joshtrichards joshtrichards added feature: media tab Media tab Version/3027 performance: ISE/IAE 😕 Illegal State Exception / Illegal Argument Exception labels Jan 24, 2024
@satvik2131
Copy link

Screencast from 20-02-24 02:42:37 PM IST.webm
Did I did it right ?

@alperozturk96
Copy link
Collaborator

alperozturk96 commented Apr 1, 2024 •
edited

@satvik2131 Could you try with this PR?

@alperozturk96 alperozturk96 self-assigned this Apr 2, 2024
@JonasMayerDev JonasMayerDev linked a pull request Apr 2, 2024 that will close this issue
Use non nullable context for PreviewMediaFragment #12792
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alperozturk96 alperozturk96

Labels
bug feature: media tab Media tab performance: ISE/IAE 😕 Illegal State Exception / Illegal Argument Exception Version/3027
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use non nullable context for PreviewMediaFragment nextcloud/android
4 participants
@joshtrichards @raffaelevall @satvik2131 @alperozturk96