mirrors/nextcloud-talk-android
1
0
Fork 0
mirror of https://github.com/nextcloud/talk-android.git synced 2024-11-24 14:05:40 +03:00
Code Issues Projects Releases Packages Wiki Activity
5978 commits 93 branches 407 tags 64 MiB
Commit graph

5978 commits

This branch
This branch
All branches
Author SHA1 Message Date
Andy Scherzinger
bb6cb228a7
use improt alias to enforce use of kotlin map function 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-30 16:58:44 +01:00
Andy Scherzinger
b5a379bc50
bump buildToolsVersion from 33.0.0 to 33.0.1 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-30 16:58:43 +01:00
Andy Scherzinger
0fa26cc137
Update to specific kotlin syntax since Java8 and Kotlin 1.8 collide now 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-30 16:58:43 +01:00
Andy Scherzinger
36744542e6
Suppress Linter since performOnCLick is called within touch event implementation 
... else it would be called twice -> false-positive

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-30 16:58:43 +01:00
Andy Scherzinger
eaf12ef4b5
Add new line at the end of kotlin file 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-30 16:58:42 +01:00
dependabot[bot]
9977a3d777
Bump kotlinVersion from 1.7.22 to 1.8.0 
Bumps `kotlinVersion` from 1.7.22 to 1.8.0.

Updates `kotlin-gradle-plugin` from 1.7.22 to 1.8.0
- [Release notes](https://github.com/JetBrains/kotlin/releases)
- [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md)
- [Commits](https://github.com/JetBrains/kotlin/compare/v1.7.22...v1.8.0)

Updates `kotlin-serialization` from 1.7.22 to 1.8.0
- [Release notes](https://github.com/JetBrains/kotlin/releases)
- [Changelog](https://github.com/JetBrains/kotlin/blob/master/ChangeLog.md)
- [Commits](https://github.com/JetBrains/kotlin/compare/v1.7.22...v1.8.0)

---
updated-dependencies:
- dependency-name: org.jetbrains.kotlin:kotlin-gradle-plugin
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: org.jetbrains.kotlin:kotlin-serialization
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
 2022-12-30 16:58:42 +01:00
Andy Scherzinger
95c3123727
Migrate to new parcelize Implementation for Kotlin 1.8, also extend chatKit MessageInput to provide access to UI elements of that component 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-30 16:58:42 +01:00
Nextcloud bot
97b1735ffd
Fix(l10n): 🔠 Update translations from Transifex 
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
 2022-12-30 10:17:31 +00:00
Nextcloud bot
9c3c48f881
Fix(l10n): 🛠️ Update Transifex configuration 
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
 2022-12-30 10:15:41 +00:00
Nextcloud bot
1e4a5ec913
Fix(l10n): 🔠 Update translations from Transifex 
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
 2022-12-29 11:23:14 +00:00
Nextcloud bot
b38213d2a1
Fix(l10n): 🛠️ Update Transifex configuration 
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
 2022-12-29 11:21:03 +00:00
Andy Scherzinger
2e26a389df
Merge pull request #2659 from nextcloud/dependabot/gradle/org.mockito-mockito-core-4.11.0 
Bump mockito-core from 4.10.0 to 4.11.0
 2022-12-29 12:10:24 +01:00
Andy Scherzinger
34cd485d4b
Merge pull request #2656 from nextcloud/chore/noid/spotbugsImprovements 
Spotbugs improvements
 2022-12-29 12:06:41 +01:00
Andy Scherzinger
2ca13f4649
Spotbugs: Defere NPE 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:37:20 +01:00
Andy Scherzinger
b36195dc9a
UI binding can never be null 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:37:20 +01:00
Andy Scherzinger
9ae722659f
Spotbugs: don't doubleCheck Map contains value, just check for null 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:37:19 +01:00
Andy Scherzinger
efdfe83507
Spotbugs: remove NPE deference 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:37:19 +01:00
Andy Scherzinger
8b9996814f
Spotbugs: NPE deference, NPE-equals, unused variable, make vars final, reformat code for line-length 120 chars 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:37:19 +01:00
Andy Scherzinger
a33f3fe400
Spotbug: Simple field is used like an enum 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:37:18 +01:00
Andy Scherzinger
f48575bfec
Spotbug: Method stores return result in local before immediately returning it 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:37:18 +01:00
Andy Scherzinger
c5067b7a60
Spotbug: split message processing to reduce complexity 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:37:00 +01:00
Andy Scherzinger
7b86b02c21
Spotbugs prevent NPR deference 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:37:00 +01:00
Andy Scherzinger
08936279b6
codacy: inefficient use of StringBuffer.toString using call StringBuffer.length instead 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:37:00 +01:00
Andy Scherzinger
b07ee434fe
housekeeping: cleanup after spotbug fixes 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:59 +01:00
Andy Scherzinger
8b61808fda
Spotbug: make constructor-called methods final 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:59 +01:00
Andy Scherzinger
09012cce6a
Spotbugs: remove superclass field masking fields inherited from BaseActivity 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:59 +01:00
Andy Scherzinger
093e6a15bc
Spotbugs: proper equals and hashCode 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:58 +01:00
Andy Scherzinger
bf8c113f9a
Spotbugs: prevent possible NPE deference 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:58 +01:00
Andy Scherzinger
946ec09315
Spotbug: don't access Array with constant index 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:58 +01:00
Andy Scherzinger
ff3dffd051
Spotbugs: literal string comparison 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:57 +01:00
Andy Scherzinger
deada5cf94
Spotbug: remove toString() output concatenation 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:57 +01:00
Andy Scherzinger
6cd0481d14
remove unneeded logging statement 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:57 +01:00
Andy Scherzinger
7eef68ef36
use proper register-method instead of reflection 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:56 +01:00
Andy Scherzinger
7a4785e083
Spotbugs: precize allocation of a collection 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:56 +01:00
Andy Scherzinger
698ebdfd1c
remove unneeded toString() call 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:56 +01:00
Andy Scherzinger
7261f75549
Spotbugs: remove needless boxing of boolean constant 
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
 2022-12-29 11:36:52 +01:00
Tim Krüger
1b70006b5b
Merge pull request #2590 from nextcloud/add-observer-for-peer-connections 
Add observer for peer connections
 2022-12-29 11:32:13 +01:00
Nextcloud bot
117f7307d1
Fix(l10n): 🔠 Update translations from Transifex 
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
 2022-12-29 02:56:17 +00:00
Nextcloud bot
f37c2b4f2a
Fix(l10n): 🛠️ Update Transifex configuration 
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
 2022-12-29 02:54:26 +00:00
dependabot[bot]
8ab39fdabe
Bump mockito-core from 4.10.0 to 4.11.0 
Bumps [mockito-core](https://github.com/mockito/mockito) from 4.10.0 to 4.11.0.
- [Release notes](https://github.com/mockito/mockito/releases)
- [Commits](https://github.com/mockito/mockito/compare/v4.10.0...v4.11.0)

---
updated-dependencies:
- dependency-name: org.mockito:mockito-core
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
 2022-12-29 02:03:46 +00:00
Daniel Calviño Sánchez
4516de4add Remove no longer needed condition 
Now that the event is posted only for proximity sensor changes the
condition is no longer needed.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
 2022-12-28 15:48:46 +01:00
Daniel Calviño Sánchez
5d7b5160b7 Rename PeerConnectionEvent to ProximitySensorEvent 
Proximity sensor events should not have been part of
PeerConnectionEvent. However, now that all the peer connection related
properties were removed the remaining event can be renamed to something
more accurate.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
 2022-12-28 15:48:45 +01:00
Daniel Calviño Sánchez
c8398695f4 Remove no longer needed code after removing EventBus message 
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
 2022-12-28 15:48:45 +01:00
Daniel Calviño Sánchez
34498efa72 Rewrite if/else chain as if/return blocks 
Just a matter of preference :-)

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
 2022-12-28 15:48:45 +01:00
Daniel Calviño Sánchez
04f1679e2a Add observer for peer connections 
The observer is just an adapter for the "PeerConnection.Observer"
provided by the WebRTC library; a custom observer is used to expose only
the events needed outside "PeerConnectionWrapper".

For now only the same events that were already handled are taken into
account, but at a later point additional events (like "onAddTrack"
instead of "onAddStream", which is deprecated) could be added too.

Note that the thread used to handle the events has changed; the EventBus
subscriber mode was "MAIN", but as the events were posted from a
PeerConnection observer, which run in a worker thread rather than in the
main thread, the subscriber was executed in the main thread rather than
in the same thread as the poster. Due to this the actions performed by
the handler now must be explicitly run in the main thread.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
 2022-12-28 15:48:45 +01:00
Daniel Calviño Sánchez
fcbfc1926d Post MediaStreamEvents for each connection state 
Rather than simplifying the states to "CONNECTED" and "DISCONNECTED" now
the raw state is posted, and the handler then decides how to treat them
(which, for now, is exactly as before).

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
 2022-12-28 15:48:45 +01:00
Daniel Calviño Sánchez
337f3d4b5e Extract methods to handle connections and disconnections of peers 
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
 2022-12-28 15:48:45 +01:00
Daniel Calviño Sánchez
4457e92504 Generalize PUBLISHER_FAILED event 
Rather than emitting PUBLISHER_FAILED when the publisher connection
fails now PEER_FAILED is emitted when any connection fails, and the
handler checks if the connection was the publisher one to apply the
specific behaviour.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
 2022-12-28 15:48:45 +01:00
Daniel Calviño Sánchez
64c4f8c7ee Remove unneeded condition 
The publisher peer connection when the HPB is used is a sender only
connection, so it never adds or removes a remote stream.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
 2022-12-28 15:48:45 +01:00
Daniel Calviño Sánchez
30aafed0e8 Post MediaStreamEvent when the stream is actually added 
The MediaStreamEvent was posted when the connection with the remote peer
was established. However, the MediaStream is added earlier (as soon as
the remote description is received), so the event is moved to better
reflect that.

Note that checking if the connection is an MCU publisher is no longer
needed, as publisher connections are sender only and therefore no remote
stream is added for publisher connections.

In any case, note that the stream will not start until the connection is
established, and a progress bar will be anyway shown on the
ParticipantDisplayItem until it is connected.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
 2022-12-28 15:48:45 +01:00