Rebuild the order after a habit deletion.

uhabits-core/src/jvmMain/java/org/isoron/uhabits/core/models/sqlite/SQLiteHabitList.kt

@@ -131,9 +131,6 @@ class SQLiteHabitList @Inject constructor(private val modelFactory: ModelFactory
     @Synchronized
     override fun remove(h: Habit) {
         loadRecords()
-        if (Order.BY_POSITION.equals(list.primaryOrder)) {
-            reorder(h, list.getByPosition(size() - 1))
-        }
         list.remove(h)
         val record = repository.find(
             h.id!!
@@ -142,6 +139,7 @@ class SQLiteHabitList @Inject constructor(private val modelFactory: ModelFactory
             h.originalEntries.clear()
             repository.remove(record)
         }
+        rebuildOrder()
         observable.notifyListeners()
     }
 

uhabits-core/src/jvmTest/java/org/isoron/uhabits/core/models/sqlite/SQLiteHabitListTest.kt

@@ -158,12 +158,18 @@ class SQLiteHabitListTest : BaseUnitTest() {
         habitList.remove(h!!)
         assertThat(habitList.indexOf(h), equalTo(-1))
 
-        habitList.add(h)
+        var rec = repository.find(2L)
-        val primaryOrder = habitList.primaryOrder
+        assertNull(rec)
+        rec = repository.find(3L)!!
+        assertThat(rec.position, equalTo(1))
+    }
+
+    @Test
+    fun testRemove_orderByName() {
         habitList.primaryOrder = HabitList.Order.BY_NAME_DESC
-        habitList.remove(h)
+        val h = habitList.getById(2)
+        habitList.remove(h!!)
         assertThat(habitList.indexOf(h), equalTo(-1))
-        habitList.primaryOrder = primaryOrder
 
         var rec = repository.find(2L)
         assertNull(rec)