drone
b7b135bd53
Drone: update Lint results to reflect reduced error/warning count [skip ci]
...
Signed-off-by: drone <drone@nextcloud.com>
2022-03-16 11:13:00 +01:00
Nextcloud bot
8b06de4546
[tx-robot] updated from transifex
...
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
2022-03-16 04:04:38 +00:00
Andy Scherzinger
eed209428e
splitting up complex methods into smaller, easier to understand methods
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-15 18:21:29 +01:00
Andy Scherzinger
c9f230a10b
fixes after rebase
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-15 17:46:41 +01:00
Andy Scherzinger
ac703a6ac7
extract magic numbers
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-15 17:46:40 +01:00
Andy Scherzinger
0ac5009242
reformat kotlin
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-15 17:45:06 +01:00
Andy Scherzinger
6b7dd29b07
migrate WebViewLoginController to kotlin
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-15 17:45:05 +01:00
Andy Scherzinger
ed4d27aff0
fix author mail
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-15 17:45:05 +01:00
Andy Scherzinger
cfe4847732
migrate RingtoneSelectionController to kotlin
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-15 17:44:52 +01:00
Tim Krueger
600780ee13
Merge pull request #1865 from nextcloud/bugfix/1861/search-bar-in-chat
...
Conversation view shows search bar when opened from notification
2022-03-15 14:45:51 +01:00
Tim Krüger
2bc5ac1461
Allow one more detekt issue
...
The source are the six parameters in
'ConductorRemapping#remapChatController". But since we're planning to
get rid of Conductor we will live with that a while.
Signed-off-by: Tim Krüger <t@timkrueger.me>
2022-03-15 14:35:08 +01:00
Tim Krüger
60e4ec9086
Use 'val' for never modified variables
...
Signed-off-by: Tim Krüger <t@timkrueger.me>
2022-03-15 14:35:08 +01:00
Dariusz Olszewski
1e76b45fb4
Switch to chat w/o animation when app opened from notification
...
Signed-off-by: Dariusz Olszewski <starypatyk@users.noreply.github.com>
2022-03-15 14:35:08 +01:00
Tim Krueger
4ee930f9d5
Merge pull request #1870 from nextcloud/bugfix/1869/crash-avatar
...
Workaround for crash when loading avatar
2022-03-15 14:02:31 +01:00
Dariusz Olszewski
ddcf1c48aa
Workaround for crash when loading avatar
...
Signed-off-by: Dariusz Olszewski <starypatyk@users.noreply.github.com>
2022-03-15 13:33:30 +01:00
Nextcloud bot
cdc4e89e54
[tx-robot] updated from transifex
...
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
2022-03-15 04:23:28 +00:00
Tim Krueger
37b950d03d
Merge pull request #1864 from nextcloud/bugfix/noid/detektImprovements
...
Detekt improvements
2022-03-14 14:43:10 +01:00
Andy Scherzinger
306e522b00
bump lint
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-14 10:23:09 +01:00
Andy Scherzinger
f36b7750de
format kotlin code
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-14 10:23:08 +01:00
Andy Scherzinger
1ef8acb153
lower detekt score
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-14 10:23:08 +01:00
Andy Scherzinger
ba3e24bbc7
replace too long paramter lists with data classes
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-14 10:23:07 +01:00
Andy Scherzinger
6e02c5de41
reformat kotlin
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-14 10:23:07 +01:00
Andy Scherzinger
6206e4910d
simplify complex conditions for readability
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-14 10:23:06 +01:00
Andy Scherzinger
454dbc5db2
unify http(s) protocoll prefix check on urls
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-14 10:23:06 +01:00
Andy Scherzinger
9223727e00
adapt to new detekt score
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-14 10:23:06 +01:00
Andy Scherzinger
a10b67456c
ignore generic exception handling
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-14 10:23:05 +01:00
Andy Scherzinger
86f8f1d5cd
make constant a constant
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-14 10:23:05 +01:00
Andy Scherzinger
2102f0fbc8
document empty blocks
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-14 10:23:01 +01:00
Nextcloud Android Bot
96b9b1083c
Weekly 13.1.0 Alpha 16
2022-03-14 03:26:18 +00:00
Nextcloud bot
72cc5d0050
[tx-robot] updated from transifex
...
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
2022-03-13 03:59:40 +00:00
Andy Scherzinger
874edc7eec
Merge pull request #1859 from nextcloud/dependabot/gradle/com.github.nextcloud-ChatKit-0.3.0-1
...
Bump ChatKit from c6a6176729 to 0.3.0-1
2022-03-11 17:44:06 +01:00
Tim Krueger
08dc22c30f
Merge pull request #1841 from nextcloud/bugfix/1839/fail-to-set-empty-status-message
...
Bugfix/1839/fail to set empty status message
2022-03-11 15:33:58 +01:00
Tim Krüger
cb1dd8e5b5
Avoid empty status message
...
Enable the 'Set status message' button only if a message is set.
Currently the API don't allow us to set an empty message [1].
See:
[1] nextcloud/server#31452
#1839
Signed-off-by: Tim Krüger <t@timkrueger.me>
2022-03-10 17:08:02 +01:00
Tim Krüger
f8e8a95b34
Use '==' instead of 'equals' in Kotlin
...
In Kotlin for a check of the structural equality '==' should be used.
Referential equality is checked with '==='.
See:
- https://kotlinlang.org/docs/equality.html
Signed-off-by: Tim Krüger <t@timkrueger.me>
2022-03-10 17:05:14 +01:00
Nextcloud bot
af09675d9d
[tx-robot] updated from transifex
...
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
2022-03-10 04:06:00 +00:00
Andy Scherzinger
e66cd756fd
Merge pull request #1857 from nextcloud/dependabot/github_actions/actions/stale-5
...
Bump actions/stale from 4.1.0 to 5
2022-03-09 16:14:29 +01:00
Andy Scherzinger
b68b94365d
Merge pull request #1858 from nextcloud/dependabot/github_actions/actions/checkout-3
...
Bump actions/checkout from 2 to 3
2022-03-09 16:14:07 +01:00
Tim Krueger
70a30f146a
Merge pull request #1851 from nextcloud/bugfix/1848/showSystemMessageAddedGroup
...
add groups and circles to system messages ("add"+"remove")
2022-03-09 10:37:19 +01:00
dependabot[bot]
298ae2d029
Bump ChatKit from c6a6176729 to 0.3.0-1
...
Bumps ChatKit from c6a6176729 to 0.3.0-1.
---
updated-dependencies:
- dependency-name: com.github.nextcloud:ChatKit
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
2022-03-08 02:06:04 +00:00
dependabot[bot]
1ab91a0318
Bump actions/checkout from 2 to 3
...
Bumps [actions/checkout](https://github.com/actions/checkout ) from 2 to 3.
- [Release notes](https://github.com/actions/checkout/releases )
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md )
- [Commits](https://github.com/actions/checkout/compare/v2...v3 )
---
updated-dependencies:
- dependency-name: actions/checkout
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com>
2022-03-07 23:22:35 +00:00
dependabot[bot]
cf6e5a9156
Bump actions/stale from 4.1.0 to 5
...
Bumps [actions/stale](https://github.com/actions/stale ) from 4.1.0 to 5.
- [Release notes](https://github.com/actions/stale/releases )
- [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md )
- [Commits](https://github.com/actions/stale/compare/v4.1.0...v5 )
---
updated-dependencies:
- dependency-name: actions/stale
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com>
2022-03-07 23:22:32 +00:00
Andy Scherzinger
1003545153
Merge pull request #1856 from nextcloud/bugfix/noid/codacyBadge
...
Update codacy badge
2022-03-07 15:27:42 +01:00
Andy Scherzinger
377f362d40
update codacy batch
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-07 15:26:38 +01:00
Andy Scherzinger
34b82f8da2
Merge pull request #1849 from nextcloud/bugfix/noid/migratePushToKotlin
...
Migrate Push to Kotlin
2022-03-07 15:23:42 +01:00
drone
1903f6cd40
Drone: update FindBugs results to reflect reduced error/warning count [skip ci]
...
Signed-off-by: drone <drone@nextcloud.com>
2022-03-07 13:23:42 +00:00
Andy Scherzinger
59e85334f7
set detekt limit to latest check run
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-07 13:43:46 +01:00
Andy Scherzinger
0fe562d415
remove unused method parameters
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-07 13:40:42 +01:00
Tim Krüger
5ad6da86f1
Use Boolean constant values directly
...
From the SpotBugs report:
> NAB_NEEDLESS_BOOLEAN_CONSTANT_CONVERSION: Method needlessly boxes a boolean
> constant
>
> This method assigns a Boxed boolean constant to a primitive boolean variable,
> or assigns a primitive boolean constant to a Boxed boolean variable. Use the
> correct constant for the variable desired. Use
>
>
> boolean b = true;
> boolean b = false;
>
> or
>
>
> Boolean b = Boolean.TRUE;
> Boolean b = Boolean.FALSE;
>
> Be aware that this boxing happens automatically when you might not expect it.
> For example,
>
>
> Map statusMap = ...
>
> public Boolean someMethod() {
> statusMap.put("foo", true); //the "true" here is boxed
> return false; //the "false" here is boxed
> }
>
> has two cases of this needless autoboxing. This can be made more efficient by
> simply substituting in the constant values:
>
>
> Map statusMap = ...
>
> public Boolean someMethod() {
> statusMap.put("foo", Boolean.TRUE);
> return Boolean.FALSE;
> }
Signed-off-by: Tim Krüger <t@timkrueger.me>
2022-03-07 13:11:23 +01:00
Andy Scherzinger
8e11f21233
adapt to java->kotlin changes regarding null-ability
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-07 09:55:30 +01:00
Andy Scherzinger
15e875f974
Migrate push models to kotlin data classes
...
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2022-03-07 09:50:19 +01:00