Commit d40cc4b9 authored by Filipe de Lima Brito's avatar Filipe de Lima Brito

Updates jitsi conference finish status.

parent 38893736
...@@ -11,9 +11,9 @@ interface JitsiVideoConferenceView { ...@@ -11,9 +11,9 @@ interface JitsiVideoConferenceView {
fun startJitsiVideoConference(url: String, name: String?) fun startJitsiVideoConference(url: String, name: String?)
/** /**
* Closes the Jitsi video conference. * Finishes the Jitsi video conference.
*/ */
fun closeJitsiVideoConference() fun finishJitsiVideoConference()
/** /**
* Logs the state of the Jitsi Meet conference displayed in a JitsiMeetView. * Logs the state of the Jitsi Meet conference displayed in a JitsiMeetView.
......
...@@ -56,7 +56,7 @@ class VideoConferencePresenter @Inject constructor( ...@@ -56,7 +56,7 @@ class VideoConferencePresenter @Inject constructor(
} }
} catch (ex: Exception) { } catch (ex: Exception) {
Timber.e(ex) Timber.e(ex)
view.closeJitsiVideoConference() view.finishJitsiVideoConference()
} }
} }
} }
......
...@@ -40,13 +40,6 @@ class VideoConferenceActivity : JitsiMeetActivity(), JitsiVideoConferenceView, ...@@ -40,13 +40,6 @@ class VideoConferenceActivity : JitsiMeetActivity(), JitsiVideoConferenceView,
presenter.initVideoConference() presenter.initVideoConference()
} }
override fun onDestroy() {
super.onDestroy()
presenter.invalidateTimer()
view?.dispose()
view = null
}
override fun onConferenceWillJoin(map: MutableMap<String, Any>?) = override fun onConferenceWillJoin(map: MutableMap<String, Any>?) =
logJitsiMeetViewState("Joining video conferencing", map) logJitsiMeetViewState("Joining video conferencing", map)
...@@ -58,7 +51,7 @@ class VideoConferenceActivity : JitsiMeetActivity(), JitsiVideoConferenceView, ...@@ -58,7 +51,7 @@ class VideoConferenceActivity : JitsiMeetActivity(), JitsiVideoConferenceView,
override fun onConferenceLeft(map: MutableMap<String, Any>?) { override fun onConferenceLeft(map: MutableMap<String, Any>?) {
logJitsiMeetViewState("Left video conferencing", map) logJitsiMeetViewState("Left video conferencing", map)
closeJitsiVideoConference() finishJitsiVideoConference()
} }
override fun onLoadConfigError(map: MutableMap<String, Any>?) = override fun onLoadConfigError(map: MutableMap<String, Any>?) =
...@@ -83,7 +76,12 @@ class VideoConferenceActivity : JitsiMeetActivity(), JitsiVideoConferenceView, ...@@ -83,7 +76,12 @@ class VideoConferenceActivity : JitsiMeetActivity(), JitsiVideoConferenceView,
) )
} }
override fun closeJitsiVideoConference() = finish() override fun finishJitsiVideoConference() {
presenter.invalidateTimer()
view?.dispose()
view = null
finish()
}
override fun logJitsiMeetViewState(message: String, map: MutableMap<String, Any>?) = override fun logJitsiMeetViewState(message: String, map: MutableMap<String, Any>?) =
Timber.i("$message: $map") Timber.i("$message: $map")
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment