From ec02fb162e84d341ceae738e0a345fde0e6d43cd Mon Sep 17 00:00:00 2001 From: tibbi Date: Sun, 20 Aug 2017 21:58:16 +0200 Subject: [PATCH] some code style updating --- .../calendar/activities/MainActivity.kt | 69 ++++++++----------- .../calendar/helpers/CalDAVEventsHandler.kt | 4 +- 2 files changed, 31 insertions(+), 42 deletions(-) diff --git a/app/src/main/kotlin/com/simplemobiletools/calendar/activities/MainActivity.kt b/app/src/main/kotlin/com/simplemobiletools/calendar/activities/MainActivity.kt index 2e106b94c..ee57a8dc4 100644 --- a/app/src/main/kotlin/com/simplemobiletools/calendar/activities/MainActivity.kt +++ b/app/src/main/kotlin/com/simplemobiletools/calendar/activities/MainActivity.kt @@ -207,16 +207,11 @@ class MainActivity : SimpleActivity(), NavigationListener { } } - private fun shouldGoToTodayBeVisible(): Boolean { - return if (config.storedView == WEEKLY_VIEW) { - week_view_view_pager.currentItem != mDefaultWeeklyPage - } else if (config.storedView == MONTHLY_VIEW) { - main_view_pager.currentItem != mDefaultMonthlyPage - } else if (config.storedView == YEARLY_VIEW) { - main_view_pager.currentItem != mDefaultYearlyPage - } else { - false - } + private fun shouldGoToTodayBeVisible() = when { + config.storedView == WEEKLY_VIEW -> week_view_view_pager.currentItem != mDefaultWeeklyPage + config.storedView == MONTHLY_VIEW -> main_view_pager.currentItem != mDefaultMonthlyPage + config.storedView == YEARLY_VIEW -> main_view_pager.currentItem != mDefaultYearlyPage + else -> false } private fun showFilterDialog() { @@ -281,14 +276,11 @@ class MainActivity : SimpleActivity(), NavigationListener { private fun updateViewPager() { resetTitle() - if (config.storedView == YEARLY_VIEW) { - fillYearlyViewPager() - } else if (config.storedView == EVENTS_LIST_VIEW) { - fillEventsList() - } else if (config.storedView == WEEKLY_VIEW) { - fillWeeklyViewPager() - } else { - openMonthlyToday() + when { + config.storedView == YEARLY_VIEW -> fillYearlyViewPager() + config.storedView == EVENTS_LIST_VIEW -> fillEventsList() + config.storedView == WEEKLY_VIEW -> fillWeeklyViewPager() + else -> openMonthlyToday() } mWeekScrollY = 0 @@ -300,14 +292,11 @@ class MainActivity : SimpleActivity(), NavigationListener { } private fun refreshViewPager() { - if (config.storedView == YEARLY_VIEW) { - (main_view_pager.adapter as MyYearPagerAdapter).refreshEvents(main_view_pager.currentItem) - } else if (config.storedView == EVENTS_LIST_VIEW) { - fillEventsList() - } else if (config.storedView == WEEKLY_VIEW) { - (week_view_view_pager.adapter as MyWeekPagerAdapter).refreshEvents(week_view_view_pager.currentItem) - } else { - (main_view_pager.adapter as MyMonthPagerAdapter).refreshEvents(main_view_pager.currentItem) + when { + config.storedView == YEARLY_VIEW -> (main_view_pager.adapter as MyYearPagerAdapter).refreshEvents(main_view_pager.currentItem) + config.storedView == EVENTS_LIST_VIEW -> fillEventsList() + config.storedView == WEEKLY_VIEW -> (week_view_view_pager.adapter as MyWeekPagerAdapter).refreshEvents(week_view_view_pager.currentItem) + else -> (main_view_pager.adapter as MyMonthPagerAdapter).refreshEvents(main_view_pager.currentItem) } } @@ -326,21 +315,21 @@ class MainActivity : SimpleActivity(), NavigationListener { } private fun tryImportEventsFromFile(uri: Uri) { - if (uri.scheme == "file") { - importEventsDialog(uri.path) - } else if (uri.scheme == "content") { - val tempFile = getTempFile() - if (tempFile == null) { - toast(R.string.unknown_error_occurred) - return - } + when { + uri.scheme == "file" -> importEventsDialog(uri.path) + uri.scheme == "content" -> { + val tempFile = getTempFile() + if (tempFile == null) { + toast(R.string.unknown_error_occurred) + return + } - val inputStream = contentResolver.openInputStream(uri) - val out = FileOutputStream(tempFile) - inputStream.copyTo(out) - importEventsDialog(tempFile.absolutePath) - } else { - toast(R.string.invalid_file_format) + val inputStream = contentResolver.openInputStream(uri) + val out = FileOutputStream(tempFile) + inputStream.copyTo(out) + importEventsDialog(tempFile.absolutePath) + } + else -> toast(R.string.invalid_file_format) } } diff --git a/app/src/main/kotlin/com/simplemobiletools/calendar/helpers/CalDAVEventsHandler.kt b/app/src/main/kotlin/com/simplemobiletools/calendar/helpers/CalDAVEventsHandler.kt index 5e4559b1f..202f6a06f 100644 --- a/app/src/main/kotlin/com/simplemobiletools/calendar/helpers/CalDAVEventsHandler.kt +++ b/app/src/main/kotlin/com/simplemobiletools/calendar/helpers/CalDAVEventsHandler.kt @@ -64,7 +64,7 @@ class CalDAVEventsHandler(val context: Context) { return calendars } - fun fetchCalDAVCalendarEvents(calendarId: Int, eventTypeId: Int) { + private fun fetchCalDAVCalendarEvents(calendarId: Int, eventTypeId: Int) { val importIdsMap = HashMap() val fetchedEventIds = ArrayList() val existingEvents = context.dbHelper.getEventsFromCalDAVCalendar(calendarId) @@ -208,7 +208,7 @@ class CalDAVEventsHandler(val context: Context) { context.contentResolver.delete(contentUri, null, null) } - fun getCalDAVEventReminders(eventId: Long): List { + private fun getCalDAVEventReminders(eventId: Long): List { val reminders = ArrayList() val uri = CalendarContract.Reminders.CONTENT_URI val projection = arrayOf(