Skip to content

Commit

Permalink
refactor: Calendar Manager refactor
Browse files Browse the repository at this point in the history
  • Loading branch information
@PavloNetrebchuk
PavloNetrebchuk committed Jun 5, 2024
1 parent f371e98 commit a068578
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 161 deletions.
7 changes: 6 additions & 1 deletion profile/src/main/java/org/openedx/profile/presentation/calendar/CalendarViewModel.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ class CalendarViewModel(

private val _uiState = MutableStateFlow(
CalendarUIState(
isCalendarExist = calendarPreferences.calendarId != CalendarManager.CALENDAR_DOES_NOT_EXIST,
isCalendarExist = isCalendarExist(),
calendarData = null,
calendarSyncState = if (networkConnection.isOnline()) CalendarSyncState.SYNCED else CalendarSyncState.OFFLINE,
isCalendarSyncEnabled = calendarPreferences.isCalendarSyncEnabled,
Expand Down Expand Up @@ -119,4 +119,9 @@ class CalendarViewModel(
}
}
}

private fun isCalendarExist(): Boolean {
return calendarPreferences.calendarId != CalendarManager.CALENDAR_DOES_NOT_EXIST &&
calendarManager.isCalendarExist(calendarPreferences.calendarId)
}
}
177 changes: 17 additions & 160 deletions profile/src/main/java/org/openedx/profile/system/CalendarManager.kt
View file
Original file line number Diff line number Diff line change
@@ -1,10 +1,8 @@
package org.openedx.profile.system

import android.annotation.SuppressLint
import android.content.ContentUris
import android.content.ContentValues
import android.content.Context
import android.content.Intent
import android.content.pm.PackageManager
import android.database.Cursor
import android.net.Uri
Expand All @@ -17,10 +15,8 @@ import org.openedx.core.domain.model.CourseDateBlock
import org.openedx.core.system.ResourceManager
import org.openedx.core.utils.Logger
import org.openedx.core.utils.toCalendar
import java.util.Calendar
import java.util.TimeZone
import java.util.concurrent.TimeUnit
import org.openedx.core.R as CoreR

