query - no need to use an explicit TX with find/findFirst() because they use a visitor

Author: vaind

objectbox/lib/src/native/query/query.dart

@@ -13,7 +13,6 @@ import '../../modelinfo/entity_definition.dart';
 import '../../modelinfo/modelproperty.dart';
 import '../../modelinfo/modelrelation.dart';
 import '../../store.dart';
-import '../../transaction.dart';
 import '../bindings/bindings.dart';
 import '../bindings/data_visitor.dart';
 import '../bindings/helpers.dart';
@@ -735,18 +734,16 @@ class Query<T> {
     }
   }
 
-  /// Finds Objects matching the query and returns the first result or null
-  /// if there are no results. Note: [offset] and [limit] are respected, if set.
+  /// Finds the first object matching the query. Returns null if there are no
+  /// results. Note: [offset] and [limit] are respected, if set.
   T? findFirst() {
     T? result;
     final visitor = dataVisitor((Pointer<Uint8> data, int size) {
       result = _entity.objectFromFB(
           _store, InternalStoreAccess.reader(_store).access(data, size));
       return false; // we only want to visit the first element
     });
-    _store.runInTransaction(TxMode.read, () {
-      checkObx(C.query_visit(_ptr, visitor, nullptr));
-    });
+    checkObx(C.query_visit(_ptr, visitor, nullptr));
     reachabilityFence(this);
     return result;
   }
@@ -769,8 +766,7 @@ class Query<T> {
   List<T> find() {
     final result = <T>[];
     final collector = objectCollector(result, _store, _entity);
-    _store.runInTransaction(
-        TxMode.read, () => checkObx(C.query_visit(_ptr, collector, nullptr)));
+    checkObx(C.query_visit(_ptr, collector, nullptr));
     reachabilityFence(this);
     return result;
   }