Replace Log_OC with timber

[abelgardep]

Implements: owncloud/android#2757
Needed for: owncloud/android#2770

[abelgardep]

@hannesa2 Thanks for the suggestions. They are already applied 👍

[hannesa2]

I would change all Log.x() to Timber.x() as well, eg

android-library/owncloudComLibrary/src/main/java/com/owncloud/android/lib/common/network/FileRequestBody.java

Line 90 in 285306e

Log.d(TAG, "File with name " + mFile.getName() + " and size " + mFile.length() +

same to

android-library/owncloudComLibrary/src/main/java/com/owncloud/android/lib/common/network/FileRequestBody.java

Line 94 in 285306e

e.printStackTrace();

[hannesa2]

More then this, I would convert lint LogNotTimber to an error.
Unfortunately printStackTrace() has no lint issue.

What do you think ?

[hannesa2]

Additionally you should build during pull request this repo with
./gradlew clean build !

Otherwise you have found this

android-library/sample_client/src/main/java/com/owncloud/android/lib/sampleclient/MainActivity.java

Line 50 in 285306e

import com.owncloud.android.lib.common.utils.Log_OC;

[hannesa2]

#297 Fix sample app

[abelgardep]

Additionally you should build during pull request this repo with
./gradlew clean build !

Otherwise you have found this

android-library/sample_client/src/main/java/com/owncloud/android/lib/sampleclient/MainActivity.java

Line 50 in 285306e

import com.owncloud.android.lib.common.utils.Log_OC;

Yes, i think we should include this check in bitrise in order not to have the same problem in the future. Don't you think @davigonz ?

I would change all Log.x() to Timber.x() as well, eg

android-library/owncloudComLibrary/src/main/java/com/owncloud/android/lib/common/network/FileRequestBody.java

Line 90 in 285306e

Log.d(TAG, "File with name " + mFile.getName() + " and size " + mFile.length() +

same to

android-library/owncloudComLibrary/src/main/java/com/owncloud/android/lib/common/network/FileRequestBody.java

Line 94 in 285306e

e.printStackTrace();

Sure, thanks for reporting this.