Search bug fixed

libpupil/src/main/java/xyz/quaver/hitomi/results.kt

@@ -15,7 +15,6 @@ fun doSearch(query: String) : List<Int> {
         .toLowerCase()
         .split(Regex("\\s+"))
 
-    val results = ArrayList<Int>()
     val positiveTerms = LinkedList<String>()
     val negativeTerms = LinkedList<String>()
 
@@ -26,33 +25,24 @@ fun doSearch(query: String) : List<Int> {
             positiveTerms.push(term)
     }
 
-    //first results
-    results.addAll(
-        if (positiveTerms.isEmpty())
-            getGalleryIDsFromNozomi(null, "index", "all")
-        else
-            getGalleryIDsForQuery(positiveTerms.poll())
-    )
+    var results = when {
+        positiveTerms.isEmpty() -> getGalleryIDsFromNozomi(null, "index", "all")
+        else -> getGalleryIDsForQuery(positiveTerms.poll())
+    }
 
     runBlocking {
         @Synchronized fun filterPositive(newResults: List<Int>) {
-            results.filter { newResults.binarySearch(it) >= 0 }.let {
-                results.clear()
-                results.addAll(it)
-            }
+            results = results.filter { newResults.binarySearch(it) >= 0 }
         }
 
         @Synchronized fun filterNegative(newResults: List<Int>) {
-            results.filterNot { newResults.binarySearch(it) >= 0 }.let {
-                results.clear()
-                results.addAll(it)
-            }
+            results = results.filter { newResults.binarySearch(it) < 0 }
         }
 
         //positive results
         positiveTerms.map {
             launch(searchDispatcher) {
-                filterPositive(getGalleryIDsForQuery(it).reversed())
+                filterPositive(getGalleryIDsForQuery(it).sorted())
             }
         }.forEach {
             it.join()
@@ -61,7 +51,7 @@ fun doSearch(query: String) : List<Int> {
         //negative results
         negativeTerms.map {
             launch(searchDispatcher) {
-                filterNegative(getGalleryIDsForQuery(it).reversed())
+                filterNegative(getGalleryIDsForQuery(it).sorted())
             }
         }.forEach {
             it.join()

libpupil/src/test/java/xyz/quaver/hitomi/UnitTest.kt

@@ -21,10 +21,9 @@ class UnitTest {
 
     @Test
     fun test_search() {
-        val ids = getGalleryIDsForQuery("female:loli").reversed()
+        val ids = getGalleryIDsForQuery("language:korean").reversed()
 
-        for (i in 0..100)
-            println(ids[i])
+        print(ids.size)
     }
 
     @Test
@@ -36,7 +35,7 @@ class UnitTest {
 
     @Test
     fun test_doSearch() {
-        val r = doSearch("type:artistcg language:korean female:loli female:mind_break -female:anal")
+        val r = doSearch("female:loli female:bondage language:korean -male:yaoi -male:guro -female:guro")
 
         print(r.size)
     }