Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Unverified Commit 4685ab6d authored by Ricki Hirner's avatar Ricki Hirner Committed by GitHub
Browse files

Revert "Update synctools" (#1600) 

Revert "Update synctools (#1579)"

This reverts commit 62a0ba35.
parent 62a0ba35

app/src/androidTest/kotlin/at/bitfire/davdroid/resource/LocalCalendarTest.kt

+5 −7
Original line number Diff line number Diff line
@@ -12,8 +12,8 @@ import android.provider.CalendarContract 
import android.provider.CalendarContract.ACCOUNT_TYPE_LOCAL

import android.provider.CalendarContract.Events

import androidx.test.platform.app.InstrumentationRegistry

import at.bitfire.ical4android.AndroidEvent

import at.bitfire.ical4android.Event

import at.bitfire.ical4android.LegacyAndroidCalendar

import at.bitfire.ical4android.util.MiscUtils.asSyncAdapter

import at.bitfire.ical4android.util.MiscUtils.closeCompat

import at.bitfire.synctools.storage.calendar.AndroidCalendarProvider
@@ -92,9 +92,8 @@ class LocalCalendarTest { 
                status = Status.VEVENT_CANCELLED

            })

        }

        val legacyCalendar = LegacyAndroidCalendar(calendar.androidCalendar)

        legacyCalendar.add(event = event, syncId = "filename.ics", flags = LocalResource.FLAG_REMOTELY_PRESENT)

        val localEvent = calendar.findByName("filename.ics")!!

        val localEvent = AndroidEvent(calendar.androidCalendar, event, "filename.ics", null, null, LocalResource.FLAG_REMOTELY_PRESENT)

        localEvent.add()

        val eventId = localEvent.id!!



        // set event as dirty
@@ -123,9 +122,8 @@ class LocalCalendarTest { 
            summary = "Event with 3 instances"

            rRules.add(RRule("FREQ=DAILY;COUNT=3"))

        }

        val legacyCalendar = LegacyAndroidCalendar(calendar.androidCalendar)

        legacyCalendar.add(event = event, syncId = "filename.ics", flags = LocalResource.FLAG_REMOTELY_PRESENT)

        val localEvent = calendar.findByName("filename.ics")!!

        val localEvent = AndroidEvent(calendar.androidCalendar, event, "filename.ics", null, null, LocalResource.FLAG_REMOTELY_PRESENT)

        localEvent.add()

        val eventId = localEvent.id!!



        // set event as dirty

app/src/androidTest/kotlin/at/bitfire/davdroid/resource/LocalEventTest.kt

+11 −17
Original line number Diff line number Diff line
@@ -14,8 +14,8 @@ import android.provider.CalendarContract.ACCOUNT_TYPE_LOCAL 
import android.provider.CalendarContract.Events

import androidx.test.platform.app.InstrumentationRegistry

import androidx.test.rule.GrantPermissionRule

import at.bitfire.ical4android.AndroidEvent

import at.bitfire.ical4android.Event

import at.bitfire.ical4android.LegacyAndroidCalendar

import at.bitfire.ical4android.util.MiscUtils.closeCompat

import at.bitfire.synctools.storage.calendar.AndroidCalendarProvider

import at.techbee.jtx.JtxContract.asSyncAdapter
@@ -74,10 +74,8 @@ class LocalEventTest { 
            dtStart = DtStart("20220120T010203Z")

            summary = "Event without uid"

        }



        val legacyCalendar = LegacyAndroidCalendar(calendar.androidCalendar)

        legacyCalendar.add(event = event, syncId = "filename.ics", flags = LocalResource.FLAG_REMOTELY_PRESENT)

        val localEvent = calendar.findByName("filename.ics")!!

        val localEvent = LocalEvent(AndroidEvent(calendar.androidCalendar, event, null))

        localEvent.add()    // save it to calendar storage



        // prepare for upload - this should generate a new random uuid, returned as filename

        val fileNameWithSuffix = localEvent.prepareForUpload()
