diff --git a/app/build.gradle b/app/build.gradle index fac7b882f..0ca8fe2a1 100644 --- a/app/build.gradle +++ b/app/build.gradle @@ -147,7 +147,7 @@ dependencies { implementation "com.android.support:support-v13:${SUPPORT_VERSION}" implementation "com.android.support:cardview-v7:${SUPPORT_VERSION}" implementation 'com.android.support.constraint:constraint-layout:1.0.2' - implementation 'com.jakewharton.timber:timber:4.5.1' + implementation 'com.jakewharton.timber:timber:4.7.0' implementation 'com.jakewharton.threetenabp:threetenabp:1.0.5' //noinspection GradleDependency implementation 'com.google.guava:guava:20.0' diff --git a/app/src/googleplay/java/org/tasks/analytics/Tracker.java b/app/src/googleplay/java/org/tasks/analytics/Tracker.java index 10c0e120d..444144d83 100644 --- a/app/src/googleplay/java/org/tasks/analytics/Tracker.java +++ b/app/src/googleplay/java/org/tasks/analytics/Tracker.java @@ -61,7 +61,7 @@ public class Tracker { } public void reportException(Thread thread, Throwable t) { - Timber.e(t, t.getMessage()); + Timber.e(t); tracker.send( new HitBuilders.ExceptionBuilder() .setDescription(exceptionParser.getDescription(thread.getName(), t)) diff --git a/app/src/googleplay/java/org/tasks/gtasks/PlayServices.java b/app/src/googleplay/java/org/tasks/gtasks/PlayServices.java index b697080c9..bbe3e9c46 100644 --- a/app/src/googleplay/java/org/tasks/gtasks/PlayServices.java +++ b/app/src/googleplay/java/org/tasks/gtasks/PlayServices.java @@ -85,10 +85,10 @@ public class PlayServices { context, credential.getSelectedAccount(), "oauth2:" + TasksScopes.TASKS, null); return true; } catch (GoogleAuthException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return false; } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return true; } } @@ -108,11 +108,11 @@ public class PlayServices { GoogleAuthUtil.getToken(activity, account, "oauth2:" + TasksScopes.TASKS, null); handler.authenticationSuccessful(accountName); } catch (UserRecoverableAuthException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); activity.startActivityForResult( e.getIntent(), GtasksLoginActivity.RC_REQUEST_OAUTH); } catch (GoogleAuthException | IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); handler.authenticationFailed(activity.getString(R.string.gtasks_GLA_errorIOAuth)); } }) diff --git a/app/src/googleplay/java/org/tasks/location/PlacePicker.java b/app/src/googleplay/java/org/tasks/location/PlacePicker.java index 25aa745e7..85bdf7d87 100644 --- a/app/src/googleplay/java/org/tasks/location/PlacePicker.java +++ b/app/src/googleplay/java/org/tasks/location/PlacePicker.java @@ -21,10 +21,10 @@ public class PlacePicker { try { return builder.build(activity); } catch (GooglePlayServicesRepairableException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); activity.startActivity(e.getIntent()); } catch (GooglePlayServicesNotAvailableException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); Toast.makeText( activity, R.string.common_google_play_services_notification_ticker, Toast.LENGTH_LONG) .show(); diff --git a/app/src/main/java/com/todoroo/andlib/utility/AndroidUtilities.java b/app/src/main/java/com/todoroo/andlib/utility/AndroidUtilities.java index 41c9176cd..65d95738e 100644 --- a/app/src/main/java/com/todoroo/andlib/utility/AndroidUtilities.java +++ b/app/src/main/java/com/todoroo/andlib/utility/AndroidUtilities.java @@ -124,10 +124,10 @@ public class AndroidUtilities { } catch (NumberFormatException e) { // failed parse to number putter.put(object, key, 's', value); - Timber.e(e, e.getMessage()); + Timber.e(e); } } catch (IndexOutOfBoundsException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } } @@ -237,7 +237,7 @@ public class AndroidUtilities { inputMethodManager.hideSoftInputFromWindow(currentFocus.getWindowToken(), 0); } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } diff --git a/app/src/main/java/com/todoroo/andlib/utility/DialogUtilities.java b/app/src/main/java/com/todoroo/andlib/utility/DialogUtilities.java index 3f6549c33..28f334be4 100644 --- a/app/src/main/java/com/todoroo/andlib/utility/DialogUtilities.java +++ b/app/src/main/java/com/todoroo/andlib/utility/DialogUtilities.java @@ -22,7 +22,7 @@ public class DialogUtilities { try { dialog.dismiss(); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } }); } diff --git a/app/src/main/java/com/todoroo/astrid/backup/TasksXmlImporter.java b/app/src/main/java/com/todoroo/astrid/backup/TasksXmlImporter.java index 743bfadb2..7c7564526 100755 --- a/app/src/main/java/com/todoroo/astrid/backup/TasksXmlImporter.java +++ b/app/src/main/java/com/todoroo/astrid/backup/TasksXmlImporter.java @@ -97,7 +97,7 @@ public class TasksXmlImporter { try { performImport(); } catch (IOException | XmlPullParserException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } }) .start(); @@ -194,7 +194,7 @@ public class TasksXmlImporter { } } catch (Exception e) { errorCount++; - Timber.e(e, e.getMessage()); + Timber.e(e); } } } @@ -304,7 +304,7 @@ public class TasksXmlImporter { } } catch (Exception e) { errorCount++; - Timber.e(e, e.getMessage()); + Timber.e(e); } } } diff --git a/app/src/main/java/com/todoroo/astrid/calls/PhoneStateChangedReceiver.java b/app/src/main/java/com/todoroo/astrid/calls/PhoneStateChangedReceiver.java index 7ac1c6ec9..4d6d5ed41 100644 --- a/app/src/main/java/com/todoroo/astrid/calls/PhoneStateChangedReceiver.java +++ b/app/src/main/java/com/todoroo/astrid/calls/PhoneStateChangedReceiver.java @@ -84,12 +84,12 @@ public class PhoneStateChangedReceiver extends InjectingBroadcastReceiver { try { calls = getMissedCalls(); } catch (Exception e) { // Sometimes database is locked, retry once - Timber.e(e, e.getMessage()); + Timber.e(e); AndroidUtilities.sleepDeep(300L); try { calls = getMissedCalls(); } catch (Exception e2) { - Timber.e(e2, e2.getMessage()); + Timber.e(e2); calls = null; } } @@ -125,7 +125,7 @@ public class PhoneStateChangedReceiver extends InjectingBroadcastReceiver { triggerMissedCallNotification(name, number, contactId); } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } finally { if (calls != null) { calls.close(); @@ -257,7 +257,7 @@ public class PhoneStateChangedReceiver extends InjectingBroadcastReceiver { try { b = BitmapFactory.decodeStream(input); } catch (OutOfMemoryError e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } return b; diff --git a/app/src/main/java/com/todoroo/astrid/dao/Database.java b/app/src/main/java/com/todoroo/astrid/dao/Database.java index 75518d138..c77187dac 100644 --- a/app/src/main/java/com/todoroo/astrid/dao/Database.java +++ b/app/src/main/java/com/todoroo/astrid/dao/Database.java @@ -127,7 +127,7 @@ public abstract class Database extends RoomDatabase { database = getOpenHelper().getWritableDatabase(); } catch (Exception e) { tracker.reportEvent(Tracking.Events.DB_OPEN_FAILED, e.getMessage()); - Timber.e(e, e.getMessage()); + Timber.e(e); throw new IllegalStateException(e); } } @@ -147,7 +147,7 @@ public abstract class Database extends RoomDatabase { try { database.close(); } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } database = null; diff --git a/app/src/main/java/com/todoroo/astrid/data/Task.java b/app/src/main/java/com/todoroo/astrid/data/Task.java index 39d7c70aa..4ec9cd221 100644 --- a/app/src/main/java/com/todoroo/astrid/data/Task.java +++ b/app/src/main/java/com/todoroo/astrid/data/Task.java @@ -458,7 +458,7 @@ public class Task implements Parcelable { long value = Long.parseLong(uuid); return value > 0; } catch (NumberFormatException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return isUuidEmpty(uuid); } } diff --git a/app/src/main/java/com/todoroo/astrid/files/FilesControlSet.java b/app/src/main/java/com/todoroo/astrid/files/FilesControlSet.java index 386e3cac8..f60dd1b0f 100644 --- a/app/src/main/java/com/todoroo/astrid/files/FilesControlSet.java +++ b/app/src/main/java/com/todoroo/astrid/files/FilesControlSet.java @@ -68,7 +68,7 @@ public class FilesControlSet extends TaskEditControlFragment { mediaPlayer.prepare(); mediaPlayer.start(); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); handler.playbackFailed(); } } @@ -202,7 +202,7 @@ public class FilesControlSet extends TaskEditControlFragment { context, filePath, TaskAttachment.FILE_TYPE_IMAGE + "*"); getActivity().startActivity(intent); } catch (ActivityNotFoundException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); Toast.makeText(context, R.string.no_application_found, Toast.LENGTH_SHORT).show(); } } else { @@ -229,7 +229,7 @@ public class FilesControlSet extends TaskEditControlFragment { Intent intent = FileHelper.getReadableActionView(context, file, type); getActivity().startActivity(intent); } catch (ActivityNotFoundException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); Toast.makeText(context, R.string.file_type_unhandled, Toast.LENGTH_LONG).show(); } } diff --git a/app/src/main/java/com/todoroo/astrid/gcal/CalendarAlarmReceiver.java b/app/src/main/java/com/todoroo/astrid/gcal/CalendarAlarmReceiver.java index f70f128df..a1aa13e94 100644 --- a/app/src/main/java/com/todoroo/astrid/gcal/CalendarAlarmReceiver.java +++ b/app/src/main/java/com/todoroo/astrid/gcal/CalendarAlarmReceiver.java @@ -59,7 +59,7 @@ public class CalendarAlarmReceiver extends InjectingBroadcastReceiver { } } catch (IllegalArgumentException e) { // Some cursor read failed, or badly formed uri - Timber.e(e, e.getMessage()); + Timber.e(e); } } diff --git a/app/src/main/java/com/todoroo/astrid/gcal/GCalHelper.java b/app/src/main/java/com/todoroo/astrid/gcal/GCalHelper.java index 04c8935fe..90e1abbe6 100644 --- a/app/src/main/java/com/todoroo/astrid/gcal/GCalHelper.java +++ b/app/src/main/java/com/todoroo/astrid/gcal/GCalHelper.java @@ -119,7 +119,7 @@ public class GCalHelper { return eventUri; } catch (Exception e) { // won't work on emulator - Timber.e(e, e.getMessage()); + Timber.e(e); } return null; diff --git a/app/src/main/java/com/todoroo/astrid/gtasks/GtasksTaskListUpdater.java b/app/src/main/java/com/todoroo/astrid/gtasks/GtasksTaskListUpdater.java index 1ea91b25a..dfc6c38bc 100644 --- a/app/src/main/java/com/todoroo/astrid/gtasks/GtasksTaskListUpdater.java +++ b/app/src/main/java/com/todoroo/astrid/gtasks/GtasksTaskListUpdater.java @@ -118,7 +118,7 @@ public class GtasksTaskListUpdater { parents.put(taskId, parent); siblings.put(taskId, sibling); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } previousTask.set(taskId); diff --git a/app/src/main/java/com/todoroo/astrid/gtasks/OrderedMetadataListFragmentHelper.java b/app/src/main/java/com/todoroo/astrid/gtasks/OrderedMetadataListFragmentHelper.java index 0daf5d75d..a6ab440fa 100644 --- a/app/src/main/java/com/todoroo/astrid/gtasks/OrderedMetadataListFragmentHelper.java +++ b/app/src/main/java/com/todoroo/astrid/gtasks/OrderedMetadataListFragmentHelper.java @@ -143,7 +143,7 @@ class OrderedMetadataListFragmentHelper { updater.moveTo(list, targetTaskId, destinationTaskId); } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } @@ -157,7 +157,7 @@ class OrderedMetadataListFragmentHelper { try { updater.indent(list, targetTaskId, delta); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } } diff --git a/app/src/main/java/com/todoroo/astrid/gtasks/api/GtasksApiUtilities.java b/app/src/main/java/com/todoroo/astrid/gtasks/api/GtasksApiUtilities.java index cbd566c81..c6570944c 100644 --- a/app/src/main/java/com/todoroo/astrid/gtasks/api/GtasksApiUtilities.java +++ b/app/src/main/java/com/todoroo/astrid/gtasks/api/GtasksApiUtilities.java @@ -55,7 +55,7 @@ public class GtasksApiUtilities { Date returnDate = new Date(date.getTime() + date.getTimezoneOffset() * 60000); return returnDate.getTime(); } catch (NumberFormatException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return 0; } } diff --git a/app/src/main/java/com/todoroo/astrid/gtasks/api/GtasksInvoker.java b/app/src/main/java/com/todoroo/astrid/gtasks/api/GtasksInvoker.java index c70d259be..b956e23db 100644 --- a/app/src/main/java/com/todoroo/astrid/gtasks/api/GtasksInvoker.java +++ b/app/src/main/java/com/todoroo/astrid/gtasks/api/GtasksInvoker.java @@ -146,7 +146,7 @@ public class GtasksInvoker { try { return Thread.currentThread().getStackTrace()[4].getMethodName(); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } return ""; diff --git a/app/src/main/java/com/todoroo/astrid/gtasks/api/MoveRequest.java b/app/src/main/java/com/todoroo/astrid/gtasks/api/MoveRequest.java index fb1f83d58..65831c0cc 100644 --- a/app/src/main/java/com/todoroo/astrid/gtasks/api/MoveRequest.java +++ b/app/src/main/java/com/todoroo/astrid/gtasks/api/MoveRequest.java @@ -39,7 +39,7 @@ public class MoveRequest { try { return executePush(); } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); recover(); return executePush(); } diff --git a/app/src/main/java/com/todoroo/astrid/gtasks/sync/GtasksSyncService.java b/app/src/main/java/com/todoroo/astrid/gtasks/sync/GtasksSyncService.java index bfdaa1cc3..66c07ac3a 100644 --- a/app/src/main/java/com/todoroo/astrid/gtasks/sync/GtasksSyncService.java +++ b/app/src/main/java/com/todoroo/astrid/gtasks/sync/GtasksSyncService.java @@ -201,7 +201,7 @@ public class GtasksSyncService { try { op = queue.take(); } catch (InterruptedException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); continue; } try { diff --git a/app/src/main/java/com/todoroo/astrid/provider/Astrid2TaskProvider.java b/app/src/main/java/com/todoroo/astrid/provider/Astrid2TaskProvider.java index 4fed860d6..dcefb33d4 100644 --- a/app/src/main/java/com/todoroo/astrid/provider/Astrid2TaskProvider.java +++ b/app/src/main/java/com/todoroo/astrid/provider/Astrid2TaskProvider.java @@ -87,7 +87,7 @@ public class Astrid2TaskProvider extends InjectingContentProvider { try { context.getContentResolver().notifyChange(CONTENT_URI, null); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } @@ -144,7 +144,7 @@ public class Astrid2TaskProvider extends InjectingContentProvider { try { m = MessageDigest.getInstance("MD5"); } catch (NoSuchAlgorithmException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return -1; } diff --git a/app/src/main/java/com/todoroo/astrid/repeats/RepeatTaskHelper.java b/app/src/main/java/com/todoroo/astrid/repeats/RepeatTaskHelper.java index 5408e7577..6d5167cd0 100644 --- a/app/src/main/java/com/todoroo/astrid/repeats/RepeatTaskHelper.java +++ b/app/src/main/java/com/todoroo/astrid/repeats/RepeatTaskHelper.java @@ -262,7 +262,7 @@ public class RepeatTaskHelper { return; } } catch (ParseException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return; } diff --git a/app/src/main/java/com/todoroo/astrid/service/TaskCreator.java b/app/src/main/java/com/todoroo/astrid/service/TaskCreator.java index 736c04f87..0fbafa10b 100644 --- a/app/src/main/java/com/todoroo/astrid/service/TaskCreator.java +++ b/app/src/main/java/com/todoroo/astrid/service/TaskCreator.java @@ -177,7 +177,7 @@ public class TaskCreator { try { TitleParser.parse(tagService, task, tags); } catch (Throwable e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } task.setTags(tags); diff --git a/app/src/main/java/com/todoroo/astrid/subtasks/AstridOrderedListFragmentHelper.java b/app/src/main/java/com/todoroo/astrid/subtasks/AstridOrderedListFragmentHelper.java index b0fdeefd6..d0f5232eb 100644 --- a/app/src/main/java/com/todoroo/astrid/subtasks/AstridOrderedListFragmentHelper.java +++ b/app/src/main/java/com/todoroo/astrid/subtasks/AstridOrderedListFragmentHelper.java @@ -143,7 +143,7 @@ class AstridOrderedListFragmentHelper { updater.moveTo(list, fragment.getFilter(), targetTaskId, destinationTaskId); } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } @@ -157,7 +157,7 @@ class AstridOrderedListFragmentHelper { try { updater.indent(list, fragment.getFilter(), targetTaskId, delta); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } } diff --git a/app/src/main/java/com/todoroo/astrid/subtasks/SubtasksFilterUpdater.java b/app/src/main/java/com/todoroo/astrid/subtasks/SubtasksFilterUpdater.java index 92a626713..f17092579 100644 --- a/app/src/main/java/com/todoroo/astrid/subtasks/SubtasksFilterUpdater.java +++ b/app/src/main/java/com/todoroo/astrid/subtasks/SubtasksFilterUpdater.java @@ -52,7 +52,7 @@ public class SubtasksFilterUpdater { JSONArray tree = new JSONArray(serializedTree); recursivelyBuildChildren(root, tree, callback); } catch (JSONException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } return root; } diff --git a/app/src/main/java/com/todoroo/astrid/subtasks/SubtasksHelper.java b/app/src/main/java/com/todoroo/astrid/subtasks/SubtasksHelper.java index ddc2319a1..25d75a33a 100644 --- a/app/src/main/java/com/todoroo/astrid/subtasks/SubtasksHelper.java +++ b/app/src/main/java/com/todoroo/astrid/subtasks/SubtasksHelper.java @@ -56,7 +56,7 @@ public class SubtasksHelper { ids.add(Long.parseLong(idString)); } } catch (NumberFormatException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } return ids; @@ -110,7 +110,7 @@ public class SubtasksHelper { try { localId = Long.parseLong(uuid); } catch (NumberFormatException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } return localId; }); diff --git a/app/src/main/java/com/todoroo/astrid/ui/MultilineListPreference.java b/app/src/main/java/com/todoroo/astrid/ui/MultilineListPreference.java index c2c220723..0f7015f62 100644 --- a/app/src/main/java/com/todoroo/astrid/ui/MultilineListPreference.java +++ b/app/src/main/java/com/todoroo/astrid/ui/MultilineListPreference.java @@ -25,7 +25,7 @@ public class MultilineListPreference extends ListPreference { super.onBindView(view); } catch (Exception e) { // happens on 4.0 emulators - Timber.e(e, e.getMessage()); + Timber.e(e); } MultilineHelper.makeMultiline(view); } diff --git a/app/src/main/java/com/todoroo/astrid/ui/NumberPicker.java b/app/src/main/java/com/todoroo/astrid/ui/NumberPicker.java index f63374fd6..54f6e5d68 100644 --- a/app/src/main/java/com/todoroo/astrid/ui/NumberPicker.java +++ b/app/src/main/java/com/todoroo/astrid/ui/NumberPicker.java @@ -305,7 +305,7 @@ public class NumberPicker extends LinearLayout try { return Integer.parseInt(str); } catch (NumberFormatException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return mStart; } } diff --git a/app/src/main/java/com/todoroo/astrid/utility/TitleParser.java b/app/src/main/java/com/todoroo/astrid/utility/TitleParser.java index 46d48e6f8..eb575ed47 100644 --- a/app/src/main/java/com/todoroo/astrid/utility/TitleParser.java +++ b/app/src/main/java/com/todoroo/astrid/utility/TitleParser.java @@ -459,7 +459,7 @@ public class TitleParser { interval = Integer.parseInt(intervalStr); } catch (NumberFormatException e) { // Ah well - Timber.e(e, e.getMessage()); + Timber.e(e); } } } diff --git a/app/src/main/java/com/todoroo/astrid/voice/VoiceOutputAssistant.java b/app/src/main/java/com/todoroo/astrid/voice/VoiceOutputAssistant.java index 65729b7b3..7bdb7756f 100644 --- a/app/src/main/java/com/todoroo/astrid/voice/VoiceOutputAssistant.java +++ b/app/src/main/java/com/todoroo/astrid/voice/VoiceOutputAssistant.java @@ -100,7 +100,7 @@ public class VoiceOutputAssistant implements OnInitListener { mTts = null; isTTSInitialized = false; } catch (VerifyError e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } } diff --git a/app/src/main/java/net/dinglisch/android/tasker/TaskerPlugin.java b/app/src/main/java/net/dinglisch/android/tasker/TaskerPlugin.java index 829649322..bd9586d5c 100644 --- a/app/src/main/java/net/dinglisch/android/tasker/TaskerPlugin.java +++ b/app/src/main/java/net/dinglisch/android/tasker/TaskerPlugin.java @@ -38,6 +38,7 @@ package net.dinglisch.android.tasker; // v1.8 20161002 // added hostSupportsKeyEncoding(), setKeyEncoding() and Host.getKeysWithEncoding() +import android.annotation.SuppressLint; import android.app.Activity; import android.content.ComponentName; import android.content.pm.PackageInfo; @@ -45,6 +46,7 @@ import android.content.pm.PackageManager; import android.os.Bundle; import android.util.Log; +@SuppressLint("LogNotTimber") public class TaskerPlugin { /** @see Setting#hostSupportsOnFireVariableReplacement(Bundle) */ diff --git a/app/src/main/java/org/tasks/activities/AddAttachmentActivity.java b/app/src/main/java/org/tasks/activities/AddAttachmentActivity.java index cf7f58d8d..e70e38415 100644 --- a/app/src/main/java/org/tasks/activities/AddAttachmentActivity.java +++ b/app/src/main/java/org/tasks/activities/AddAttachmentActivity.java @@ -170,7 +170,7 @@ public class AddAttachmentActivity extends InjectingAppCompatActivity } return file; } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } return null; } @@ -196,7 +196,7 @@ public class AddAttachmentActivity extends InjectingAppCompatActivity try { AndroidUtilities.copyFile(src, dst); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); Toast.makeText(this, R.string.file_err_copy, Toast.LENGTH_LONG).show(); return null; } diff --git a/app/src/main/java/org/tasks/activities/CameraActivity.java b/app/src/main/java/org/tasks/activities/CameraActivity.java index 9c169d0ac..f839578e1 100644 --- a/app/src/main/java/org/tasks/activities/CameraActivity.java +++ b/app/src/main/java/org/tasks/activities/CameraActivity.java @@ -107,7 +107,7 @@ public class CameraActivity extends InjectingAppCompatActivity { } return file; } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } return null; } diff --git a/app/src/main/java/org/tasks/backup/TasksJsonExporter.java b/app/src/main/java/org/tasks/backup/TasksJsonExporter.java index c69839938..23fa151df 100755 --- a/app/src/main/java/org/tasks/backup/TasksJsonExporter.java +++ b/app/src/main/java/org/tasks/backup/TasksJsonExporter.java @@ -135,7 +135,7 @@ public class TasksJsonExporter { onFinishExport(output); } } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } finally { post( () -> { diff --git a/app/src/main/java/org/tasks/backup/TasksJsonImporter.java b/app/src/main/java/org/tasks/backup/TasksJsonImporter.java index 49007702e..c92cb3029 100644 --- a/app/src/main/java/org/tasks/backup/TasksJsonImporter.java +++ b/app/src/main/java/org/tasks/backup/TasksJsonImporter.java @@ -110,7 +110,7 @@ public class TasksJsonImporter { try { performImport(); } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } }) .start(); diff --git a/app/src/main/java/org/tasks/billing/Inventory.java b/app/src/main/java/org/tasks/billing/Inventory.java index 89d06edb0..945311225 100644 --- a/app/src/main/java/org/tasks/billing/Inventory.java +++ b/app/src/main/java/org/tasks/billing/Inventory.java @@ -87,7 +87,7 @@ public class Inventory { return Security.verifyPurchase( billingKey, purchase.getOriginalJson(), purchase.getSignature()); } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return false; } } diff --git a/app/src/main/java/org/tasks/caldav/CaldavSettingsActivity.java b/app/src/main/java/org/tasks/caldav/CaldavSettingsActivity.java index a0f84c8d5..fd13cb52b 100644 --- a/app/src/main/java/org/tasks/caldav/CaldavSettingsActivity.java +++ b/app/src/main/java/org/tasks/caldav/CaldavSettingsActivity.java @@ -240,7 +240,7 @@ public class CaldavSettingsActivity extends ThemedInjectingAppCompatActivity try { host = IDN.toASCII(host); } catch (Exception e) { - Timber.e(e.getMessage(), e); + Timber.e(e); } String path = baseURL.getEncodedPath(); int port = baseURL.getPort(); diff --git a/app/src/main/java/org/tasks/caldav/CaldavSynchronizer.java b/app/src/main/java/org/tasks/caldav/CaldavSynchronizer.java index 6ad605a46..aac3e8af8 100644 --- a/app/src/main/java/org/tasks/caldav/CaldavSynchronizer.java +++ b/app/src/main/java/org/tasks/caldav/CaldavSynchronizer.java @@ -215,9 +215,9 @@ public class CaldavSynchronizer { Timber.d("UPDATE %s", caldavAccount); caldavDao.update(caldavAccount); } catch (IOException | HttpException | DavException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } localBroadcastManager.broadcastRefresh(); @@ -230,7 +230,7 @@ public class CaldavSynchronizer { try { pushTask(task, caldavAccount, httpClient, httpUrl); } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } } @@ -246,11 +246,11 @@ public class CaldavSynchronizer { } } catch (HttpException e) { if (e.getStatus() != 404) { - Timber.e(e, e.getMessage()); + Timber.e(e); return false; } } catch (IOException e) { - Timber.e(e.getMessage(), e); + Timber.e(e); return false; } caldavDao.delete(caldavTask); @@ -308,7 +308,7 @@ public class CaldavSynchronizer { caldavTask.setVtodo(new String(data)); } } catch (HttpException e) { - Timber.e(e.getMessage(), e); + Timber.e(e); return; } @@ -327,7 +327,7 @@ public class CaldavSynchronizer { try { tasks = at.bitfire.ical4android.Task.fromReader(new StringReader(vtodo)); } catch (InvalidCalendarException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return; } diff --git a/app/src/main/java/org/tasks/caldav/TaskConverter.java b/app/src/main/java/org/tasks/caldav/TaskConverter.java index f556f15ff..6dee687d6 100644 --- a/app/src/main/java/org/tasks/caldav/TaskConverter.java +++ b/app/src/main/java/org/tasks/caldav/TaskConverter.java @@ -53,7 +53,7 @@ class TaskConverter { Task.createDueDate( URGENCY_SPECIFIC_DAY, DUE_DATE_FORMAT.parse(due.getValue()).getTime())); } catch (ParseException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } } @@ -93,7 +93,7 @@ class TaskConverter { at.bitfire.ical4android.Task.fromReader(new StringReader(caldavTask.getVtodo())).get(0); } } catch (IOException | InvalidCalendarException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } if (remote == null) { remote = new at.bitfire.ical4android.Task(); @@ -107,7 +107,7 @@ class TaskConverter { try { remote.setDue(new Due(newDateTime(task.getDueDate()).toString("yyyyMMdd"))); } catch (ParseException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } remote.setDue( @@ -124,7 +124,7 @@ class TaskConverter { String rrule = task.getRecurrenceWithoutFrom().replace("RRULE:", ""); remote.setRRule(new RRule(rrule)); } catch (ParseException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } remote.setLastModified(newDateTime(task.getModificationDate()).toUTC().getMillis()); diff --git a/app/src/main/java/org/tasks/calendars/CalendarEventAttendeeProvider.java b/app/src/main/java/org/tasks/calendars/CalendarEventAttendeeProvider.java index 7574213d5..4dbe25012 100644 --- a/app/src/main/java/org/tasks/calendars/CalendarEventAttendeeProvider.java +++ b/app/src/main/java/org/tasks/calendars/CalendarEventAttendeeProvider.java @@ -54,7 +54,7 @@ public class CalendarEventAttendeeProvider { } } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } finally { if (cursor != null) { cursor.close(); diff --git a/app/src/main/java/org/tasks/calendars/CalendarEventProvider.java b/app/src/main/java/org/tasks/calendars/CalendarEventProvider.java index a022037c2..6e547d042 100644 --- a/app/src/main/java/org/tasks/calendars/CalendarEventProvider.java +++ b/app/src/main/java/org/tasks/calendars/CalendarEventProvider.java @@ -118,7 +118,7 @@ public class CalendarEventProvider { } } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } finally { if (cursor != null) { cursor.close(); diff --git a/app/src/main/java/org/tasks/calendars/CalendarProvider.java b/app/src/main/java/org/tasks/calendars/CalendarProvider.java index 20c8fec43..7c91119dc 100644 --- a/app/src/main/java/org/tasks/calendars/CalendarProvider.java +++ b/app/src/main/java/org/tasks/calendars/CalendarProvider.java @@ -74,7 +74,7 @@ public class CalendarProvider { } } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } finally { if (cursor != null) { cursor.close(); diff --git a/app/src/main/java/org/tasks/dashclock/DashClockExtension.java b/app/src/main/java/org/tasks/dashclock/DashClockExtension.java index d622a3210..e7f6a06ca 100644 --- a/app/src/main/java/org/tasks/dashclock/DashClockExtension.java +++ b/app/src/main/java/org/tasks/dashclock/DashClockExtension.java @@ -97,7 +97,7 @@ public class DashClockExtension extends com.google.android.apps.dashclock.api.Da try { publishUpdate(data); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } } diff --git a/app/src/main/java/org/tasks/data/UserActivity.java b/app/src/main/java/org/tasks/data/UserActivity.java index f26bf0427..551254a38 100644 --- a/app/src/main/java/org/tasks/data/UserActivity.java +++ b/app/src/main/java/org/tasks/data/UserActivity.java @@ -87,7 +87,7 @@ public class UserActivity implements Parcelable { } return null; } catch (JSONException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return null; } } diff --git a/app/src/main/java/org/tasks/db/Migrations.java b/app/src/main/java/org/tasks/db/Migrations.java index 414bc65ac..38b0a7944 100644 --- a/app/src/main/java/org/tasks/db/Migrations.java +++ b/app/src/main/java/org/tasks/db/Migrations.java @@ -31,7 +31,7 @@ public class Migrations { try { database.execSQL("ALTER TABLE `store` ADD COLUMN `value4` TEXT DEFAULT -1"); } catch (SQLiteException e) { - Timber.w(e, e.getMessage()); + Timber.w(e); } } }; diff --git a/app/src/main/java/org/tasks/fragments/CommentBarFragment.java b/app/src/main/java/org/tasks/fragments/CommentBarFragment.java index 4feb9001e..ebe6915b0 100644 --- a/app/src/main/java/org/tasks/fragments/CommentBarFragment.java +++ b/app/src/main/java/org/tasks/fragments/CommentBarFragment.java @@ -77,7 +77,7 @@ public class CommentBarFragment extends TaskEditControlFragment { json.put("uri", uri.toString()); return json; } catch (JSONException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } return null; } diff --git a/app/src/main/java/org/tasks/gtasks/CreateListDialog.java b/app/src/main/java/org/tasks/gtasks/CreateListDialog.java index 9fcac2177..f86f9e7dd 100644 --- a/app/src/main/java/org/tasks/gtasks/CreateListDialog.java +++ b/app/src/main/java/org/tasks/gtasks/CreateListDialog.java @@ -69,7 +69,7 @@ public class CreateListDialog extends InjectingDialogFragment { try { return gtasksInvoker.createGtaskList(name); } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return null; } } diff --git a/app/src/main/java/org/tasks/gtasks/DeleteListDialog.java b/app/src/main/java/org/tasks/gtasks/DeleteListDialog.java index de79cb1fc..1ca8846c4 100644 --- a/app/src/main/java/org/tasks/gtasks/DeleteListDialog.java +++ b/app/src/main/java/org/tasks/gtasks/DeleteListDialog.java @@ -69,7 +69,7 @@ public class DeleteListDialog extends InjectingDialogFragment { gtasksInvoker.deleteGtaskList(id); return true; } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return false; } } diff --git a/app/src/main/java/org/tasks/gtasks/GoogleTaskSynchronizer.java b/app/src/main/java/org/tasks/gtasks/GoogleTaskSynchronizer.java index db613e277..d76478a97 100644 --- a/app/src/main/java/org/tasks/gtasks/GoogleTaskSynchronizer.java +++ b/app/src/main/java/org/tasks/gtasks/GoogleTaskSynchronizer.java @@ -129,10 +129,10 @@ public class GoogleTaskSynchronizer { synchronize(); gtasksPreferenceService.recordSuccessfulSync(); } catch (UserRecoverableAuthIOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); sendNotification(context, e.getIntent()); } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } catch (Exception e) { tracker.reportException(e); } finally { @@ -197,7 +197,7 @@ public class GoogleTaskSynchronizer { } catch (UserRecoverableAuthIOException e) { throw e; } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } } @@ -268,7 +268,7 @@ public class GoogleTaskSynchronizer { try { invoker.updateGtask(listId, remoteModel); } catch (HttpNotFoundException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); googleTaskDao.delete(gtasksMetadata); return; } @@ -353,7 +353,7 @@ public class GoogleTaskSynchronizer { } catch (UserRecoverableAuthIOException e) { throw e; } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } diff --git a/app/src/main/java/org/tasks/gtasks/GoogleTasksUnsuccessfulResponseHandler.java b/app/src/main/java/org/tasks/gtasks/GoogleTasksUnsuccessfulResponseHandler.java index c878bfae8..c29ab77d6 100644 --- a/app/src/main/java/org/tasks/gtasks/GoogleTasksUnsuccessfulResponseHandler.java +++ b/app/src/main/java/org/tasks/gtasks/GoogleTasksUnsuccessfulResponseHandler.java @@ -32,7 +32,7 @@ public class GoogleTasksUnsuccessfulResponseHandler implements HttpUnsuccessfulR public boolean handleResponse(HttpRequest request, HttpResponse response, boolean supportsRetry) throws IOException { HttpResponseException httpResponseException = new HttpResponseException(response); - Timber.e(httpResponseException, httpResponseException.getMessage()); + Timber.e(httpResponseException); if (!supportsRetry) { return false; } diff --git a/app/src/main/java/org/tasks/gtasks/RenameListDialog.java b/app/src/main/java/org/tasks/gtasks/RenameListDialog.java index 13c0b663a..7bc94bf5f 100644 --- a/app/src/main/java/org/tasks/gtasks/RenameListDialog.java +++ b/app/src/main/java/org/tasks/gtasks/RenameListDialog.java @@ -73,7 +73,7 @@ public class RenameListDialog extends InjectingDialogFragment { try { return gtasksInvoker.renameGtaskList(id, name); } catch (IOException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return null; } } diff --git a/app/src/main/java/org/tasks/injection/Dagger.java b/app/src/main/java/org/tasks/injection/Dagger.java index 4ab82f6d7..d89ea8b4c 100644 --- a/app/src/main/java/org/tasks/injection/Dagger.java +++ b/app/src/main/java/org/tasks/injection/Dagger.java @@ -16,7 +16,7 @@ class Dagger { try { localeContext = Locale.getInstance(localeContext).createConfigurationContext(localeContext); } catch (Exception e) { - Timber.e(e.getMessage(), e); + Timber.e(e); } applicationComponent = diff --git a/app/src/main/java/org/tasks/injection/InjectingJobIntentService.java b/app/src/main/java/org/tasks/injection/InjectingJobIntentService.java index 43ebf5a37..5b0f7375c 100644 --- a/app/src/main/java/org/tasks/injection/InjectingJobIntentService.java +++ b/app/src/main/java/org/tasks/injection/InjectingJobIntentService.java @@ -16,7 +16,7 @@ public abstract class InjectingJobIntentService extends JobIntentService { try { doWork(intent); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } diff --git a/app/src/main/java/org/tasks/injection/InjectingPreferenceActivity.java b/app/src/main/java/org/tasks/injection/InjectingPreferenceActivity.java index 318baca01..229007f87 100644 --- a/app/src/main/java/org/tasks/injection/InjectingPreferenceActivity.java +++ b/app/src/main/java/org/tasks/injection/InjectingPreferenceActivity.java @@ -67,7 +67,7 @@ public abstract class InjectingPreferenceActivity extends AppCompatPreferenceAct ActivityInfo activityInfo = getPackageManager().getActivityInfo(componentName, 0); toolbar.setTitle(activityInfo.labelRes); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); toolbar.setTitle(getTitle()); } toolbar.setNavigationIcon(ContextCompat.getDrawable(this, R.drawable.ic_arrow_back_24dp)); diff --git a/app/src/main/java/org/tasks/jobs/AfterSaveIntentService.java b/app/src/main/java/org/tasks/jobs/AfterSaveIntentService.java index 1749abc5c..afae8a720 100644 --- a/app/src/main/java/org/tasks/jobs/AfterSaveIntentService.java +++ b/app/src/main/java/org/tasks/jobs/AfterSaveIntentService.java @@ -109,7 +109,7 @@ public class AfterSaveIntentService extends InjectingJobIntentService { context.getString(R.string.gcal_completed_title, task.getTitle())); cr.update(Uri.parse(calendarUri), values, null, null); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } } diff --git a/app/src/main/java/org/tasks/jobs/BackupJob.java b/app/src/main/java/org/tasks/jobs/BackupJob.java index d804d2abd..421ef4c7c 100644 --- a/app/src/main/java/org/tasks/jobs/BackupJob.java +++ b/app/src/main/java/org/tasks/jobs/BackupJob.java @@ -56,14 +56,14 @@ public class BackupJob extends Job { try { deleteOldBackups(); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } try { tasksJsonExporter.exportTasks( context, TasksJsonExporter.ExportType.EXPORT_TYPE_SERVICE, null); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } diff --git a/app/src/main/java/org/tasks/jobs/JobCreator.java b/app/src/main/java/org/tasks/jobs/JobCreator.java index 966051bd8..590f3c1e0 100644 --- a/app/src/main/java/org/tasks/jobs/JobCreator.java +++ b/app/src/main/java/org/tasks/jobs/JobCreator.java @@ -72,7 +72,7 @@ public class JobCreator implements com.evernote.android.job.JobCreator { case TAG_REFRESH: return new RefreshJob(refreshScheduler, localBroadcastManager); default: - Timber.e("Unhandled tag: " + tag); + Timber.e("Unhandled tag: %s", tag); return null; } } diff --git a/app/src/main/java/org/tasks/locale/receiver/TaskerTaskCreator.java b/app/src/main/java/org/tasks/locale/receiver/TaskerTaskCreator.java index 8bdba737b..2223cbe86 100644 --- a/app/src/main/java/org/tasks/locale/receiver/TaskerTaskCreator.java +++ b/app/src/main/java/org/tasks/locale/receiver/TaskerTaskCreator.java @@ -39,7 +39,7 @@ public class TaskerTaskCreator { new DateTime(dueDate.getYear(), dueDate.getMonthValue(), dueDate.getDayOfMonth()); task.setDueDate(Task.createDueDate(Task.URGENCY_SPECIFIC_DAY, dt.getMillis())); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } @@ -55,7 +55,7 @@ public class TaskerTaskCreator { .withMinuteOfHour(dueTime.getMinute()) .getMillis())); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } @@ -65,7 +65,7 @@ public class TaskerTaskCreator { int priority = Integer.parseInt(priorityString); task.setImportance(Math.max(0, Math.min(3, priority))); } catch (NumberFormatException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } diff --git a/app/src/main/java/org/tasks/preferences/DefaultFilterProvider.java b/app/src/main/java/org/tasks/preferences/DefaultFilterProvider.java index 9762339b2..330c61061 100644 --- a/app/src/main/java/org/tasks/preferences/DefaultFilterProvider.java +++ b/app/src/main/java/org/tasks/preferences/DefaultFilterProvider.java @@ -120,7 +120,7 @@ public class DefaultFilterProvider { return filter; } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } return def; diff --git a/app/src/main/java/org/tasks/preferences/Device.java b/app/src/main/java/org/tasks/preferences/Device.java index 416b0c1ff..cfd6402d1 100644 --- a/app/src/main/java/org/tasks/preferences/Device.java +++ b/app/src/main/java/org/tasks/preferences/Device.java @@ -84,7 +84,7 @@ public class Device { "----------", "")); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } return ""; } diff --git a/app/src/main/java/org/tasks/preferences/MiscellaneousPreferences.java b/app/src/main/java/org/tasks/preferences/MiscellaneousPreferences.java index c86628f25..3eb7cb7da 100644 --- a/app/src/main/java/org/tasks/preferences/MiscellaneousPreferences.java +++ b/app/src/main/java/org/tasks/preferences/MiscellaneousPreferences.java @@ -67,7 +67,7 @@ public class MiscellaneousPreferences extends InjectingPreferenceActivity { } } catch (VerifyError e) { // unavailable - Timber.e(e, e.getMessage()); + Timber.e(e); } super.onActivityResult(requestCode, resultCode, data); } @@ -135,7 +135,7 @@ public class MiscellaneousPreferences extends InjectingPreferenceActivity { voiceOutputAssistant.shutdown(); } } catch (VerifyError e) { - Timber.e(e, e.getMessage()); + Timber.e(e); preference.setEnabled(false); preferences.setBoolean(preference.getKey(), false); } diff --git a/app/src/main/java/org/tasks/preferences/Preferences.java b/app/src/main/java/org/tasks/preferences/Preferences.java index c86210825..2a49d4432 100644 --- a/app/src/main/java/org/tasks/preferences/Preferences.java +++ b/app/src/main/java/org/tasks/preferences/Preferences.java @@ -136,7 +136,7 @@ public class Preferences { p -> new GsonBuilder().create().fromJson(p, com.android.billingclient.api.Purchase.class)); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return emptySet(); } } @@ -149,7 +149,7 @@ public class Preferences { newHashSet(transform(purchases, p -> new GsonBuilder().create().toJson(p)))); editor.apply(); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } @@ -254,7 +254,7 @@ public class Preferences { try { return Integer.parseInt(value); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return defaultValue; } } @@ -279,7 +279,7 @@ public class Preferences { try { return prefs.getBoolean(key, defValue); } catch (ClassCastException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); return defValue; } } diff --git a/app/src/main/java/org/tasks/receivers/Badger.java b/app/src/main/java/org/tasks/receivers/Badger.java index f9cb186a5..75b2896dd 100644 --- a/app/src/main/java/org/tasks/receivers/Badger.java +++ b/app/src/main/java/org/tasks/receivers/Badger.java @@ -52,7 +52,7 @@ public class Badger { } enabled = newValue; } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } diff --git a/app/src/main/java/org/tasks/receivers/RepeatConfirmationReceiver.java b/app/src/main/java/org/tasks/receivers/RepeatConfirmationReceiver.java index 975a9dcd9..4a9290b59 100644 --- a/app/src/main/java/org/tasks/receivers/RepeatConfirmationReceiver.java +++ b/app/src/main/java/org/tasks/receivers/RepeatConfirmationReceiver.java @@ -78,7 +78,7 @@ public class RepeatConfirmationReceiver extends BroadcastReceiver { } task.setRecurrence(rrule, task.repeatAfterCompletion()); } catch (ParseException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } taskDao.save(task); }) diff --git a/app/src/main/java/org/tasks/repeats/CustomRecurrenceDialog.java b/app/src/main/java/org/tasks/repeats/CustomRecurrenceDialog.java index ca6e29c10..43a002129 100644 --- a/app/src/main/java/org/tasks/repeats/CustomRecurrenceDialog.java +++ b/app/src/main/java/org/tasks/repeats/CustomRecurrenceDialog.java @@ -186,7 +186,7 @@ public class CustomRecurrenceDialog extends InjectingDialogFragment { rrule = new RRule(rule); } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } if (rrule == null) { rrule = new RRule(); diff --git a/app/src/main/java/org/tasks/sync/SyncExecutor.java b/app/src/main/java/org/tasks/sync/SyncExecutor.java index 76ad3b1ab..95532e96f 100644 --- a/app/src/main/java/org/tasks/sync/SyncExecutor.java +++ b/app/src/main/java/org/tasks/sync/SyncExecutor.java @@ -28,7 +28,7 @@ public class SyncExecutor { try { executor.execute(wrapWithExceptionHandling(callback, command)); } catch (RejectedExecutionException e) { - Timber.e(e, e.getMessage()); + Timber.e(e); tracker.reportException(e); callback.finished(); } @@ -40,7 +40,7 @@ public class SyncExecutor { try { command.run(); } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); tracker.reportException(e); executor.shutdownNow(); callback.finished(); diff --git a/app/src/main/java/org/tasks/tasklist/ViewHolder.java b/app/src/main/java/org/tasks/tasklist/ViewHolder.java index 753aab66c..459233569 100644 --- a/app/src/main/java/org/tasks/tasklist/ViewHolder.java +++ b/app/src/main/java/org/tasks/tasklist/ViewHolder.java @@ -332,7 +332,7 @@ class ViewHolder extends RecyclerView.ViewHolder { action.intent.send(); } catch (PendingIntent.CanceledException e) { // Oh well - Timber.e(e, e.getMessage()); + Timber.e(e); } } } diff --git a/app/src/main/java/org/tasks/ui/CalendarControlSet.java b/app/src/main/java/org/tasks/ui/CalendarControlSet.java index 6ee06529f..9597a8484 100644 --- a/app/src/main/java/org/tasks/ui/CalendarControlSet.java +++ b/app/src/main/java/org/tasks/ui/CalendarControlSet.java @@ -89,7 +89,7 @@ public class CalendarControlSet extends TaskEditControlFragment { calendarId = null; } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); tracker.reportException(e); calendarId = null; } @@ -186,7 +186,7 @@ public class CalendarControlSet extends TaskEditControlFragment { startActivity(intent); } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); } } } @@ -261,7 +261,7 @@ public class CalendarControlSet extends TaskEditControlFragment { startActivity(intent); } } catch (Exception e) { - Timber.e(e, e.getMessage()); + Timber.e(e); Toast.makeText(getActivity(), R.string.gcal_TEA_error, Toast.LENGTH_LONG).show(); } finally { cursor.close(); diff --git a/app/src/main/java/org/tasks/ui/NavigationDrawerFragment.java b/app/src/main/java/org/tasks/ui/NavigationDrawerFragment.java index db186f893..8167ea710 100644 --- a/app/src/main/java/org/tasks/ui/NavigationDrawerFragment.java +++ b/app/src/main/java/org/tasks/ui/NavigationDrawerFragment.java @@ -152,7 +152,7 @@ public class NavigationDrawerFragment extends InjectingFragment { localBroadcastManager.unregisterReceiver(refreshReceiver); } catch (IllegalArgumentException e) { // Might not have fully initialized - Timber.e(e, e.getMessage()); + Timber.e(e); } } diff --git a/app/src/main/java/org/tasks/ui/ProgressDialogAsyncTask.java b/app/src/main/java/org/tasks/ui/ProgressDialogAsyncTask.java index 00fc8f17c..d1a227516 100644 --- a/app/src/main/java/org/tasks/ui/ProgressDialogAsyncTask.java +++ b/app/src/main/java/org/tasks/ui/ProgressDialogAsyncTask.java @@ -31,7 +31,7 @@ public abstract class ProgressDialogAsyncTask extends AsyncTask