Simplify ctag check

app/src/main/java/org/tasks/caldav/CaldavSynchronizer.kt

@@ -55,11 +55,6 @@ class CaldavSynchronizer @Inject constructor(
         private val firebase: Firebase,
         private val provider: CaldavClientProvider,
         private val iCal: iCalendar) {
-    companion object {
-        init {
-            prodId = ProdId("+//IDN tasks.org//android-" + BuildConfig.VERSION_CODE + "//EN")
-        }
-    }
 
     suspend fun sync(account: CaldavAccount) {
         Thread.currentThread().contextClassLoader = context.classLoader
@@ -162,16 +157,8 @@ class CaldavSynchronizer @Inject constructor(
         Timber.d("sync(%s)", caldavCalendar)
         val httpUrl = resource.href
         pushLocalChanges(caldavCalendar, httpClient, httpUrl)
-        val syncToken = resource[SyncToken::class.java]
+        val remoteCtag = resource.ctag
-        val ctag = resource[GetCTag::class.java]
+        if (caldavCalendar.ctag?.equals(remoteCtag) == true) {
-        var remoteCtag: String? = null
-        if (syncToken != null) {
-            remoteCtag = syncToken.token
-        } else if (ctag != null) {
-            remoteCtag = ctag.cTag
-        }
-        val localCtag = caldavCalendar.ctag
-        if (localCtag != null && localCtag == remoteCtag) {
             Timber.d("%s up to date", caldavCalendar.name)
             return
         }
@@ -298,4 +285,13 @@ class CaldavSynchronizer @Inject constructor(
         caldavDao.update(caldavTask)
         Timber.d("SENT %s", caldavTask)
     }
+
+    companion object {
+        init {
+            prodId = ProdId("+//IDN tasks.org//android-" + BuildConfig.VERSION_CODE + "//EN")
+        }
+
+        val Response.ctag: String?
+            get() = this[SyncToken::class.java]?.token ?: this[GetCTag::class.java]?.cTag
+    }
 }