@@ -104,9 +102,8 @@ class LocalEventTest { 
            summary = "Event with normal uid"

            uid = "some-event@hostname.tld"     // old UID format, UUID would be new format

        }

        val legacyCalendar = LegacyAndroidCalendar(calendar.androidCalendar)

        legacyCalendar.add(event = event, syncId = "filename.ics", flags = LocalResource.FLAG_REMOTELY_PRESENT)

        val localEvent = calendar.findByName("filename.ics")!!

        val localEvent = LocalEvent(AndroidEvent(calendar.androidCalendar, event, null))

        localEvent.add() // save it to calendar storage



        // prepare for upload - this should use the UID for the file name

        val fileNameWithSuffix = localEvent.prepareForUpload()
@@ -132,9 +129,8 @@ class LocalEventTest { 
            summary = "Event with funny uid"

            uid = "https://www.example.com/events/asdfewfe-cxyb-ewrws-sadfrwerxyvser-asdfxye-"

        }

        val legacyCalendar = LegacyAndroidCalendar(calendar.androidCalendar)

        legacyCalendar.add(event = event, syncId = "filename.ics", flags = LocalResource.FLAG_REMOTELY_PRESENT)

        val localEvent = calendar.findByName("filename.ics")!!

        val localEvent = LocalEvent(AndroidEvent(calendar.androidCalendar, event, null))

        localEvent.add() // save it to calendar storage



        // prepare for upload - this should generate a new random uuid, returned as filename

        val fileNameWithSuffix = localEvent.prepareForUpload()
@@ -185,9 +181,8 @@ class LocalEventTest { 
                status = Status.VEVENT_CANCELLED

            })

        }

        val legacyCalendar = LegacyAndroidCalendar(calendar.androidCalendar)

        legacyCalendar.add(event = event, syncId = "filename.ics", flags = LocalResource.FLAG_REMOTELY_PRESENT)

        val localEvent = calendar.findByName("filename.ics")!!

        val localEvent = LocalEvent(AndroidEvent(calendar.androidCalendar, event, "filename.ics", null, null, LocalResource.FLAG_REMOTELY_PRESENT))

        localEvent.add()

        val eventId = localEvent.id!!



        // set event as dirty
@@ -215,9 +210,8 @@ class LocalEventTest { 
            summary = "Event with 3 instances"

            rRules.add(RRule("FREQ=DAILY;COUNT=3"))

        }

        val legacyCalendar = LegacyAndroidCalendar(calendar.androidCalendar)

        legacyCalendar.add(event = event, syncId = "filename.ics", flags = LocalResource.FLAG_REMOTELY_PRESENT)

        val localEvent = calendar.findByName("filename.ics")!!

        val localEvent = LocalEvent(AndroidEvent(calendar.androidCalendar, event, "filename.ics", null, null, LocalResource.FLAG_REMOTELY_PRESENT))

        localEvent.add()

        val eventId = localEvent.id!!



        // set event as dirty

app/src/main/kotlin/at/bitfire/davdroid/resource/LocalCalendar.kt

+27 −35
Original line number Diff line number Diff line
@@ -12,7 +12,6 @@ import at.bitfire.ical4android.AndroidEvent 
import at.bitfire.ical4android.util.MiscUtils.asSyncAdapter

import at.bitfire.synctools.storage.BatchOperation

import at.bitfire.synctools.storage.calendar.AndroidCalendar

import at.bitfire.synctools.storage.calendar.AndroidEvent2

import at.bitfire.synctools.storage.calendar.CalendarBatchOperation

import dagger.assisted.Assisted

