Skip to content

Conversation

@backportbot-nextcloud
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented May 25, 2020

⚠️ This backport had conflicts and is incomplete ⚠️

backport of #6139

@nextcloud-android-bot
Copy link
Collaborator

Unit test failed, but no output was generated. Maybe a preliminary stage failed.

@nextcloud-android-bot
Copy link
Collaborator

Lint

Typestable-3.12PR
Warnings9494
Errors11

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings10
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings77
Security Warnings44
Dodgy code Warnings142
Total381

SpotBugs (stable-3.12)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings10
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings77
Security Warnings44
Dodgy code Warnings142
Total381

@nextcloud-android-bot
Copy link
Collaborator

Job interval was set in ms, when API requred minutes.
Use of invalid time unit resulted in period of ~164 years.

This change removes ms interval constants and sets
proper contact backup job period.

Fixes #6041

Signed-off-by: Chris Narkiewicz <hello@ezaquarii.com>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@tobiasKaminsky tobiasKaminsky force-pushed the backport/6139/stable-3.12 branch from 75c3da3 to 5ff4856 Compare May 25, 2020 11:12
@tobiasKaminsky
Copy link
Member

warning This backport had conflicts and is incomplete warning

Resolved :+1

@nextcloud-android-bot
Copy link
Collaborator

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/14150.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@nextcloud-android-bot
Copy link
Collaborator

Unit test failed, but no output was generated. Maybe a preliminary stage failed.

@nextcloud-android-bot
Copy link
Collaborator

Lint

Typestable-3.12PR
Warnings9494
Errors11

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings10
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings77
Security Warnings44
Dodgy code Warnings142
Total381

SpotBugs (stable-3.12)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings10
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings77
Security Warnings44
Dodgy code Warnings142
Total381

@nextcloud-android-bot
Copy link
Collaborator

@AndyScherzinger AndyScherzinger merged commit ad25c8d into stable-3.12 May 25, 2020
@delete-merged-branch delete-merged-branch bot deleted the backport/6139/stable-3.12 branch May 25, 2020 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants