v0.3.1 Fix longPress bugs caused by collectionViewCell reuse

Author: zjfjack

Example/JZCalendarWeekViewExample/Source/CustomViews/CustomWeekView.swift

@@ -23,7 +23,7 @@ class CustomWeekView: JZBaseWeekView {
     override func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
         let date = flowLayout.dateForColumnHeader(at: indexPath)
         let cell = collectionView.dequeueReusableCell(withReuseIdentifier: EventCell.className, for: indexPath) as! EventCell
-        cell.updateView(event: allEventsBySection[date]![indexPath.row] as! Event)
+        cell.configureCell(event: allEventsBySection[date]![indexPath.row] as! Event)
         return cell
     }
 

Example/JZCalendarWeekViewExample/Source/DefaultViewController.swift

@@ -87,7 +87,6 @@ extension DefaultViewController: OptionsViewDelegate {
             optionsButton.heightAnchor.constraint(equalToConstant: 25).isActive = true
         }
         optionsButton.addTarget(self, action: #selector(presentOptionsVC), for: .touchUpInside)
-        self.navigationController?.navigationBar.tintColor = JZWeekViewColors.today
         self.navigationItem.rightBarButtonItem = UIBarButtonItem(customView: optionsButton)
     }
 

Example/JZCalendarWeekViewExample/Source/DefaultWeekView.swift

@@ -19,7 +19,7 @@ class DefaultWeekView: JZBaseWeekView {
 
     override func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
         let cell = collectionView.dequeueReusableCell(withReuseIdentifier: EventCell.className, for: indexPath) as! EventCell
-        cell.updateView(event: getCurrentEvent(with: indexPath) as! Event)
+        cell.configureCell(event: getCurrentEvent(with: indexPath) as! Event)
         return cell
     }
 

Example/JZCalendarWeekViewExample/Source/Event.swift