import dagger.assisted.AssistedFactory
@@ -57,15 +56,11 @@ class LocalCalendar @AssistedInject constructor( 
            androidCalendar.writeSyncState(state.toString())

        }



    override fun findDeleted(): List<LocalEvent> {

        val result = LinkedList<LocalEvent>()

        androidCalendar.iterateEventRows(null, "${Events.DELETED} AND ${Events.ORIGINAL_ID} IS NULL", null) { values ->

            // create legacy AndroidEvent from AndroidEvent2's content values

            val legacyEvent = AndroidEvent(androidCalendar, values)

            result += LocalEvent(legacyEvent)

        }

        return result

    }



    override fun findDeleted() =

        androidCalendar

            .findEvents("${Events.DELETED} AND ${Events.ORIGINAL_ID} IS NULL", null)

            .map { LocalEvent(it) }



    override fun findDirty(): List<LocalEvent> {

        val dirty = LinkedList<LocalEvent>()
@@ -75,11 +70,10 @@ class LocalCalendar @AssistedInject constructor( 
         * When a calendar component is created, its sequence number is 0. It is monotonically incremented by the "Organizer's"

         * CUA each time the "Organizer" makes a significant revision to the calendar component.

         */

        androidCalendar.iterateEventRows(null, "${Events.DIRTY} AND ${Events.ORIGINAL_ID} IS NULL", null) { values ->

            val legacyEvent = AndroidEvent(androidCalendar, values)

            val localEvent = LocalEvent(legacyEvent)

        for (androidEvent in androidCalendar.findEvents("${Events.DIRTY} AND ${Events.ORIGINAL_ID} IS NULL", null)) {

            val localEvent = LocalEvent(androidEvent)

            try {

                val event = localEvent.event

                val event = requireNotNull(androidEvent.event)



                val nonGroupScheduled = event.attendees.isEmpty()

                val weAreOrganizer = localEvent.weAreOrganizer
@@ -101,14 +95,12 @@ class LocalCalendar @AssistedInject constructor( 
    }



    override fun findByName(name: String) =

        androidCalendar.findEventRow(null, "${Events._SYNC_ID}=?", arrayOf(name))?.let { values ->

            val legacyEvent = AndroidEvent(androidCalendar, values)

            LocalEvent(legacyEvent)

        }

        androidCalendar.findEvents("${Events._SYNC_ID}=?", arrayOf(name)).firstOrNull()?.let { LocalEvent(it) }





    override fun markNotDirty(flags: Int) =

        androidCalendar.updateEventRows(

            contentValuesOf(AndroidEvent2.COLUMN_FLAGS to flags),

        androidCalendar.updateEvents(

            contentValuesOf(AndroidEvent.COLUMN_FLAGS to flags),

            "${Events.CALENDAR_ID}=? AND NOT ${Events.DIRTY} AND ${Events.ORIGINAL_ID} IS NULL",

            arrayOf(androidCalendar.id.toString())

        )
@@ -116,9 +108,9 @@ class LocalCalendar @AssistedInject constructor( 
    override fun removeNotDirtyMarked(flags: Int): Int {

        // list all non-dirty events with the given flags and delete every row + its exceptions

        val batch = CalendarBatchOperation(androidCalendar.client)

        androidCalendar.iterateEventRows(

        androidCalendar.iterateEvents(

            arrayOf(Events._ID),

            "${Events.CALENDAR_ID}=? AND NOT ${Events.DIRTY} AND ${Events.ORIGINAL_ID} IS NULL AND ${AndroidEvent2.COLUMN_FLAGS}=?",

            "${Events.CALENDAR_ID}=? AND NOT ${Events.DIRTY} AND ${Events.ORIGINAL_ID} IS NULL AND ${AndroidEvent.COLUMN_FLAGS}=?",

            arrayOf(androidCalendar.id.toString(), flags.toString())

        ) { values ->

            val id = values.getAsInteger(Events._ID)
@@ -132,8 +124,8 @@ class LocalCalendar @AssistedInject constructor( 
    }



    override fun forgetETags() {

        androidCalendar.updateEventRows(

            contentValuesOf(AndroidEvent2.COLUMN_ETAG to null),

        androidCalendar.updateEvents(

            contentValuesOf(AndroidEvent.COLUMN_ETAG to null),

            "${Events.CALENDAR_ID}=?", arrayOf(androidCalendar.id.toString())

        )

    }
@@ -143,8 +135,8 @@ class LocalCalendar @AssistedInject constructor( 
        // process deleted exceptions

        logger.info("Processing deleted exceptions")



        androidCalendar.iterateEventRows(

            arrayOf(Events._ID, Events.ORIGINAL_ID, AndroidEvent2.COLUMN_SEQUENCE),

        androidCalendar.iterateEvents(

            arrayOf(Events._ID, Events.ORIGINAL_ID, AndroidEvent.COLUMN_SEQUENCE),

            "${Events.CALENDAR_ID}=? AND ${Events.DELETED} AND ${Events.ORIGINAL_ID} IS NOT NULL",

            arrayOf(androidCalendar.id.toString())

        ) { values ->
@@ -156,12 +148,12 @@ class LocalCalendar @AssistedInject constructor( 
            val batch = CalendarBatchOperation(androidCalendar.client)



            // enqueue: increase sequence of main event

            val originalEventValues = androidCalendar.getEventRow(originalID, arrayOf(AndroidEvent2.COLUMN_SEQUENCE))

            val originalSequence = originalEventValues?.getAsInteger(AndroidEvent2.COLUMN_SEQUENCE) ?: 0

            val originalEventValues = androidCalendar.getEventValues(originalID, arrayOf(AndroidEvent.COLUMN_SEQUENCE))

            val originalSequence = originalEventValues?.getAsInteger(AndroidEvent.COLUMN_SEQUENCE) ?: 0



            batch += BatchOperation.CpoBuilder

                .newUpdate(ContentUris.withAppendedId(Events.CONTENT_URI, originalID).asSyncAdapter(androidCalendar.account))

                .withValue(AndroidEvent2.COLUMN_SEQUENCE, originalSequence + 1)

                .withValue(AndroidEvent.COLUMN_SEQUENCE, originalSequence + 1)

                .withValue(Events.DIRTY, 1)



            // completely remove deleted exception
@@ -171,8 +163,8 @@ class LocalCalendar @AssistedInject constructor( 


        // process dirty exceptions

        logger.info("Processing dirty exceptions")

        androidCalendar.iterateEventRows(

            arrayOf(Events._ID, Events.ORIGINAL_ID, AndroidEvent2.COLUMN_SEQUENCE),

        androidCalendar.iterateEvents(

            arrayOf(Events._ID, Events.ORIGINAL_ID, AndroidEvent.COLUMN_SEQUENCE),

            "${Events.CALENDAR_ID}=? AND ${Events.DIRTY} AND ${Events.ORIGINAL_ID} IS NOT NULL",

            arrayOf(androidCalendar.id.toString())

        ) { values ->
@@ -180,7 +172,7 @@ class LocalCalendar @AssistedInject constructor( 


            val id = values.getAsLong(Events._ID)                   // can't be null (by definition)

            val originalID = values.getAsLong(Events.ORIGINAL_ID)   // can't be null (by query)

            val sequence = values.getAsInteger(AndroidEvent2.COLUMN_SEQUENCE) ?: 0

            val sequence = values.getAsInteger(AndroidEvent.COLUMN_SEQUENCE) ?: 0



            val batch = CalendarBatchOperation(androidCalendar.client)
@@ -192,7 +184,7 @@ class LocalCalendar @AssistedInject constructor( 
            // enqueue: increase exception SEQUENCE and set DIRTY to 0

            batch += BatchOperation.CpoBuilder

                .newUpdate(androidCalendar.eventUri(id))

                .withValue(AndroidEvent2.COLUMN_SEQUENCE, sequence + 1)

                .withValue(AndroidEvent.COLUMN_SEQUENCE, sequence + 1)

                .withValue(Events.DIRTY, 0)



            batch.commit()
@@ -206,7 +198,7 @@ class LocalCalendar @AssistedInject constructor( 
     */

    fun deleteDirtyEventsWithoutInstances() {

        // Iterate dirty main events without exceptions

        androidCalendar.iterateEventRows(

        androidCalendar.iterateEvents(

            arrayOf(Events._ID),

            "${Events.DIRTY} AND NOT ${Events.DELETED} AND ${Events.ORIGINAL_ID} IS NULL",

            null
@@ -219,7 +211,7 @@ class LocalCalendar @AssistedInject constructor( 
            // delete event if there are no instances

            if (numEventInstances == 0) {

                logger.fine("Marking event #$eventId without instances as deleted")

                androidCalendar.updateEventRow(eventId, contentValuesOf(Events.DELETED to 1))

                androidCalendar.updateEvent(eventId, contentValuesOf(Events.DELETED to 1))

            }

        }

    }

app/src/main/kotlin/at/bitfire/davdroid/resource/LocalEvent.kt

+11 −18
Original line number Diff line number Diff line
@@ -9,9 +9,6 @@ import android.provider.CalendarContract.Events 
import androidx.core.content.contentValuesOf

import at.bitfire.ical4android.AndroidEvent

import at.bitfire.ical4android.Event

import at.bitfire.ical4android.LegacyAndroidCalendar

import at.bitfire.synctools.storage.LocalStorageException

import at.bitfire.synctools.storage.calendar.AndroidEvent2

import java.util.UUID



class LocalEvent(
@@ -40,6 +37,8 @@ class LocalEvent( 
    override val flags: Int

        get() = androidEvent.flags



    override fun add() = androidEvent.add()



    override fun update(data: Event) = androidEvent.update(data)



    override fun delete() = androidEvent.delete()
@@ -47,12 +46,8 @@ class LocalEvent( 


    // other methods



    val event: Event by lazy {

        val legacyCalendar = LegacyAndroidCalendar(androidEvent.calendar)

        legacyCalendar.getEvent(androidEvent.id) ?: throw LocalStorageException("Event ${androidEvent.id} not found")

    }



    val weAreOrganizer: Boolean = event.isOrganizer == true

    val weAreOrganizer

        get() = androidEvent.event!!.isOrganizer == true





    /**
@@ -63,7 +58,7 @@ class LocalEvent( 
     */

    override fun prepareForUpload(): String {

        // make sure that UID is set

        val uid: String = event.uid ?: run {

        val uid: String = androidEvent.event!!.uid ?: run {

            // generate new UID

            val newUid = UUID.randomUUID().toString()
@@ -71,8 +66,8 @@ class LocalEvent( 
            val values = contentValuesOf(Events.UID_2445 to newUid)

            androidEvent.update(values)



            // update in event data object (does not write to calendar store!)

            event.uid = newUid

            // update this event

            androidEvent.event?.uid = newUid



            newUid

        }
@@ -90,16 +85,14 @@ class LocalEvent( 
            "${UUID.randomUUID()}.ics"      // UID would be dangerous as file name, use random UUID instead

    }



    @Deprecated("Use add...() of specific collection implementation", level = DeprecationLevel.ERROR)

    override fun add() = throw NotImplementedError()



    override fun clearDirty(fileName: String?, eTag: String?, scheduleTag: String?) {

        val values = ContentValues(5)

        if (fileName != null)

            values.put(Events._SYNC_ID, fileName)

        values.put(AndroidEvent2.COLUMN_ETAG, eTag)

        values.put(AndroidEvent2.COLUMN_SCHEDULE_TAG, scheduleTag)

        values.put(AndroidEvent2.COLUMN_SEQUENCE, event.sequence)

        values.put(AndroidEvent.COLUMN_ETAG, eTag)

        values.put(AndroidEvent.COLUMN_SCHEDULE_TAG, scheduleTag)

        values.put(AndroidEvent.COLUMN_SEQUENCE, androidEvent.event!!.sequence)

        values.put(Events.DIRTY, 0)

        androidEvent.update(values)
@@ -110,7 +103,7 @@ class LocalEvent( 
    }



    override fun updateFlags(flags: Int) {

        val values = contentValuesOf(AndroidEvent2.COLUMN_FLAGS to flags)

        val values = contentValuesOf(AndroidEvent.COLUMN_FLAGS to flags)

        androidEvent.update(values)



        androidEvent.flags = flags

app/src/main/kotlin/at/bitfire/davdroid/resource/LocalResource.kt

+0 −5
Original line number Diff line number Diff line
@@ -5,11 +5,7 @@ 
package at.bitfire.davdroid.resource



import android.net.Uri

import at.bitfire.davdroid.resource.LocalResource.Companion.FLAG_REMOTELY_PRESENT



/**

 * Defines operations that are used by SyncManager for all sync data types.

 */

interface LocalResource<in TData: Any> {



    companion object {
@@ -78,7 +74,6 @@ interface LocalResource<in TData: Any> { 
     *

     * @return content URI of the created row (e.g. event URI)

     */

    @Deprecated("Use add...() of specific collection implementation")

    fun add(): Uri



    /**