mirrors/SchildiChat-android
1
0
Fork 0
mirror of https://github.com/SchildiChat/SchildiChat-android.git synced 2024-11-26 03:16:02 +03:00
Code Issues Projects Releases Packages Wiki Activity

Fix crash on lollipop devices due to onVisibilityChanged is called before constructor.

Browse source
This commit is contained in:
Onuray Sahin 2021-08-26 13:50:23 +03:00
parent 4ad542f0d3
commit 41c0d4d14f
2 changed files with 17 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
changelog.d/3897.bugfix Normal file
View file

@ -0,0 +1 @@
Crash on opening a room on Android 5.0 and 5.1 - Regression with Voice message

23
vector/src/main/java/im/vector/app/features/home/room/detail/composer/VoiceMessageRecorderView.kt
View file

@ -40,11 +40,7 @@ import kotlin.math.floor
/** /**
* Encapsulates the voice message recording view and animations. * Encapsulates the voice message recording view and animations.
*/ */
class VoiceMessageRecorderView @JvmOverloads constructor( class VoiceMessageRecorderView: ConstraintLayout, VoiceMessagePlaybackTracker.Listener {
context: Context,
attrs: AttributeSet? = null,
defStyleAttr: Int = 0
) : ConstraintLayout(context, attrs, defStyleAttr), VoiceMessagePlaybackTracker.Listener {
interface Callback { interface Callback {
// Return true if the recording is started // Return true if the recording is started
@ -54,7 +50,7 @@ class VoiceMessageRecorderView @JvmOverloads constructor(
fun onVoicePlaybackButtonClicked() fun onVoicePlaybackButtonClicked()
} }
private val views: ViewVoiceMessageRecorderBinding private lateinit var views: ViewVoiceMessageRecorderBinding
var callback: Callback? = null var callback: Callback? = null
var voiceMessagePlaybackTracker: VoiceMessagePlaybackTracker? = null var voiceMessagePlaybackTracker: VoiceMessagePlaybackTracker? = null
@ -80,7 +76,17 @@ class VoiceMessageRecorderView @JvmOverloads constructor(
private val distanceToCancel = dimensionConverter.dpToPx(120).toFloat() private val distanceToCancel = dimensionConverter.dpToPx(120).toFloat()
private val rtlXMultiplier = context.resources.getInteger(R.integer.rtl_x_multiplier) private val rtlXMultiplier = context.resources.getInteger(R.integer.rtl_x_multiplier)
init { // Don't convert to primary constructor.
// We need to define views as lateinit var to be able to check if initialized for the bug fix on api 21 and 22.
@JvmOverloads constructor(
context: Context,
attrs: AttributeSet? = null,
defStyleAttr: Int = 0
) : super(context, attrs, defStyleAttr) {
initialize()
}
fun initialize() {
inflate(context, R.layout.view_voice_message_recorder, this) inflate(context, R.layout.view_voice_message_recorder, this)
views = ViewVoiceMessageRecorderBinding.bind(this) views = ViewVoiceMessageRecorderBinding.bind(this)
@ -90,6 +96,9 @@ class VoiceMessageRecorderView @JvmOverloads constructor(
override fun onVisibilityChanged(changedView: View, visibility: Int) { override fun onVisibilityChanged(changedView: View, visibility: Int) {
super.onVisibilityChanged(changedView, visibility) super.onVisibilityChanged(changedView, visibility)
// onVisibilityChanged is called before constructor on api 21 and 22.
if (!this::views.isInitialized) return
if (changedView == this && visibility == VISIBLE) { if (changedView == this && visibility == VISIBLE) {
views.voiceMessageMicButton.contentDescription = context.getString(R.string.a11y_start_voice_message) views.voiceMessageMicButton.contentDescription = context.getString(R.string.a11y_start_voice_message)
} else { } else {