diff --git a/app/src/main/java/org/tasks/widget/WidgetClickActivity.kt b/app/src/main/java/org/tasks/widget/WidgetClickActivity.kt index 10049f0cb..590c5d31a 100644 --- a/app/src/main/java/org/tasks/widget/WidgetClickActivity.kt +++ b/app/src/main/java/org/tasks/widget/WidgetClickActivity.kt @@ -14,8 +14,10 @@ import org.tasks.analytics.Firebase import org.tasks.data.entity.Task import org.tasks.dialogs.BaseDateTimePicker.OnDismissHandler import org.tasks.dialogs.DateTimePicker.Companion.newDateTimePicker +import org.tasks.filters.Filter import org.tasks.intents.TaskIntents import org.tasks.preferences.Preferences +import timber.log.Timber import javax.inject.Inject @AndroidEntryPoint @@ -35,6 +37,8 @@ class WidgetClickActivity : AppCompatActivity(), OnDismissHandler { } when (action) { COMPLETE_TASK -> { + val task = task + Timber.tag("$action task=$task") lifecycleScope.launch(NonCancellable) { taskCompleter.setComplete(task, !task.isCompleted) firebase.completeTask("widget") @@ -42,20 +46,24 @@ class WidgetClickActivity : AppCompatActivity(), OnDismissHandler { finish() } EDIT_TASK -> { - startActivity( - TaskIntents.getEditTaskIntent( - this, - intent.getParcelableExtra(EXTRA_FILTER), - intent.getParcelableExtra(EXTRA_TASK))) + val filter = intent.getParcelableExtra(EXTRA_FILTER) + val task = task + Timber.tag("$action task=$task filter=$filter") + startActivity(TaskIntents.getEditTaskIntent(this, filter, task)) finish() } TOGGLE_SUBTASKS -> { + val task = task + val collapsed = intent.getBooleanExtra(EXTRA_COLLAPSED, false) + Timber.d("$action collapsed=$collapsed task=$task") lifecycleScope.launch(NonCancellable) { - taskDao.setCollapsed(task.id, intent.getBooleanExtra(EXTRA_COLLAPSED, false)) + taskDao.setCollapsed(task.id, collapsed) } finish() } RESCHEDULE_TASK -> { + val task = task + Timber.d("$action task=$task") val fragmentManager = supportFragmentManager if (fragmentManager.findFragmentByTag(FRAG_TAG_DATE_TIME_PICKER) == null) { newDateTimePicker( @@ -65,14 +73,14 @@ class WidgetClickActivity : AppCompatActivity(), OnDismissHandler { } } TOGGLE_GROUP -> { - val widgetPreferences = WidgetPreferences( - applicationContext, - preferences, - intent.getIntExtra(EXTRA_WIDGET, -1) - ) - val collapsed = widgetPreferences.collapsed.toMutableSet() + val widgetId = intent.getIntExtra(EXTRA_WIDGET, -1) val group = intent.getLongExtra(EXTRA_GROUP, -1) - if (intent.getBooleanExtra(EXTRA_COLLAPSED, false)) { + val setCollapsed = intent.getBooleanExtra(EXTRA_COLLAPSED, false) + Timber.d("$action widgetId=$widgetId group=$group collapsed=$setCollapsed") + val widgetPreferences = + WidgetPreferences(applicationContext, preferences, widgetId) + val collapsed = widgetPreferences.collapsed.toMutableSet() + if (setCollapsed) { collapsed.add(group) } else { collapsed.remove(group) @@ -81,6 +89,9 @@ class WidgetClickActivity : AppCompatActivity(), OnDismissHandler { localBroadcastManager.broadcastRefresh() finish() } + else -> { + Timber.e("Unknown action $action") + } } } @@ -104,4 +115,4 @@ class WidgetClickActivity : AppCompatActivity(), OnDismissHandler { const val EXTRA_WIDGET = "extra_widget" private const val FRAG_TAG_DATE_TIME_PICKER = "frag_tag_date_time_picker" } -} \ No newline at end of file +}