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.

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

Refactor Kotlin code to improve error handling #317

Merged
merged 50 commits into from
Dec 7, 2023
Merged
Changes from 1 commit
Commits
Show all changes
50 commits
Select commit Hold shift + click to select a range
373b236
refactor: improve null safety
krocard Nov 8, 2023
a2e29de
refactor: report errors instead of ignoring invalid states
krocard Nov 9, 2023
e0f0244
refactor: move helpers in base module
krocard Nov 10, 2023
5fc530a
refactor: reformat
krocard Nov 10, 2023
09db8f3
fixup! method not called toJson
krocard Nov 10, 2023
2cb51f8
refactor: use getPlayer instead of direct map access
krocard Nov 10, 2023
46a7535
refactor: introduce module accessors
krocard Nov 13, 2023
14fc208
refactor: rename addUIBlock
krocard Nov 13, 2023
b39491e
refactor: introduce throw safe block
krocard Nov 13, 2023
f2ff634
fix: remove extra promise.resolve
krocard Nov 13, 2023
f8e3611
refactor: compile time safe getPlayer
krocard Nov 13, 2023
899d4f0
refactor: add withArray
krocard Nov 13, 2023
f60caee
chore: add documentation
krocard Nov 13, 2023
7b81d83
refactor: make map&array helpers type safe
krocard Nov 13, 2023
508caaa
refactor: cast manager module
krocard Nov 13, 2023
ce3d5b8
chore: prepare release 0.14.0
Nov 14, 2023
5cd38c8
Remove unnecessary dot from CHANGELOG.md
123mpozzi Nov 14, 2023
6883489
Merge pull request #321 from bitmovin/release/v0.14.0
123mpozzi Nov 14, 2023
9c86447
fix: don't return `Unit`
krocard Nov 14, 2023
c969a41
fix: remove unnecessary promise resolve
krocard Nov 14, 2023
598d8bf
fix: factorize with* methods
krocard Nov 14, 2023
efe8065
refactor: move helper method to extension
krocard Nov 14, 2023
d3c9572
refactor: rename PromiseRejectOnExceptionBlock to RejectPromiseOnExce…
krocard Nov 14, 2023
a6528bb
refactor: migrate SourceModule to BitmovinBaseModule
krocard Nov 14, 2023
ef81c51
Merge remote-tracking branch 'origin/main' into refactor--improve-nul…
krocard Nov 14, 2023
fe244b9
refactor: migrate offline module
krocard Nov 17, 2023
9f8abdb
refactor: migrate drm module
krocard Nov 17, 2023
3fd0906
refactor: remove duplicated promise.resolve
krocard Nov 17, 2023
edd2bfa
refactor: remove duplicated promise.resolve
krocard Nov 17, 2023
2aff015
fix: allow duration not propagated
krocard Nov 22, 2023
8d1d05c
fix: don't resolve Unit
krocard Nov 22, 2023
d733d09
refactor: introduce a type safe promise
krocard Nov 29, 2023
4e6f164
fix: code formating
krocard Nov 29, 2023
cb07f50
refactor: prefer early return on failure
krocard Dec 1, 2023
8773de7
fix: reject invalid command
krocard Dec 1, 2023
c30b6fe
fix: formating
krocard Dec 1, 2023
6231fd9
refactor: improve documentation
krocard Dec 1, 2023
f32a92f
fix: reject if analytics is disable
krocard Dec 4, 2023
4373ab6
refactor: mark noop function as inline
krocard Dec 4, 2023
08ab485
fix: made all array non nullable (empty on null)
krocard Dec 4, 2023
2c0d887
fix: serializing methods
krocard Dec 4, 2023
779b1e3
fix: don't skip player creation if config is missing
krocard Dec 4, 2023
d77c955
fix: don't send scaling mode if undefined
krocard Dec 6, 2023
4994562
fix: rename&use runOnMainLooperAndLogException
krocard Dec 6, 2023
d36c14d
fix: log invalid playerId
krocard Dec 6, 2023
ff1148c
fix: error message
krocard Dec 6, 2023
d272236
refactor: avoid creating a handler for each request
krocard Dec 6, 2023
e035a36
refactor: use single line function
krocard Dec 6, 2023
653fcd2
refactor: invalid ref in kdoc
krocard Dec 6, 2023
ae6f3da
fix: inverted condition breaking PiP
krocard Dec 7, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: error message
krocard committed Dec 6, 2023
commit ff1148c842a51b853899828471020ea43ebb791b
Original file line number Diff line number Diff line change
@@ -290,7 +290,7 @@ class RNPlayerViewManager(private val context: ReactApplicationContext) : Simple
try {
block()
} catch (e: Exception) {
Log.e(MODULE_NAME, "Error while command", e)
Log.e(MODULE_NAME, "Error while executing command", e)
}
}
}