Commit graph

12541 commits

Author SHA1 Message Date
Andy Scherzinger
8cb9a76b93
bump dagger to 2.22.1
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2019-04-03 08:47:49 +02:00
Tobias Kaminsky
06e710d67c daily dev 20190403 2019-04-03 03:34:35 +02:00
Nextcloud bot
e88df10983
[tx-robot] updated from transifex 2019-04-03 00:53:20 +00:00
Andy Scherzinger
b9c9954ad0
Merge pull request #3829 from nextcloud/370-alpha1
bump master to 3.7.0-alpha1
2019-04-02 13:04:16 +02:00
Andy Scherzinger
15a944933b
bump master to 3.7.0-alpha1
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
2019-04-02 12:45:15 +02:00
Andy Scherzinger
cbb4234771
Merge pull request #3791 from nextcloud/ezaquarii/inject-app-preferences-into-activities
Inject AppPreferences into activities and fragments
2019-04-02 12:33:08 +02:00
Nextcloud bot
ed84655a05
[tx-robot] updated from transifex 2019-04-02 09:11:34 +00:00
Andy Scherzinger
2dfbe3dfe1
Merge pull request #3826 from nextcloud/dependabot/gradle/com.mebigfatguy.fb-contrib-fb-contrib-7.4.3
Bump fb-contrib from 7.4.3.sb to 7.4.3
2019-04-02 08:23:10 +02:00
Tobias Kaminsky
677bd759df daily dev 20190402 2019-04-02 03:34:18 +02:00
dependabot[bot]
9e63184aca
Bump fb-contrib from 7.4.3.sb to 7.4.3
Bumps [fb-contrib](https://github.com/mebigfatguy/fb-contrib) from 7.4.3.sb to 7.4.3.
- [Release notes](https://github.com/mebigfatguy/fb-contrib/releases)
- [Commits](https://github.com/mebigfatguy/fb-contrib/compare/v7.4.3.sb...v7.4.3)

Signed-off-by: dependabot[bot] <support@dependabot.com>
2019-04-02 01:11:33 +00:00
nextcloud-android-bot
d9c33be55a Drone: update FindBugs results to reflect reduced error/warning count [skip ci]
Signed-off-by: nextcloud-android-bot <android@nextcloud.com>
2019-04-01 15:03:59 +00:00
ezaquarii
518f5c8d52
Merge 8761e50b0f into 0ba9cc6f6b 2019-04-01 14:52:05 +00:00
Chris Narkiewicz
8761e50b0f
Inject AppPreferences into activities and fragments
Signed-off-by: Chris Narkiewicz <hello@ezaquarii.com>
2019-04-01 15:51:47 +01:00
Andy Scherzinger
0ba9cc6f6b
Merge pull request #3824 from nextcloud/spotbugs
Use spotbugs for analytics
2019-04-01 13:55:41 +02:00
tobiasKaminsky
20802780c7
use spotbugs for analytics
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
2019-04-01 10:30:49 +02:00
Andy Scherzinger
78323554e6
Merge pull request #3821 from nextcloud/npeCheckOnNote
NPE check on note
2019-04-01 09:47:22 +02:00
Tobias Kaminsky
7cba1afd12
Merge pull request #3813 from nextcloud/ezaquarii/integrate-spotbugs
Add SpotBugs tasks
2019-04-01 09:45:58 +02:00
Andy Scherzinger
b4e1e57859
Merge pull request #3820 from nextcloud/fixNotificationResultCheck
Check result also for 202 (accepted)
2019-04-01 09:43:59 +02:00
Tobias Kaminsky
caf89285eb
Merge pull request #3818 from nextcloud/filedatastoragemanager-string-comparison
datamodel: use equals for string comparison.
2019-04-01 09:14:22 +02:00
tobiasKaminsky
b0f9c5ea68
NPE check on note, it should never be null due to our fetch, which fails back to "" (empty string), but there are few crash reports on google…
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
2019-04-01 08:31:26 +02:00
tobiasKaminsky
b2a2fa494c
check result also for 202 (accepted)
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
2019-04-01 08:21:24 +02:00
Nextcloud bot
52ab9b95b3
[tx-robot] updated from transifex 2019-04-01 00:53:38 +00:00
ardevd
3968be4a58
fixed inverted comparison
Signed-off-by: ardevd <edvard.holst@gmail.com>
2019-03-31 22:24:33 +02:00
nextcloud-android-bot
f26eecacec Drone: update FindBugs results to reflect reduced error/warning count [skip ci]
Signed-off-by: nextcloud-android-bot <android@nextcloud.com>
2019-03-31 20:01:06 +00:00
ardevd
81987b9b25
Merge e797e5ced8 into 7f523dfbcd 2019-03-31 19:56:19 +00:00
ardevd
e797e5ced8
datamodel: use equals for string comparison.
It is almost always a mistake to compare two instances of java.lang.String using reference equality == or !=, because it is not comparing actual value but locations in memory.

Signed-off-by: ardevd <edvard.holst@gmail.com>
2019-03-31 21:55:47 +02:00
Nextcloud bot
7f523dfbcd
[tx-robot] updated from transifex 2019-03-31 01:52:59 +00:00
Chris Narkiewicz
d4788b0ad0
Add SpotBugs tasks
Closes #3759

Signed-off-by: Chris Narkiewicz <hello@ezaquarii.com>
2019-03-30 22:55:57 +00:00
Nextcloud bot
83cdb1f28b
[tx-robot] updated from transifex 2019-03-30 01:52:07 +00:00
Tobias Kaminsky
7608587e87 daily dev 20190329 2019-03-29 03:34:19 +01:00
Nextcloud bot
789ccd9cea
[tx-robot] updated from transifex 2019-03-29 01:52:28 +00:00
Andy Scherzinger
3d5f74bfe5
Merge pull request #2796 from nextcloud/png2svg
Unify icons, remove png in favor of xml
2019-03-28 11:33:18 +01:00
tobiasKaminsky
816a10735a
- unify icons
- add svg, convert to android xml, removed png

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

fix after rebase

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

renamed user -> ic_user (icon)
make favorite mutable

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

replace further png resources with xml drawables

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

fixes after rebase

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

allow for vector compat

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

add missing icon

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

add yet another missing icon

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

lint: When using VectorDrawableCompat, you need to use app:srcCompat

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

lint: When using VectorDrawableCompat, you need to use app:srcCompat

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

fix lint

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>

Drone: update Lint results to reflect reduced error/warning count [skip ci]

Signed-off-by: nextcloud-android-bot <android@nextcloud.com>

improve findbugs score

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

use correct icon for user account settings within drawer menu

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

icons needs to be grey600 by default, don't tint grey600 to not tint avatars

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

codacy: organize imports

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

programmatically set in-View drawables, in-view drawables can't be set via XML when supporting pre-lollipop

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

codacy: Useless parentheses & organize imports

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

organize imports

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

codacy: useless paranthesis

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

revert programmatically set
do not use vectorDrawables.useSupportLibrary

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>

use 128dp for talk

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>

Revert "lint: When using VectorDrawableCompat, you need to use app:srcCompat"

This reverts commit 80068e7d37f3a8c6b7096c51543bda68443e4341.

Revert "lint: When using VectorDrawableCompat, you need to use app:srcCompat"

This reverts commit 0130e9a96d2602ad4b5d33aae67f34a458366d70.

revert srcCompat

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
2019-03-28 10:14:01 +01:00
Tobias Kaminsky
49df3c14dc
Merge pull request #3796 from nextcloud/npeInOwnerId
Do not show avatar if ownerId is null
2019-03-28 07:35:49 +01:00
Andy Scherzinger
f91b4d22cd
Merge pull request #3795 from nextcloud/dependabot/gradle/com.h3xstream.findsecbugs-findsecbugs-plugin-1.9.0
Bump findsecbugs-plugin from 1.8.0 to 1.9.0
2019-03-28 07:22:37 +01:00
tobiasKaminsky
d14ce0218e
do not show avatar if ownerId is null
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
2019-03-28 06:57:32 +01:00
Tobias Kaminsky
e6f32d794c daily dev 20190328 2019-03-28 03:33:31 +01:00
dependabot[bot]
f95f9fcb39
Bump findsecbugs-plugin from 1.8.0 to 1.9.0
Bumps [findsecbugs-plugin](https://github.com/find-sec-bugs/find-sec-bugs) from 1.8.0 to 1.9.0.
- [Release notes](https://github.com/find-sec-bugs/find-sec-bugs/releases)
- [Changelog](https://github.com/find-sec-bugs/find-sec-bugs/blob/master/CHANGELOG.md)
- [Commits](https://github.com/find-sec-bugs/find-sec-bugs/commits)

Signed-off-by: dependabot[bot] <support@dependabot.com>
2019-03-28 02:15:17 +00:00
Tobias Kaminsky
505be99946
Merge pull request #3788 from nextcloud/filemenufilter-null-passed-to-method
files: avoid possible dereference of null pointer.
2019-03-27 10:53:32 +01:00
Tobias Kaminsky
3b88b7b464
Merge pull request #3787 from nextcloud/authtokentype_always_null
authentication: Added missing variable assignment.
2019-03-27 10:33:01 +01:00
ardevd
f1f3d74ce6
getCapability marked with @NonNull and associated npe check removed.
Signed-off-by: ardevd <edvard.holst@gmail.com>
2019-03-27 10:29:52 +01:00
Andy Scherzinger
d4c008188a
Merge pull request #3789 from nextcloud/try-with-resources-in-download-file-operation
operations: properly close FileOutputStream.
2019-03-27 10:11:23 +01:00
ardevd
c59b41d761
operations: properly close FileOutputStream.
Classes that implement the Closeable interface or its super-interface, AutoCloseable, needs to be closed after use. try-with-resources does this for you automatically.

Signed-off-by: ardevd <edvard.holst@gmail.com>
Signed-off-by: nextcloud-android-bot <android@nextcloud.com>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
2019-03-27 09:34:19 +01:00
Andy Scherzinger
23c9ea0cf4
Merge pull request #3790 from nextcloud/private-constructor-in-util-classes
Added private constructors to hide implicit public ones.
2019-03-27 08:57:35 +01:00
Tobias Kaminsky
9b7a354c2e daily dev 20190327 2019-03-27 03:33:33 +01:00
Nextcloud bot
43ccc1f39f
[tx-robot] updated from transifex 2019-03-27 01:45:03 +00:00
ardevd
37165b2ac5
Added private constructors to hide implicit public ones.
Utility classes, which are collections of static members, are not meant to be instantiated. Even abstract utility classes, which can be extended, should not have public constructors.

Java adds an implicit public constructor to every class which does not define at least one explicitly. Hence, at least one non-public constructor should be defined.

Signed-off-by: ardevd <edvard.holst@gmail.com>
2019-03-26 22:32:25 +01:00
ardevd
878de49381
Merge branch 'authtokentype_always_null' of github.com:nextcloud/android into authtokentype_always_null
Signed-off-by: ardevd <edvard.holst@gmail.com>
2019-03-26 14:35:09 +01:00
ardevd
b96414d794
files: avoid possible dereference of null pointer.
it seems the `capability` object can be null. In which case it is probably a bad idea to pass a null object into `getRichDocumentsDirectEditing()`.

Signed-off-by: ardevd <edvard.holst@gmail.com>
2019-03-26 14:34:29 +01:00
ardevd
dd04bf9e93
authentication: Added missing variable assignment.
The `mAuthTokenType` variable is declared and assigned a null value but no value is subsequently assigned, making the null check later on redundant. The `AccountAuthenticator.KEY_AUTH_TOKEN_TYPE` extra value should therefore be assigned the `mAuthTokenType` value I'm assuming.

Signed-off-by: ardevd <edvard.holst@gmail.com>
2019-03-26 14:31:59 +01:00