Revert "During migration, only look at new tickets instead of updated ones (#…" (#850)

This reverts commit de488fe.

Author: urielsalis

src/main/kotlin/io/github/mojira/arisa/ExecutionTimeframe.kt

@@ -49,7 +49,7 @@ class ExecutionTimeframe(
      * Creates a JQL query for freshly updated issues.
      */
     fun getFreshlyUpdatedJql() =
-        "created > ${lastRunTime.toEpochMilli()}${capIfNotOpenEnded()}"
+        "updated > ${lastRunTime.toEpochMilli()}${capIfNotOpenEnded()}"
 
     /**
      * Creates a JQL query for issues which have been updated in the execution timeframe, shifted to the past
@@ -59,7 +59,7 @@ class ExecutionTimeframe(
         require(!offset.isNegative)
         val checkStart = lastRunTime.minus(offset)
         val checkEnd = currentRunTime.minus(offset)
-        return "created > ${checkStart.toEpochMilli()} AND created <= ${checkEnd.toEpochMilli()}"
+        return "updated > ${checkStart.toEpochMilli()} AND updated <= ${checkEnd.toEpochMilli()}"
     }
 
     fun contains(instant: Instant): Boolean = instant in lastRunTime..currentRunTime
@@ -70,7 +70,7 @@ class ExecutionTimeframe(
      * @return If open ended: empty string. Otherwise: ` AND updated <= [currentRunTime]`
      */
     private fun capIfNotOpenEnded(): String =
-        if (openEnded) "" else " AND created <= ${currentRunTime.toEpochMilli()}"
+        if (openEnded) "" else " AND updated <= ${currentRunTime.toEpochMilli()}"
 
     override fun toString(): String {
         val formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss")

src/test/kotlin/io/github/mojira/arisa/ExecutionTimeframeTest.kt

@@ -25,7 +25,7 @@ class ExecutionTimeframeTest : StringSpec({
         timeframe.lastRunTime shouldBe lastRunTime
         timeframe.currentRunTime.isAfter(Instant.now()) shouldBe false
         // Should not be capped
-        timeframe.getFreshlyUpdatedJql() shouldNotContain " AND created <= "
+        timeframe.getFreshlyUpdatedJql() shouldNotContain " AND updated <= "
 
         val delayedStart = LocalDateTime.of(2021, 1, 1, 12, 0, 0)
             .atZone(ZoneOffset.UTC)
@@ -35,7 +35,7 @@ class ExecutionTimeframeTest : StringSpec({
         // Shift timeframe to start at `offsetBaseInstant`
         // Note: Cannot hardcode `delayedEnd` value in string because it depends on how fast
         // `ExecutionTimeframe.getTimeframeFromLastRun` executes
-        timeframe.getDelayedUpdatedJql(offset) shouldBe "created > 1609502400000 AND created <= ${delayedEnd.toEpochMilli()}"
+        timeframe.getDelayedUpdatedJql(offset) shouldBe "updated > 1609502400000 AND updated <= ${delayedEnd.toEpochMilli()}"
     }
 
     "getTimeframeFromLastRun should return the correct timeframe if last run was a while ago" {
@@ -56,7 +56,7 @@ class ExecutionTimeframeTest : StringSpec({
 
         timeframe.lastRunTime shouldBe lastRunTime
         timeframe.currentRunTime shouldBe timeframeEnd
-        timeframe.getFreshlyUpdatedJql() shouldBe "created > 1609502400000 AND created <= 1609503000000"
-        timeframe.getDelayedUpdatedJql(Duration.ofHours(1)) shouldBe "created > 1609498800000 AND created <= 1609499400000"
+        timeframe.getFreshlyUpdatedJql() shouldBe "updated > 1609502400000 AND updated <= 1609503000000"
+        timeframe.getDelayedUpdatedJql(Duration.ofHours(1)) shouldBe "updated > 1609498800000 AND updated <= 1609499400000"
     }
 })