class CalendarManager(
private val context: Context,
Expand All @@ -47,11 +43,23 @@ class CalendarManager(
/**
* Check if the calendar is already existed in mobile calendar app or not
*/
fun isCalendarExists(calendarTitle: String): Boolean {
if (hasPermissions()) {
return getCalendarId(calendarTitle) != CALENDAR_DOES_NOT_EXIST
}
return false
fun isCalendarExist(calendarId: Long): Boolean {
val projection = arrayOf(CalendarContract.Calendars._ID)
val selection = "${CalendarContract.Calendars._ID} = ?"
val selectionArgs = arrayOf(calendarId.toString())

val cursor = context.contentResolver.query(
CalendarContract.Calendars.CONTENT_URI,
projection,
selection,
selectionArgs,
null
)

val exists = cursor != null && cursor.count > 0
cursor?.close()

return exists
}

/**
Expand Down Expand Up @@ -113,39 +121,6 @@ class CalendarManager(
return CALENDAR_DOES_NOT_EXIST
}

/**
* Method to check if the calendar with the course name exist in the mobile calendar app or not
*/
@SuppressLint("Range")
fun getCalendarId(calendarTitle: String): Long {
var calendarId = CALENDAR_DOES_NOT_EXIST
val projection = arrayOf(
CalendarContract.Calendars._ID,
CalendarContract.Calendars.ACCOUNT_NAME,
CalendarContract.Calendars.NAME
)
val calendarContentResolver = context.contentResolver
val cursor = calendarContentResolver.query(
CalendarContract.Calendars.CONTENT_URI, projection,
CalendarContract.Calendars.ACCOUNT_NAME + "=? and (" +
CalendarContract.Calendars.NAME + "=? or " +
CalendarContract.Calendars.CALENDAR_DISPLAY_NAME + "=?)", arrayOf(
accountName, calendarTitle,
calendarTitle
), null
)
if (cursor?.moveToFirst() == true) {
if (cursor.getString(cursor.getColumnIndex(CalendarContract.Calendars.NAME))
.equals(calendarTitle)
) {
calendarId =
cursor.getInt(cursor.getColumnIndex(CalendarContract.Calendars._ID)).toLong()
}
}
cursor?.close()
return calendarId
}

/**
* Method to add important dates of course as calendar event into calendar of mobile app
*/
Expand Down Expand Up @@ -247,82 +222,6 @@ class CalendarManager(
context.contentResolver.insert(CalendarContract.Reminders.CONTENT_URI, eventValues)
}

/**
* Method to query the events for the given calendar id
*
* @param calendarId calendarId to query the events
*
* @return [Cursor]
*
* */
private fun getCalendarEvents(calendarId: Long): Cursor? {
val calendarContentResolver = context.contentResolver
val projection = arrayOf(
CalendarContract.Events._ID,
CalendarContract.Events.DTEND,
CalendarContract.Events.DESCRIPTION
)
val selection = CalendarContract.Events.CALENDAR_ID + "=?"
return calendarContentResolver.query(
CalendarContract.Events.CONTENT_URI,
projection,
selection,
arrayOf(calendarId.toString()),
null
)
}

/**
* Method to compare the calendar events with course dates
* @return true if the events are the same as calendar dates otherwise false
*/
@SuppressLint("Range")
private fun compareEvents(
calendarId: Long,
courseDateBlocks: List<CourseDateBlock>
): Boolean {
val cursor = getCalendarEvents(calendarId) ?: return false

val datesList = ArrayList(courseDateBlocks)
val dueDateColumnIndex = cursor.getColumnIndex(CalendarContract.Events.DTEND)
val descriptionColumnIndex = cursor.getColumnIndex(CalendarContract.Events.DESCRIPTION)

while (cursor.moveToNext()) {
val dueDateInMillis = cursor.getLong(dueDateColumnIndex)

val description = cursor.getString(descriptionColumnIndex)
if (description != null) {
val matchedDate = datesList.find { unit ->
description.contains(unit.title, ignoreCase = true)
}

matchedDate?.let { unit ->
val dueDateCalendar = Calendar.getInstance().apply {
timeInMillis = dueDateInMillis
set(Calendar.SECOND, 0)
set(Calendar.MILLISECOND, 0)
}

val unitDateCalendar = unit.date.toCalendar().apply {
set(Calendar.SECOND, 0)
set(Calendar.MILLISECOND, 0)
}

if (dueDateCalendar == unitDateCalendar) {
datesList.remove(unit)
} else {
// If any single value isn't matched, return false
cursor.close()
return false
}
}
}
}

cursor.close()
return datesList.isEmpty()
}

/**
* Method to delete the course calendar from the mobile calendar app
*/
Expand Down Expand Up @@ -353,37 +252,6 @@ class CalendarManager(
).build()
}

fun openCalendarApp() {
val builder: Uri.Builder = CalendarContract.CONTENT_URI.buildUpon()
.appendPath("time")
ContentUris.appendId(builder, Calendar.getInstance().timeInMillis)
val intent = Intent(Intent.ACTION_VIEW).setData(builder.build())
intent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK)
context.startActivity(intent)
}

/**
* Helper method used to check that the calendar if outdated for the course or not
*
* @param calendarTitle Title for the course Calendar
* @param courseDateBlocks Course dates events
*
* @return Calendar Id if Calendar is outdated otherwise -1 or CALENDAR_DOES_NOT_EXIST
*
*/
fun isCalendarOutOfDate(
calendarTitle: String,
courseDateBlocks: List<CourseDateBlock>
): Long {
if (isCalendarExists(calendarTitle)) {
val calendarId = getCalendarId(calendarTitle)
if (compareEvents(calendarId, courseDateBlocks).not()) {
return calendarId
}
}
return CALENDAR_DOES_NOT_EXIST
}

/**
* Method to get the current user account as the Calendar owner
*
Expand All @@ -393,17 +261,6 @@ class CalendarManager(
return corePreferences.user?.email ?: LOCAL_USER
}

/**
* Method to create the Calendar title for the platform against the course
*
* @param courseName Name of the course for that creating the Calendar events.
*
* @return title of the Calendar against the course
*/
fun getCourseCalendarTitle(courseName: String): String {
return "${resourceManager.getString(id = CoreR.string.platform_name)} - $courseName"
}

fun getCalendarData(calendarId: Long): CalendarData? {
val projection = arrayOf(
CalendarContract.Calendars.CALENDAR_DISPLAY_NAME,
Expand Down

0 comments on commit a068578

Please sign in to comment.