search

vhaudiquet / blade-player

Android music player that plays from local library, Spotify or Deezer
GNU General Public License v3.0
68 stars 12 forks source link

The app is crashing for deleting the newly/latest created playlist #32

Closed sagorahmed closed 4 years ago

sagorahmed commented 5 years ago

Actual behavior

The application crashes when I try to delete the newly created playlist.

Expected behavior

The users should be able to delete the playlist successfully without the crashing.

How to reproduce

  1. Open the app and go to the "Songs" menu.
  2. Choose a song and click on ⋮ (Three dots/ Vertical ellipsis) at the side of the song.
  3. Then click on "Add to playlist".
  4. After that click on the"New Playlist" to create a new playlist.
  5. Give a name and click ok. (Your playlist will successfully be created)
  6. Then delete your newly created playlist.
  7. See error.

Recording Of The Bug

bladebug

Logcat

D AndroidRuntime: Shutting down VM

04-03 14:53:04.258 30746 30746 E AndroidRuntime: FATAL EXCEPTION: main

04-03 14:53:04.258 30746 30746 E AndroidRuntime: Process: v.blade, PID: 30746

04-03 14:53:04.258 30746 30746 E AndroidRuntime: java.lang.NullPointerException: Attempt to invoke virtual method 'boolean java.lang.String.equals(java.lang.Object)' on a null object reference

04-03 14:53:04.258 30746 30746 E AndroidRuntime:    at v.blade.library.sources.Source$1.removePlaylist(Source.java:462)

04-03 14:53:04.258 30746 30746 E AndroidRuntime:    at v.blade.ui.MainActivity$4$1$2.onClick(MainActivity.java:316)

04-03 14:53:04.258 30746 30746 E AndroidRuntime:    at android.support.v7.app.AlertController$ButtonHandler.handleMessage(AlertController.java:166)

04-03 14:53:04.258 30746 30746 E AndroidRuntime:    at android.os.Handler.dispatchMessage(Handler.java:102)

04-03 14:53:04.258 30746 30746 E AndroidRuntime:    at android.os.Looper.loop(Looper.java:148)

04-03 14:53:04.258 30746 30746 E AndroidRuntime:    at android.app.ActivityThread.main(ActivityThread.java:7325)

04-03 14:53:04.258 30746 30746 E AndroidRuntime:    at java.lang.reflect.Method.invoke(Native Method)

04-03 14:53:04.258 30746 30746 E AndroidRuntime:    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230)

04-03 14:53:04.258 30746 30746 E AndroidRuntime:    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120)
vhaudiquet commented 5 years ago

Thanks for the report ! This should be easy to fix. Will be fixed in next release for sure.

sagorahmed commented 5 years ago

Hello @Valou3433, Thanks for your response. Glad you want it to be fixed and hope that you will release the latest version as soon as possible with more flexibility. Best regards, Sagor Ahmed