fixed possible OOM when setting a time of 0:00 to events, due to splitting them into multiple events

Author: AndroidDeveloperLB

library/src/main/java/com/alamkanak/weekview/WeekView.kt

@@ -1849,9 +1849,6 @@ class WeekView @JvmOverloads constructor(context: Context, attrs: AttributeSet?
         if (mEvents == null)
             mEvents = ArrayList()
 
-        if (this.weekViewLoader == null && !isInEditMode)
-            throw IllegalStateException("You must provide a MonthChangeListener")
-
         // If a refresh was requested then reset some variables.
         if (mRefreshEvents) {
             this.clearEvents()
@@ -1910,9 +1907,9 @@ class WeekView @JvmOverloads constructor(context: Context, attrs: AttributeSet?
     private fun cacheEvent(event: WeekViewEvent) {
         if (!event.isAllDay && event.startTime!! >= event.endTime)
             return
-        val splitedEvents = event.splitWeekViewEvents()
-        for (splitedEvent in splitedEvents) {
-            mEventRects!!.add(EventRect(splitedEvent, event, null))
+        val splitEvents = event.splitWeekViewEvents()
+        for (splitEvent in splitEvents) {
+            mEventRects!!.add(EventRect(splitEvent, event, null))
         }
 
         mEvents!!.add(event)

library/src/main/java/com/alamkanak/weekview/WeekViewEvent.kt

@@ -127,30 +127,31 @@ class WeekViewEvent {
             events.add(event1)
 
             // Add other days.
-            val otherDay = this.startTime!!.clone() as Calendar
-            otherDay.add(Calendar.DATE, 1)
-            while (!isSameDay(otherDay, this.endTime!!)) {
-                val overDay = otherDay.clone() as Calendar
-                overDay.set(Calendar.HOUR_OF_DAY, 0)
-                overDay.set(Calendar.MINUTE, 0)
-                val endOfOverDay = overDay.clone() as Calendar
-                endOfOverDay.set(Calendar.HOUR_OF_DAY, 23)
-                endOfOverDay.set(Calendar.MINUTE, 59)
-                val eventMore = WeekViewEvent(this.identifier!!, this.name, null, overDay, endOfOverDay, this.isAllDay)
-                eventMore.color = this.color
-                events.add(eventMore)
-
-                // Add next day.
+            if (!isSameDay(this.startTime!!, this.endTime!!)) {
+                val otherDay = this.startTime!!.clone() as Calendar
                 otherDay.add(Calendar.DATE, 1)
+                while (!isSameDay(otherDay, this.endTime!!)) {
+                    val overDay = otherDay.clone() as Calendar
+                    overDay.set(Calendar.HOUR_OF_DAY, 0)
+                    overDay.set(Calendar.MINUTE, 0)
+                    val endOfOverDay = overDay.clone() as Calendar
+                    endOfOverDay.set(Calendar.HOUR_OF_DAY, 23)
+                    endOfOverDay.set(Calendar.MINUTE, 59)
+                    val eventMore = WeekViewEvent(this.identifier!!, this.name, null, overDay, endOfOverDay, this.isAllDay)
+                    eventMore.color = this.color
+                    events.add(eventMore)
+
+                    // Add next day.
+                    otherDay.add(Calendar.DATE, 1)
+                }
+                // Add last day.
+                val startTime = this.endTime!!.clone() as Calendar
+                startTime.set(Calendar.HOUR_OF_DAY, 0)
+                startTime.set(Calendar.MINUTE, 0)
+                val event2 = WeekViewEvent(this.identifier!!, this.name, this.location, startTime, this.endTime!!, this.isAllDay)
+                event2.color = this.color
+                events.add(event2)
             }
-
-            // Add last day.
-            val startTime = this.endTime!!.clone() as Calendar
-            startTime.set(Calendar.HOUR_OF_DAY, 0)
-            startTime.set(Calendar.MINUTE, 0)
-            val event2 = WeekViewEvent(this.identifier!!, this.name, this.location, startTime, this.endTime!!, this.isAllDay)
-            event2.color = this.color
-            events.add(event2)
         } else {
             events.add(this)
         }

library/src/main/java/com/alamkanak/weekview/WeekViewUtil.kt

@@ -27,6 +27,8 @@ object WeekViewUtil {
      */
     @JvmStatic
     fun isSameDay(dateOne: Calendar, dateTwo: Calendar): Boolean {
+        if (dateOne === dateTwo)
+            return true
         return dateOne.get(Calendar.YEAR) == dateTwo.get(Calendar.YEAR) && dateOne.get(Calendar.DAY_OF_YEAR) == dateTwo.get(Calendar.DAY_OF_YEAR)
     }