@@ -12,21 +12,19 @@ import JZCalendarWeekView
 class Event: JZBaseEvent {
 
     var location: String
-    /// Not used for now
-    var eventType: Int
     var title: String
-    var id: String
 
-    init(id: String, title: String, startDate: Date, endDate: Date, location: String, eventType: Int) {
-        self.id = id
+    
+    init(id: String, title: String, startDate: Date, endDate: Date, location: String) {
         self.location = location
-        self.eventType = eventType
         self.title = title
-        super.init(startDate: startDate, endDate: endDate)
+        
+        // If you want to have you custom uid, you can set the parent class's id with your uid or UUID().uuidString (In this case, we just use the base class id)
+        super.init(id: id, startDate: startDate, endDate: endDate)
     }
 
     override func copy(with zone: NSZone?) -> Any {
-        return Event(id: id, title: title, startDate: startDate, endDate: endDate, location: location, eventType: eventType)
+        return Event(id: id, title: title, startDate: startDate, endDate: endDate, location: location)
     }
 
 

Example/JZCalendarWeekViewExample/Source/EventCell.swift

@@ -34,7 +34,7 @@ class EventCell: UICollectionViewCell {
         borderView.backgroundColor = UIColor(hex: 0x0899FF)
     }
 
-    func updateView(event: Event) {
+    func configureCell(event: Event) {
         self.event = event
         locationLabel.text = event.location
         titleLabel.text = event.title

Example/JZCalendarWeekViewExample/Source/LongPressViews/LongPressEventCell.swift

@@ -9,7 +9,8 @@
 import UIKit
 import JZCalendarWeekView
 
-class LongPressEventCell: JZBaseEventCell {
+// If you want to use Move Type LongPressWeekView, you have to inherit from JZLongPressEventCell and update event when you configure cell every time
+class LongPressEventCell: JZLongPressEventCell {
 
     @IBOutlet weak var titleLabel: UILabel!
     @IBOutlet weak var locationLabel: UILabel!
@@ -19,6 +20,9 @@ class LongPressEventCell: JZBaseEventCell {
         super.awakeFromNib()
 
         setupBasic()
+        // You have to set the background color in contentView instead of cell background color, because cell reuse problems in collectionview
+        // When setting alpha to cell, the alpha will back to 1 when collectionview scrolled, which means that moving cell will not be translucent
+        self.contentView.backgroundColor = UIColor(hex: 0xEEF7FF)
     }
 
     func setupBasic() {
@@ -29,11 +33,10 @@ class LongPressEventCell: JZBaseEventCell {
         layer.shadowOpacity = 0
         locationLabel.font = UIFont.systemFont(ofSize: 12)
         titleLabel.font = UIFont.systemFont(ofSize: 14, weight: .medium)
-        self.backgroundColor = UIColor(hex: 0xEEF7FF)
         borderView.backgroundColor = UIColor(hex: 0x0899FF)
     }
-    
-    func updateView(event: Event) {
+
+    func configureCell(event: Event) {
         self.event = event
         locationLabel.text = event.location
         titleLabel.text = event.title

Example/JZCalendarWeekViewExample/Source/LongPressViews/LongPressViewController.swift

@@ -78,7 +78,7 @@ extension LongPressViewController: JZLongPressViewDelegate, JZLongPressViewDataS
 
     func weekView(_ weekView: JZLongPressWeekView, didEndAddNewLongPressAt startDate: Date) {
         let newEvent = Event(id: UUID().uuidString, title: "New Event", startDate: startDate, endDate: startDate.add(component: .hour, value: weekView.addNewDurationMins/60),
-                             location: "Melbourne", eventType: 5)
+                             location: "Melbourne")
 
         if viewModel.eventsByDate[startDate.startOfDay] == nil {
             viewModel.eventsByDate[startDate.startOfDay] = [Event]()

Example/JZCalendarWeekViewExample/Source/LongPressViews/LongPressWeekView.swift

@@ -19,7 +19,7 @@ class LongPressWeekView: JZLongPressWeekView {
 
     override func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
         let cell = collectionView.dequeueReusableCell(withReuseIdentifier: LongPressEventCell.className, for: indexPath) as! LongPressEventCell
-        cell.updateView(event: getCurrentEvent(with: indexPath) as! Event)
+        cell.configureCell(event: getCurrentEvent(with: indexPath) as! Event)
         return cell
     }
 

Example/JZCalendarWeekViewExample/Source/RootViewModel.swift

@@ -15,10 +15,10 @@ class RootViewModel: NSObject {
     private let secondDate = Date().add(component: .day, value: 1)
     private let thirdDate = Date().add(component: .day, value: 2)
 
-    lazy var events = [Event(id: "0", title: "One", startDate: firstDate, endDate: firstDate.add(component: .hour, value: 1), location: "Melbourne", eventType: 0),
-                       Event(id: "1", title: "Two", startDate: secondDate, endDate: secondDate.add(component: .hour, value: 4), location: "Sydney", eventType: 0),
-                       Event(id: "2", title: "Three", startDate: thirdDate, endDate: thirdDate.add(component: .hour, value: 2), location: "Tasmania", eventType: 1),
-                       Event(id: "3", title: "Four", startDate: thirdDate, endDate: thirdDate.add(component: .day, value: 1), location: "Canberra", eventType: 1)]
+    lazy var events = [Event(id: "0", title: "One", startDate: firstDate, endDate: firstDate.add(component: .hour, value: 1), location: "Melbourne"),
+                       Event(id: "1", title: "Two", startDate: secondDate, endDate: secondDate.add(component: .hour, value: 4), location: "Sydney"),
+                       Event(id: "2", title: "Three", startDate: thirdDate, endDate: thirdDate.add(component: .hour, value: 2), location: "Tasmania"),
+                       Event(id: "3", title: "Four", startDate: thirdDate, endDate: thirdDate.add(component: .hour, value: 26), location: "Canberra")]
 
     lazy var eventsByDate: EventsByDate = JZWeekViewHelper.getIntraEventsByDate(originalEvents: events)
 

Example/JZCalendarWeekViewExample/Supporting Files/Info.plist

@@ -15,9 +15,9 @@
 	<key>CFBundlePackageType</key>
 	<string>APPL</string>
 	<key>CFBundleShortVersionString</key>
-	<string>0.3.0</string>
+	<string>0.3.1</string>
 	<key>CFBundleVersion</key>
-	<string>1805031223</string>
+	<string>1805041708</string>
 	<key>LSRequiresIPhoneOS</key>
 	<true/>
 	<key>UILaunchStoryboardName</key>