CappielloAntonio / tempo

An open source and lightweight music client for Subsonic, designed and built natively for Android.
GNU General Public License v3.0
867 stars 39 forks source link

fix: null checking for song without duration info #122

Closed DelightLane closed 5 months ago

DelightLane commented 6 months ago

Crash when to read playlist with sound file without id3 info

Process: com.cappielloantonio.notquitemy.tempo PID: 12645 UID: 10688 Flags: 0x28c83e46 Package: com.cappielloantonio.notquitemy.tempo v1 (1.0.0) Foreground: Yes Process-Runtime: 1469181614 Build: samsung/winnerxks/winner:12/SP2A.220305.013/F907NKOU3IWH1:user/release-keys Loading-Progress: 1.0

java.lang.NullPointerException: Attempt to invoke virtual method 'int java.lang.Integer.intValue()' on a null object reference at com.cappielloantonio.tempo.ui.adapter.PlayerSongQueueAdapter.onBindViewHolder(PlayerSongQueueAdapter.java:57) at com.cappielloantonio.tempo.ui.adapter.PlayerSongQueueAdapter.onBindViewHolder(PlayerSongQueueAdapter.java:27) at androidx.recyclerview.widget.RecyclerView$Adapter.onBindViewHolder(RecyclerView.java:7747) at androidx.recyclerview.widget.RecyclerView$Adapter.bindViewHolder(RecyclerView.java:7847) at androidx.recyclerview.widget.RecyclerView$Recycler.tryBindViewHolderByDeadline(RecyclerView.java:6646) at androidx.recyclerview.widget.RecyclerView$Recycler.tryGetViewHolderForPositionByDeadline(RecyclerView.java:6917) at androidx.recyclerview.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:6757) at androidx.recyclerview.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:6753) at androidx.recyclerview.widget.LinearLayoutManager$LayoutState.next(LinearLayoutManager.java:2362) at androidx.recyclerview.widget.LinearLayoutManager.layoutChunk(LinearLayoutManager.java:1662) at androidx.recyclerview.widget.LinearLayoutManager.fill(LinearLayoutManager.java:1622) at androidx.recyclerview.widget.LinearLayoutManager.onLayoutChildren(LinearLayoutManager.java:687) at androidx.recyclerview.widget.RecyclerView.dispatchLayoutStep2(RecyclerView.java:4645) at androidx.recyclerview.widget.RecyclerView.onMeasure(RecyclerView.java:4022) at android.view.View.measure(View.java:27238) at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7977) at android.widget.LinearLayout.measureChildBeforeLayout(LinearLayout.java:1552) at android.widget.LinearLayout.measureVertical(LinearLayout.java:842) at android.widget.LinearLayout.onMeasure(LinearLayout.java:721) at android.view.View.measure(View.java:27238) at androidx.core.widget.NestedScrollView.measureChildWithMargins(NestedScrollView.java:1792) at android.widget.FrameLayout.onMeasure(FrameLayout.java:197) at androidx.core.widget.NestedScrollView.onMeasure(NestedScrollView.java:627) at android.view.View.measure(View.java:27238) at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7977) at android.widget.FrameLayout.onMeasure(FrameLayout.java:197) at android.view.View.measure(View.java:27238) at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7977) at androidx.coordinatorlayout.widget.CoordinatorLayout.onMeasureChild(CoordinatorLayout.java:795) at androidx.coordinatorlayout.widget.CoordinatorLayout.onMeasure(CoordinatorLayout.java:868) at android.view.View.measure(View.java:27238) at android.view.ViewGroup.measureChildWithMargins(ViewGroup.java:7977) at android.widget.FrameLayout.onMeasure(FrameLayout.java:197) at android.view.View.measure(View.java:27238) at androidx.recyclerview.widget.RecyclerView$LayoutManager.measureChildWithMargins(RecyclerView.java:10210) at androidx.recyclerview.widget.LinearLayoutManager.layoutChunk(LinearLayoutManager.java:1688) at androidx.recyclerview.widget.LinearLayoutManager.fill(LinearLayoutManager.java:1622) at androidx.recyclerview.widget.LinearLayoutManager.onLayoutChildren(LinearLayoutManager.java:687) at androidx.recyclerview.widget.RecyclerView.dispatchLayoutStep2(RecyclerView.java:4645) at androidx.recyclerview.widget.RecyclerView.dispatchLayout(RecyclerView.java:4348) at androidx.recyclerview.widget.RecyclerView.onLayout(RecyclerView.java:4919) at android.view.View.layout(View.java:24562) at android.view.ViewGroup.layout(ViewGroup.java:7409) at androidx.viewpager2.widget.ViewPager2.onLayout(ViewPager2.java:535) at android.view.View.layout(View.java:24562) at android.view.ViewGroup.layout(ViewGroup.java:7409) at android.widget.FrameLayout.layoutChildren(FrameLayout.java:374) at android.widget.FrameLayout.onLayout(FrameLayout.java:312) at android.view.View.layout(View.java:24562) at android.view.ViewGroup.layout(ViewGroup.java:7409) at android.widget.FrameLayout.layoutChildren(FrameLayout.java:374) at android.widget.FrameLayout.onLayout(FrameLayout.java:312) at android.view.View.layout(View.java:24562) at android.view.ViewGroup.layout(ViewGroup.java:7409) at androidx.coordinatorlayout.widget.CoordinatorLayout.layoutChild(CoordinatorLayout.java:1248) at androidx.coordinatorlayout.widget.CoordinatorLayout.onLayoutChild(CoordinatorLayout.java:934) at com.google.android.material.bottomsheet.BottomSheetBehavior.onLayoutChild(BottomSheetBehavior.java:571) at androidx.coordinatorlayout.widget.CoordinatorLayout.onLayout(CoordinatorLayout.java:953) at android.view.View.layout(View.java:24562) at android.view.ViewGroup.layout(ViewGroup.java:7409) at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1829) at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1673) at android.widget.LinearLayout.onLayout(LinearLayout.java:1582) at android.view.View.layout(View.java:24562) at android.view.ViewGroup.layout(ViewGroup.java:7409) at android.widget.FrameLayout.layoutChildren(FrameLayout.java:374) at android.widget.FrameLayout.onLayout(FrameLayout.java:312) at android.view.View.layout(View.java:24562) at android.view.ViewGroup.layout(ViewGroup.java:7409) at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1829) at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1673) at android.widget.LinearLayout.onLayout(LinearLayout.java:1582) at android.view.View.layout(View.java:24562) at android.view.ViewGroup.layout(ViewGroup.java:7409) at android.widget.FrameLayout.layoutChildren(FrameLayout.java:374) at android.widget.FrameLayout.onLayout(FrameLayout.java:312) at android.view.View.layout(View.java:24562) at android.view.ViewGroup.layout(ViewGroup.java:7409) at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1829) at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1673) at android.widget.LinearLayout.onLayout(LinearLayout.java:1582) at android.view.View.layout(View.java:24562) at android.view.ViewGroup.layout(ViewGroup.java:7409) at android.widget.FrameLayout.layoutChildren(FrameLayout.java:374) at android.widget.FrameLayout.onLayout(FrameLayout.java:312) at com.android.internal.policy.DecorView.onLayout(DecorView.java:1396) at android.view.View.layout(View.java:24562) at android.view.ViewGroup.layout(ViewGroup.java:7409) at android.view.ViewRootImpl.performLayout(ViewRootImpl.java:4697) at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:4078) at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:2948) at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:10675) at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1111) at android.view.Choreographer.doCallbacks(Choreographer.java:866) at android.view.Choreographer.doFrame(Choreographer.java:797) at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1095) at android.os.Handler.handleCallback(Handler.java:938) at android.os.Handler.dispatchMessage(Handler.java:99) at android.os.Looper.loopOnce(Looper.java:226) at android.os.Looper.loop(Looper.java:313) at android.app.ActivityThread.main(ActivityThread.java:8855) at java.lang.reflect.Method.invoke(Native Method) at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:571) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1135)

CappielloAntonio commented 5 months ago

Thank you very much!