Merge pull request #4603 from vector-im/feature/aris/remove_not_needed_job_cancel_4602

There is no need to call job.cancel() when we are using viewModelScope()

changelog.d/4602.misc

@@ -0,0 +1 @@
+There is no need to call job.cancel() when we are using viewModelScope()

vector/src/main/java/im/vector/app/features/home/room/detail/search/SearchViewModel.kt

@@ -141,9 +141,4 @@ class SearchViewModel @AssistedInject constructor(
             )
         }
     }
-
-    override fun onCleared() {
-        currentTask?.cancel()
-        super.onCleared()
-    }
 }

vector/src/main/java/im/vector/app/features/roomdirectory/RoomDirectoryViewModel.kt

@@ -231,9 +231,4 @@ class RoomDirectoryViewModel @AssistedInject constructor(
             }
         }
     }
-
-    override fun onCleared() {
-        currentJob?.cancel()
-        super.onCleared()
-    }
 }