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
Andy Scherzinger
72569cf2fd
Merge pull request #1855 from nextcloud/dependabot/gradle/com.fasterxml.jackson.core-jackson-core-2.13.2
...
Bump jackson-core from 2.13.1 to 2.13.2
2022-03-07 09:49:49 +01:00
Nextcloud Android Bot
35e2c7e857
Weekly 13.1.0 Alpha 15
2022-03-07 03:13:49 +00:00
dependabot[bot]
3284cecb38
Bump jackson-core from 2.13.1 to 2.13.2
...
Bumps [jackson-core](https://github.com/FasterXML/jackson-core ) from 2.13.1 to 2.13.2.
- [Release notes](https://github.com/FasterXML/jackson-core/releases )
- [Commits](https://github.com/FasterXML/jackson-core/compare/jackson-core-2.13.1...jackson-core-2.13.2 )
---
updated-dependencies:
- dependency-name: com.fasterxml.jackson.core:jackson-core
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
2022-03-07 02:08:30 +00:00
Nextcloud bot
d327e7e738
[tx-robot] updated from transifex
...
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
2022-03-06 03:58:18 +00:00
Andy Scherzinger
8f4fb84d8e
Merge pull request #1854 from nextcloud/feat/workflow-auto-update-command-rebase.yml
...
Updating command-rebase.yml workflow from template
2022-03-05 12:49:03 +01:00
Nextcloud bot
a519087fec
Updating command-rebase.yml workflow from template
...
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
2022-03-05 08:36:48 +00:00
Nextcloud bot
5faa31600e
[tx-robot] updated from transifex
...
Signed-off-by: Nextcloud bot <bot@nextcloud.com>
2022-03-05 03:57:56 +00:00
Marcel Hibbe
a1e694104c
add groups and circles to system messages ("add"+"remove")
...
Signed-off-by: Marcel Hibbe <dev@mhibbe.de>
2022-03-04 14:58:46 +01:00
Marcel Hibbe
a75cb7fa5a
Merge pull request #1846 from nextcloud/bugfix/noid/fixCanLeaveApiCheck
...
Bugfix/noid/fix can leave api check
2022-03-04 09:54:16 +01:00