Commit graph

785 commits

Author SHA1 Message Date
Stypox
8eec9b55bf Remove PlayerErrorHandler and correctly set ErrorInfo msg 2021-12-04 10:36:36 +01:00
Stypox
9ac1add14c Prevent exception from being serialized in ErrorInfo
The wrong @Decorator was put in the wrong place to mark the throwable fieldd as transient, now this is fixed and the exception is not serialized. So if a non-serializable throwable is passed, that's not an issue, since it's not going to be serialized. The need for EnsureExceptionSerializable is also gone.
2021-12-04 10:36:36 +01:00
Stypox
5583f71a8e Replace ErrorActivity with ErrorUtil 2021-12-04 10:36:36 +01:00
litetex
e9468197cb Added a workaround for not serializable exceptions 2021-11-23 20:12:14 +01:00
litetex
5f56989161 PlayerErrorHandler refactor + docs 2021-11-23 20:12:12 +01:00
litetex
95d5274b7c Added option to report player errors
* Added a new setting so that player errors are reported (under Video and Audio > Player)
* Moved the player error logic to separate class specially created for this purpose
2021-11-23 20:12:07 +01:00
Stypox
3427253824 Merge pull request #7412 from litetex/code-cleanup
Some code cleanup(s)
2021-11-23 08:59:34 +01:00
litetex
8a1018e8b2 Removed useless lines 2021-11-22 19:49:52 +01:00
litetex
dcfa4511f7 Formatted code 2021-11-21 19:56:50 +01:00
litetex
010c38f389 Removed useless doc 2021-11-21 19:46:11 +01:00
litetex
cbd0fc6ac5 Merge pull request #7406 from Redirion/usedefaultloadcontrol
Use DefaultLoadcontrol
2021-11-21 15:11:21 +01:00
XiangRongLin
b785c1ccc2 Merge pull request #7381 from litetex/prevent-automatic-replay-after-returning-from-background
Prevent automatic replay after returning from background
2021-11-17 09:38:09 +01:00
litetex
5c1315641d setRecovery: Remove checks and use Math.min/max 2021-11-15 19:56:14 +01:00
litetex
ca37d7cb66 Prevent automatic replay after returning from background
See also https://github.com/TeamNewPipe/NewPipe/pull/7195#issuecomment-962624380
2021-11-15 19:47:08 +01:00
Robin
b85196aa4f Fixed Period Transition 2021-11-14 21:19:36 +01:00
Robin
7d8e1e43be Use DefaultLoadcontrol 2021-11-14 20:12:12 +01:00
Robin
a86e819a35 Merge pull request #7363 from litetex/playback-speed-ctrls-simple-landscape-improvements
Simple playback-speed-controls improvements
2021-11-12 21:19:17 +01:00
litetex
2a9d3f5f53 Refactored `PlaybackResolver`
* fixes the deprecation of ``setTag``
* makes the code more consistent
* de-duplicates some code
2021-11-12 20:14:39 +01:00
litetex
24bf4487d0 De-duplicated PlayerDataSource-code 2021-11-12 19:40:00 +01:00
Robin
38360b5c92 Use Android11+ extractors 2021-11-12 19:17:52 +01:00
Robin
6205dce2c8 Update ExoPlayer to 2.14.2 2021-11-12 19:17:51 +01:00
litetex
2d21925dae Removed obvious title from the "Playback Speed Controls" 2021-11-07 13:51:43 +01:00
litetex
a2fb490924 Simplified some code and added some comments 2021-11-05 19:07:56 +01:00
litetex
c219bdf8e2 Merged extremely similar code together / parity between video and popup player
* Removed ``player.getPlayPauseButton().requestFocus();`` as there is no reason why it was introduced there documented
* Use the same delay to hide the controls on both players
2021-11-05 18:10:55 +01:00
litetex
9f49d535b4 De-Duplicated some code 2021-11-05 18:07:21 +01:00
litetex
7afec6feb2 Why log double? 2021-11-05 18:06:32 +01:00
litetex
5b6353d018 Added some doc 2021-11-05 18:06:10 +01:00
litetex
9080d3d6dd Removed `e.printStacktrace` and used an proper logger 2021-11-05 18:04:49 +01:00
litetex
7858970446 Added a custom title
to also save some margin/padding/etc
2021-11-04 19:47:08 +01:00
litetex
5bc68ffd1d Merge pull request #6844 from 0x416c6578/shuffle-mode-ui-fix
Fixed shuffle button opacity UI bug
2021-11-03 18:18:31 +01:00
litetex
3103ebab71 Fixed typo 2021-11-03 17:30:30 +01:00
litetex
1625c4b2a7 Always create a backup list when shuffling
The backup-list has to be created at all cost (even when current list size <= 2). Otherwise it's not possible to enter shuffle-mode (as ``isShuffled()`` always returns false)!
2021-10-23 17:35:42 +02:00
litetex
be83a6bd0e Fixing the shuffle button on the UI is enough.
No need for doing the heavier method ``onShuffleModeEnabledChanged(false);``
2021-10-23 16:46:56 +02:00
0x416c6578
128b079f4f Removed changes to the intent handler 2021-10-23 16:43:29 +02:00
0x416c6578
085df96ad8 Fixed shuffle button opacity bug
Parameterised shuffle state into initPlayback for potentially passing the shuffle state into the player in the future
2021-10-23 16:43:28 +02:00
TacoTheDank
8f7a36dc48 Update RecyclerView & Groupie 2021-10-19 17:31:59 -04:00
litetex
617cde9930 Merge pull request #7194 from KalleStruik/add-to-playlist-in-share
Add a "add to playlist" option in the share menu
2021-10-13 20:34:07 +02:00
TobiGr
3a7f39d684 Simplify code and add annotations 2021-10-10 20:33:05 +02:00
litetex
2361d32f75 Cleaned up PlaylistDialog-related code 2021-10-09 18:46:20 +02:00
Tobi
c1444b75cd Merge pull request #7176 from litetex/fix-progress-bar-end-not-synced
Fixed seekbar not completed after video end
2021-10-02 09:24:59 +02:00
litetex
e9669884f5 Referenced issue in code 2021-09-26 20:58:49 +02:00
litetex
085990c609 Fixed progress bar not completing after video end 2021-09-26 20:50:24 +02:00
litetex
9aaa75fd3d Improved streamType check and documentation 2021-09-25 10:31:42 +02:00
litetex
45a1900b27 Removed useless check for livestreams 2021-09-25 00:04:53 +02:00
litetex
2f5f052aa4 Reformatted MediaSessionManager code 2021-09-24 23:55:06 +02:00
litetex
b44e9ccd7f Fixed performance problems with mediaSessionManager
* Built on top of Redirons changes
* MediaSessionManager-Metadata get's only update when metadata changes - not every second as it only changes when the metadata changes
* Reworked mediasessionmanager
2021-09-24 23:40:12 +02:00
Robin
52a83f3f47 Two fixes for setMetadata 2021-09-24 23:28:55 +02:00
Stypox
890b5b3895 Merge branch 'master' into dev 2021-09-19 22:37:00 +02:00
z3r0r4
ac3a8214b2 Add play next to long press menu & refactor enqueue methods (#6872)
* added mvp play next button in long press menu; new intent handling, new long press dialog entry, new dialog functions, new strings

* changed line length for checkstyle pass

* cleaned comments, moved strings

* Update app/src/main/res/values/strings.xml

to make long press entry more descriptive

Co-authored-by: opusforlife2 <53176348+opusforlife2@users.noreply.github.com>

* Update app/src/main/res/values/strings.xml

Co-authored-by: Stypox <stypox@pm.me>

* replace redundant nextOnVideoPlayer methods

Co-authored-by: Stypox <stypox@pm.me>

* add enqueueNextOnPlayer and enqueueOnPlayer without selectOnAppend and RESUME_PLAYBACK/ deprecate enqueueNextOn*Player and enqueueOn*Player methods
add getPlayerIntent, getPlayerEnqueueIntent and getPlayerEnqueueNextIntent without selectOnAppend and RESUME_PLAYBACK/ deprecate those with
add section comments

* removed deprecated methods
removed redundant methods

* removed deprecated methods
removed redundant methods

* replaced APPEND_ONLY, removed SELECT_ON_APPEND / replaced remaining enqueueOn*Player methods

* now works with playlists

* renamed dialog entry

* checking for >1 items in the queue using the PlayerHolder

* making enqueue*OnPlayer safe to call when no video is playing (defaulting to audio)

* corrected strings

* improve getQueueSize in PlayerHolder

* long press to enqueue only if queue isnt empty

* add Whitespace

Co-authored-by: Stypox <stypox@pm.me>

* clarify comments / add spaces

* PlayerType as parameter of the enqueueOnPlayer method
add Helper method

* using the helper function everywhere (except for the background and popup long-press actions (also on playlists, history, ...)), so basically nowhere
/ passing checkstyle

* assimilated the enqueue*OnPlayer methods

* removed redundant comment, variable

* simplify code line

Co-authored-by: Stypox <stypox@pm.me>

* move if

* replace workaround for isPlayerOpen()

Co-authored-by: Stypox <stypox@pm.me>

* replaced workarounds (getType), corrected static access with getInstance

* remove unused imports

* changed method call to original, new method doesnt exist yet.

* Use getter method instead of property access syntax.

* improve conditional for play next entry

Co-authored-by: Stypox <stypox@pm.me>

* show play next btn in feed fragment

Co-authored-by: Stypox <stypox@pm.me>

* add play next to local playlist and statistics fragment

Co-authored-by: Stypox <stypox@pm.me>

* formating

Co-authored-by: Stypox <stypox@pm.me>

* correcting logic

Co-authored-by: Stypox <stypox@pm.me>

* remove 2 year old unused string, formating

Co-authored-by: Stypox <stypox@pm.me>

* correct enqueue (next) conditionals, default to background if no player is open. Dont generally default to background play.

* remove player open checks from button long press enqueue actions

* improve log msg

* Rename next to enqueue_next

* Refactor kotlin

Co-authored-by: opusforlife2 <53176348+opusforlife2@users.noreply.github.com>
Co-authored-by: Stypox <stypox@pm.me>
2021-09-18 11:22:49 +02:00
Tobi
01c7041d6f Merge pull request #7056 from TeamNewPipe/fix/playOnPopup
Fix handling exception in `playOnPopup` and toggle description tab
2021-09-10 18:21:41 +02:00